Issue Details (XML | Word | Printable)

Key: MBS-4634
Type: Bug Bug
Status: Closed Closed
Resolution: Duplicate
Priority: Normal Normal
Assignee: Unassigned
Reporter: nikki
Votes: 0
Watchers: 0
Operations

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

Internal server error when uploading multiple covers in the same position

Created: 01/May/12 01:58 PM   Updated: 04/May/12 10:08 AM   Resolved: 04/May/12 10:08 AM
Component/s: None
Affects Version/s: None
Fix Version/s: None

Issue Links:
Duplicate
 


 Description  « Hide

Open multiple "add cover art" tabs. Select an image in each. Submit, one by one. Get error:

Caught exception in MusicBrainz::Server::Controller::Release->add_cover_art "The edit could not be created.
POST: $VAR1 = {
          'add-cover-art.id' => '835050629',
          'add-cover-art.position' => '1',
          'add-cover-art.comment' => 'right inside',
          'add-cover-art.edit_note' => '',
          'add-cover-art.as_auto_editor' => '1'
        };

Exception:$VAR1 = 'Failed query:
	\'INSERT INTO cover_art_archive.cover_art (release, edit, ordering, id, comment)
         VALUES (?, ?, ?, ?, ?)\'
	(124271 17470567 1 835050629 right inside)
