Edit to add and reorder medium fails

XMLWordPrintable

    • Type: Bug
    • Resolution: Unresolved
    • Priority: Normal
    • None
    • Affects Version/s: None
    • Component/s: Release editor
    • None

      A release has four Hybrid SACD (SACD layer, 2 channels) mediums. I wanted to add four Hybrid SACD (CD layer) that have the same tracklists with the existing mediums. I did that by using the "Add Medium" tool in the release editor to duplicate the four existing mediums. I then reordered the eight mediums so that each CD layer is placed before its corresponding SACD layer:

      1. Hybrid SACD 1, CD layer
      2. Hybrid SACD 1, SACD layer, 2 channels
      3. Hybrid SACD 2, CD layer
      4. Hybrid SACD 2, SACD layer, 2 channels
      5. Hybrid SACD 3, CD layer
      6. Hybrid SACD 3, SACD layer, 2 channels
      7. Hybrid SACD 4, CD layer
      8. Hybrid SACD 4, SACD layer, 2 channels

      When I click submit on the edit note tab of the release editor, the following error appears.

      Failed query: 'UPDATE medium SET position = -position WHERE id IN (?,?,?,?,?,?)' (5429595 5429594 5501619 5429596 5429593) 22P02 DBD::Pg::st execute failed: ERROR: invalid input syntax for type integer: "" LINE 2: WHERE id IN ('5429595','','5429594','5501619','542... ^ [for Statement "UPDATE medium SET position = -position WHERE id IN (?,?,?,?,?,?)" with ParamValues: 1='5429595', 2='', 3='5429594', 4='5501619', 5='5429596', 6='5429593'] at lib/Sql.pm line 129. Sql::catch {...} (MusicBrainz::Server::Exceptions::DatabaseError=HASH(0x556b6a392dc8)) called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 123 Try::Tiny::try(CODE(0x556b6a39d428), Try::Tiny::Catch=REF(0x556b6a0ac8c0)) called at lib/Sql.pm line 130 Sql::do(Sql=HASH(0x556b6a55ecc8), "UPDATE medium SET position = -position\x{a} WHERE id IN "..., 5429595, "", 5429594, 5501619, 5429596, 5429593) called at lib/MusicBrainz/Server/Data/Medium.pm line 297 MusicBrainz::Server::Data::Medium::reorder(MusicBrainz::Server::Data::Medium=HASH(0x556b6222c718), 5429596, 8, 5429593, 2, 5501619, 1, 5429595, ...) called at lib/MusicBrainz/Server/Edit/Release/ReorderMediums.pm line 139 MusicBrainz::Server::Edit::Release::ReorderMediums::accept(MusicBrainz::Server::Edit::Release::ReorderMediums=HASH(0x556b63b7cea0)) called at lib/MusicBrainz/Server/Data/Edit.pm line 813 MusicBrainz::Server::Data::Edit::try {...} () called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 100 eval {...} called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 93 Try::Tiny::try(CODE(0x556b6a6a0770), Try::Tiny::Catch=REF(0x556b6a154a20)) called at lib/MusicBrainz/Server/Data/Edit.pm line 876 MusicBrainz::Server::Data::Edit::_do_accept(MusicBrainz::Server::Data::Edit=HASH(0x556b61393250), MusicBrainz::Server::Edit::Release::ReorderMediums=HASH(0x556b63b7cea0)) called at lib/MusicBrainz/Server/Data/Edit.pm line 915 MusicBrainz::Server::Data::Edit::__ANON__(MusicBrainz::Server::Edit::Release::ReorderMediums=HASH(0x556b63b7cea0)) called at lib/MusicBrainz/Server/Data/Edit.pm line 938 MusicBrainz::Server::Data::Edit::_close(MusicBrainz::Server::Data::Edit=HASH(0x556b61393250), MusicBrainz::Server::Edit::Release::ReorderMediums=HASH(0x556b63b7cea0), CODE(0x556b6a15ac98)) called at lib/MusicBrainz/Server/Data/Edit.pm line 915 MusicBrainz::Server::Data::Edit::accept(MusicBrainz::Server::Data::Edit=HASH(0x556b61393250), MusicBrainz::Server::Edit::Release::ReorderMediums=HASH(0x556b63b7cea0)) called at lib/MusicBrainz/Server/Data/Edit.pm line 652 MusicBrainz::Server::Data::Edit::create(MusicBrainz::Server::Data::Edit=HASH(0x556b61393250), "release", MusicBrainz::Server::Entity::Release=HASH(0x556b6a10f520), "edit_type", 313, "medium_positions", ARRAY(0x556b6a21fe48), "editor", ...) called at lib/MusicBrainz/Server/Controller/WS/js/Edit.pm line 697 MusicBrainz::Server::Controller::WS::js::Edit::try {...} () called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 102 eval {...} called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 93 Try::Tiny::try(CODE(0x556b6a69d2c0), Try::Tiny::Catch=REF(0x556b6a66e7b8)) called at lib/MusicBrainz/Server/Controller/WS/js/Edit.pm line 715 MusicBrainz::Server::Controller::WS::js::Edit::create_edits(MusicBrainz::Server::Controller::WS::js::Edit=HASH(0x556b633f6590), MusicBrainz::Server=HASH(0x556b6a30f228), HASH(0x556b6a2ddaf8)) called at lib/MusicBrainz/Server/Controller/WS/js/Edit.pm line 779 MusicBrainz::Server::Controller::WS::js::Edit::__ANON__() called at lib/Sql.pm line 352 Sql::try {...} () called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 102 eval {...} called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 93 Try::Tiny::try(CODE(0x556b6a3696a8), Try::Tiny::Catch=REF(0x556b6a6ab750)) called at lib/Sql.pm line 364 Sql::_auto_transaction(CODE(0x556b6a250088), Sql=HASH(0x556b6a55ecc8)) called at lib/Sql.pm line 370 Sql::_run_in_transaction_one(CODE(0x556b6a250088), Sql=HASH(0x556b6a55ecc8)) called at lib/Sql.pm line 385 Sql::run_in_transaction(CODE(0x556b6a250088), Sql=HASH(0x556b6a55ecc8)) called at lib/MusicBrainz/Server/Model/MB.pm line 20 MusicBrainz::Server::Model::MB::with_transaction(MusicBrainz::Server::Model::MB=HASH(0x556b60c33c88), CODE(0x556b6a250088)) called at lib/MusicBrainz/Server/Controller/WS/js/Edit.pm line 790 MusicBrainz::Server::Controller::WS::js::Edit::submit_edits(MusicBrainz::Server::Controller::WS::js::Edit=HASH(0x556b633f6590), MusicBrainz::Server=HASH(0x556b6a30f228), HASH(0x556b6a2ddaf8)) called at lib/MusicBrainz/Server/Controller/WS/js/Edit.pm line 750 MusicBrainz::Server::Controller::WS::js::Edit::create(MusicBrainz::Server::Controller::WS::js::Edit=HASH(0x556b633f6590), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Action.pm line 374 Catalyst::Action::execute(Catalyst::Action=HASH(0x556b6390bdc8), MusicBrainz::Server::Controller::WS::js::Edit=HASH(0x556b633f6590), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2067 eval {...} called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2067 Catalyst::execute(MusicBrainz::Server=HASH(0x556b6a30f228), "MusicBrainz::Server::Controller::WS::js::Edit", Catalyst::Action=HASH(0x556b6390bdc8)) called at lib/Catalyst/Plugin/ErrorInfo.pm line 34 Catalyst::Plugin::ErrorInfo::execute(MusicBrainz::Server=HASH(0x556b6a30f228), "MusicBrainz::Server::Controller::WS::js::Edit", Catalyst::Action=HASH(0x556b6390bdc8)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Action.pm line 368 Catalyst::Action::dispatch(Catalyst::Action=HASH(0x556b6390bdc8), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/ActionChain.pm line 74 Catalyst::ActionChain::_dispatch_chain_actions(Catalyst::ActionChain=HASH(0x556b6a199c50), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/ActionChain.pm line 43 Catalyst::ActionChain::dispatch(Catalyst::ActionChain=HASH(0x556b6a199c50), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Controller.pm line 170 Catalyst::Controller::_ACTION(MusicBrainz::Server::Controller::WS::js::Edit=HASH(0x556b633f6590), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Action.pm line 374 Catalyst::Action::execute(Catalyst::Action=HASH(0x556b63908088), MusicBrainz::Server::Controller::WS::js::Edit=HASH(0x556b633f6590), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2067 eval {...} called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2067 Catalyst::execute(MusicBrainz::Server=HASH(0x556b6a30f228), "MusicBrainz::Server::Controller::WS::js::Edit", Catalyst::Action=HASH(0x556b63908088)) called at lib/Catalyst/Plugin/ErrorInfo.pm line 34 Catalyst::Plugin::ErrorInfo::execute(MusicBrainz::Server=HASH(0x556b6a30f228), "MusicBrainz::Server::Controller::WS::js::Edit", Catalyst::Action=HASH(0x556b63908088)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Action.pm line 368 Catalyst::Action::dispatch(Catalyst::Action=HASH(0x556b63908088), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Dispatcher.pm line 261 Catalyst::Dispatcher::_do_forward(Catalyst::Dispatcher=HASH(0x556b569bd198), "forward", MusicBrainz::Server=HASH(0x556b6a30f228), "_ACTION") called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Dispatcher.pm line 241 Catalyst::Dispatcher::forward(Catalyst::Dispatcher=HASH(0x556b569bd198), MusicBrainz::Server=HASH(0x556b6a30f228), "_ACTION") called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 489 Catalyst::forward(MusicBrainz::Server=HASH(0x556b6a30f228), "_ACTION") called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Controller.pm line 134 Catalyst::Controller::_DISPATCH(MusicBrainz::Server::Controller::WS::js::Edit=HASH(0x556b633f6590), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Action.pm line 374 Catalyst::Action::execute(Catalyst::Action=HASH(0x556b639003e0), MusicBrainz::Server::Controller::WS::js::Edit=HASH(0x556b633f6590), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2067 eval {...} called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2067 Catalyst::execute(MusicBrainz::Server=HASH(0x556b6a30f228), "MusicBrainz::Server::Controller::WS::js::Edit", Catalyst::Action=HASH(0x556b639003e0)) called at lib/Catalyst/Plugin/ErrorInfo.pm line 34 Catalyst::Plugin::ErrorInfo::execute(MusicBrainz::Server=HASH(0x556b6a30f228), "MusicBrainz::Server::Controller::WS::js::Edit", Catalyst::Action=HASH(0x556b639003e0)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Action.pm line 368 Catalyst::Action::dispatch(Catalyst::Action=HASH(0x556b639003e0), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Dispatcher.pm line 261 Catalyst::Dispatcher::_do_forward(Catalyst::Dispatcher=HASH(0x556b569bd198), "forward", MusicBrainz::Server=HASH(0x556b6a30f228), "/ws/js/edit/_DISPATCH") called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Dispatcher.pm line 241 Catalyst::Dispatcher::forward(Catalyst::Dispatcher=HASH(0x556b569bd198), MusicBrainz::Server=HASH(0x556b6a30f228), "/ws/js/edit/_DISPATCH") called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 489 Catalyst::forward(MusicBrainz::Server=HASH(0x556b6a30f228), "/ws/js/edit/_DISPATCH") called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Dispatcher.pm line 107 Catalyst::Dispatcher::dispatch(Catalyst::Dispatcher=HASH(0x556b569bd198), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2013 Catalyst::dispatch(MusicBrainz::Server=HASH(0x556b6a30f228)) called at lib/MusicBrainz/Server.pm line 402 MusicBrainz::Server::__ANON__() called at lib/MusicBrainz/Server.pm line 367 MusicBrainz::Server::with_translations(MusicBrainz::Server=HASH(0x556b6a30f228), CODE(0x556b6a1735e8)) called at lib/MusicBrainz/Server.pm line 403 Class::MOP::Class:::around(CODE(0x556b5602bcc0), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 166 Class::MOP::Method::Wrapped::__ANON__(MusicBrainz::Server=HASH(0x556b6a30f228)) called at lib/MusicBrainz/Server.pm line 417 Class::MOP::Class:::around(CODE(0x556b6395f5e0), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 166 Class::MOP::Method::Wrapped::__ANON__(MusicBrainz::Server=HASH(0x556b6a30f228)) called at lib/MusicBrainz/Server.pm line 536 Class::MOP::Class:::around(CODE(0x556b63467990), MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 166 Class::MOP::Method::Wrapped::__ANON__(MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 35 MusicBrainz::Server::_wrapped_dispatch(MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/x86_64-linux-gnu-thread-multi/Class/MOP/Method/Wrapped.pm line 97 MusicBrainz::Server::dispatch(MusicBrainz::Server=HASH(0x556b6a30f228)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2419 Catalyst::try {...} () called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 102 eval {...} called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 93 Try::Tiny::try(CODE(0x556b6a6780f0), Try::Tiny::Catch=REF(0x556b6a281bd8)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst.pm line 2428 Catalyst::handle_request("MusicBrainz::Server", "env", HASH(0x556b6a14ac50), "response_cb", CODE(0x556b6a202600)) called at /home/musicbrainz/carton-local/lib/perl5/Catalyst/Engine.pm line 807 Catalyst::Engine::__ANON__(CODE(0x556b6a202600)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Middleware/HTTPExceptions.pm line 32 Plack::Middleware::HTTPExceptions::try {...} () called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 102 eval {...} called at /home/musicbrainz/carton-local/lib/perl5/Try/Tiny.pm line 93 Try::Tiny::try(CODE(0x556b6a1432a0), Try::Tiny::Catch=REF(0x556b6a239f20)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Middleware/HTTPExceptions.pm line 41 Plack::Middleware::HTTPExceptions::__ANON__(CODE(0x556b66159448)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Util.pm line 339 Plack::Util::__ANON__(CODE(0x556b6a0b92d8)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Util.pm line 339 Plack::Util::__ANON__(CODE(0x556b6a21bb50)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Util.pm line 339 Plack::Util::__ANON__(CODE(0x556b6a5539e0)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Util.pm line 339 Plack::Util::__ANON__(CODE(0x556b6a0788e0)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Util.pm line 339 Plack::Util::__ANON__(CODE(0x556b6a316e18)) called at /home/musicbrainz/carton-local/lib/perl5/Starlet/Server.pm line 377 Starlet::Server::handle_connection(Plack::Handler::Starlet=HASH(0x556b60c339a0), HASH(0x556b6a14ac50), IO::Socket::INET=GLOB(0x556b6a3572a0), CODE(0x556b639eaff8), "", "", "") called at /home/musicbrainz/carton-local/lib/perl5/Starlet/Server.pm line 190 Starlet::Server::accept_loop(Plack::Handler::Starlet=HASH(0x556b60c339a0), CODE(0x556b639eaff8), 63) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Handler/Starlet.pm line 80 Plack::Handler::Starlet::run(Plack::Handler::Starlet=HASH(0x556b60c339a0), CODE(0x556b639eaff8)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Loader.pm line 84 Plack::Loader::run(Plack::Loader=HASH(0x556b4f888540), Plack::Handler::Starlet=HASH(0x556b60c339a0)) called at /home/musicbrainz/carton-local/lib/perl5/Plack/Runner.pm line 279 Plack::Runner::run(Plack::Runner=HASH(0x556b4f61c940)) called at /home/musicbrainz/carton-local/bin/plackup line 7 

      I am not redirected back to the release page, although the four CD layer mediums were added (https://musicbrainz.org/edit/128110866, https://musicbrainz.org/edit/128110867, https://musicbrainz.org/edit/128110868, https://musicbrainz.org/edit/128110869).

      However, they were all positioned after the four SACD layers as so, which is not what I wanted:

      1. Hybrid SACD 1, SACD layer, 2 channels
      2. Hybrid SACD 2, SACD layer, 2 channels
      3. Hybrid SACD 3, SACD layer, 2 channels
      4. Hybrid SACD 4, SACD layer, 2 channels
      5. Hybrid SACD 3, CD layer
      6. Hybrid SACD 4, CD layer
      7. Hybrid SACD 1, CD layer
      8. Hybrid SACD 2, CD layer

      Note that the latter four mediums are positioned as medium 5, 7, 9, and 10.

            Assignee:
            Unassigned
            Reporter:
            silentbird
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:

                Version Package