The user reports a fatal error on her upgraded site after she has gone to an external link where the option 'New Without Navigation' produces a popup window. I am unable to reproduce the error in my 2.5.7 test site, installed from scratch, even though the code in line 36 of modules/mod_menu/temp/default_url.php is identical, but it looks like get() is called on a non-object.
Here is a fix which makes sure that get() is performed on an object:
Code: Select all
// $options = 'toolbar=no,location=no,status=no,menubar=no,scrollbars=yes,resizable=yes,'.$params->get('window_open');
$options = 'toolbar=no,location=no,status=no,menubar=no,scrollbars=yes,resizable=yes,'.$item->params->get('window_open');
Even though this behavior could be a by-product of the upgrade, should I still report it?