LP# 1337550 Remove AggregateOffer schema.org instance
authorDan Scott <dscott@laurentian.ca>
Thu, 3 Jul 2014 19:53:25 +0000 (15:53 -0400)
committerBen Shum <bshum@biblio.org>
Thu, 10 Jul 2014 19:45:14 +0000 (15:45 -0400)
As our record details page lists each copy as a separate Offer (well, in
many cases anyway, unless paging gets invoked), the AggregateOffer
entity is not really necessary--and using it on the same page as
separate Offer entities is not recommended by at least one search engine
(per Google https://support.google.com/webmasters/answer/146750?hl=en
says a given page should include "_one_ of the following: Offer or
Offer-aggregate").

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
Open-ILS/src/templates/opac/parts/record/copy_counts.tt2

index 9f86bc7..6e95ae7 100644 (file)
             ou_name = cp_org_unit.name;
             displayed_ous.$ou_name = 1;
     %]
-    <li property="offers" typeof="AggregateOffer">
-        <link property="itemOffered" href="#schemarecord">
-        <meta property="offerCount" content="[% ou_avail %]">
-        <meta property="seller" content="[% ou_name | html %]">
+    <li>
     [% l('[quant,_1,copy,copies] at [_2].', ou_avail, ou_name) | html %]
     [%- this_depth = ctx.get_aou(ou_id).ou_type.depth;
         IF ou_avail > 0 && this_depth != ctx.copy_depth %]
         UNLESS depth < 0 || displayed_ous.exists(ou_name);
     %]
     [%- IF attrs.plib_copy_counts.$depth.count > 0; %]
-    <li class="preferred" property="offers" typeof="AggregateOffer">
-        <link property="itemOffered" href="#schemarecord">
-        <meta property="offerCount" content="[% ou_avail %]">
-        <meta property="seller" content="[% ou_name | html %]">
+    <li class="preferred">
     [%-
         l('[_1] of [quant,_2,copy,copies] available at [_3].',
             attrs.plib_copy_counts.$depth.available,