Uploaded image for project: 'MusicBrainz Server'
  1. MusicBrainz Server
  2. MBS-5801

ISE when going back to "edit note" after having 'tabbed' back to fix an AC

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: High
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2014-02-17
    • Component/s: Release editor
    • Labels:
      None
    • Environment:
      User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:18.0) Gecko/20100101 Firefox/18.0

      Description

      I'm not entirely sure how I triggered this ISE, but based on 'INSERT INTO artist_credit_name (join_phrase,artist,name,position,artist_credit) VALUES (?,?,?,?,?)', I'm guessing it's because I originally had entered an artist credit for one of the tracks, then switched to the edit note tab, noticed the wrong AC, switched back, then switched back to edit note tab (the following may or may not be related: here I noticed the recordings changes were no longer included, so I switched to the recordings tab and "copy all changes", switched back to edit note tab,) filled out the edit note, pressed submit... and got this ISE:

      Internal Server Error
      
      Oops, something went wrong!
      
      Error:
      
      Caught exception in MusicBrainz::Server::Controller::ReleaseEditor::Add->add "Failed query:
      	'INSERT INTO artist_credit_name (join_phrase,artist,name,position,artist_credit) VALUES (?,?,?,?,?)'
      	( 961864 1382817 0 1119693)
      DBD::Pg::st execute failed: ERROR:  insert or update on table "artist_credit_name" violates foreign key constraint "artist_credit_name_fk_artist"
      DETAIL:  Key (artist)=(961864) is not present in table "artist". at lib/Sql.pm line 107.
      
       at lib/Sql.pm line 113.
      	Sql::__ANON__('DBD::Pg::st execute failed: ERROR:  insert or update on table...') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Try/Tiny.pm line 100
      	Try::Tiny::try('CODE(0x119fee00)', 'Try::Tiny::Catch=REF(0x119fec50)') called at lib/Sql.pm line 114
      	Sql::do('Sql=HASH(0xdb9cad0)', 'INSERT INTO artist_credit_name (join_phrase,artist,name,posit...', '', 961864, 1382817, 0, 1119693) called at lib/Sql.pm line 153
      	Sql::insert_row('Sql=HASH(0xdb9cad0)', 'artist_credit_name', 'HASH(0x119f7728)') called at lib/MusicBrainz/Server/Data/ArtistCredit.pm line 165
      	MusicBrainz::Server::Data::ArtistCredit::find_or_insert('MusicBrainz::Server::Data::ArtistCredit=HASH(0xa56f5e8)', 'HASH(0x119d5cf8)') called at lib/MusicBrainz/Server/Data/Tracklist.pm line 232
      	MusicBrainz::Server::Data::Tracklist::find_or_insert('MusicBrainz::Server::Data::Tracklist=HASH(0xb4af8b0)', 'ARRAY(0x119c5ea0)') called at lib/MusicBrainz/Server/Edit/Medium/Create.pm line 149
      	MusicBrainz::Server::Edit::Medium::Create::_insert_hash('MusicBrainz::Server::Edit::Medium::Create=HASH(0xf2007c8)', 'HASH(0x119cd9d0)') called at lib/MusicBrainz/Server/Edit/Generic/Create.pm line 44
      	MusicBrainz::Server::Edit::Generic::Create::insert('MusicBrainz::Server::Edit::Medium::Create=HASH(0xf2007c8)') called at lib/MusicBrainz/Server/Data/Edit.pm line 392
      	MusicBrainz::Server::Data::Edit::create('MusicBrainz::Server::Data::Edit=HASH(0xaa68608)', 'edit_type', 51, 'editor_id', 229784, 'format_id', 1, 'privileges', 1, ...) called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 1326
      	MusicBrainz::Server::Wizard::ReleaseEditor::__ANON__('edit_type', 51, 'editor_id', 229784, 'format_id', 1, 'privileges', 1, 'position', ...) called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 1353
      	MusicBrainz::Server::Wizard::ReleaseEditor::__ANON__() called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Try/Tiny.pm line 76
      	eval {...} called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Try/Tiny.pm line 67
      	Try::Tiny::try('CODE(0x10cfcb90)', 'Try::Tiny::Catch=REF(0xf200378)') called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 1362
      	MusicBrainz::Server::Wizard::ReleaseEditor::_create_edit('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)', 'CODE(0x10d17900)', 51, 229784, 'privileges', 1, 'format_id', 1, 'position', ...) called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 1327
      	MusicBrainz::Server::Wizard::ReleaseEditor::_submit_edit('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)', 51, 'Per library borrowed physical copy.\x{d}\x{a}\x{d}\x{a}Creating a new release...', 'format_id', 1, 'position', 1, 'release', 'MusicBrainz::Server::Entity::Release=HASH(0x119c60e0)', ...) called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 864
      	MusicBrainz::Server::Wizard::ReleaseEditor::__ANON__(51, 'Per library borrowed physical copy.\x{d}\x{a}\x{d}\x{a}Creating a new release...', 'format_id', 1, 'position', 1, 'release', 'MusicBrainz::Server::Entity::Release=HASH(0x119c60e0)', 'tracklist', ...) called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 1246
      	MusicBrainz::Server::Wizard::ReleaseEditor::_edit_release_track_edits('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)', 'edit_note', 'Per library borrowed physical copy.\x{d}\x{a}\x{d}\x{a}Creating a new release...', 'data', 'HASH(0xf1f0d58)', 'create_edit', 'CODE(0xf28be80)', 'previewing', '', ...) called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 963
      	MusicBrainz::Server::Wizard::ReleaseEditor::create_common_edits('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)', 'edit_note', 'Per library borrowed physical copy.\x{d}\x{a}\x{d}\x{a}Creating a new release...', 'data', 'HASH(0xf1f0d58)', 'create_edit', 'CODE(0xf28be80)', 'previewing', '', ...) called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 941
      	MusicBrainz::Server::Wizard::ReleaseEditor::create_edits('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)', 'data', 'HASH(0xf1f0d58)', 'create_edit', 'CODE(0xf28be80)', 'edit_note', 'Per library borrowed physical copy.\x{d}\x{a}\x{d}\x{a}Creating a new release...', 'previewing', '', ...) called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/x86_64-linux-gnu-thread-multi/Moose/Meta/Method/Augmented.pm line 48
      	MusicBrainz::Server::Wizard::ReleaseEditor::Add::create_edits('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)', 'data', 'HASH(0xf1f0d58)', 'create_edit', 'CODE(0xf28be80)', 'edit_note', 'Per library borrowed physical copy.\x{d}\x{a}\x{d}\x{a}Creating a new release...', 'previewing', '', ...) called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 56
      	Class::MOP::Method::Wrapped::__ANON__('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)', 'data', 'HASH(0xf1f0d58)', 'create_edit', 'CODE(0xf28be80)', 'edit_note', 'Per library borrowed physical copy.\x{d}\x{a}\x{d}\x{a}Creating a new release...', 'previewing', '', ...) called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 91
      	MusicBrainz::Server::Wizard::ReleaseEditor::Add::create_edits('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)', 'data', 'HASH(0xf1f0d58)', 'create_edit', 'CODE(0xf28be80)', 'edit_note', 'Per library borrowed physical copy.\x{d}\x{a}\x{d}\x{a}Creating a new release...', 'previewing', '', ...) called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 865
      	MusicBrainz::Server::Wizard::ReleaseEditor::prepare_edits('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)') called at lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 95
      	MusicBrainz::Server::Wizard::ReleaseEditor::run('MusicBrainz::Server::Wizard::ReleaseEditor::Add=HASH(0xe46dce8)') called at lib/MusicBrainz/Server/Controller/ReleaseEditor/Add.pm line 23
      	MusicBrainz::Server::Controller::ReleaseEditor::Add::add('MusicBrainz::Server::Controller::ReleaseEditor::Add=HASH(0xbc...', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Action.pm line 65
      	Catalyst::Action::execute('Catalyst::Action=HASH(0xc026950)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add=HASH(0xbc...', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1672
      	eval {...} called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1672
      	Catalyst::execute('MusicBrainz::Server=HASH(0xeaa6c00)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add', 'Catalyst::Action=HASH(0xc026950)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Plugin/StackTrace.pm line 94
      	Catalyst::Plugin::StackTrace::execute('MusicBrainz::Server=HASH(0xeaa6c00)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add', 'Catalyst::Action=HASH(0xc026950)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Action.pm line 60
      	Catalyst::Action::dispatch('Catalyst::Action=HASH(0xc026950)', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Controller.pm line 158
      	Catalyst::Controller::_ACTION('MusicBrainz::Server::Controller::ReleaseEditor::Add=HASH(0xbc...', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Action.pm line 65
      	Catalyst::Action::execute('Catalyst::Action=HASH(0xbecc220)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add=HASH(0xbc...', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1672
      	eval {...} called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1672
      	Catalyst::execute('MusicBrainz::Server=HASH(0xeaa6c00)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add', 'Catalyst::Action=HASH(0xbecc220)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Plugin/StackTrace.pm line 94
      	Catalyst::Plugin::StackTrace::execute('MusicBrainz::Server=HASH(0xeaa6c00)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add', 'Catalyst::Action=HASH(0xbecc220)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Action.pm line 60
      	Catalyst::Action::dispatch('Catalyst::Action=HASH(0xbecc220)', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Dispatcher.pm line 257
      	Catalyst::Dispatcher::_do_forward('Catalyst::Dispatcher=HASH(0x3dab370)', 'forward', 'MusicBrainz::Server=HASH(0xeaa6c00)', '_ACTION') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Dispatcher.pm line 237
      	Catalyst::Dispatcher::forward('Catalyst::Dispatcher=HASH(0x3dab370)', 'MusicBrainz::Server=HASH(0xeaa6c00)', '_ACTION') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 362
      	Catalyst::forward('MusicBrainz::Server=HASH(0xeaa6c00)', '_ACTION') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Controller.pm line 132
      	Catalyst::Controller::_DISPATCH('MusicBrainz::Server::Controller::ReleaseEditor::Add=HASH(0xbc...', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Action.pm line 65
      	Catalyst::Action::execute('Catalyst::Action=HASH(0xc026740)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add=HASH(0xbc...', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1672
      	eval {...} called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1672
      	Catalyst::execute('MusicBrainz::Server=HASH(0xeaa6c00)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add', 'Catalyst::Action=HASH(0xc026740)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Plugin/StackTrace.pm line 94
      	Catalyst::Plugin::StackTrace::execute('MusicBrainz::Server=HASH(0xeaa6c00)', 'MusicBrainz::Server::Controller::ReleaseEditor::Add', 'Catalyst::Action=HASH(0xc026740)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Action.pm line 60
      	Catalyst::Action::dispatch('Catalyst::Action=HASH(0xc026740)', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Dispatcher.pm line 257
      	Catalyst::Dispatcher::_do_forward('Catalyst::Dispatcher=HASH(0x3dab370)', 'forward', 'MusicBrainz::Server=HASH(0xeaa6c00)', '/release_editor/_DISPATCH') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Dispatcher.pm line 237
      	Catalyst::Dispatcher::forward('Catalyst::Dispatcher=HASH(0x3dab370)', 'MusicBrainz::Server=HASH(0xeaa6c00)', '/release_editor/_DISPATCH') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 362
      	Catalyst::forward('MusicBrainz::Server=HASH(0xeaa6c00)', '/release_editor/_DISPATCH') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Dispatcher.pm line 105
      	Catalyst::Dispatcher::dispatch('Catalyst::Dispatcher=HASH(0x3dab370)', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1618
      	Catalyst::dispatch('MusicBrainz::Server=HASH(0xeaa6c00)') called at lib/MusicBrainz/Server.pm line 304
      	MusicBrainz::Server::__ANON__() called at lib/MusicBrainz/Server.pm line 296
      	MusicBrainz::Server::with_translations('MusicBrainz::Server=HASH(0xeaa6c00)', 'CODE(0xe276138)') called at lib/MusicBrainz/Server.pm line 305
      	Class::MOP::Class:::around('CODE(0x2f4d140)', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 162
      	Class::MOP::Method::Wrapped::__ANON__('MusicBrainz::Server=HASH(0xeaa6c00)') called at lib/MusicBrainz/Server.pm line 318
      	Class::MOP::Class:::around('CODE(0x35f05d0)', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 162
      	Class::MOP::Method::Wrapped::__ANON__('MusicBrainz::Server=HASH(0xeaa6c00)') called at lib/MusicBrainz/Server.pm line 363
      	Class::MOP::Class:::around('CODE(0xc226090)', 'MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 162
      	Class::MOP::Method::Wrapped::__ANON__('MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 35
      	Class::MOP::Method::Wrapped::__ANON__('MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 91
      	MusicBrainz::Server::dispatch('MusicBrainz::Server=HASH(0xeaa6c00)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1968
      	Catalyst::__ANON__() called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Try/Tiny.pm line 76
      	eval {...} called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Try/Tiny.pm line 67
      	Try::Tiny::try('CODE(0xea37e70)', 'Try::Tiny::Catch=REF(0xe32fe60)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst.pm line 1974
      	Catalyst::handle_request('MusicBrainz::Server', 'env', 'HASH(0xe8c8d28)', 'response_cb', 'CODE(0xe8b9a90)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Plugin/AutoRestart.pm line 117
      	Catalyst::Plugin::AutoRestart::handle_request('MusicBrainz::Server', 'env', 'HASH(0xe8c8d28)', 'response_cb', 'CODE(0xe8b9a90)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Catalyst/Engine.pm line 688
      	Catalyst::Engine::__ANON__('CODE(0xe8b9a90)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Plack/Handler/FCGI.pm line 130
      	Plack::Handler::FCGI::run('Plack::Handler::FCGI=HASH(0x11caeb8)', 'CODE(0xc22bc78)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Plack/Loader.pm line 84
      	Plack::Loader::run('Plack::Loader=HASH(0x10f2290)', 'Plack::Handler::FCGI=HASH(0x11caeb8)') called at /home/beta/musicbrainz-server/admin/../local/lib/perl5/Plack/Runner.pm line 267
      	Plack::Runner::run('Plack::Runner=HASH(0xf74700)') called at /home/beta/musicbrainz-server/admin/../local/bin/plackup line 10"
      
      Stack trace:
      
          line 1361 MusicBrainz::Server::Wizard::ReleaseEditor
          line 1362 MusicBrainz::Server::Wizard::ReleaseEditor
          line 1327 MusicBrainz::Server::Wizard::ReleaseEditor
          line 864 MusicBrainz::Server::Wizard::ReleaseEditor
          line 1246 MusicBrainz::Server::Wizard::ReleaseEditor
          line 963 MusicBrainz::Server::Wizard::ReleaseEditor
          line 941 MusicBrainz::Server::Wizard::ReleaseEditor
          line 865 MusicBrainz::Server::Wizard::ReleaseEditor
          line 95 MusicBrainz::Server::Wizard::ReleaseEditor
          line 23 MusicBrainz::Server::Controller::ReleaseEditor::Add
          line 304 MusicBrainz::Server
          line 296 MusicBrainz::Server
          line 305 MusicBrainz::Server
          line 318 MusicBrainz::Server
          line 363 MusicBrainz::Server
      
      Request data:
      
      $VAR1 = {
                'query_parameters' => {},
                'body_parameters' => {
                                       'save' => 'Enter edit',
                                       'wizard_page_id' => '5',
                                       'edit_note' => 'Per library borrowed physical copy.
      
      Creating a new release to be safe; the barcode didn\'t match 100% with the already existing release with a matching barcode.',
                                       'id' => '',
                                       'as_auto_editor' => '1',
                                       'wizard_session_id' => '3'
                                     }
              };
                  
      
      Host: pino
      
      Interface language: en
      
      URL: https://beta.musicbrainz.org/release/add
      
      We're terribly sorry for this problem. Please wait a few minutes and repeat your request — the problem may go away.
      
      If the problem persists, please report a bug and include any error message that is shown above.
      
      Please be sure to include the stack trace and request data shown above!
      

      ( Also available as a Gist with HTML code: https://gist.github.com/a841f957bbf5b86c6233 )

        Attachments

          Activity

            People

            • Assignee:
              bitmap Michael Wiencek
              Reporter:
              freso Frederik “Freso” S. Olesen
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Due:
                Created:
                Updated:
                Resolved:

                Packages

                Version Package
                2014-02-17