Patch attached fixes many bug in strings and allows easier translation for translators.

CommentFileSizeAuthor
#5 D6_image_string_review_2008122501.patch36.72 KBhass
#4 D6_image_string_review_2008122201.patch36.29 KBhass
D6_image_string_review_2008101201.patch37.44 KBhass
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hass’s picture

hass CreditAttribution: hass commented

Bumping

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented

+1

drewish’s picture

drewish CreditAttribution: drewish commented
Status: Needs review » Needs work

if you're going to bump it at least make sure it still applies. the image_attach and image_import changes fail and it looks like image.module has a syntax error.

hass’s picture

hass CreditAttribution: hass commented
Status: Needs work » Needs review
FileSize
D6_image_string_review_2008122201.patch36.29 KB

Seems like some of the issues are already fixed. Re-roled that patch + fixed the syntax errors - sorry for this. :-(

hass’s picture

hass CreditAttribution: hass commented
FileSize
D6_image_string_review_2008122501.patch36.72 KB

Added two missing strings and chasing head.

drewish’s picture

drewish CreditAttribution: drewish commented
Status: Needs review » Fixed

Much better patch. Thanks, looks great. Committed to HEAD.

Status: Fixed » Closed (fixed)

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