Text "Patron reviews" appears even if no review available

[Eva Cerninakova]

Text "Patron reviews" appears in record detail page in OPAC (in "Awards, Reviews, & Suggested Reads" section even if no review available (e.g. http://eg-test.osvobozena-knihovna.cz/eg/opac/record/16358?query=golem;qtype=keyword;locg=1;detail_record_view=0;expand=awards;ac=reviews#awards).
The text concerns Chilifresh patron review and occurs in the TPAC template /openils/var/templates/opac/parts/record/awards.tt2 (line 5).
As it is quite confusing for catalog users, the text should not be displayed when Chilifresh patron reviews are not used in particular Evergren catalog (may be also the section Awards, Reviews, & Suggested Reads should not be displayed when empty).
(Evergren 2.10.7)

[Linda Jansova (skolkova-s)]

We have tested a solution which is inspired by Novelist (in /templates/opac/parts/record/awards.tt2) and consists of adding

[% IF ENV.OILS_CHILIFRESH_URL %]

before the following code:

    <strong>[% l('Patron Reviews:') %]</strong>
    <div style="margin-bottom:20px;">
        <span class="chili_review" id="isbn_[% attrs.isbn_clean | html %]"> </span>
        <div id="chili_review_[% attrs.isbn_clean | html %]" style="display: none;" align="center" width="100%"></div>
    </div>

and

  [% END %]

after the code in the awards.tt2 file.

It seems to work okay (when Chilifresh is switched off, no "Patron Reviews:" text appears).

The customized version of the awards.tt2 file is attached.

[Linda Jansova (skolkova-s)]

Apologies if the file is not in the correct patch format, it is simply an edited version of the original awards.tt2 file.

[Kathy Lussier (klussier)]

I've put Linda's suggested changes into a git branch and pushed it to the working repo:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/lp1628655-chilifresh-tpac

I've added Linda as the author of the patch. Linda, if I should be designating somebody else as the author, let me know and I'll update the branch.

[Linda Jansova (skolkova-s)]

Thank you, Kathy! It looks perfect as it is :-)!

[Jeff Davis (jdavis-sitka)]

Works for me. Signoff pushed to working repo:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1628655-chilifresh-tpac-signoff

Thanks, Linda!

[Linda Jansova (skolkova-s)]

Thank you, Jeff!

[Kathy Lussier (klussier)]

Thank you Linda and Jeff! Merged to master, release 2.12 and release 2.11