Issue Details (XML | Word | Printable)

Key: MBS-5428
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Unassigned
Reporter: Alex Mauer
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
MusicBrainz Server

Modbot: falsely failed recording edit

Created: 07/Oct/12 03:14 PM   Updated: 26/Jan/13 08:04 PM   Resolved: 26/Jan/13 08:04 PM
Component/s: Edit system
Affects Version/s: None
Fix Version/s: None

Issue Links:
Relates
 


 Description  « Hide

http://musicbrainz.org/edit/19173440

Modbot says "Data has changed since this edit was created, and now conflicts with changes made in this edit."

This is not correct.



Sort Order: Ascending order - Click to sort in descending order
nikki added a comment - 07/Oct/12 03:55 PM

http://beta.musicbrainz.org/edit/19194010 was entered after your edit and appears to have made a change to the artist, although I can't tell what.


Alex Mauer added a comment - 07/Oct/12 03:57 PM

I'm pretty sure it just changed the recording title, not the artist (and therefore shouldn't conflict)


nikki added a comment - 07/Oct/12 04:01 PM

If it had only changed the recording title, the artist section wouldn't be shown. Compare it with http://beta.musicbrainz.org/edit/19245554 for example.


Ian McEwen added a comment - 07/Oct/12 07:41 PM

Join phrase – the initial edit changed a join phrase of "" to " ".


Ulrich Klauer added a comment - 07/Oct/12 09:00 PM

So, should this be closed as invalid?


Ian McEwen added a comment - 07/Oct/12 09:20 PM

I'm not sure. I'd be curious to know why it is that the join phrase was getting changed, and if it's still possible for it to change that way. I think we have a patch in the pipeline that would make this impossible (having whitespace in the final join phrase), at the database level. However, it still seems like the conflicting edit shouldn't be possible (rather than it just being impossible to apply the edit, like the patch I mention does). So I guess more investigation is needed


Ian McEwen added a comment - 26/Jan/13 08:04 PM

This is no longer possible due to the controlled_for_whitespace checks.