Bugzilla – Attachment 7900 Details for
Bug 5503
comments shouldn't show patron's full name
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 5503 [REVISED] comments shouldn't show patron's full name
Bug-5503-REVISED-comments-shouldnt-show-patrons-fu.patch (text/plain), 9.58 KB, created by
Owen Leonard
on 2012-02-27 13:32:56 UTC
(
hide
)
Description:
Bug 5503 [REVISED] comments shouldn't show patron's full name
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2012-02-27 13:32:56 UTC
Size:
9.58 KB
patch
obsolete
>From ca81fd936b740191296149c400f1513a901bc4f6 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Fri, 16 Dec 2011 16:32:40 -0500 >Subject: [PATCH] Bug 5503 [REVISED] comments shouldn't show patron's full name >Content-Type: text/plain; charset="utf-8" > >Adding a few choices for what information can be displayed >alongside comments in the OPAC: > >- nothing >- full name >- first name >- last name >- first name and last name first initial >- username > >Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> > >Tested by choosing each choice and previewing an individual title >with a comment and confirming the name was displayed properly. Then >choosing each option and prevewing the recent comments page. > >All tests passed. >--- > installer/data/mysql/sysprefs.sql | 2 +- > installer/data/mysql/updatedatabase.pl | 8 +++++ > .../prog/en/modules/admin/preferences/opac.pref | 11 +++++-- > koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 30 ++++++++++++------- > .../opac-tmpl/prog/en/modules/opac-showreviews.tt | 16 +++++++++- > opac/opac-showreviews.pl | 2 + > 6 files changed, 52 insertions(+), 17 deletions(-) > >diff --git a/installer/data/mysql/sysprefs.sql b/installer/data/mysql/sysprefs.sql >index 0cc79b2..aae5527 100644 >--- a/installer/data/mysql/sysprefs.sql >+++ b/installer/data/mysql/sysprefs.sql >@@ -103,7 +103,7 @@ INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES > INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('ReturnBeforeExpiry',0,'If ON, checkout will be prevented if returndate is after patron card expiry',NULL,'YesNo'); > INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('ReturnLog',1,'If ON, enables the circulation (returns) log',NULL,'YesNo'); > INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('reviewson',1,'If ON, enables patron reviews of bibliographic records in the OPAC','','YesNo'); >-INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('ShowReviewer',1,'If ON, name of reviewer will be shown above comments in OPAC','','YesNo'); >+INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('ShowReviewer','full','Choose how a commenter''s identity is presented alongside comments in the OPAC','none|full|first|surname|firstandinitial|username','Choice'); > INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('ShowReviewerPhoto',1,'If ON, photo of reviewer will be shown beside comments in OPAC','','YesNo'); > INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('SpecifyDueDate',1,'Define whether to display \"Specify Due Date\" form in Circulation','','YesNo'); > INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('SubscriptionHistory','simplified','Define the display preference for serials issue history in OPAC','simplified|full','Choice'); >diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl >index d2d9d97..b890a41 100755 >--- a/installer/data/mysql/updatedatabase.pl >+++ b/installer/data/mysql/updatedatabase.pl >@@ -4712,6 +4712,14 @@ if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { > SetVersion($DBversion); > } > >+$DBversion = "3.07.00.XXX"; >+if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { >+ $dbh->do(" UPDATE `systempreferences` SET `value` = 'none', `options` = 'none|full|first|surname|firstandinitial|username', `explanation` = 'Choose how a commenter''s identity is presented alongside comments in the OPAC', `type` = 'Choice' WHERE `systempreferences`.`variable` = 'ShowReviewer' AND `systempreferences`.`variable` = 0"); >+ $dbh->do(" UPDATE `systempreferences` SET `value` = 'full', `options` = 'none|full|first|surname|firstandinitial|username', `explanation` = 'Choose how a commenter''s identity is presented alongside comments in the OPAC', `type` = 'Choice' WHERE `systempreferences`.`variable` = 'ShowReviewer' AND `systempreferences`.`variable` = 1"); >+ print "Adding additional options for the display of commenter's identity in the OPAC: Full name, first name, last name, first name and last name first initial, username, or no information\n"; >+ SetVersion($DBversion); >+} >+ > =head1 FUNCTIONS > > =head2 DropAllForeignKeys($table) >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref >index 24856b8..bca7a71 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref >@@ -277,11 +277,16 @@ OPAC: > no: "Don't allow" > - patrons to make comments on items on the OPAC. > - >+ - Show > - pref: ShowReviewer > choices: >- yes: Show >- no: Hide >- - reviewer's name above comments in OPAC. >+ none: no name >+ full: full name >+ first: first name >+ surname: last name >+ firstandinitial: first name and last name initial >+ username: username >+ - of commenter with comments in OPAC. > - > - pref: ShowReviewerPhoto > choices: >diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >index 25480c8..7af60a0 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >@@ -873,18 +873,26 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > </p></div> > [% ELSE %] > <div class="commentline"> >- [% IF ( ShowReviewer ) %] >- [% IF ( review.avatarurl ) %] >- <img class="avatar" src="[% review.avatarurl %]" height="80" width="80" alt="" /> >- [% END %] >- <h5> >- Comment by >- [% review.title %] >- [% review.firstname %] >- [% review.surname %] >- </h5> >+ [% IF ( ShowReviewer != "none" ) %] >+ [% IF ( review.avatarurl ) %] >+ <img class="avatar" src="[% review.avatarurl %]" height="80" width="80" alt="" /> >+ [% END %] >+ [% SWITCH ShowReviewer %] >+ [% CASE 'full' %] >+ <h5>Comment by [% review.title %] [% review.firstname %] [% review.surname %]</h5> >+ [% CASE 'first' %] >+ <h5>Comment by [% review.firstname %]</h5> >+ [% CASE 'surname' %] >+ <h5>Comment by [% review.surname %]</h5> >+ [% CASE 'firstandinitial' %] >+ <h5>Comment by [% review.firstname %] [% review.surname|truncate(2,'.') %]</h5> >+ [% CASE 'username' %] >+ <h5>Comment by [% review.userid %]</h5> >+ [% END %] >+ <small>[% review.datereviewed %]</small> >+ [% ELSIF ( ShowReviewer == "none") %] >+ <h5>Patron comment on [% review.datereviewed %]</h5> > [% END %] >- <small>[% review.datereviewed %]</small> > <p> > [% FILTER html_break %] > [% review.review |html %] >diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-showreviews.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-showreviews.tt >index fdea679..d548361 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-showreviews.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-showreviews.tt >@@ -54,8 +54,20 @@ $(document).ready(function(){ > [% review.review |html %] > [% END %] > <span style="font-size:87%;font-color:#CCC;">Added [% review.datereviewed %] [% IF ( review.your_comment ) %] by <strong>you</strong>[% ELSE %] >- [% IF ( ShowReviewer ) %] by >- [% review.firstname %] [% review.surname %][% END %][% END %]</span></p> >+ [% IF ( ShowReviewer != "none" ) %] by >+ [% SWITCH ShowReviewer %] >+ [% CASE 'full' %] >+ [% review.borrtitle %] [% review.firstname %] [% review.surname %] >+ [% CASE 'first' %] >+ [% review.firstname %] >+ [% CASE 'surname' %] >+ [% review.surname %] >+ [% CASE 'firstandinitial' %] >+ [% review.firstname %] [% review.surname|truncate(2,'.') %] >+ [% CASE 'username' %] >+ [% review.userid %] >+ [% END %] >+ [% END %][% END %]</span></p> > </td> > <td> > [% IF ( review.BiblioDefaultViewmarc ) %]<a class="title" href="/cgi-bin/koha/opac-MARCdetail.pl?biblionumber=[% review.biblionumber |url %]" title="View details for this title"> >diff --git a/opac/opac-showreviews.pl b/opac/opac-showreviews.pl >index 74f4684..449ffcd 100755 >--- a/opac/opac-showreviews.pl >+++ b/opac/opac-showreviews.pl >@@ -101,8 +101,10 @@ for my $result (@$reviews){ > $result->{size} = $bib->{'size'}; > $result->{notes} = $bib->{'notes'}; > $result->{timestamp} = $bib->{'timestamp'}; >+ $result->{borrtitle} = $borr->{'title'}; > $result->{firstname} = $borr->{'firstname'}; > $result->{surname} = $borr->{'surname'}; >+ $result->{userid} = $borr->{'userid'}; > if ($libravatar_enabled and $borr->{'email'}) { > $result->{avatarurl} = libravatar_url(email => $borr->{'email'}, size => 40, https => $ENV{HTTPS}); > } >-- >1.7.3
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 5503
:
6844
|
6845
|
7202
|
7632
|
7900
|
7902
|
7968