the standard in core for includes/requires is to have a leading period like ./sites/all/modules/views/arguments/ (for example). the benefit of the leading period+slash is that you avoid php searching the include path for the file.

Comments

merlinofchaos’s picture

merlinofchaos commented
Status: Active » Fixed

Fixed in cvs.

htalvitie’s picture

htalvitie commented

Related: http://drupal.org/node/230929 (Use consistent syntax (./path/, not path/) for includes)

@moshe: The "./path" syntax requirement is not mentioned currently on http://drupal.org/coding-standards (section "Including Code"). Maybe this should be added?

moshe weitzman’s picture

moshe weitzman commented

@htalvitie - yes, sounds good. please join the docs team (if you haven't already) and add it. i like the way you think.

Anonymous’s picture

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

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

c960657’s picture

c960657 commented

Related: #259623: Broken autoloader: convert includes/requires to use absolute paths