role names shouldn't be gettexted in schema.php

[tenpura]

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

[markjaquith]

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.

[tenpura]

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.

[ryan]

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.

[nbachiyski]

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?

[ryan]

Yes, let's document. Plugins that deal with roles will need to account for the context .

[tenpura]

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') {

?

[ryan]

(In [6916]) Don't save translated role names to the DB. Instead, translate them on the fly. fixes #3442 #5537

[ryan]

I defaulted the domain args to avoid the missing arg warnings.