Issue Details (XML | Word | Printable)

Key: MBS-5187
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Oliver Charles
Reporter: John Russell
Votes: 1
Watchers: 1
Operations

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

Cannot add release

Created: 20/Aug/12 09:34 PM   Updated: 30/Aug/12 02:44 PM   Resolved: 30/Aug/12 02:44 PM
Component/s: None
Affects Version/s: None
Fix Version/s: Bug fixes, 2012-09-03

Environment: Mac OS X 10.8, Chrome
Issue Links:
Duplicate
 
Relates
 


 Description  « Hide

Error:

Caught exception in MusicBrainz::Server::Controller::ReleaseEditor::Add->add "Wide character in subroutine entry at lib/MusicBrainz/Server/Wizard/ReleaseEditor/Add.pm line 83."
Stack trace:
line 85 MusicBrainz::Server::Wizard::ReleaseEditor::Add
line 132 MusicBrainz::Server::Wizard::ReleaseEditor::Add
line 88 MusicBrainz::Server::Wizard
line 401 MusicBrainz::Server::Wizard
line 132 MusicBrainz::Server::Wizard
line 92 MusicBrainz::Server::Wizard::ReleaseEditor
line 23 MusicBrainz::Server::Controller::ReleaseEditor::Add
line 322 MusicBrainz::Server
line 322 MusicBrainz::Server
line 322 MusicBrainz::Server
line 286 MusicBrainz::Server
Request data:

$VAR1 = {
'query_parameters' => {},
'body_parameters' => {
'artist_credit.names.0.name' => 'Various Artists',
'packaging_id' => '',
'next' => "Next \x{bb}",
'date.month' => '',
'release_group_id' => '',
'labels.0.catalog_number' => '',
'wizard_page_id' => '0',
'release_group.name' => '',
'artist_credit.names.0.artist.id' => '',
'labels.0.deleted' => '0',
'country_id' => '',
'primary_type_id' => '',
'labels.0.name' => '',
'date.year' => '',
'id' => '',
'script_id' => '',
'status_id' => '',
'date.day' => '',
'name' => 'The What CD Volume 6: The B-Sides',
'gc-keepuppercase' => 'on',
'barcode' => '',
'artist_credit.names.0.join_phrase' => '',
'labels.0.label_id' => '',
'wizard_session_id' => '33720',
'language_id' => '',
'comment' => '',
'gc-mode' => 'English',
'artist_credit.names.0.artist.name' => 'Various Artists',
'annotation' => ''
}
};

URL: http://musicbrainz.org/release/add



Sort Order: Ascending order - Click to sort in descending order
Oliver Charles added a comment - 28/Aug/12 04:57 PM

Seems very similar to MBS-5184


Ian McEwen added a comment - 29/Aug/12 04:17 AM - edited

The commit adding the offending line is 57694ff7ee6b3f68dbf8f6d0cc0647e5002e9d45 – MBS-2345 ("When you seed release editor with tracks it should bypass the Duplicates Page"), which indeed shipped with the last release. It appears we're using decode_json in some sort of incorrect fashion and need to be encoding it to utf8: http://cpanratings.perl.org/dist/JSON has some comments in this vein.

(edit: let me note that I'm not actually working on this at present though – don't take this comment as an indication fixing the bug is getting looked at (looking at you, other devs ))


Oliver Charles added a comment - 29/Aug/12 10:10 AM

This is in review at http://codereview.musicbrainz.org/r/2217/. I'd like to get this in for the next release on monday though if possible, because this seems quite urgent.


Kuno Woudt added a comment - 30/Aug/12 02:44 PM

commit 39498282caa92752d12cc4cb466f42694fe99385