imagecrop.admin.inc:209:

watchdog('imagecrop', 'Access denied for user @name on fid @fid', array('@name' => $user->name, '@fid' => $imagecrop->file->fid));

At imagecrop.admin.inc:198, $imagecrop is instantiated as an object: $imagecrop = new ImageCrop();

ImageCrop::$file is a private member. The class's public file getter, ImageCrop::getFile(), should be used instead:

- watchdog('imagecrop', 'Access denied for user @name on fid @fid', array('@name' => $user->name, '@fid' => $imagecrop->file->fid)); 
+ watchdog('imagecrop', 'Access denied for user @name on fid @fid', array('@name' => $user->name, '@fid' => $imagecrop->getFile()->fid));

Comments

nils.destoop’s picture

Status: Needs review » Fixed

Thx for notifying about this. The change has been committed.

Status: Fixed » Closed (fixed)

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