From: Dan Scott <dscott@laurentian.ca>
Date: Sun, 27 Oct 2013 17:24:41 +0000 (-0400)
Subject: More granular seller property
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=53bf2ca2d02f9804393c521dbe8bdf59807f586a;p=working%2FEvergreen.git

More granular seller property

Rather than wrapping everything in the table cell as the seller, focus
on the <a> and give it an explicit Library typeof attribute; then we can
offer up the name of the library as the name attribute, should the
crawler be too lazy to follow the link.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
---

diff --git a/Open-ILS/src/templates/opac/parts/record/copy_table.tt2 b/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
index 5678da1e78..736ee0673f 100644
--- a/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
@@ -93,7 +93,7 @@ END; # FOREACH bib
             [%- IF serial_holdings %]<td headers='copy_header_holding_label' class='rdetail-issue-issue'>
                 [%- copy_info.holding_label | html; -%]
             </td>
-            [%- ELSE %]<td headers='copy_header_library' property="seller">
+            [%- ELSE %]<td headers='copy_header_library'>
             [%-
                 org_name = ctx.get_aou(copy_info.circ_lib).name;
                 org_sname = ctx.get_aou(copy_info.circ_lib).shortname;
@@ -101,8 +101,8 @@ END; # FOREACH bib
                 UNLESS lib_url;
                     lib_url = mkurl(ctx.opac_root _ '/library/' _ ctx.get_aou(copy_info.circ_lib).shortname, {}, 1);
                 END; 
-                IF lib_url; '<a href="'; lib_url | url; '">'; END;
-                org_name | html;
+                IF lib_url; '<a property="seller" typeof="Library" href="'; lib_url | url; '">'; END;
+                '<span property="name">'; org_name | html; '</span>';
                 IF lib_url; '</a>'; END;
             -%]
                 <link property="businessFunction" href="http://purl.org/goodrelations/v1#LeaseOut">