Page MenuHomePhabricator

Editing last diff causes re-edit of that diff instead of creating a new edit
Closed, ResolvedPublic

Description

Author: eep

Description:
The last diff(erence) "edit" link does not have an "&oldid" so, one would think, editing it would be like editing the page normally. Apparently not so, as I've been finding out lately. If it's edited, the result is the combination of the previous diff and the new edit, making it seem like the latest edit did both edits. I don't know if it's a bug with MW 1.11a or what, but I didn't used to have this problem in Wikipedia before a few weeks ago (when I was banned--ahem). Anyway, I keep noticing it on MediaWiki.org and Meta and am getting sick of it so here's the bug report.


Version: 1.11.x
Severity: normal

Details

Reference
bz10916

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:52 PM
bzimport set Reference to bz10916.
bzimport added a subscriber: Unknown Object (MLST).

robchur wrote:

What "last difference" link, where? Please give precise steps to reproduce, explain what the actual behaviour is, what the expected behaviour is, and state how what is happening is wrong.

eep wrote:

  1. Go to the history of any page.
  2. Click the "last" (difference) link (to compare the last and previous edits).
  3. Hover the mouse over the page's main "edit" tab link and note the URL has "&oldid=#" in it, denoting that link would edit an older version and not the current version.

3a. Hover the mouse over the left side's "edit" link and note the URL has an "&oldid=#" of 1 less than the main page's edit link.
3b. Hover the mouse over the right side's "edit" link and note the URL does not have an "&oldid=#" at all, implying it is the latest version and, if clicked, would edit the latest version.

  1. Click that right side's "edit" link.
  2. Edit the page.
  3. Save it.
  4. Note how your edit merged with the previous edit to become the last edit.
  • Actual result: see above
  • Expected result: a new edit should be just that--a new edit and not merge with the previous edit
  • It's wrong because that's not how editing is supposed to work and changed in MW 1.11.x sometime

robchur wrote:

Behaviour appears to be consistent; "left side" edit link brings up the edit page with the text from that old version, right side brings up current edit page, as it should, and the main "edit this page" link brings up the current edit page too.

All working as it should be.

eep wrote:

Um, have you even tried it, Rob? Edit the right side and save it; your edit will merge with the previous edit... I just edited http://www.mediawiki.org/w/index.php?title=User:Robchurch but, as you can plainly see, your last edit (you added "* /Tips" to line 34) merged with mine ("** Fix diff editing..." on line 35). Damn MediaWiki.org is slow today... Well that's weird...it didn't do it that time. That is so weird because even http://www.mediawiki.org/w/index.php?title=Extension:Mp3&diff=next&oldid=122219 no longer shows the combined previous edit http://www.mediawiki.org/w/index.php?title=Extension%3AMp3&diff=122219&oldid=115004 anymore. Maybe it's a caching issue? I swear it's been doing this for me for a while now... Ah well...I'll update this if it happens to me again and, hopefully, it'll stay in the page history long enough for someone else to see it...

robchur wrote:

Of course I tried it. I can't see the incorrect behaviour; all the edit links on a standard diff page do what's expected of them.

eep wrote:

OK, it just happened with http://www.mediawiki.org/w/index.php?title=Project:Extension_requests&diff=0&oldid=123405 which I only added the last paragraph but the edit implies I added everything else too.

robchur wrote:

Aha! This isn't a problem with the editor, or the links, it's a problem with the difference engine; I purged the diff provided, and it showed up less changes.

*** This bug has been marked as a duplicate of bug 9112 ***