Last modified: 2013-11-21 23:56:25 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T57704, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 55704 - OAuth MediaWiki extension's "Privacy policy" link is wrong on Wikimedia wikis
OAuth MediaWiki extension's "Privacy policy" link is wrong on Wikimedia wikis
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
OAuth (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 55679
  Show dependency treegraph
 
Reported: 2013-10-14 17:30 UTC by MZMcBride
Modified: 2013-11-21 23:56 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description MZMcBride 2013-10-14 17:30:09 UTC
[[mw:Extension:OAuth]] prompts a user to authorize an app at a URL similar to <https://www.mediawiki.org/w/index.php?title=Special:MWOAuth/authorize&oauth_token=0cd1232bf697849cedc5d95f77d61144&oauth_consumer_key=f26337df9e626952ba65123be86f8e62>.

This prompt looks like this:

---
Hi MZMcBride,

Gerrit Patch Uploader would like to have basic access on your behalf on www.mediawiki.org.

Privacy Policy                                 [Cancel] [Allow]
---

"Privacy Policy" should probably be in sentence case, but more worryingly, it currently links to [[mw:Project:Privacy policy]], which doesn't exist. It should link to [[wmf:Privacy policy]] on Wikimedia wikis.

This may be an issue for the WikimediaMessages extension. I'm not sure.
Comment 1 Brad Jorsch 2013-10-15 16:41:48 UTC
(In reply to comment #0)
> This may be an issue for the WikimediaMessages extension. I'm not sure.

That is probably the case. We don't want non-WMF wikis linking to the WMF privacy policy by default.

I'm not sure how overriding a message from one extension in another actually works, though.
Comment 2 Chris Steipp 2013-10-15 19:11:57 UTC
This seems to be fixed. The link goes to https://wikimediafoundation.org/wiki/Privacy_policy on mw.o. Is the wrong link showing up anywhere else?
Comment 3 Aaron Schulz 2013-10-15 19:14:02 UTC
(In reply to comment #2)
> This seems to be fixed. The link goes to
> https://wikimediafoundation.org/wiki/Privacy_policy on mw.o. Is the wrong
> link
> showing up anywhere else?

Is that a local override of the MediaWiki: page?
Comment 4 Brad Jorsch 2013-10-15 19:16:20 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > This seems to be fixed. The link goes to
> > https://wikimediafoundation.org/wiki/Privacy_policy on mw.o. Is the wrong
> > link
> > showing up anywhere else?
> 
> Is that a local override of the MediaWiki: page?

Yes, it is. [[mw:MediaWiki:Mwoauth-form-privacypolicy-link]]. I don't think we want to do that on every wiki and for every language.
Comment 5 MZMcBride 2013-10-15 23:20:49 UTC
(In reply to comment #4)
> Yes, it is. [[mw:MediaWiki:Mwoauth-form-privacypolicy-link]]. I don't think
> we want to do that on every wiki and for every language.

Indeed. We need a sane default here.
Comment 6 Kunal Mehta (Legoktm) 2013-10-15 23:24:21 UTC
> I'm not sure how overriding a message from one extension in another actually
> works, though.

The copyright warning for the edit page uses a hook, see https://github.com/wikimedia/mediawiki-extensions-WikimediaMessages/blob/master/WikimediaMessages.php#L38 for how it works.
Comment 7 Aaron Schulz 2013-10-21 17:21:25 UTC
It would be nice if there was a generic hook for messages that could be overrided (the key name would be passed in).
Comment 8 Gerrit Notification Bot 2013-11-08 20:23:09 UTC
Change 94416 had a related patch set uploaded by CSteipp:
Add hook to override message

https://gerrit.wikimedia.org/r/94416
Comment 9 Gerrit Notification Bot 2013-11-08 21:58:07 UTC
Change 94416 merged by jenkins-bot:
Add hook to override message

https://gerrit.wikimedia.org/r/94416
Comment 10 Gerrit Notification Bot 2013-11-21 22:03:28 UTC
Change 96894 had a related patch set uploaded by CSteipp:
Add site-wide privacy link

https://gerrit.wikimedia.org/r/96894
Comment 11 Gerrit Notification Bot 2013-11-21 22:04:41 UTC
Change 96894 merged by jenkins-bot:
Add site-wide privacy link

https://gerrit.wikimedia.org/r/96894
Comment 12 MZMcBride 2013-11-21 23:56:25 UTC
I believe this bug is now fixed. Thanks, all!

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links