there are certain circumstances where drupal_get_path() is not loaded, but the module code is. attached simply wraps the code using drupal_get_path() in a function_exists() check. i believe this will work fine, as i don't think those .inc files need to be loaded in cases where drupal_get_path() isn't available.

CommentFileSizeAuthor
#1 path_fix.patch1.52 KBhunmonk
path_fix.patch981 byteshunmonk

Comments

hunmonk’s picture

hunmonk commented
StatusFileSize
new path_fix.patch1.52 KB

chx pointed out that we can't have any functions run in project_issue_exit() that are in the included files, in the case where drupal_get_path() isn't available (ie, cached pages), so i've added a check to test for that case.

chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community
hunmonk’s picture

hunmonk commented
Status: Reviewed & tested by the community » Patch (to be ported)

fixed in HEAD. this problem exists in all other branches, but since project_issue didn't have an exit() hook before, it wasn't causing any headaches. leaving to be ported if anybody wants to clean that up.

dww’s picture

dww
we/he/they
commented
Status: Patch (to be ported) » Fixed

Backported and committed to DRUPAL-5, DRUPAL-4-7--2 and DRUPAL-4-7. Just cleaning out low-hanging fruit. ;)

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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