MBS-11846: Display release artist on release group view (metabrainz#2201)

Right now, there's no way to see if a release inside a release group
has a different artist credit than the release group itself.
This adds a release artist column to that view.
It also adds an "artist-credit-variation" class for cases
where the release and release group artist credits differ,
to help userscript users highlight that (matching what we do
for track artists on recording pages).

Author: reosarevok

lib/MusicBrainz/Server/Controller/ReleaseGroup.pm

@@ -78,6 +78,7 @@ sub show : Chained('load') PathPart('') {
 
     $c->model('Release')->load_related_info(@$releases);
     $c->model('Release')->load_meta(@$releases);
+    $c->model('ArtistCredit')->load(@$releases);
     $c->model('ReleaseStatus')->load(@$releases);
     $c->model('CritiqueBrainz')->load_display_reviews($rg)
         unless $self->should_return_jsonld($c);

root/release_group/ReleaseGroupIndex.js

@@ -18,6 +18,8 @@ import CritiqueBrainzReview, {type CritiqueBrainzReviewT}
 import PaginatedResults from '../components/PaginatedResults';
 import TaggerIcon from '../static/scripts/common/components/TaggerIcon';
 import loopParity from '../utility/loopParity';
+import ArtistCreditLink
+  from '../static/scripts/common/components/ArtistCreditLink';
 import EntityLink from '../static/scripts/common/components/EntityLink';
 import CleanupBanner from '../components/CleanupBanner';
 import FormRow from '../components/FormRow';
@@ -45,13 +47,17 @@ type Props = {
   +wikipediaExtract: WikipediaExtractT | null,
 };
 
-function buildReleaseStatusTable($c, releaseStatusGroup) {
+function buildReleaseStatusTable(
+  $c,
+  releaseStatusGroup,
+  releaseGroupCreditId,
+) {
   const status = releaseStatusGroup[0].status;
   return (
     <React.Fragment key={status ? status.name : 'no-status'}>
       <tr className="subh">
         {$c.user ? <th /> : null}
-        <th colSpan={$c.session && $c.session.tport ? 8 : 7}>
+        <th colSpan={$c.session && $c.session.tport ? 9 : 8}>
           {status?.name
             ? lp_attributes(status.name, 'release_status')
             : lp('(unknown)', 'release status')}
@@ -72,6 +78,14 @@ function buildReleaseStatusTable($c, releaseStatusGroup) {
           <td>
             <EntityLink entity={release} showCaaPresence />
           </td>
+          {/* The class being added is for usage with userscripts */}
+          <td className={
+            releaseGroupCreditId === release.artistCredit.id
+              ? null
+              : 'artist-credit-variation'}
+          >
+            <ArtistCreditLink artistCredit={release.artistCredit} />
+          </td>
           <td>{release.combined_format_name || l('[missing media]')}</td>
           <td>{release.combined_track_count || lp('-', 'missing data')}</td>
           <td>
@@ -139,6 +153,7 @@ const ReleaseGroupIndex = ({
                     </th>
                   ) : null}
                   <th>{l('Release')}</th>
+                  <th>{l('Artist')}</th>
                   <th>{l('Format')}</th>
                   <th>{l('Tracks')}</th>
                   <th>{l('Country') + lp('/', 'and') + l('Date')}</th>
@@ -150,7 +165,11 @@ const ReleaseGroupIndex = ({
                 </tr>
               </thead>
               <tbody>
-                {releases.map(r => buildReleaseStatusTable($c, r))}
+                {releases.map(releaseStatusGroup => buildReleaseStatusTable(
+                  $c,
+                  releaseStatusGroup,
+                  releaseGroup.artistCredit.id,
+                ))}
               </tbody>
             </table>
           </PaginatedResults>