The Key will be not saved in the $user Object without the hook http://api.drupal.org/api/function/hook_user_presave/7.

CommentFileSizeAuthor
#9 hook_user_presave.patch2.9 KBgábor hojtsy
#9 localekeys.patch1.08 KBgábor hojtsy
#4 748664-l10n-save.patch3.05 KBandypost
#3 748664-l10n-save.patch3.05 KBandypost
#2 748664-l10n-save.patch2.31 KBandypost
#1 l10n_client_748664.patch1.62 KBtobiasb

Comments

tobiasb’s picture

tobiasb
Primary language German
Location Berlin
commented
Status: Active » Needs review
StatusFileSize
new l10n_client_748664.patch1.62 KB
andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 748664-l10n-save.patch2.31 KB

A bit improved patch after #763048: Remove drupal_unpack($user) due to namespacing collisions

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new 748664-l10n-save.patch3.05 KB

Now it works again

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 748664-l10n-save.patch3.05 KB

Same patch with trim() for key as proposed in #323917-10: Support for the 2.0 remote server API

tobiasb’s picture

tobiasb
Primary language German
Location Berlin
commented
Status: Needs review » Reviewed & tested by the community
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

The trimming functionality was just added in #http://drupal.org/node/323917#comment-2857042 to D6 as well, so the functionality is in sync with this patch, no need to backport that.

andypost’s picture

andypost
he/him
Primary language Russian
commented

@Gábor Hojtsy: I've just included trim() because this patch fixes saving functionality with hook_user_presave() and sync D7 with #323917-10: Support for the 2.0 remote server API

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

Ha, BTW _locale_import_one_string_db() also changed API in D6, so I've just committed that one line to the D6 version already. Fixes some notices.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed
StatusFileSize
new localekeys.patch1.08 KB
new hook_user_presave.patch2.9 KB

Looks good, works good. Committed to D7. Also noticed that the 'deletes' key way missing from the _locale_import_one_string_db() arrays in both D7 and D6. So also committed a small update to D6. Thanks for your great help!

Status: Fixed » Closed (fixed)

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