Issue Details (XML | Word | Printable)

Key: MBS-1411
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: nikki
Reporter: nikki
Votes: 0
Watchers: 0

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

Don't lowercase relationship attributes

Created: 02/Feb/11 03:55 PM   Updated: 21/Sep/11 12:57 PM   Resolved: 21/Sep/11 12:57 PM
Component/s: Data display
Affects Version/s: NGS - Release Candidate 1
Fix Version/s: Bug fixes, 2011-09-19

 Description  « Hide


at the moment we lowercase attributes when displaying them which (wrongly) turns things like "Hawaiian guitar" and "Hammond organ" into "hawaiian guitar" and "hammond organ", instead we should just store them capitalised normally and then display them without changing the capitalisation.

the capitalisation would need fixing for the current ones which are unnecessarily capitalised, but I can do that

voiceinsideyou made changes - 02/Sep/11 08:03 AM
Field Original Value New Value
Component/s Data display [ 10006 ]
Oliver Charles made changes - 14/Sep/11 12:17 PM
Status Open [ 1 ] Review Submitted [ 5 ]
Oliver Charles added a comment - 14/Sep/11 12:18 PM

I've pushed out a review that stops calling lc on relationships. In that branch, there is an SQL update file that lower cases all attributes, so if you could have a look and see which need fixing, then hand the branch back to me, that would be great.

Oliver Charles made changes - 14/Sep/11 12:18 PM
Assignee Oliver Charles [ acid2 ] nikki [ nikki ]
Oliver Charles made changes - 14/Sep/11 12:19 PM
Fix Version/s Bug fixes, 2011-09-19 [ 10101 ]
Oliver Charles made changes - 21/Sep/11 12:57 PM
Status Review Submitted [ 5 ] Closed [ 6 ]
Resolution Fixed [ 1 ]