I have an imagefield_crop that I would like to have use one of three possible aspect ratios. Instead of making three fields and hiding the inactive ones, I'd rather just change the resolution with a hook.

Patch to follow.

CommentFileSizeAuthor
#4 imagefield_crop-hook_imagefield_crop_instance_alter-1915510-4.patch1.96 KBroysegall
#3 imagefield_crop-hook_imagefield_crop_instance_alter-1915510-2.patch1.78 KBjsagotsky
#1 imagefield_crop-hook_imagefield_crop_instance_alter-1915510-1.patch1.82 KBjsagotsky

Comments

jsagotsky’s picture

jsagotsky commented
StatusFileSize
new imagefield_crop-hook_imagefield_crop_instance_alter-1915510-1.patch1.82 KB

Patch provides hook_imagefield_crop_instance_alter, which allows you to modify the field instance right after it's loaded.

joetsuihk’s picture

joetsuihk commented
Status: Active » Needs review

Thanks! Please have someone review this patch before I can commit this.

jsagotsky’s picture

jsagotsky commented
StatusFileSize
new imagefield_crop-hook_imagefield_crop_instance_alter-1915510-2.patch1.78 KB

Missed an instance where I needed to alter. Here's a rerolled patch.

roysegall’s picture

roysegall commented
StatusFileSize
new imagefield_crop-hook_imagefield_crop_instance_alter-1915510-4.patch1.96 KB

Re rolling

ram4nd’s picture

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

  • ram4nd committed 58aed3a on 7.x-2.x authored by jsagotsky 
    Issue #1915510 by jsagotsky, RoySegall: Provide a hook so a field's...

Status: Fixed » Closed (fixed)

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