Issue Details (XML | Word | Printable)

Key: MBS-3512
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Johannes Weißl
Reporter: patate12
Votes: 2
Watchers: 2
Operations

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

Release editor added a track duration out of the blue

Created: 23/Sep/11 04:10 PM   Updated: 15/Dec/11 01:51 PM   Resolved: 15/Dec/11 01:51 PM
Component/s: Release editor
Affects Version/s: None
Fix Version/s: Bug fixes, 2011-12-22

Issue Links:
Duplicate
 


 Description  « Hide

http://musicbrainz.org/edit/15252404

I just hit guess case and fixed a '.
Then in the edit review I see this duratoin added out of the blue.
back in the tracklist tab, still no track duration visible, but it's in the edit anyway... (._.?)



Sort Order: Ascending order - Click to sort in descending order
Luca Salini added a comment - 23/Sep/11 04:14 PM - edited

I can confirm that this bug is triggered when the tracklist has no length for a track, but the linked recording has one.
I'll move my previous comment to a separate ticket.


Johannes Weißl added a comment - 23/Sep/11 06:10 PM

I noticed this, too... I thought the track times come from the recordings, which shouldn't be. Recording times should not automatically get transfered to the tracklist.


voiceinsideyou added a comment - 07/Oct/11 05:12 AM

This may be related to MBS-3607 - but not really sure


Kuno Woudt added a comment - 08/Nov/11 07:16 PM

This was explicitly requested in MBS-3355, I have no strong opinion either way.. but I cannot act on conflicting tickets. Please discuss this in the weekly developer meeting or on the mb-users mailinglist. Depending on the outcome of such discussion, either close this ticket or assign it back to me when there is enough consensus on NOT automatically setting track times.


Johannes Weißl added a comment - 08/Nov/11 09:27 PM

I will start a thread on the musicbrainz-users list and also attend the next dev meeting. I'm sorry that I didn't notice MBS-3355 before it got fixed.

I have multiple albums in my TODO collection that need to be fixed (errors in tracklists), but I can't do it because that would add track timings that I haven't verified.

In my opinion durations should only automatically float in this direction:
tracklists -> recordings
never the other way round.

It is also not at all a bad thing to have vinyl singles without track timings set. I would consider it quite normal for vinyl releases.


Nicolás Tamargo added a comment - 08/Nov/11 11:16 PM

Kuno, I actually don't think this conflicts with what Paul wanted. We could have a way of sourcing the track times from recordings; it just shouldn't do it without asking and as part of a different edit by an editor who isn't trying to do that.


patate12 added a comment - 14/Nov/11 04:08 PM

Hi hrglgrmpf, I re-assign this ticket to you, just in case you know how to manage (I don't).
If you're as little used to that sort of things as me, just re-open the ticket unassigned, maybe.


Johannes Weißl added a comment - 14/Nov/11 09:11 PM

It was decided in the dev meeting to add a checkbox. I see how difficult it is to create a patch, if it turns out too much work for me, I'll assign it to warp!


Johannes Weißl added a comment - 14/Nov/11 10:47 PM

A quickly wrote a patch, see http://codereview.musicbrainz.org/r/1617/