Project:Localization client
Version:7.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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

Comments

#1

Status:active» needs review
AttachmentSize
l10n_client_748664.patch 1.62 KB

#2

Status:needs review» reviewed & tested by the community

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

AttachmentSize
748664-l10n-save.patch 2.31 KB

#3

Now it works again

AttachmentSize
748664-l10n-save.patch 3.05 KB

#4

Status:reviewed & tested by the community» needs review

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

AttachmentSize
748664-l10n-save.patch 3.05 KB

#5

Status:needs review» reviewed & tested by the community

#6

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.

#7

@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

#8

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.

#9

Status:reviewed & tested by the community» fixed

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!

AttachmentSize
hook_user_presave.patch 2.9 KB
localekeys.patch 1.08 KB

#10

Status:fixed» closed (fixed)

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