Locale module for D7 should not need the locale_inc_callback function since the registry already handles this.

CommentFileSizeAuthor
#4 locale_283201.patch5.44 KBdrewish
#1 locale-registry-283201-1.patch5.2 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Title: use code registry properly in locale page callbacks » use code registry properly for locale page callbacks
Status: Active » Needs review
StatusFileSize
new locale-registry-283201-1.patch5.2 KB

simple patch - tested, but not thoroughly.

pwolanin’s picture

pwolanin commented

note - this critical issue needs to be fixed before this patch is usable: http://drupal.org/node/280628

meba’s picture

meba commented

Applied (without errors) and clicked around locale interface pages, import form, etc. Works for me.

drewish’s picture

drewish commented
StatusFileSize
new locale_283201.patch5.44 KB

in comment #2 pwolanin mentioned #280628: locale.module missing confirm form which has been committed.

i re-rolled to clean up some fuzz. ran the local and translation test and they all passed.

drewish’s picture

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

meant to bump the status.

dries’s picture

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

I've updated/corrected the code comment of locale_menu() and committed this to CVS HEAD. Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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