Attached is a patch that helps avoid E_NOTICE problems and adds some text cleanups (messages should be sentences, and adds the @todo keyword for a todo comment).

CommentFileSizeAuthor
#2 796278_imagefield_crop_cleanups_2.patch3.67 KBgreggles
#1 796278_imagefield_crop_cleanups.patch3.01 KBgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
StatusFileSize
new 796278_imagefield_crop_cleanups.patch3.01 KB
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
StatusFileSize
new 796278_imagefield_crop_cleanups_2.patch3.67 KB

And more fixups.

yhager’s picture

yhager commented
Status: Needs review » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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