From 928965f40b645f7c4c63b7a0179020e218597995 Mon Sep 17 00:00:00 2001
From: Claas Kazzer <kazzer@ub.uni-leipzig.de>
Date: Mon, 9 Jul 2018 15:18:56 +0200
Subject: [PATCH] refs #13567 * improves schema.org tagging on author and place
 of publication at data-publicationDetails.phtml

---
 .../templates/RecordDriver/SolrDefault/data-authors.phtml     | 4 +++-
 .../RecordDriver/SolrDefault/data-publicationDetails.phtml    | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/themes/finc/templates/RecordDriver/SolrDefault/data-authors.phtml b/themes/finc/templates/RecordDriver/SolrDefault/data-authors.phtml
index 3ecfd3cd378..31cad9b7eaa 100644
--- a/themes/finc/templates/RecordDriver/SolrDefault/data-authors.phtml
+++ b/themes/finc/templates/RecordDriver/SolrDefault/data-authors.phtml
@@ -22,10 +22,12 @@ $types = [
     <? if (!empty($data[$type])): ?>
         <? foreach ($data[$type] as $author => $roles): ?>
             <? ob_start(); ?>
-            <span class="author-data" property="<?=$this->escapeHtml($schemaLabel)?>">
+            <span class="author-data" property="<?=$this->escapeHtml($schemaLabel)?>" typeof="Person">
+              <span property="name">
                 <a href="<?=$this->record($this->driver)->getLink('author', $author)?>">
             <?=$this->escapeHtml($author)?>
                 </a>
+              </span>
                 <?
                 // Display additional data using the appropriate translation prefix
                 // (for example, to render author roles correctly):
diff --git a/themes/finc/templates/RecordDriver/SolrDefault/data-publicationDetails.phtml b/themes/finc/templates/RecordDriver/SolrDefault/data-publicationDetails.phtml
index 1992d50d9bc..fa14eede7c5 100644
--- a/themes/finc/templates/RecordDriver/SolrDefault/data-publicationDetails.phtml
+++ b/themes/finc/templates/RecordDriver/SolrDefault/data-publicationDetails.phtml
@@ -4,7 +4,9 @@
     <? foreach ($data as $field): ?>
     <span property="publisher" typeof="Organization">
         <? $pubPlace = $field->getPlace(); if (!empty($pubPlace)): ?>
-            <span property="location"><?=$this->escapeHtml($pubPlace)?></span>
+          <span property="location" typeof="Place">
+            <span property="name"><?=$this->escapeHtml($pubPlace)?></span>
+          </span>
         <? endif; ?>
         <? $pubName = $field->getName(); if (!empty($pubName)): ?>
             <span property="name"><?=$this->escapeHtml($pubName)?></span>
-- 
GitLab