[BUGFIX] Restore changes removed by musical sources (kitodo#1780)

Author: beatrycze-volk

Classes/Controller/MetadataController.php

@@ -314,7 +314,7 @@ private function buildUrlFromMetadata(array $metadata): array
         $buildUrl = [];
 
         foreach ($metadata as $i => $section) {
-            if ($this->settings['linkTitle'] && $section['_id'] && isset($section['title']) && !empty($section['title'])) {
+            if ($this->settings['linkTitle'] && $section['_id'] && !empty($section['title'])) {
                 $details = $this->currentDocument->getLogicalStructure($section['_id'][0]);
                 $buildUrl[$i]['title'] = [
                     'id' => $this->document->getUid(),

Resources/Private/Partials/Metadata/Entries.html

@@ -14,7 +14,7 @@
       data-namespace-typo3-fluid="true">
 
 {configObject.wrap -> kitodo:metadataWrapVariable(name: 'metadataWrap')}
-<f:variable name="metaSectionCObj" value="{metaCObjData.{sectionIterator.index}}" />
+<f:variable name="metaSectionConfigObject" value="{metaConfigObjectData.{sectionIterator.index}}" />
 
 <f:variable name="wrappedValues"><f:spaceless>
     <f:if condition="{kitodo:isArray(subject: '{documentMetadataSection.{configObject.indexName}}')}">
@@ -32,16 +32,16 @@
                     <f:else>
                         <f:if condition="{kitodo:isArray(subject: '{value}')}">
                             <f:then>
-                                <kitodo:stdWrap wrap="{metadataWrap.value}" data="{metaSectionCObj}">
+                                <kitodo:stdWrap wrap="{metadataWrap.value}" data="{metaSectionConfigObject}">
                                     <f:for each="{value}" key="key" iteration="subiterator" as="val">
                                         <f:for each="{configObject.format.0.subentries}" as="subentry">
                                             <f:if condition="{subentry.indexName} === {key}">
                                                 <f:variable name="subConfigObject">{subentry}</f:variable>
                                                 {subConfigObject.wrap -> kitodo:metadataWrapVariable(name: 'subMetadataWrap')}
-                                                <kitodo:stdWrap wrap="{subMetadataWrap.all}" data="{metaSectionCObj}">
-                                                    <kitodo:stdWrap wrap="{subMetadataWrap.key}" data="{metaSectionCObj}">{subConfigObject.label}</kitodo:stdWrap>
+                                                <kitodo:stdWrap wrap="{subMetadataWrap.all}" data="{metaSectionConfigObject}">
+                                                    <kitodo:stdWrap wrap="{subMetadataWrap.key}" data="{metaSectionConfigObject}">{subConfigObject.label}</kitodo:stdWrap>
                                                     <f:for each="{val}" as="subvalue">
-                                                        <kitodo:stdWrap wrap="{subMetadataWrap.value}" data="{metaSectionCObj}">
+                                                        <kitodo:stdWrap wrap="{subMetadataWrap.value}" data="{metaSectionConfigObject}">
                                                             {subvalue}
                                                         </kitodo:stdWrap>
                                                     </f:for>
@@ -52,22 +52,22 @@
                                 </kitodo:stdWrap>
                             </f:then>
                             <f:else>
-                                <kitodo:stdWrap wrap="{metadataWrap.value}" data="{metaSectionCObj}">{value}</kitodo:stdWrap>
+                                <kitodo:stdWrap wrap="{metadataWrap.value}" data="{metaSectionConfigObject}">{value}</kitodo:stdWrap>
                             </f:else>
                         </f:if>
                     </f:else>
                 </f:if>
             </f:for>
         </f:then>
         <f:else>
-            <kitodo:stdWrap wrap="{metadataWrap.value}" data="{metaSectionCObj}">{documentMetadataSection.{configObject.indexName}}</kitodo:stdWrap>
+            <kitodo:stdWrap wrap="{metadataWrap.value}" data="{metaSectionConfigObject}">{documentMetadataSection.{configObject.indexName}}</kitodo:stdWrap>
         </f:else>
     </f:if>
 </f:spaceless></f:variable>
 
 <f:if condition="{wrappedValues}">
-    <kitodo:stdWrap wrap="{metadataWrap.all}" data="{metaSectionCObj}">
-        <kitodo:stdWrap wrap="{metadataWrap.key}" data="{metaSectionCObj}">{configObject.label}</kitodo:stdWrap>
+    <kitodo:stdWrap wrap="{metadataWrap.all}" data="{metaSectionConfigObject}">
+        <kitodo:stdWrap wrap="{metadataWrap.key}" data="{metaSectionConfigObject}">{configObject.label}</kitodo:stdWrap>
         {wrappedValues -> f:format.raw()}
     </kitodo:stdWrap>
 </f:if>

Tests/Functional/ViewHelpers/StdWrapViewHelperTest.php

@@ -41,8 +41,8 @@ public function renderWithStdWrap(): void
         );
 
         // A fully filled array with correct values does not make any difference. The rendering result
-        // is not been influenced by the viewhelpers data parameter.
-        $view->assign('metaSectionCObj', [0 => ['tilte' => 'A test title']]);
+        // has not been influenced by the viewhelpers data parameter.
+        $view->assign('metaSectionConfigObject', [0 => ['tilte' => 'A test title']]);
 
         $view->setTemplateSource(
             '<html xmlns:kitodo="http://typo3.org/ns/Kitodo/Dlf/ViewHelpers">