// update timestamp, see http://drupal.org/node/353405
  $file->timestamp = REQUEST_TIME;
  drupal_write_record('files', $file, 'fid');

This is updating every file timestamp on a given node, everytime a new file is added. Even for non-image files. I've fixed this locally by simply commenting out those 2 lines, but this probably isn't the appropriate fix given #353405: Need help with imagefield_crop 6.x implementation.

CommentFileSizeAuthor
#1 imagefield_crop-1426206-01.patch507 bytesjhedstrom

Comments

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Active » Needs review
StatusFileSize
new imagefield_crop-1426206-01.patch507 bytes

Patch that removes those lines, but like I said in the original post, I'm guessing there's a better way to do this. At a minimum, this hook should verify that the file in question is using this module's widget.

ram4nd’s picture

ram4nd commented
Issue summary: View changes
Status: Needs review » Fixed

Can't find this code from D7 branch, marking this fixed.

Status: Fixed » Closed (fixed)

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