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

Don't add credit to relationship on entity merge if it would match destination entity name

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Normal Normal
    • 2021-10-25
    • None
    • Edit system
    • None

      When two entities that can have relationship credits are merged and relationship credits haven't been chosen for renaming, the name of the old entity is added as a credit for any relationships it had that had no credit already, in order to save the name: https://github.com/metabrainz/musicbrainz-server/blob/9942e134f71df5b0c34ab7f020e498fdb5ea2df8/lib/MusicBrainz/Server/Data/Relationship.pm#L545-L558

      This makes sense in most cases, but it does not make sense for the merge of two entities of the same name (but with different disambiguation or whatever). In that case, we end up with a relationship with a credit that already matches the merged entity name. As such, it just looks a bit dumb and is not required at all to save the name, which is the only reason we're doing this to begin with. We should probably make sure the name is different before we save it.

      Original text:

      I had to remove some manually today:

      https://musicbrainz.org/edit/59878295
      https://musicbrainz.org/edit/59878294
      https://musicbrainz.org/edit/59878293
      https://musicbrainz.org/edit/59878292

      I don’t undesratnd why… maybe the relationships were added while https://musicbrainz.org/edit/49295603 was pending?
      But AC should still have been cleaned up.

            reosarevok Nicolás Tamargo
            jesus2099 jesus2099
            Votes:
            5 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              Resolved:

                Version Package
                2021-10-25