More granular seller property
authorDan Scott <dscott@laurentian.ca>
Sun, 27 Oct 2013 17:24:41 +0000 (13:24 -0400)
committerDan Scott <dscott@laurentian.ca>
Fri, 1 Nov 2013 20:31:13 +0000 (16:31 -0400)
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>
Open-ILS/src/templates/opac/parts/record/copy_table.tt2

index 5678da1..736ee06 100644 (file)
@@ -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">