Page MenuHomePhabricator

Old revisions of Items should not have edit buttons
Closed, ResolvedPublic8 Estimated Story PointsBUG REPORT

Description

List of steps to reproduce

What happens?:

  • There are (non-js) edit buttons for the Terms and Sitelinks.

What should have happened instead?:

  • There should not be any edit-buttons of any kind on old revisions.

History:

ACs:

  • after the fix create integration tests that make sure this does not happen (again)/does not go uncheked
  • reenable the browser test in a stable way (see patch, T227266)

Notes:

  • There is a test but it didn't catch the problem. We should make sure that this test (or a new one) can detect regressions of this sort in the future.

Event Timeline

Change 689067 had a related patch set uploaded (by Hoo man; author: Hoo man):

[mediawiki/core@master] Article::generateContentOutput: Set revision id if loading from PC

https://gerrit.wikimedia.org/r/689067

Change 683914 had a related patch set uploaded (by Hoo man; author: Michael Große):

[mediawiki/extensions/Wikibase@master] Reenable test to check no edit links on old revision

https://gerrit.wikimedia.org/r/683914

Change 689067 merged by jenkins-bot:

[mediawiki/core@master] Article::generateContentOutput: Ensure OutputPage has rev id set

https://gerrit.wikimedia.org/r/689067

Change 683914 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Reenable test to check no edit links on old revision

https://gerrit.wikimedia.org/r/683914

Change 690044 had a related patch set uploaded (by Hoo man; author: Hoo man):

[mediawiki/extensions/Wikibase@master] Make "item old revisions do not have an edit link" more robust

https://gerrit.wikimedia.org/r/690044

Change 690806 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/Wikibase@master] Revert "Reenable test to check no edit links on old revision"

https://gerrit.wikimedia.org/r/690806

Change 690806 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Revert "Reenable test to check no edit links on old revision"

https://gerrit.wikimedia.org/r/690806

Ladsgroup subscribed.

Technically this is done but it's test renable patch got reverted. I let you decide how to proceed here.

Technically this is done but it's test renable patch got reverted. I let you decide how to proceed here.

I changed my "make the test more robust" patch (which sadly didn't get considered instead of the revert) to also revert the revert, as I think the test serves value.

Change 690044 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Re-Add "item old revisions do not have an edit link"

https://gerrit.wikimedia.org/r/690044