#3442 closed defect (bug) (fixed)
role names shouldn't be gettexted in schema.php
Reported by: | tenpura | Owned by: | ryan |
---|---|---|---|
Milestone: | 2.5 | Priority: | normal |
Severity: | normal | Version: | 2.5 |
Component: | Administration | Keywords: | i18n needs-patch |
Focuses: | Cc: |
Description
Description:
Currently role names are stored into database in localized character encodings. If somehow those get garbled, unserialize() will fail to retrieve the data (because of the string length mismatch) and then a login failure occurs.
Suggested Solution:
Instead of using __() for the storing data in upgrade-schema.php, use __() when we display the data. (atleast for the role names, it seems to be a common problem.)
References:
http://www.google.com/search?hl=en&q=%22capabilities.php+on+line+19%22
Attachments (1)
Change History (14)
#5
@
16 years ago
- Milestone set to 2.5
- Resolution invalid deleted
- Status changed from closed to reopened
- Summary changed from Garbled role names cause login failure to role names shouldn't be gettexted in schema.php
- Version changed from 2.0.5 to 2.5
The description is a bit old but this is still a ongoing problem, so please don't close.
I said "garbled" but this actually could be anything (e.g. database encoding conversion) that causes string length mismatch of serialized data.
#8
@
16 years ago
Patch saves the untranslated strings to the roles array stored in the DB and translates them on the fly later. Dummy _c() calls are made to get the strings in the message catalog. translate_with_context() is used for on-the-fly translation since strings are not extracted from it when creating the message catalog.
#9
@
16 years ago
Very nice! It preserves backward compatibility and the common roles' translation will change with locale change.
There is a samll problem: artificially added roles will be left as-is (good thing), except for stripping the text after the last |. Probably we should document having | in role names?
#10
@
16 years ago
Yes, let's document. Plugins that deal with roles will need to account for the context .
#11
@
16 years ago
It's cool that we can change roles' translation anytime we want. Thank you!
BTW, I get Warning: Missing argument 2 for translate_with_context().
function translate_with_context($text, $domain) {
should be like
function translate_with_context($text, $domain = 'default') {
?
Sounds like a good solution, although it wouldn't fix blogs that are already broken.
Issues:
We'd need to change the role names on upgrade, and thanks to our muddled roles/capabilities system, that's not a simple SQL query... it'd have to be done in PHP.
Blogs sharing centralized user/role tables would have to be upgraded in unison (they probably would be anyway).
Maybe we can get the role/cap system changed in 2.2 to something simpler so we don't have to keep limping along with the one we have.