|
Lines 35-50
t::lib::Mocks::mock_userenv({ branchcode => $library1->{branchcode} });
Link Here
|
| 35 |
|
35 |
|
| 36 |
my $bib_title = "Test Title"; |
36 |
my $bib_title = "Test Title"; |
| 37 |
|
37 |
|
| 38 |
my $borrower1 = $builder->build({ |
38 |
my $patron1 = $builder->build_object({ |
| 39 |
source => 'Borrower', |
39 |
class => 'Koha::Patrons', |
| 40 |
value => { |
40 |
value => { |
| 41 |
branchcode => $library1->{branchcode}, |
41 |
branchcode => $library1->{branchcode}, |
| 42 |
dateexpiry => '3000-01-01', |
42 |
dateexpiry => '3000-01-01', |
| 43 |
} |
43 |
} |
| 44 |
}); |
44 |
}); |
|
|
45 |
my $borrower1 = $patron1->unblessed; |
| 45 |
|
46 |
|
| 46 |
my $borrower2 = $builder->build({ |
47 |
my $patron2 = $builder->build_object({ |
| 47 |
source => 'Borrower', |
48 |
class => 'Koha::Patrons', |
| 48 |
value => { |
49 |
value => { |
| 49 |
branchcode => $library1->{branchcode}, |
50 |
branchcode => $library1->{branchcode}, |
| 50 |
dateexpiry => '3000-01-01', |
51 |
dateexpiry => '3000-01-01', |
|
Lines 53-60
my $borrower2 = $builder->build({
Link Here
|
| 53 |
|
54 |
|
| 54 |
# Test hold_fulfillment_policy |
55 |
# Test hold_fulfillment_policy |
| 55 |
my ( $itemtype ) = @{ $dbh->selectrow_arrayref("SELECT itemtype FROM itemtypes LIMIT 1") }; |
56 |
my ( $itemtype ) = @{ $dbh->selectrow_arrayref("SELECT itemtype FROM itemtypes LIMIT 1") }; |
| 56 |
my $borrowernumber1 = $borrower1->{borrowernumber}; |
|
|
| 57 |
my $borrowernumber2 = $borrower2->{borrowernumber}; |
| 58 |
my $library_A = $library1->{branchcode}; |
57 |
my $library_A = $library1->{branchcode}; |
| 59 |
my $library_B = $library2->{branchcode}; |
58 |
my $library_B = $library2->{branchcode}; |
| 60 |
|
59 |
|
|
Lines 69-95
my $biblioitemnumber =
Link Here
|
| 69 |
$dbh->selectrow_array("SELECT biblioitemnumber FROM biblioitems WHERE biblionumber = $biblionumber") |
68 |
$dbh->selectrow_array("SELECT biblioitemnumber FROM biblioitems WHERE biblionumber = $biblionumber") |
| 70 |
or BAIL_OUT("Cannot find newly created biblioitems record"); |
69 |
or BAIL_OUT("Cannot find newly created biblioitems record"); |
| 71 |
|
70 |
|
| 72 |
$dbh->do(" |
71 |
my $item1 = $builder->build_object({ class => 'Koha::Items', value => { barcode => 'AllowHoldIf1', biblionumber => $biblionumber, biblioitemnumber => $biblioitemnumber, homebranch => $library_A, holdingbranch => $library_A, notforloan => 0, damaged => 0, itemlost => 0, withdrawn => 0, onloan => undef, itype => $itemtype } }); |
| 73 |
INSERT INTO items (barcode, biblionumber, biblioitemnumber, homebranch, holdingbranch, notforloan, damaged, itemlost, withdrawn, onloan, itype) |
|
|
| 74 |
VALUES ('AllowHoldIf1', $biblionumber, $biblioitemnumber, '$library_A', '$library_A', 0, 0, 0, 0, NULL, '$itemtype') |
| 75 |
"); |
| 76 |
|
72 |
|
| 77 |
my $itemnumber1 = |
73 |
my $item2 = $builder->build_object({ class => 'Koha::Items', value => { barcode => 'AllowHoldIf2', biblionumber => $biblionumber, biblioitemnumber => $biblioitemnumber, homebranch => $library_A, holdingbranch => $library_A, notforloan => 0, damaged => 0, itemlost => 0, withdrawn => 0, onloan => undef, itype => $itemtype } }); |
| 78 |
$dbh->selectrow_array("SELECT itemnumber FROM items WHERE biblionumber = $biblionumber") |
|
|
| 79 |
or BAIL_OUT("Cannot find newly created item"); |
| 80 |
|
| 81 |
my $item1 = Koha::Items->find( $itemnumber1 )->unblessed; |
| 82 |
|
| 83 |
$dbh->do(" |
| 84 |
INSERT INTO items (barcode, biblionumber, biblioitemnumber, homebranch, holdingbranch, notforloan, damaged, itemlost, withdrawn, onloan, itype) |
| 85 |
VALUES ('AllowHoldIf2', $biblionumber, $biblioitemnumber, '$library_A', '$library_A', 0, 0, 0, 0, NULL, '$itemtype') |
| 86 |
"); |
| 87 |
|
| 88 |
my $itemnumber2 = |
| 89 |
$dbh->selectrow_array("SELECT itemnumber FROM items WHERE biblionumber = $biblionumber ORDER BY itemnumber DESC") |
| 90 |
or BAIL_OUT("Cannot find newly created item"); |
| 91 |
|
| 92 |
my $item2 = Koha::Items->find( $itemnumber2 )->unblessed; |
| 93 |
|
74 |
|
| 94 |
$dbh->do("DELETE FROM issuingrules"); |
75 |
$dbh->do("DELETE FROM issuingrules"); |
| 95 |
my $rule = Koha::IssuingRule->new( |
76 |
my $rule = Koha::IssuingRule->new( |
|
Lines 106-125
my $rule = Koha::IssuingRule->new(
Link Here
|
| 106 |
); |
87 |
); |
| 107 |
$rule->store(); |
88 |
$rule->store(); |
| 108 |
|
89 |
|
| 109 |
my $is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
90 |
my $is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 110 |
is( $is, 0, "Item cannot be held, 2 items available" ); |
91 |
is( $is, 0, "Item cannot be held, 2 items available" ); |
| 111 |
|
92 |
|
| 112 |
my $issue1 = AddIssue( $borrower2, $item1->{barcode} ); |
93 |
my $issue1 = AddIssue( $patron2->unblessed, $item1->barcode ); |
| 113 |
|
94 |
|
| 114 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
95 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 115 |
is( $is, 0, "Item cannot be held, 1 item available" ); |
96 |
is( $is, 0, "Item cannot be held, 1 item available" ); |
| 116 |
|
97 |
|
| 117 |
AddIssue( $borrower2, $item2->{barcode} ); |
98 |
AddIssue( $patron2->unblessed, $item2->barcode ); |
| 118 |
|
99 |
|
| 119 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
100 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 120 |
is( $is, 1, "Item can be held, no items available" ); |
101 |
is( $is, 1, "Item can be held, no items available" ); |
| 121 |
|
102 |
|
| 122 |
AddReturn( $item1->{barcode} ); |
103 |
AddReturn( $item1->barcode ); |
| 123 |
|
104 |
|
| 124 |
{ # Remove the issue for the first patron, and modify the branch for item1 |
105 |
{ # Remove the issue for the first patron, and modify the branch for item1 |
| 125 |
subtest 'IsAvailableForItemLevelRequest behaviours depending on ReservesControlBranch + holdallowed' => sub { |
106 |
subtest 'IsAvailableForItemLevelRequest behaviours depending on ReservesControlBranch + holdallowed' => sub { |
|
Lines 133-141
AddReturn( $item1->{barcode} );
Link Here
|
| 133 |
subtest 'Item is available at a different library' => sub { |
114 |
subtest 'Item is available at a different library' => sub { |
| 134 |
plan tests => 4; |
115 |
plan tests => 4; |
| 135 |
|
116 |
|
| 136 |
$item1 = Koha::Items->find( $item1->{itemnumber} ); |
|
|
| 137 |
$item1->set({homebranch => $library_B, holdingbranch => $library_B })->store; |
117 |
$item1->set({homebranch => $library_B, holdingbranch => $library_B })->store; |
| 138 |
$item1 = $item1->unblessed; |
|
|
| 139 |
#Scenario is: |
118 |
#Scenario is: |
| 140 |
#One shelf holds is 'If all unavailable'/2 |
119 |
#One shelf holds is 'If all unavailable'/2 |
| 141 |
#Item 1 homebranch library B is available |
120 |
#Item 1 homebranch library B is available |
|
Lines 149-159
AddReturn( $item1->{barcode} );
Link Here
|
| 149 |
$sth_insert_rule->execute( $hold_allowed_from_home_library ); |
128 |
$sth_insert_rule->execute( $hold_allowed_from_home_library ); |
| 150 |
|
129 |
|
| 151 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); |
130 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); |
| 152 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
131 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 153 |
is( $is, 1, "Hold allowed from home library + ReservesControlBranch=ItemHomeLibrary, One item is available at different library, not holdable = none available => the hold is allowed at item level" ); |
132 |
is( $is, 1, "Hold allowed from home library + ReservesControlBranch=ItemHomeLibrary, One item is available at different library, not holdable = none available => the hold is allowed at item level" ); |
| 154 |
|
133 |
|
| 155 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'PatronLibrary'); |
134 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'PatronLibrary'); |
| 156 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
135 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 157 |
is( $is, 1, "Hold allowed from home library + ReservesControlBranch=PatronLibrary, One item is available at different library, not holdable = none available => the hold is allowed at item level" ); |
136 |
is( $is, 1, "Hold allowed from home library + ReservesControlBranch=PatronLibrary, One item is available at different library, not holdable = none available => the hold is allowed at item level" ); |
| 158 |
} |
137 |
} |
| 159 |
|
138 |
|
|
Lines 162-172
AddReturn( $item1->{barcode} );
Link Here
|
| 162 |
$sth_insert_rule->execute( $hold_allowed_from_any_libraries ); |
141 |
$sth_insert_rule->execute( $hold_allowed_from_any_libraries ); |
| 163 |
|
142 |
|
| 164 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); |
143 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); |
| 165 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
144 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 166 |
is( $is, 0, "Hold allowed from any library + ReservesControlBranch=ItemHomeLibrary, One item is available at the diff library, holdable = 1 available => the hold is not allowed at item level" ); |
145 |
is( $is, 0, "Hold allowed from any library + ReservesControlBranch=ItemHomeLibrary, One item is available at the diff library, holdable = 1 available => the hold is not allowed at item level" ); |
| 167 |
|
146 |
|
| 168 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'PatronLibrary'); |
147 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'PatronLibrary'); |
| 169 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
148 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 170 |
is( $is, 0, "Hold allowed from any library + ReservesControlBranch=PatronLibrary, One item is available at the diff library, holdable = 1 available => the hold is not allowed at item level" ); |
149 |
is( $is, 0, "Hold allowed from any library + ReservesControlBranch=PatronLibrary, One item is available at the diff library, holdable = 1 available => the hold is not allowed at item level" ); |
| 171 |
} |
150 |
} |
| 172 |
}; |
151 |
}; |
|
Lines 174-182
AddReturn( $item1->{barcode} );
Link Here
|
| 174 |
subtest 'Item is available at the same library' => sub { |
153 |
subtest 'Item is available at the same library' => sub { |
| 175 |
plan tests => 4; |
154 |
plan tests => 4; |
| 176 |
|
155 |
|
| 177 |
$item1 = Koha::Items->find( $item1->{itemnumber} ); |
|
|
| 178 |
$item1->set({homebranch => $library_A, holdingbranch => $library_A })->store; |
156 |
$item1->set({homebranch => $library_A, holdingbranch => $library_A })->store; |
| 179 |
$item1 = $item1->unblessed; |
|
|
| 180 |
#Scenario is: |
157 |
#Scenario is: |
| 181 |
#One shelf holds is 'If all unavailable'/2 |
158 |
#One shelf holds is 'If all unavailable'/2 |
| 182 |
#Item 1 homebranch library A is available |
159 |
#Item 1 homebranch library A is available |
|
Lines 190-200
AddReturn( $item1->{barcode} );
Link Here
|
| 190 |
$sth_insert_rule->execute( $hold_allowed_from_home_library ); |
167 |
$sth_insert_rule->execute( $hold_allowed_from_home_library ); |
| 191 |
|
168 |
|
| 192 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); |
169 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); |
| 193 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
170 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 194 |
is( $is, 0, "Hold allowed from home library + ReservesControlBranch=ItemHomeLibrary, One item is available at the same library, holdable = 1 available => the hold is not allowed at item level" ); |
171 |
is( $is, 0, "Hold allowed from home library + ReservesControlBranch=ItemHomeLibrary, One item is available at the same library, holdable = 1 available => the hold is not allowed at item level" ); |
| 195 |
|
172 |
|
| 196 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'PatronLibrary'); |
173 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'PatronLibrary'); |
| 197 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
174 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 198 |
is( $is, 0, "Hold allowed from home library + ReservesControlBranch=PatronLibrary, One item is available at the same library, holdable = 1 available => the hold is not allowed at item level" ); |
175 |
is( $is, 0, "Hold allowed from home library + ReservesControlBranch=PatronLibrary, One item is available at the same library, holdable = 1 available => the hold is not allowed at item level" ); |
| 199 |
} |
176 |
} |
| 200 |
|
177 |
|
|
Lines 203-213
AddReturn( $item1->{barcode} );
Link Here
|
| 203 |
$sth_insert_rule->execute( $hold_allowed_from_any_libraries ); |
180 |
$sth_insert_rule->execute( $hold_allowed_from_any_libraries ); |
| 204 |
|
181 |
|
| 205 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); |
182 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); |
| 206 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
183 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 207 |
is( $is, 0, "Hold allowed from any library + ReservesControlBranch=ItemHomeLibrary, One item is available at the same library, holdable = 1 available => the hold is not allowed at item level" ); |
184 |
is( $is, 0, "Hold allowed from any library + ReservesControlBranch=ItemHomeLibrary, One item is available at the same library, holdable = 1 available => the hold is not allowed at item level" ); |
| 208 |
|
185 |
|
| 209 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'PatronLibrary'); |
186 |
t::lib::Mocks::mock_preference('ReservesControlBranch', 'PatronLibrary'); |
| 210 |
$is = IsAvailableForItemLevelRequest( $item1, $borrower1); |
187 |
$is = IsAvailableForItemLevelRequest( $item1, $patron1); |
| 211 |
is( $is, 0, "Hold allowed from any library + ReservesControlBranch=PatronLibrary, One item is available at the same library, holdable = 1 available => the hold is not allowed at item level" ); |
188 |
is( $is, 0, "Hold allowed from any library + ReservesControlBranch=PatronLibrary, One item is available at the same library, holdable = 1 available => the hold is not allowed at item level" ); |
| 212 |
} |
189 |
} |
| 213 |
}; |
190 |
}; |
|
Lines 218-225
my $biblio = $builder->build({
Link Here
|
| 218 |
source => 'Biblio', |
195 |
source => 'Biblio', |
| 219 |
}); |
196 |
}); |
| 220 |
|
197 |
|
| 221 |
my $item3 = $builder->build({ |
198 |
my $item3 = $builder->build_object({ |
| 222 |
source => 'Item', |
199 |
class => 'Koha::Items', |
| 223 |
value => { |
200 |
value => { |
| 224 |
biblionumber => $biblio->{biblionumber}, |
201 |
biblionumber => $biblio->{biblionumber}, |
| 225 |
itemlost => 0, |
202 |
itemlost => 0, |
|
Lines 233-239
my $item3 = $builder->build({
Link Here
|
| 233 |
my $hold = $builder->build({ |
210 |
my $hold = $builder->build({ |
| 234 |
source => 'Reserve', |
211 |
source => 'Reserve', |
| 235 |
value =>{ |
212 |
value =>{ |
| 236 |
itemnumber => $item3->{itemnumber}, |
213 |
itemnumber => $item3->itemnumber, |
| 237 |
found => 'T' |
214 |
found => 'T' |
| 238 |
} |
215 |
} |
| 239 |
}); |
216 |
}); |
|
Lines 253-259
$rule = Koha::IssuingRule->new(
Link Here
|
| 253 |
); |
230 |
); |
| 254 |
$rule->store(); |
231 |
$rule->store(); |
| 255 |
|
232 |
|
| 256 |
$is = IsAvailableForItemLevelRequest( $item3, $borrower1); |
233 |
$is = IsAvailableForItemLevelRequest( $item3, $patron1); |
| 257 |
is( $is, 1, "Item can be held, items in transit are not available" ); |
234 |
is( $is, 1, "Item can be held, items in transit are not available" ); |
| 258 |
|
235 |
|
| 259 |
# Cleanup |
236 |
# Cleanup |