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

    Details

    • Type: Bug
    • Status: Closed
    • Priority: High
    • Resolution: Fixed
    • Affects Version/s: Bug fixes, 2011-08-22
    • Fix Version/s: Bug fixes, 2011-09-19
    • Component/s: Release editor
    • Labels:
      None

      Description

      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
      

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Packages

                  Version Package
                  Bug fixes, 2011-09-19