Replace deprecated usage of entity_load('image_style') with a direct call to ImageStyle::load.

CommentFileSizeAuthor
#2 2645016-2.patch2.05 KBheykarthikwithu

Comments

heykarthikwithu created an issue. See original summary.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Assigned: heykarthikwithu » Unassigned
Status: Active » Needs review
StatusFileSize
new 2645016-2.patch2.05 KB

replaced the usage of deprecated entity_load('image_style') with a direct call to ImageStyle::load.

Status: Needs review » Needs work

The last submitted patch, 2: 2645016-2.patch, failed testing.

The last submitted patch, 2: 2645016-2.patch, failed testing.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Status: Needs work » Needs review
frjo’s picture

frjo commented
Status: Needs review » Fixed

I have just committed code that sync up the Colorbox formatter with core ImageFormatter so this should be fixed. Did this before looking at the issue queue.

Thanks for taking the time to improve the Colorbox module!

frjo’s picture

frjo commented

Missed one instance of this in template_preprocess_colorbox_formatter(). Saw "entity_load('image_style'", remembers this post and fixed it. Thanks!

Status: Fixed » Closed (fixed)

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