Issue Details (XML | Word | Printable)

Key: MBS-3208
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Kuno Woudt
Reporter: voiceinsideyou
Votes: 1
Watchers: 0
Operations

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

Trim leading/trailing whitespace in release editor prior to Add Missing Entities check

Created: 10/Aug/11 03:06 AM   Updated: 09/Jul/12 07:27 AM   Resolved: 09/Jul/12 07:27 AM
Component/s: Release editor
Affects Version/s: None
Fix Version/s: Bug fixes, 2011-08-15, Bug fixes, 2012-04-10, Bug fixes, 2012-07-09

Issue Links:
Relates
 
Resolution
 


 Description  « Hide

As raised in MBS-2681, if you have two artist entries (for a VA release, normally) that differ only by leading or trailing whitespace it will ask you to add missing entities twice on the AME tab. e.g.

1. SomeTrack by "SomeArtist"
2. SomeOtherTrack by "SomeArtist "

Because of the whitespace, it's not apparent why it's asking you to add-or-select the same artist twice.

I believe such names get auto-trimmed by the system when entering edits, so doing this prior to the AME step shouldn't be an issue, I don't think?



Sort Order: Ascending order - Click to sort in descending order
Kuno Woudt added a comment - 11/Aug/11 08:06 PM

This issue is fixed in my changes for MBS-2817, which is currently in code review.


nikki added a comment - 23/Jan/12 02:00 PM

Broken again. All fields in the release editor are not stripping unnecessary spaces. See http://test.musicbrainz.org/release/5f1dd945-603d-468f-8352-378ebd306890 for example.


Kuno Woudt added a comment - 10/Apr/12 03:10 PM

commit 69d00ba46fafc0f3bac3c6f3f6347c538634a5c1


nikki added a comment - 22/May/12 06:31 AM

Broken again, see the disambiguation comment on http://musicbrainz.org/label/b21cb401-8658-4dc7-80df-86b01ac26257


Kuno Woudt added a comment - 27/Jun/12 12:18 PM

commit a850948dc71abe3e289e001a192b3cbe43a16382