Last modified: 2014-02-14 19:52:05 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 T62741, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 60741 - Remove Easter egg from conf/highlight.php
Remove Easter egg from conf/highlight.php
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
wmf-deployment
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-02 18:55 UTC by db [inactive,noenotif]
Modified: 2014-02-14 19:52 UTC (History)
7 users (show)

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


Attachments

Description db [inactive,noenotif] 2014-02-02 18:55:43 UTC
[1] shows a $secretSitePassword, looking at the (public) code at [2] gives a comment "Easter egg". In my opinion this is very confusing, because some people who see this, can think about a secret problem, so please remove the easter egg and give better explain what the user should do or has done wrong when viewing the highlight.php without a param. Thanks.

[1] https://noc.wikimedia.org/conf/highlight.php
[2] https://git.wikimedia.org/blob/operations%2Fmediawiki-config/c815099256d9669aded81b6efca721e7e57c9965/docroot%2Fnoc%2Fconf%2Fhighlight.php#L43
Comment 1 Gerrit Notification Bot 2014-02-02 19:13:17 UTC
Change 110874 had a related patch set uploaded by Se4598:
Replace easter egg by a more explaining message

https://gerrit.wikimedia.org/r/110874
Comment 2 MZMcBride 2014-02-02 21:22:37 UTC
I don't think there's consensus to remove this Easter egg for now. Though making the fake password longer would be nice. The current password looks a bit too short to be real. :-)
Comment 3 se4598 2014-02-02 21:25:37 UTC
no consensus :)

	Odder		22:20
I for one think the patch is valid.

As much as I like easter eggs, this particular one leads to people making requests such as linked to by Se4598, which in itself is a waste of our volunteers' precious time.

	Matanya		22:24
i'm with odder here.
Comment 4 Gerrit Notification Bot 2014-02-02 21:36:17 UTC
Change 110874 merged by Ori.livneh:
Replace easter egg by a more explaining message

https://gerrit.wikimedia.org/r/110874
Comment 5 se4598 2014-02-02 21:46:21 UTC
Vote was 3 against 2 for the easter egg, but then suddenly Ori:
"Oh no, someone hacked my Gerrit account and merged this change"

-- end of story --
Comment 6 Gerrit Notification Bot 2014-02-03 09:12:33 UTC
Change 110926 had a related patch set uploaded by Hashar:
Replace easter egg by a more explaining message

https://gerrit.wikimedia.org/r/110926
Comment 7 Tomasz W. Kozlowski 2014-02-14 19:50:17 UTC
This was merged & deployed by Tim ages ago.

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


Navigation
Links