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

Internal Server Errror in VA advanced tracklist editor if setting "Artist as Credited" without setting artist search field

XMLWordPrintable

      I noticed that release http://musicbrainz.org/release/60ded2f3-8fc7-4c3d-85c5-5c5b318a2970 had "(feat. Iris Trevisan)" in a few of the track names, and AFAIK that convention has been deprecated. Artist credits should be used for marking featuring artists instead.

      So I started editing the artist credits (first time for me, btw). Iris Trevisan didn't seem to exist in the database, so I tried what happens if I leave the "Artist in MusicBrainz" field empty and just write the name to the "Artist as credited" field. It seemed to work alright, at least in the tracklist editor. At the edit notes page the diff seemed a bit weird [1], but I entered the edit anyway. That resulted in an Internal Server Error. I'll paste the page contents here:

      Internal Server Error

Oops, something went wrong!

Error:

Caught exception in MusicBrainz::Server::Controller::ReleaseEditor::Edit->edit "No artist was created for Iris Trevisan at /home/musicbrainz/musicbrainz-server/script/../lib/MusicBrainz/Server/Wizard/ReleaseEditor.pm line 832."

Stack trace:

    line 832 MusicBrainz::Server::Wizard::ReleaseEditor
    line 777 MusicBrainz::Server::Wizard::ReleaseEditor
    line 93 MusicBrainz::Server::Wizard::ReleaseEditor
    line 17 MusicBrainz::Server::Controller::ReleaseEditor::Edit
    line 270 MusicBrainz::Server

Request data:

$VAR1 = {
          'query_parameters' => {},
          'body_parameters' => {
                                 'save' => 'Enter edit',
                                 'edit_note' => '',
                                 'wizard_session_id' => '0.5548511227964'
                               }
        };
            

URL: http://musicbrainz.org/release/60ded2f3-8fc7-4c3d-85c5-5c5b318a2970/edit

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 all of the information that is shown above.

Please be sure to include the stack trace and request data shown above!

      [1] One strange thing was that the old artist was empty for all but the last track. Another thing was that the new artist was presented in a confusing way, but maybe that's as designed (first the artist as credited and then the artists in the MB database). The non-existing Iris Trevisan was shown as [removed] - maybe that's ok, but the artist isn't actually removed, because it (probably) never existed in the first place. This is how the diff was shown (I'll try to edit this to be at least semi-readable):

      Artist credits: 	
# Old Artist     New Artist
1                The Free feat. Iris Trevisan
                 The Free (eurodance group)
                 [removed] as Iris Trevisan

4                The Free feat. Iris Trevisan
                 The Free (eurodance group)
                 [removed] as Iris Trevisan

7                The Free feat. Iris Trevisan
                 The Free (eurodance group)
                 [removed] as Iris Trevisan

10               The Free feat. Iris Trevisan
                 The Free (eurodance group)
                 [removed] as Iris Trevisan

11               The Free feat. Iris Trevisan
                 The Free (eurodance group)
                 [removed] as Iris Trevisan

13 The Free      The Free feat. Iris Trevisan
                 The Free (eurodance group)
                 [removed] as Iris Trevisan

            warp Kuno Woudt
            tanu Tanu Kaskinen
            Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved:

                Version Package
                Bug fixes, 2011-09-19