We should be using module_load_include() instead of require_once(drupal_get_path('module', 'sheetnode') . '/socialcalc.inc'); to load the include file.

I will attach a patch shortly.

CommentFileSizeAuthor
#3 replace-require-once-with-module-load-include-2631668.patch9.3 KBwillhallonline

Comments

willhallonline created an issue. See original summary.

willhallonline’s picture

willhallonline
Primary language English
Location Cambridge, UK
commented
Issue summary: View changes
willhallonline’s picture

willhallonline
Primary language English
Location Cambridge, UK
commented
StatusFileSize
new replace-require-once-with-module-load-include-2631668.patch9.3 KB

All instances of:

require_once(drupal_get_path('module', 'sheetnode') . '/socialcalc.inc');

Replaced with

module_load_include('inc', 'sheetnode', 'socialcalc');

willhallonline’s picture

willhallonline
Primary language English
Location Cambridge, UK
commented
Status: Active » Needs review

  • willhallonline committed 2051a01 on 7.x-1.x 
    Issue #2631668 by willhallonline: Replace require_once() with...
willhallonline’s picture

willhallonline
Primary language English
Location Cambridge, UK
commented
Status: Needs review » Fixed
willhallonline’s picture

willhallonline
Primary language English
Location Cambridge, UK
commented
Status: Fixed » Closed (fixed)