About 10 OAuth messages point to MediaWiki pages such as https://www.mediawiki.org/wiki/Help:OAuth/Errors . These links don't really need to be translated, so they shouldn't be in the messages. They should be inserted by the software.
Version: unspecified
Severity: normal
The following 8 messages:
http://translatewiki.net/wiki/MediaWiki:Mwoauth-bad-request-missing-params/en
http://translatewiki.net/wiki/MediaWiki:Mwoauth-bad-request-invalid-action/en
http://translatewiki.net/wiki/MediaWiki:Mwoauth-bad-request-invalid-action-contact/en
http://translatewiki.net/wiki/MediaWiki:Mwoauthdatastore-request-token-not-found/en
http://translatewiki.net/wiki/MediaWiki:Mwoauthserver-bad-consumer/en
http://translatewiki.net/wiki/MediaWiki:Mwoauthserver-bad-consumer-key/en
http://translatewiki.net/wiki/MediaWiki:Mwoauthserver-insufficient-rights/en
http://translatewiki.net/wiki/MediaWiki:Mwoauthserver-invalid-user/en
Imported to GCI website: https://codein.withgoogle.com/dashboard/tasks/5644495049719808/.
Change 394859 had a related patch set uploaded (by Albert221; owner: Albert221):
[mediawiki/extensions/OAuth@master] [WIP]Un-i18n of exception messages
@Amire80 For clarity, you want the https://www.mediawiki.org/wiki/Help:OAuth/Errors turning into a message parameter, and just passed in via MediaWiki, rather than hard coded into the i18n messages?
Change 394859 abandoned by Albert221:
[WIP]Un-i18n of exception messages
Reason:
Misunderstood task
@Reedy, looking at other extensions, extension messages are hardcoded in the function call and not put in i18n. Examples;
I think this is what @Amire80 wants? Or are my missing something?
Exceptions can be hardcoded when they truly represent exceptional conditions (i.e. The enduser should never see them). In this case, these messages are user orienited so they should be translated.
However, the long url/markup is confusing to translators, so instead of things like [https://www.mediawiki.org/wiki/Help:OAuth/Errors#E009 E009] the message should instead just have $1, which would then be fed using ->rawParams( Linker::makeExternalLink(... ) ) (or Message::rawParam as this case is due to weird structure of extension).
That said, I don't understand why we link at all. I think we should just include the actual error in the message. See T181943
Okay @Bawolff, got it! @Albert221, do you understand the ticket better now and can you still work on it?
That said, I don't understand why we link at all.
I'm almost sure that this is because of the UX, look at T58492
Change 394860 had a related patch set uploaded (by Albert221; owner: Albert221):
[mediawiki/extensions/OAuth@master] Move links from i18n to software
Hardcoding URLs in the software doesn't seem nice - maybe some site wants to link to its own documentation instead if mediawiki.org. The nice way to handle that would be to have a separate message for the URL (with {{notranslate}} in the message description). Personally I don't feel it's worth the effort, the links are not a problem for translators.
Change 394860 merged by jenkins-bot:
[mediawiki/extensions/OAuth@master] Move links from i18n to software
The other three messages with EXXX error codes in them should probably also be updated for uniformity. Which is a good chance to fix the duplication of E009 - I created E011 for mwoauthserver-consumer-no-secret on the help page, that should be fixed in the message.
Change 399974 had a related patch set uploaded (by Albert221; owner: Albert221):
[mediawiki/extensions/OAuth@master] Replace MediaWiki links with parameters
Change 399974 merged by jenkins-bot:
[mediawiki/extensions/OAuth@master] Replace MediaWiki links with parameters