Avoid redundant linked resources captions

omeka · Apr 3, 2024 · c96c0d0 · c96c0d0
1 parent 360f62b
commit c96c0d0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/application/src/Api/Adapter/AbstractResourceEntityAdapter.php b/application/src/Api/Adapter/AbstractResourceEntityAdapter.php
@@ -575,16 +575,14 @@ public function getSubjectValues(Resource $resource, $page = null, $perPage = nu
         $offset = (is_numeric($page) && is_numeric($perPage)) ? (($page - 1) * $perPage) : null;
         $qb = $this->getSubjectValuesQueryBuilder($resource, $propertyId, $resourceType, $siteId)
             ->join('value.property', 'property')
-            ->join('property.vocabulary', 'vocabulary')
             ->select([
                 'value val',
                 'property.id property_id',
-                'resource_template_property.id resource_template_property_id',
                 'property.label property_label',
+                'resource_template_property.id resource_template_property_id',
                 'resource_template_property.alternateLabel property_alternate_label',
-                "CONCAT(vocabulary.prefix, ':', property.localName) term",
             ])
-            ->orderBy('property.id, resource_template_property.alternateLabel, resource.title')
+            ->orderBy('resource_template_property.alternateLabel, property.label, property.id, resource.title')
             ->setMaxResults($perPage)
             ->setFirstResult($offset);
         $event = new Event('api.subject_values.query', $this, [

diff --git a/application/src/Api/Representation/AbstractResourceEntityRepresentation.php b/application/src/Api/Representation/AbstractResourceEntityRepresentation.php
@@ -402,7 +402,7 @@ public function subjectValues($page = null, $perPage = null, $propertyId = null,
         $results = $this->getAdapter()->getSubjectValues($this->resource, $page, $perPage, $propertyId, $resourceType, $siteId);
         $subjectValues = [];
         foreach ($results as $result) {
-            $index = sprintf('%s-%s', $result['property_id'], $result['resource_template_property_id']);
+            $index = $result['property_alternate_label'] ?: $result['property_label'];
             $result['val'] = new ValueRepresentation($result['val'], $this->getServiceLocator());
             $subjectValues[$index][] = $result;
         }

diff --git a/application/view/common/linked-resources.phtml b/application/view/common/linked-resources.phtml
@@ -63,11 +63,10 @@ $resourcePropertiesSelect->setValueOptions($valueOptions);
 <?php foreach ($subjectValues as $values): ?>
 <?php
 $caption = sprintf(
-    $this->translate('%s with "%s: %s" (%s)'),
+    $this->translate('%s with "%s: %s"'),
     $this->translate($labelMap[$resourceType]),
     $values[0]['property_alternate_label'] ?? $this->translate($values[0]['property_label']),
-    $objectResource->displayTitle(),
-    $values[0]['term']
+    $objectResource->displayTitle()
 );
 ?>
 <table>