Line 0
Link Here
|
0 |
- |
1 |
#!/usr/bin/perl |
|
|
2 |
|
3 |
# This file is part of Koha. |
4 |
# |
5 |
# Copyright (C) 2013 Mark Tompsett |
6 |
# |
7 |
# This Koha test module is intended for Patron Category |
8 |
# tweaks to the OpacHiddenItems system preference. |
9 |
# Related to Bug 10859 |
10 |
# |
11 |
# Koha is free software; you can redistribute it and/or modify it |
12 |
# under the terms of the GNU General Public License as published by |
13 |
# the Free Software Foundation; either version 3 of the License, or |
14 |
# (at your option) any later version. |
15 |
# |
16 |
# Koha is distributed in the hope that it will be useful, but |
17 |
# WITHOUT ANY WARRANTY; without even the implied warranty of |
18 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
19 |
# GNU General Public License for more details. |
20 |
# |
21 |
# You should have received a copy of the GNU General Public License |
22 |
# along with Koha; if not, see <http://www.gnu.org/licenses>. |
23 |
# |
24 |
|
25 |
use Modern::Perl; |
26 |
use MARC::Record; |
27 |
use Carp; |
28 |
use C4::Biblio; |
29 |
use C4::Context; |
30 |
use C4::Members; |
31 |
use Data::Dumper; |
32 |
use YAML qw/Load/; # due to bug 10872 not being in master yet. |
33 |
use Test::More tests => 22; |
34 |
|
35 |
BEGIN { |
36 |
use_ok('C4::Items'); |
37 |
} |
38 |
|
39 |
my $dbh = C4::Context->dbh; |
40 |
# Start transaction |
41 |
$dbh->{AutoCommit} = 0; |
42 |
$dbh->{RaiseError} = 1; |
43 |
|
44 |
# Helper biblio. |
45 |
my ($bibnum, $bibitemnum) = get_biblio(); |
46 |
|
47 |
# Add dummy branches. |
48 |
$dbh->do('INSERT INTO branches (branchcode,branchname) VALUES ("LIB1","LIB1");'); |
49 |
$dbh->do('INSERT INTO branches (branchcode,branchname) VALUES ("LIB2","LIB2");'); |
50 |
|
51 |
# Add dummy patron categories |
52 |
$dbh->do("INSERT INTO categories (categorycode, description) VALUES ('SPT','Sample Patron Code');"); |
53 |
$dbh->do("INSERT INTO categories (categorycode, description) VALUES ('SS','Sample Staff Code');"); |
54 |
|
55 |
# Add same item for two branches. |
56 |
my ($item_bibnum, $item_bibitemnum, $itemnumber) = AddItem({ homebranch => 'LIB1', holdingbranch => 'LIB1' } , $bibnum); |
57 |
cmp_ok($item_bibnum, '==', $bibnum, "LIB1 Item linked to correct biblionumber."); |
58 |
cmp_ok($item_bibitemnum, '==', $bibitemnum, "LIB1 Item linked to correct biblioitemnumber."); |
59 |
($item_bibnum, $item_bibitemnum, $itemnumber) = AddItem({ homebranch => 'LIB2', holdingbranch => 'LIB2' } , $bibnum); |
60 |
cmp_ok($item_bibnum, '==', $bibnum, "LIB2 Item linked to correct biblionumber."); |
61 |
cmp_ok($item_bibitemnum, '==', $bibitemnum, "LIB2 Item linked to correct biblioitemnumber."); |
62 |
|
63 |
# Add dummy patron. |
64 |
C4::Context->set_preference('PatronSelfRegistrationDefaultCategory','PT'); |
65 |
my %borrower; |
66 |
$borrower{'surname'} = 'Borrower'; |
67 |
$borrower{'firstname'} = 'Sample'; |
68 |
$borrower{'address'} = '123 Bogus Street'; |
69 |
$borrower{'city'} = 'Bogusville'; |
70 |
$borrower{'branchcode'} = 'LIB1'; |
71 |
$borrower{'categorycode'} = 'PT'; |
72 |
$borrower{'zipcode'} = 'H0H 0H0'; |
73 |
$borrower{'userid'} = 'Sample.Borrower'; |
74 |
$borrower{'email'} = 'santa@northpole.com'; |
75 |
$borrower{'phone'} = '967-1111'; |
76 |
print Dumper(\%borrower) . "\n"; |
77 |
my ( $borrowernumber, $password ) = C4::Members::AddMember_Opac(%borrower) or |
78 |
croak "AddMember_Opac failed.\n"; |
79 |
diag("Sample Borrower added.\n"); |
80 |
print STDERR "CHECK: $borrowernumber\n$password\n"; |
81 |
|
82 |
# Get the member's card number. |
83 |
my $sql = "SELECT cardnumber FROM borrowers WHERE borrowernumber=?;"; |
84 |
my $sth = $dbh->prepare($sql); |
85 |
$sth->execute($borrowernumber); |
86 |
my $cardnumber = $sth->fetchrow; |
87 |
|
88 |
# Get an array of hashes of Item information. |
89 |
my @all_items = C4::Items::GetItemsInfo($bibnum); |
90 |
diag($all_items[0]->{'itemnumber'} . " - " . $all_items[0]->{'homebranch'}); |
91 |
diag($all_items[1]->{'itemnumber'} . " - " . $all_items[1]->{'homebranch'}); |
92 |
|
93 |
# Hide everything for this record. |
94 |
C4::Context->set_preference('OpacHiddenItems',"homebranch: [LIB1,LIB2]\n"); |
95 |
|
96 |
# Check what's hidden when a user isn't logged in. |
97 |
my @items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
98 |
cmp_ok( scalar @items2hide, '==', 2, "OpacHiddenItems hid everything correctly."); |
99 |
diag(join(',',@items2hide)); |
100 |
|
101 |
diag("Borrower number: $borrowernumber"); |
102 |
# Set up dummy patron as if logged in. |
103 |
my @USERENV = ( |
104 |
$borrowernumber, |
105 |
'Sample.Borrower', |
106 |
$cardnumber, |
107 |
'Sample', |
108 |
'Borrower', |
109 |
'LIB1', |
110 |
'LIB1', |
111 |
0, |
112 |
); |
113 |
C4::Context->_new_userenv('DUMMY_SESSION_ID'); |
114 |
C4::Context::set_userenv( @USERENV ); |
115 |
diag("Sample Borrower environment variables set up.\n"); |
116 |
|
117 |
# 4 blocks of 4 cases: 'SPT','SS','','SS|SPT' |
118 |
C4::Context->set_preference('PatronSingleBranch','SPT'); |
119 |
C4::Context->set_preference('PatronEveryBranch','SPT'); |
120 |
$sql = "UPDATE borrowers SET categorycode='SPT' WHERE borrowernumber=?;"; |
121 |
$sth = $dbh->prepare($sql); |
122 |
$sth->execute($borrowernumber); |
123 |
|
124 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
125 |
cmp_ok( scalar @items2hide, '==', 0, "OpacHiddenItems unhid everything correctly."); |
126 |
diag("PatronSingleBranch: SPT\nPatronEveryBranch: SPT\n"); |
127 |
diag(join(',',@items2hide)); |
128 |
|
129 |
# PatronSingleBranch toggle |
130 |
C4::Context->set_preference('PatronSingleBranch','SPT'); |
131 |
C4::Context->set_preference('PatronEveryBranch','SS'); |
132 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
133 |
cmp_ok( scalar @items2hide, '==', 1, "OpacHiddenItems hid LIB2 correctly."); |
134 |
diag("PatronSingleBranch: SPT\nPatronEveryBranch: SS\n"); |
135 |
diag(join(',',@items2hide)); |
136 |
|
137 |
# PatronSingleBranch toggle |
138 |
C4::Context->set_preference('PatronSingleBranch','SPT'); |
139 |
C4::Context->set_preference('PatronEveryBranch',''); |
140 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
141 |
cmp_ok( scalar @items2hide, '==', 1, "OpacHiddenItems hid LIB2 correctly."); |
142 |
diag("PatronSingleBranch: SPT\nPatronEveryBranch: ''\n"); |
143 |
diag(join(',',@items2hide)); |
144 |
|
145 |
# PatronSingleBranch toggle |
146 |
C4::Context->set_preference('PatronSingleBranch','SPT'); |
147 |
C4::Context->set_preference('PatronEveryBranch','SS|SPT'); |
148 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
149 |
cmp_ok( scalar @items2hide, '==', 0, "OpacHiddenItems unhid everything correctly."); |
150 |
diag("PatronSingleBranch: SPT\nPatronEveryBranch: SS|SPT\n"); |
151 |
diag(join(',',@items2hide)); |
152 |
|
153 |
# PatronSingleBranch toggle |
154 |
C4::Context->set_preference('PatronSingleBranch','SS'); |
155 |
C4::Context->set_preference('PatronEveryBranch','SPT'); |
156 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
157 |
cmp_ok( scalar @items2hide, '==', 0, "OpacHiddenItems unhid everything correctly."); |
158 |
diag("PatronSingleBranch: SS\nPatronEveryBranch: SPT\n"); |
159 |
diag(join(',',@items2hide)); |
160 |
|
161 |
# PatronSingleBranch toggle |
162 |
C4::Context->set_preference('PatronSingleBranch','SS'); |
163 |
C4::Context->set_preference('PatronEveryBranch','SS'); |
164 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
165 |
cmp_ok( scalar @items2hide, '==', 2, "OpacHiddenItems hid everything correctly."); |
166 |
diag("PatronSingleBranch: SS\nPatronEveryBranch: SS\n"); |
167 |
diag(join(',',@items2hide)); |
168 |
|
169 |
# PatronSingleBranch toggle |
170 |
C4::Context->set_preference('PatronSingleBranch','SS'); |
171 |
C4::Context->set_preference('PatronEveryBranch',''); |
172 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
173 |
cmp_ok( scalar @items2hide, '==', 2, "OpacHiddenItems hid everything correctly."); |
174 |
diag("PatronSingleBranch: SS\nPatronEveryBranch: ''\n"); |
175 |
diag(join(',',@items2hide)); |
176 |
|
177 |
# PatronSingleBranch toggle |
178 |
C4::Context->set_preference('PatronSingleBranch','SS'); |
179 |
C4::Context->set_preference('PatronEveryBranch','SS|SPT'); |
180 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
181 |
cmp_ok( scalar @items2hide, '==', 0, "OpacHiddenItems unhid everything correctly."); |
182 |
diag("PatronSingleBranch: SS\nPatronEveryBranch: SS|SPT\n"); |
183 |
diag(join(',',@items2hide)); |
184 |
|
185 |
# PatronSingleBranch toggle |
186 |
C4::Context->set_preference('PatronSingleBranch',''); |
187 |
C4::Context->set_preference('PatronEveryBranch','SPT'); |
188 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
189 |
cmp_ok( scalar @items2hide, '==', 0, "OpacHiddenItems unhid everything correctly."); |
190 |
diag("PatronSingleBranch: ''\nPatronEveryBranch: SPT\n"); |
191 |
diag(join(',',@items2hide)); |
192 |
|
193 |
# PatronSingleBranch toggle |
194 |
C4::Context->set_preference('PatronSingleBranch',''); |
195 |
C4::Context->set_preference('PatronEveryBranch','SS'); |
196 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
197 |
cmp_ok( scalar @items2hide, '==', 2, "OpacHiddenItems hid everything correctly."); |
198 |
diag("PatronSingleBranch: ''\nPatronEveryBranch: SS\n"); |
199 |
diag(join(',',@items2hide)); |
200 |
|
201 |
# PatronSingleBranch toggle |
202 |
C4::Context->set_preference('PatronSingleBranch',''); |
203 |
C4::Context->set_preference('PatronEveryBranch',''); |
204 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
205 |
cmp_ok( scalar @items2hide, '==', 2, "OpacHiddenItems hid everything correctly."); |
206 |
diag("PatronSingleBranch: ''\nPatronEveryBranch: ''\n"); |
207 |
diag(join(',',@items2hide)); |
208 |
|
209 |
# PatronSingleBranch toggle |
210 |
C4::Context->set_preference('PatronSingleBranch',''); |
211 |
C4::Context->set_preference('PatronEveryBranch','SS|SPT'); |
212 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
213 |
cmp_ok( scalar @items2hide, '==', 0, "OpacHiddenItems unhid everything correctly."); |
214 |
diag("PatronSingleBranch: ''\nPatronEveryBranch: SS|SPT\n"); |
215 |
diag(join(',',@items2hide)); |
216 |
|
217 |
# PatronSingleBranch toggle |
218 |
C4::Context->set_preference('PatronSingleBranch','SS|SPT'); |
219 |
C4::Context->set_preference('PatronEveryBranch','SPT'); |
220 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
221 |
cmp_ok( scalar @items2hide, '==', 0, "OpacHiddenItems unhid everything correctly."); |
222 |
diag("PatronSingleBranch: SS|SPT\nPatronEveryBranch: SPT\n"); |
223 |
diag(join(',',@items2hide)); |
224 |
|
225 |
# PatronSingleBranch toggle |
226 |
C4::Context->set_preference('PatronSingleBranch','SS|SPT'); |
227 |
C4::Context->set_preference('PatronEveryBranch','SS'); |
228 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
229 |
cmp_ok( scalar @items2hide, '==', 1, "OpacHiddenItems hid LIB2 correctly."); |
230 |
diag("PatronSingleBranch: SS|SPT\nPatronEveryBranch: SS\n"); |
231 |
diag(join(',',@items2hide)); |
232 |
|
233 |
# PatronSingleBranch toggle |
234 |
C4::Context->set_preference('PatronSingleBranch','SS|SPT'); |
235 |
C4::Context->set_preference('PatronEveryBranch',''); |
236 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
237 |
cmp_ok( scalar @items2hide, '==', 1, "OpacHiddenItems hid LIB2 correctly."); |
238 |
diag("PatronSingleBranch: SS|SPT\nPatronEveryBranch: ''\n"); |
239 |
diag(join(',',@items2hide)); |
240 |
|
241 |
# PatronSingleBranch toggle |
242 |
C4::Context->set_preference('PatronSingleBranch','SS|SPT'); |
243 |
C4::Context->set_preference('PatronEveryBranch','SS|SPT'); |
244 |
@items2hide = C4::Items::GetHiddenItemnumbers(@all_items); |
245 |
cmp_ok( scalar @items2hide, '==', 0, "OpacHiddenItems unhid everything correctly."); |
246 |
diag("PatronSingleBranch: SS|SPT\nPatronEveryBranch: SS|SPT\n"); |
247 |
diag(join(',',@items2hide)); |
248 |
|
249 |
# Nothing wrong with being explicit. :) |
250 |
$dbh->rollback(); |
251 |
|
252 |
# Helper method to set up a Biblio. |
253 |
sub get_biblio { |
254 |
my $bib = MARC::Record->new(); |
255 |
$bib->append_fields( |
256 |
MARC::Field->new('100', ' ', ' ', a => 'Moffat, Steven'), |
257 |
MARC::Field->new('245', ' ', ' ', a => 'Silence in the library'), |
258 |
); |
259 |
return ($bibnum, $bibitemnum) = AddBiblio($bib, ''); |
260 |
} |