hook_image_default_styles() was removed in the move to CMI, however it is still listed in image.api.php. It should be removed.

CommentFileSizeAuthor
#2 image_default_styles_removed-1787156.patch2.3 KBjaffaralia

Comments

xjm’s picture

xjm
she/her
Primary language English
commented

jaffaralia is working on this issue during core mentoring. (I forgot to suggest assigning the issue to himself.)

jaffaralia’s picture

jaffaralia commented
Status: Active » Needs review
StatusFileSize
new image_default_styles_removed-1787156.patch2.3 KB

I removed hook_image_default_styles() function in image.api.php. Here i attached patch for this issue.

Thanks,
Jaffar

gdd’s picture

gdd
he/him
Primary language English
Location Portland, OR
commented
Status: Needs review » Reviewed & tested by the community

This looks perfect, thank you!

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thanks -- committed to 8.x. jaffaralia: congratulations on (probably?) your first core patch!

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