Hi there,

I noticed today that I'd get an error when I turned on the php E_NOTICE flag (more info here: #761780: Plenty of E_ALL notices in modules developed on installation with Drupal tarball)

If you could please fix it, it would be great! 8-)

* notice: Undefined variable: path in /usr/clients/www/drupal/sites/all/modules/themekey/themekey_base.inc on line 133.

Thank you.
Alexis Wilke

CommentFileSizeAuthor
#1 themekey-global_params-6.x.patch547 bytesAlexisWilke

Comments

AlexisWilke’s picture

AlexisWilke commented
Assigned: Unassigned » AlexisWilke
StatusFileSize
new themekey-global_params-6.x.patch547 bytes

Okay, there is a patch. Note that $path was undefined so NULL (which is the default for arg() second parameter) so removing it completely is the same as what you had before. However, you may want to review your code and make sure that it was really meant that way.

Thank you.
Alexis Wilke

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Status: Active » Fixed

You're right. Committed to CVS.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.