Last modified: 2013-10-03 14:41:09 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 T56506, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 54506 - $wgHideOpenIDLoginLink was not completely renamed to $wgOpenIDHideOpenIDLoginLink
$wgHideOpenIDLoginLink was not completely renamed to $wgOpenIDHideOpenIDLogin...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
OpenID (Other open bugs)
master
All All
: Unprioritized normal (vote)
: ---
Assigned To: T. Gries
:
Depends on:
Blocks: 9604
  Show dependency treegraph
 
Reported: 2013-09-24 16:19 UTC by Brad Jorsch
Modified: 2013-10-03 14:41 UTC (History)
1 user (show)

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


Attachments

Description Brad Jorsch 2013-09-24 16:19:17 UTC
Grepping through the code shows no use of this variable outside of its declaration. It certainly doesn't remove the login link.
Comment 1 Chris Steipp 2013-09-25 23:44:33 UTC
It looks like the name was changed to $wgHideOpenIDLoginLink. Setting $wgHideOpenIDLoginLink=true hides the link correctly. https://www.mediawiki.org/wiki/Extension:OpenID#Configuration references the correct name.
Comment 2 T. Gries 2013-09-26 05:31:54 UTC
Thanks for reporting.

All OpenID parameter names should have the form $wgOpenID...... 

It appears, that the renaming of this specific variable has been overlooked in the past. I will fix this begin of October.

Sorry for the inconvenience.
Comment 3 Brad Jorsch 2013-09-26 15:08:43 UTC
The bug is still a bug, it's not invalid. If the name should be $wgHideOpenIDLoginLink, then that should be listed in OpenID.php and $wgOpenIDHideOpenIDLoginLink shouldn't be. And if the name should be $wgOpenIDHideOpenIDLoginLink, then $wgHideOpenIDLoginLink needs to be replaced in the code and the documentation.
Comment 4 T. Gries 2013-10-03 07:59:21 UTC
fixed, tested, merged, closed
Comment 5 T. Gries 2013-10-03 08:03:35 UTC
solved with https://gerrit.wikimedia.org/r/#/c/87321/ (why is gerrit not adding this information any more ?)
Comment 6 Andre Klapper 2013-10-03 14:28:26 UTC
(In reply to comment #5)
> solved with https://gerrit.wikimedia.org/r/#/c/87321/ (why is gerrit not
> adding this information any more ?)

Because you did not follow http://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines
Comment 7 T. Gries 2013-10-03 14:41:09 UTC
http://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines :
added there "No text must follow the bugnumber. "

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


Navigation
Links