DBD::Pg::st execute failed: ERROR:  duplicate key value violates unique constraint "cover_art_pkey"
DETAIL:  Key (id)=(835050629) already exists. at lib/Sql.pm line 103.

 at lib/Sql.pm line 109
	Sql::__ANON__(\'DBD::Pg::st execute failed: ERROR:  duplicate key value viola...\') called at local/lib/perl5/Try/Tiny.pm line 100
	Try::Tiny::try(\'CODE(0xf9f0708)\', \'Try::Tiny::Catch=REF(0xf9f6e28)\') called at lib/Sql.pm line 110
	Sql::do(\'Sql=HASH(0xc8ab838)\', \'INSERT INTO cover_art_archive.cover_art (release, edit, order...\', 124271, 17470567, 1, 835050629, \'right inside\') called at lib/MusicBrainz/Server/Data/CoverArtArchive.pm line 90
	MusicBrainz::Server::Data::CoverArtArchive::insert_cover_art(\'MusicBrainz::Server::Data::CoverArtArchive=HASH(0xa951af8)\', 124271, 17470567, 835050629, 1, \'ARRAY(0xd89d630)\', \'right inside\') called at lib/MusicBrainz/Server/Edit/Release/AddCoverArt.pm line 75
	MusicBrainz::Server::Edit::Release::AddCoverArt::post_insert(\'MusicBrainz::Server::Edit::Release::AddCoverArt=HASH(0xf9a4fb0)\') called at lib/MusicBrainz/Server/Data/Edit.pm line 408
	MusicBrainz::Server::Data::Edit::__ANON__() called at lib/Sql.pm line 264
	Sql::__ANON__() called at local/lib/perl5/Try/Tiny.pm line 76
	eval {...} called at local/lib/perl5/Try/Tiny.pm line 67
	Try::Tiny::try(\'CODE(0xf9f69a8)\', \'Try::Tiny::Catch=REF(0xf9d77b0)\') called at lib/Sql.pm line 277
	Sql::_auto_transaction(\'CODE(0xf9e4590)\', \'Sql=HASH(0xc8ab838)\') called at lib/Sql.pm line 282
	Sql::_run_in_transaction_one(\'CODE(0xf9e4590)\', \'Sql=HASH(0xc8ab838)\') called at lib/Sql.pm line 296
	Sql::run_in_transaction(\'CODE(0xf9e4590)\', \'Sql=HASH(0xc8ab838)\') called at lib/MusicBrainz/Server/Data/Edit.pm line 436
	MusicBrainz::Server::Data::Edit::create(\'MusicBrainz::Server::Data::Edit=HASH(0xabe6a10)\', \'editor_id\', 53705, \'privileges\', 185, \'cover_art_types\', \'ARRAY(0xd89d630)\', \'cover_art_id\', 835050629, ...) called at lib/MusicBrainz/Server/Controller.pm line 85
	MusicBrainz::Server::Controller::__ANON__() called at local/lib/perl5/Try/Tiny.pm line 76
	eval {...} called at local/lib/perl5/Try/Tiny.pm line 67
	Try::Tiny::try(\'CODE(0xd89da20)\', \'Try::Tiny::Catch=REF(0xd8003f0)\') called at lib/MusicBrainz/Server/Controller.pm line 101
	MusicBrainz::Server::Controller::_insert_edit(\'MusicBrainz::Server::Controller::Release=HASH(0xc1291d0)\', \'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Form::Release::AddCoverArt=HASH(0xd5085b8)\', \'edit_type\', 314, \'release\', \'MusicBrainz::Server::Entity::Release=HASH(0xd48ed20)\', \'cover_art_types\', \'ARRAY(0xd89d630)\', ...) called at lib/MusicBrainz/Server/Controller/Release.pm line 451
	MusicBrainz::Server::Controller::Release::add_cover_art(\'MusicBrainz::Server::Controller::Release=HASH(0xc1291d0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst/Action.pm line 65
	Catalyst::Action::execute(\'Catalyst::Action=HASH(0xc4cc038)\', \'MusicBrainz::Server::Controller::Release=HASH(0xc1291d0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst.pm line 1667
	eval {...} called at local/lib/perl5/Catalyst.pm line 1667
	Catalyst::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4cc038)\') called at local/lib/perl5/Catalyst/Plugin/StackTrace.pm line 94
	Catalyst::Plugin::StackTrace::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4cc038)\') called at lib/MusicBrainz/Server.pm line 316
	MusicBrainz::Server::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4cc038)\') called at local/lib/perl5/Catalyst/Action.pm line 60
	Catalyst::Action::dispatch(\'Catalyst::Action=HASH(0xc4cc038)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst/ActionChain.pm line 38
	Catalyst::ActionChain::dispatch(\'Catalyst::ActionChain=HASH(0xcf9e868)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst/Controller.pm line 125
	Catalyst::Controller::_ACTION(\'MusicBrainz::Server::Controller::Release=HASH(0xc1291d0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst/Action.pm line 65
	Catalyst::Action::execute(\'Catalyst::Action=HASH(0xc4c61c0)\', \'MusicBrainz::Server::Controller::Release=HASH(0xc1291d0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst.pm line 1667
	eval {...} called at local/lib/perl5/Catalyst.pm line 1667
	Catalyst::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4c61c0)\') called at local/lib/perl5/Catalyst/Plugin/StackTrace.pm line 94
	Catalyst::Plugin::StackTrace::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4c61c0)\') called at lib/MusicBrainz/Server.pm line 316
	MusicBrainz::Server::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4c61c0)\') called at local/lib/perl5/Catalyst/Action.pm line 60
	Catalyst::Action::dispatch(\'Catalyst::Action=HASH(0xc4c61c0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst/Dispatcher.pm line 257
	Catalyst::Dispatcher::_do_forward(\'Catalyst::Dispatcher=HASH(0x42903f0)\', \'forward\', \'MusicBrainz::Server=HASH(0xce000a8)\', \'_ACTION\') called at local/lib/perl5/Catalyst/Dispatcher.pm line 237
	Catalyst::Dispatcher::forward(\'Catalyst::Dispatcher=HASH(0x42903f0)\', \'MusicBrainz::Server=HASH(0xce000a8)\', \'_ACTION\') called at local/lib/perl5/Catalyst.pm line 342
	Catalyst::forward(\'MusicBrainz::Server=HASH(0xce000a8)\', \'_ACTION\') called at local/lib/perl5/Catalyst/Controller.pm line 99
	Catalyst::Controller::_DISPATCH(\'MusicBrainz::Server::Controller::Release=HASH(0xc1291d0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst/Action.pm line 65
	Catalyst::Action::execute(\'Catalyst::Action=HASH(0xc4c5ae8)\', \'MusicBrainz::Server::Controller::Release=HASH(0xc1291d0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst.pm line 1667
	eval {...} called at local/lib/perl5/Catalyst.pm line 1667
	Catalyst::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4c5ae8)\') called at local/lib/perl5/Catalyst/Plugin/StackTrace.pm line 94
	Catalyst::Plugin::StackTrace::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4c5ae8)\') called at lib/MusicBrainz/Server.pm line 316
	MusicBrainz::Server::execute(\'MusicBrainz::Server=HASH(0xce000a8)\', \'MusicBrainz::Server::Controller::Release\', \'Catalyst::Action=HASH(0xc4c5ae8)\') called at local/lib/perl5/Catalyst/Action.pm line 60
	Catalyst::Action::dispatch(\'Catalyst::Action=HASH(0xc4c5ae8)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst/Dispatcher.pm line 257
	Catalyst::Dispatcher::_do_forward(\'Catalyst::Dispatcher=HASH(0x42903f0)\', \'forward\', \'MusicBrainz::Server=HASH(0xce000a8)\', \'/release/_DISPATCH\') called at local/lib/perl5/Catalyst/Dispatcher.pm line 237
	Catalyst::Dispatcher::forward(\'Catalyst::Dispatcher=HASH(0x42903f0)\', \'MusicBrainz::Server=HASH(0xce000a8)\', \'/release/_DISPATCH\') called at local/lib/perl5/Catalyst.pm line 342
	Catalyst::forward(\'MusicBrainz::Server=HASH(0xce000a8)\', \'/release/_DISPATCH\') called at local/lib/perl5/Catalyst/Dispatcher.pm line 105
	Catalyst::Dispatcher::dispatch(\'Catalyst::Dispatcher=HASH(0x42903f0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst.pm line 1613
	Catalyst::dispatch(\'MusicBrainz::Server=HASH(0xce000a8)\') called at lib/MusicBrainz/Server.pm line 280
	Class::MOP::Class:::around(\'CODE(0x3e46ea0)\', \'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 162
	Class::MOP::Method::Wrapped::__ANON__(\'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 91
	MusicBrainz::Server::dispatch(\'MusicBrainz::Server=HASH(0xce000a8)\') called at local/lib/perl5/Catalyst.pm line 1963
	Catalyst::__ANON__() called at local/lib/perl5/Try/Tiny.pm line 76
	eval {...} called at local/lib/perl5/Try/Tiny.pm line 67
	Try::Tiny::try(\'CODE(0xc6f7580)\', \'Try::Tiny::Catch=REF(0xc6f7508)\') called at local/lib/perl5/Catalyst.pm line 1969
	Catalyst::handle_request(\'MusicBrainz::Server\', \'env\', \'HASH(0xce334e8)\') called at lib/Catalyst/Plugin/AutoRestart.pm line 119
	Catalyst::Plugin::AutoRestart::handle_request(\'MusicBrainz::Server\', \'env\', \'HASH(0xce334e8)\') called at local/lib/perl5/Catalyst/Engine.pm line 856
	Catalyst::Engine::__ANON__(\'CODE(0xce47c08)\') called at local/lib/perl5/Plack/Handler/FCGI.pm line 134
	Plack::Handler::FCGI::run(\'Plack::Handler::FCGI=HASH(0x22e1a30)\', \'CODE(0xc6921d0)\') called at local/lib/perl5/Plack/Loader.pm line 88
	Plack::Loader::run(\'Plack::Loader=HASH(0x22aae00)\', \'Plack::Handler::FCGI=HASH(0x22e1a30)\') called at local/lib/perl5/Plack/Runner.pm line 263
	Plack::Runner::run(\'Plack::Runner=HASH(0x20ad808)\') called at local/bin/plackup line 10
 at lib/Sql.pm line 277
';
 at lib/MusicBrainz/Server/Controller.pm line 101"


Stack trace:
line 97 MusicBrainz::Server::Controller
line 101 MusicBrainz::Server::Controller
line 451 MusicBrainz::Server::Controller::Release
line 316 MusicBrainz::Server
line 316 MusicBrainz::Server
line 316 MusicBrainz::Server
line 280 MusicBrainz::Server


Request data: 
$VAR1 = {
          'query_parameters' => {},
          'body_parameters' => {
                                 'add-cover-art.id' => '835050629',
                                 'add-cover-art.edit_note' => '',
                                 'add-cover-art.comment' => 'right inside',
                                 'add-cover-art.position' => '1',
                                 'add-cover-art.as_auto_editor' => '1'
                               }
        };


Sort Order: Ascending order - Click to sort in descending order
Oliver Charles added a comment - 01/May/12 02:54 PM

If you open the tabs "slowly" - ie, waiting a second between each middle/ctrl click, does this problem still happen?


nikki added a comment - 02/May/12 05:39 PM

As far as I can tell, it always happens in Opera even if I open the tabs slowly. In Safari the ISEs occur randomly (i.e. it works fine except for when it doesn't) so I'm not sure if it's linked to how fast I open the windows.


Oliver Charles added a comment - 04/May/12 10:08 AM

Dupe of MBS-4514