Bugzilla – Attachment 46628 Details for
Bug 15096
Export today's checked in barcodes: Display warning if reading history is set to "never"
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 15096: Export today's checked in barcodes: Display warning if reading history is set to "never"
Bug-15096-Export-todays-checked-in-barcodes-Displa.patch (text/plain), 6.28 KB, created by
Aleisha Amohia
on 2016-01-13 23:56:47 UTC
(
hide
)
Description:
Bug 15096: Export today's checked in barcodes: Display warning if reading history is set to "never"
Filename:
MIME Type:
Creator:
Aleisha Amohia
Created:
2016-01-13 23:56:47 UTC
Size:
6.28 KB
patch
obsolete
>From e69e6982760e02f6774cabc549491093359afd6f Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch> >Date: Sat, 31 Oct 2015 21:24:44 +0100 >Subject: [PATCH] Bug 15096: Export today's checked in barcodes: Display > warning if reading history is set to "never" > >If a patron's privacy settings are set to "Never" for keeping a reading history, "Export today's >checked in barcodes" returns an empty file. This patch does not allow to export in such case. > >To test: > >- Apply patch >- Check out / check in in some items > >-Test toolbar: >- On user's detail page, go to More->Export today's checked in barcodes >- Verify that the menu item does not appear if syspref 'intranetreadinghistory' > is set to 'Don't allow' >- Verify that the menu item appears if syspref 'intranetreadinghistory' > is set to allow and > - that the menu item is grayed out with a tooltip if the user has set privacy > settings to never keep a reading history > - that the menu item works as before if user's privacy settings allow reading history. > >- Test left tab "Circulation history": >- Verify that the left tab "Circulation history" does not appear if syspref > 'intranetreadinghistory' is set to 'Don't allow' >- Verify that the tab appars if syspref 'intranetreadinghistory' is set to 'Allow' and > - that a message appears if user's privacy settings do not allow to keep the reading history > - that the export works as before if user's privacy settings allow to keep the reading history > >(Amended and changed test plan for comment #9) > >Signed-off-by: Aleisha <aleishaamohia@hotmail.com> >--- > .../prog/en/includes/members-toolbar.inc | 10 ++++-- > .../prog/en/modules/members/readingrec.tt | 2 ++ > members/readingrec.pl | 34 +++++++++++--------- > 3 files changed, 29 insertions(+), 17 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc >index 641bbad..3341601 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc >@@ -38,7 +38,7 @@ $(document).ready(function(){ > $(".btn-group").removeClass("open"); > });[% END %] > [% END %] >- $("#updatechild, #patronflags, #renewpatron, #deletepatron").tooltip(); >+ $("#updatechild, #patronflags, #renewpatron, #deletepatron, #exportbarcodes").tooltip(); > $("#exportcheckins").click(function(){ > export_barcodes(); > $(".btn-group").removeClass("open"); >@@ -195,7 +195,13 @@ function searchToHold(){ > [% ELSE %] > <li class="disabled"><a data-toggle="tooltip" data-placement="left" title="Patron is an adult" id="updatechild" href="#">Update child to adult patron</a></li></li> > [% END %] >- <li><a id="exportcheckins" href="#">Export today's checked in barcodes</a></li> >+ [% IF Koha.Preference('intranetreadinghistory') %] >+ [%IF ( privacy == 2 ) %] >+ <li class="disabled"><a data-toggle="tooltip" data-placement="left" title="Not allowed by patron's privacy settings" id="exportbarcodes" href="#">Export today's checked in barcodes</a></li> >+ [% ELSE %] >+ <li><a id="exportcheckins" href="#">Export today's checked in barcodes</a></li> >+ [% END %] >+ [% END %] > </ul> > </div> > </div> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/readingrec.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/readingrec.tt >index 6fb109e..e13a0d8 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/readingrec.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/readingrec.tt >@@ -52,6 +52,8 @@ > <div class="dialog alert">Staff members are not allowed to access patron's checkout history</div> > [% ELSIF is_anonymous %] > <div class="dialog alert">This is the anonymous patron, so no circulation history is displayed. To get a list of anonymized loans, please run a report.</div> >+[% ELSIF ( privacy == 2) %] >+ <div class="dialog message">This patron has set the privacy rules to never keeping a circulation history.</div> > [% ELSIF ( !loop_reading ) %] > <div class="dialog message">This patron has no circulation history.</div> > [% ELSE %] >diff --git a/members/readingrec.pl b/members/readingrec.pl >index 20860c2..0ae8808 100755 >--- a/members/readingrec.pl >+++ b/members/readingrec.pl >@@ -75,21 +75,24 @@ my $branches = GetBranches(); > > # barcode export > if ( $op eq 'export_barcodes' ) { >- my $today = output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 1 }); >- my @barcodes = >- map { $_->{barcode} } grep { $_->{returndate} =~ m/^$today/o } @{$issues}; >- my $borrowercardnumber = >- GetMember( borrowernumber => $borrowernumber )->{'cardnumber'}; >- my $delimiter = "\n"; >- binmode( STDOUT, ":encoding(UTF-8)" ); >- print $input->header( >- -type => 'application/octet-stream', >- -charset => 'utf-8', >- -attachment => "$today-$borrowercardnumber-checkinexport.txt" >- ); >- my $content = join $delimiter, uniq(@barcodes); >- print $content; >- exit; >+ if ( $data->{'privacy'} < 2) { >+ my $today = output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 1 }); >+ my @barcodes = >+ map { $_->{barcode} } grep { $_->{returndate} =~ m/^$today/o } @{$issues}; >+ my $borrowercardnumber = >+ GetMember( borrowernumber => $borrowernumber )->{'cardnumber'}; >+ my $delimiter = "\n"; >+ binmode( STDOUT, ":encoding(UTF-8)" ); >+ print $input->header( >+ -type => 'application/octet-stream', >+ -charset => 'utf-8', >+ -attachment => "$today-$borrowercardnumber-checkinexport.txt" >+ ); >+ >+ my $content = join $delimiter, uniq(@barcodes); >+ print $content; >+ exit; >+ } > } > > if ( $data->{'category_type'} eq 'C') { >@@ -123,6 +126,7 @@ $template->param(%$data); > $template->param( > readingrecordview => 1, > borrowernumber => $borrowernumber, >+ privacy => $data->{'privacy'}, > categoryname => $data->{description}, > roadtype => $roadtype, > is_child => ( $data->{category_type} eq 'C' ), >-- >1.7.10.4
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 15096
:
44269
|
45508
|
46501
|
46591
|
46628
|
47212