Undefined variable

emmajane - September 3, 2008 - 15:33
Project:Super Nav
Version:6.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

I have a mostly-empty Drupal 6 installation and I'm having problems with variable errors appearing on my pages:

* notice: Undefined index: supernav_logout in /home/emmajane/websites/drupal/drupal-6/sites/all/modules/supernav/supernav.module on line 1161.
* notice: Undefined variable: script in /home/emmajane/websites/drupal/drupal-6/sites/all/modules/supernav/supernav.module on line 1184.
* notice: Undefined variable: comma in /home/emmajane/websites/drupal/drupal-6/sites/all/modules/supernav/supernav.module on line 74.
* notice: Undefined variable: shortcut_keys in /home/emmajane/websites/drupal/drupal-6/sites/all/modules/supernav/supernav.module on line 74.
* notice: Undefined variable: comma in /home/emmajane/websites/drupal/drupal-6/sites/all/modules/supernav/supernav.module on line 74.
* notice: Undefined variable: shortcut_keys in /home/emmajane/websites/drupal/drupal-6/sites/all/modules/supernav/supernav.module on line 74.

These appear to be bits of injected Javascript? I am using the default Garland theme when these problems arise ($closure is printed at the bottom of the page).

#1

emmajane - September 3, 2008 - 16:02
Status:active» needs review

I think I've found the spots where the variables are getting confused. Please see attached patch for the module.

AttachmentSize
supernav.patch 1.6 KB

#2

chrisshattuck - November 6, 2008 - 00:18

Please see http://drupal.org/node/285027#comment-1094346. Thank you for the patch, emmajane!

 
 

Drupal is a registered trademark of Dries Buytaert.