Problem:
admin/settings/node-image produces blank page. No errors are generated (checked with different page; no messages);
node/x/images same problem

Server: php5, apache2.2, linux Fedora Code 4, drupal 5.1

Bug traced to function _node_images_mkdir_recursive

Erronous function call on line 798 (line 5 inside function definition):

if (!file_check_directory(implode("/", $dirs), $mode, $form_item)) {

drupal 5.1 function file_check_directory: first parameter is a pointer. You can't point to the result of a function (appearantly)

Solution: put function result in a variable first;

$dir = implode("/", $dirs);
if (!file_check_directory($dir, $mode, $form_item)) {

pseudo-patch:
@798
- if (!file_check_directory(implode("/", $dirs), $mode, $form_item)) {
+ $dir = implode("/", $dirs);
+ if (!file_check_directory($dir, $mode, $form_item)) {

Comments

rmpel’s picture

Forgot to add; the version this is based on is the 1.9 version of the module.

stefano73’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

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