The attached D6 and D7 patch fixes a context sensitive issue in system module.

CommentFileSizeAuthor
#3 D7_core_system_context_sensitive_bug_2008101401.patch2.41 KBhass
#3 D6_core_system_context_sensitive_bug_2008101401.patch2.3 KBhass
D7_core_system_context_sensitive_bug_2008101301.patch2.46 KBhass
D6_core_system_context_sensitive_bug_2008101301.patch2.35 KBhass

Comments

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Reviewed & tested by the community

Looks great, thanks for all of us non english-speaking :)

dries’s picture

dries commented
Status: Reviewed & tested by the community » Needs work

Elsewhere in Drupal, we tend to use @module instead of @module_name. Can we reroll with such a change?

hass’s picture

hass commented
Status: Needs work » Needs review
StatusFileSize
new D6_core_system_context_sensitive_bug_2008101401.patch2.3 KB
new D7_core_system_context_sensitive_bug_2008101401.patch2.41 KB

re-roled with @module.

drewish’s picture

drewish commented

looks like a simple enough change to me.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Reviewed & tested by the community

Thanks hass.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD and DRUPAL-6. Thanks!

hass’s picture

hass commented

THX :-)

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Status: Fixed » Postponed (maintainer needs more info)

I understand that this is better than it was.
I just would like to know, why do we accept patches like this for D6? What is the policy on D6 string changes?

There could be many similar small fixes for D6, but the D6 is translated, and I think this patch just made more work for D6 translators and D6 users..

arhak’s picture

arhak commented

subscribing

hass’s picture

hass commented
Status: Postponed (maintainer needs more info) » Closed (fixed)

The issue has been resolved very long time ago. Marking case closed.