Last modified: 2014-03-18 13:20:00 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 T57960, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 55960 - wbmergeitems api modules did not correctly merge Q7 and Q194
wbmergeitems api modules did not correctly merge Q7 and Q194
Status: VERIFIED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
master
All All
: High major (vote)
: ---
Assigned To: Wikidata bugs
u=dev c=backend p=8
:
: 60390 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-21 08:28 UTC by Addshore
Modified: 2014-03-18 13:20 UTC (History)
7 users (show)

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


Attachments

Description Addshore 2013-10-21 08:28:25 UTC
https://test.wikidata.org/w/index.php?title=Q7&action=history
https://test.wikidata.org/w/index.php?title=Q194&action=history

See https://www.wikidata.org/wiki/MediaWiki_talk:Gadget-Merge.js#New_merge_API

It seems the first of the two edits was made (removing the data from the first item). but the second edit (adding the data to the second item) never went through.
Comment 1 Marius Hoch 2013-12-28 09:04:25 UTC
Problem is that we don't undo our changes to the first item (merge from) if the second item (merge target) can't be saved. There's also a TODO about that in the code "//todo if the second result is not a success we should probably undo the first change".

Solving that in a clean manner is hard, I guess... but this is actually causing trouble in production.
Comment 2 Gerrit Notification Bot 2014-01-06 11:49:45 UTC
Change 105666 had a related patch set uploaded by Addshore:
Undo any edits if a Merge Fails!

https://gerrit.wikimedia.org/r/105666
Comment 3 Gerrit Notification Bot 2014-01-06 12:16:49 UTC
Change 105666 abandoned by Addshore:
Undo any edits if a Merge Fails!

Reason:
per comment

https://gerrit.wikimedia.org/r/105666
Comment 4 Addshore 2014-01-06 12:17:11 UTC
After further discussion with Hoo it turns out the failed merges are due to the database having bad data in it already, i.e. two items that have the same label and description.
Because of this we can not simply undo the edit as wikibase will not allow this edit to happen.
Instead we should check that both edits will be possible before merging, this might require some further refactoring of content handler classes
Comment 5 Addshore 2014-01-24 14:48:50 UTC
*** Bug 60390 has been marked as a duplicate of this bug. ***
Comment 6 Gerrit Notification Bot 2014-01-30 15:47:26 UTC
Change 110380 had a related patch set uploaded by Addshore:
Check for Label/Description conflicts in ChangeOpsMerge

https://gerrit.wikimedia.org/r/110380
Comment 7 Gerrit Notification Bot 2014-03-12 11:36:03 UTC
Change 110380 merged by jenkins-bot:
Check for conflicts in ChangeOpsMerge preSave

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

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


Navigation
Links