Closed (fixed)
Project:
Views (for Drupal 7)
Version:
6.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Mar 2006 at 01:43 UTC
Updated:
1 Apr 2006 at 19:15 UTC
... raising :
Fatal error: Call to undefined function: drupal_get_path() in (...)\modules\views\views.module on line 6
Comments
Comment #1
yched commentedI would add : _highly_ critical, since once you logged out, you can't access ?q=user/login to log in, and have to disable views module through the database
Comment #2
RayZ commentedCan you give the steps to reproduce this? I'm not seeing it.
Comment #3
yched commentedTurn page caching on.
Empty cache (devel module or plain sql "TRUNCATE TABLE cache").
Log out.
Go to any page : everything's fine
Hit F5 (or CTRL-F5 maybe ?) : Fatal error
Since no-one has confirmed yet on something so big, I'm beginnig to wonder if it's something weird in my own instal ?
Comment #4
merlinofchaos commentedNo, it's not you; I was able to reproduce this.
I added a check in hook_init to see if drupal_get_path exists. If it doesn't, we just exit. Cached pages don't need to load that stuff anyway.
Sorry about this one, I misunderstood hook_init.
Comment #5
merlinofchaos commentedComment #6
yched commentedThanks :-)
BTW, could you remove the trailing spaces in views.module ?
They get in the way when you compare file versions / modifications
(other files seem OK)
Comment #7
merlinofchaos commentedHow's that?
Comment #8
yched commentedWell, for instance line 218 in views.module is
" _views_sanitize_view($view); "there's a useless space at the end of the line, which is not adviced in drupal coding style guideline.
Some editors (I use Zend Studio) automatically trim those spaces on file save / file close
Subsequent file comparison between local copy and cvs copy brings a "false positive", since the files are marked "different" when the only differences are these meaningless spaces.
They also come out when you make a patch, etc...
No big deal, really, but it just gets in the way...
Comment #9
merlinofchaos commentedI don't see any extra whitespace on that line in CVS.
Comment #10
yched commentedSorry, I guess I misunderstood your "how's that ?" comment.
I thought it was a request for explanations, when you actually had done the job.
Thx :-)
Comment #11
(not verified) commented