Last modified: 2013-03-20 17:53:48 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 T47037, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 45037 - Suppressing interlanguage links inclusion should also hide "Edit links"
Suppressing interlanguage links inclusion should also hide "Edit links"
Status: VERIFIED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataClient (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Wikidata bugs
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-02-15 09:00 UTC by Krinkle
Modified: 2013-03-20 17:53 UTC (History)
9 users (show)

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


Attachments

Description Krinkle 2013-02-15 09:00:04 UTC
This edit on en.wikipedia concerns me:

https://en.wikipedia.org/w/index.php?title=MediaWiki:Common.css&curid=2213345&diff=538111527&oldid=535671838

body.page-Main_Page .wbc-editpage { 
    display: none;
}

Something like that should never be necessary.

If it isn't really turned off, then the link should be visible.

If it is turned off (as the author of that revision claims), then the link should be invisible.

Is there a way to turn off interlanguage inclusion on a page? And if so, does it also hide the link? And if so, why is that functionality not used for the Main Page?
Comment 1 Aude 2013-02-15 09:58:18 UTC
It is a bug indeed and that just temporarily hides the link in the most obvious spot

we will fix in the right way ASAP
Comment 3 jeblad 2013-03-13 00:48:49 UTC
Something strange happened while doing reviews. I also did some stuff in another closed terminal.
This is the history from the terminal where I checked out the latest stuff. I run tests in another terminal.

 1342  git branch | fgrep '45150'
 1343  git checkout master
 1344  git pull
 1345  git checkout maintenance-script-for-summary-45150
 1346  git status
 1347  git checkout master # at this point I left the summary-stuff in its branch
 1348  git review -d 51179 # (bug 45037) show edit link only if we have repo links
 1349  git checkout master
 1350  git pull
 1351  git review -d 53381 # (Bug 43994) New Special:UnconnectedPages
 1352  git rebase origin/master
 1353  git status
 1354  git commit --amend
 1355  git review
 1356  git pull
 1357  git checkout master
 1358  git pull
 1359  git review -d 53395 # Moved ClaimSaver and ExceptionWithCode to their own files
 1360  git checkout master
 1361  git pull
 1362  git review -d 53414 # Show/hide the wikibase watchlist entries server side
 1363  git rebase origin/master
 1364  git checkout master # checkout of master to find the errors
 1365  git pull
 1366  history

After 1362 I found that tests failed to run, did a checkout of master and found a lot of errors. 1359 should not introduce errors in the client. At 1348 I did not find any errors. Perhaps some rebase and/or merge has triggered something.

This is the testrun that found the errors
john@loke[master] ~/Workspace/core/tests/phpunit> env MW_INSTALL_PATH="/var/www/client" php phpunit.php --verbose --group WikibaseClient --conf /var/www/client/LocalSettings.php

john@loke[master] ~/Workspace/Wikibase> git reset --hard HEAD^
HEAD is now at 0905d95 Merge "(bug 45037) show edit link only if we have repo links"
john@loke[master] ~/Workspace/Wikibase> git reset --hard HEAD^^
HEAD is now at 7d3cb0f (Bug 41882) Followup for restrictions for special pages

Going one back did not remove the errors, going two back did remove them. So it seems like the merge at 0905d95 "(bug 45037) show edit link only if we have repo links" triggered the errors. That is line 1348 in my history and I did not do a local rebase before the merge.

I'm going to revert the changeset and then we must try to figure out what went wrong.
Comment 4 jeblad 2013-03-13 00:58:36 UTC
I guess I was to focused on testing page_props in the UI and by MySQL Workbench, and simply forgot to run the tests on this specific review.
Comment 5 Anja Jentzsch 2013-03-15 15:45:25 UTC
Original changeset got reverted, new changeset: https://gerrit.wikimedia.org/r/#/c/53543/
Comment 6 Anja Jentzsch 2013-03-20 14:25:35 UTC
Change I97cc9378: (bug 45037) show edit link only if we have repo links

Changeset is merged.
Comment 7 abraham.taherivand 2013-03-20 17:53:48 UTC
Verified in Wikidata demo sprint 35-2

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


Navigation
Links