Bug 17642: Do not explode if no authorised value exist
[koha.git] / t / db_dependent / Acquisition.t
1 #!/usr/bin/perl
2
3 # This file is part of Koha.
4 #
5 # Koha is free software; you can redistribute it and/or modify it
6 # under the terms of the GNU General Public License as published by
7 # the Free Software Foundation; either version 3 of the License, or
8 # (at your option) any later version.
9 #
10 # Koha is distributed in the hope that it will be useful, but
11 # WITHOUT ANY WARRANTY; without even the implied warranty of
12 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 # GNU General Public License for more details.
14 #
15 # You should have received a copy of the GNU General Public License
16 # along with Koha; if not, see <http://www.gnu.org/licenses>.
17
18 use Modern::Perl;
19
20 use POSIX qw(strftime);
21
22 use Test::More tests => 65;
23 use Koha::Database;
24
25 BEGIN {
26     use_ok('C4::Acquisition');
27     use_ok('C4::Bookseller');
28     use_ok('C4::Biblio');
29     use_ok('C4::Budgets');
30     use_ok('C4::Bookseller');
31     use_ok('Koha::Acquisition::Order');
32     use_ok('Koha::Acquisition::Bookseller');
33 }
34
35 # Sub used for testing C4::Acquisition subs returning order(s):
36 #    GetOrdersByStatus, GetOrders, GetDeletedOrders, GetOrder etc.
37 # (\@test_missing_fields,\@test_extra_fields,\@test_different_fields,$test_nbr_fields) =
38 #  _check_fields_of_order ($exp_fields, $original_order_content, $order_to_check);
39 # params :
40 # $exp_fields             : arrayref whose elements are the keys we expect to find
41 # $original_order_content : hashref whose 2 keys str and num contains hashrefs
42 #                           containing content fields of the order created with Koha::Acquisition::Order
43 # $order_to_check         : hashref whose keys/values are the content of an order
44 #                           returned by the C4::Acquisition sub we are testing
45 # returns :
46 # \@test_missing_fields   : arrayref void if ok ; otherwise contains the list of
47 #                           fields missing in $order_to_check
48 # \@test_extra_fields     : arrayref void if ok ; otherwise contains the list of
49 #                           fields unexpected in $order_to_check
50 # \@test_different_fields : arrayref void if ok ; otherwise contains the list of
51 #                           fields which value is not the same in between $order_to_check and
52 # $test_nbr_fields        : contains the number of fields of $order_to_check
53
54 sub _check_fields_of_order {
55     my ( $exp_fields, $original_order_content, $order_to_check ) = @_;
56     my @test_missing_fields   = ();
57     my @test_extra_fields     = ();
58     my @test_different_fields = ();
59     my $test_nbr_fields       = scalar( keys %$order_to_check );
60     foreach my $field (@$exp_fields) {
61         push @test_missing_fields, $field
62           unless exists( $order_to_check->{$field} );
63     }
64     foreach my $field ( keys %$order_to_check ) {
65         push @test_extra_fields, $field
66           unless grep ( /^$field$/, @$exp_fields );
67     }
68     foreach my $field ( keys %{ $original_order_content->{str} } ) {
69         push @test_different_fields, $field
70           unless ( !exists $order_to_check->{$field} )
71           or ( $original_order_content->{str}->{$field} eq
72             $order_to_check->{$field} );
73     }
74     foreach my $field ( keys %{ $original_order_content->{num} } ) {
75         push @test_different_fields, $field
76           unless ( !exists $order_to_check->{$field} )
77           or ( $original_order_content->{num}->{$field} ==
78             $order_to_check->{$field} );
79     }
80     return (
81         \@test_missing_fields,   \@test_extra_fields,
82         \@test_different_fields, $test_nbr_fields
83     );
84 }
85
86 # Sub used for testing C4::Acquisition subs returning several orders
87 # (\@test_missing_fields,\@test_extra_fields,\@test_different_fields,\@test_nbr_fields) =
88 #   _check_fields_of_orders ($exp_fields, $original_orders_content, $orders_to_check)
89 sub _check_fields_of_orders {
90     my ( $exp_fields, $original_orders_content, $orders_to_check ) = @_;
91     my @test_missing_fields   = ();
92     my @test_extra_fields     = ();
93     my @test_different_fields = ();
94     my @test_nbr_fields       = ();
95     foreach my $order_to_check (@$orders_to_check) {
96         my $original_order_content =
97           ( grep { $_->{str}->{ordernumber} eq $order_to_check->{ordernumber} }
98               @$original_orders_content )[0];
99         my (
100             $t_missing_fields,   $t_extra_fields,
101             $t_different_fields, $t_nbr_fields
102           )
103           = _check_fields_of_order( $exp_fields, $original_order_content,
104             $order_to_check );
105         push @test_missing_fields,   @$t_missing_fields;
106         push @test_extra_fields,     @$t_extra_fields;
107         push @test_different_fields, @$t_different_fields;
108         push @test_nbr_fields,       $t_nbr_fields;
109     }
110     @test_missing_fields = keys %{ { map { $_ => 1 } @test_missing_fields } };
111     @test_extra_fields   = keys %{ { map { $_ => 1 } @test_extra_fields } };
112     @test_different_fields =
113       keys %{ { map { $_ => 1 } @test_different_fields } };
114     return (
115         \@test_missing_fields,   \@test_extra_fields,
116         \@test_different_fields, \@test_nbr_fields
117     );
118 }
119
120
121 my $schema = Koha::Database->new()->schema();
122 $schema->storage->txn_begin();
123
124 my $dbh = C4::Context->dbh;
125 $dbh->{RaiseError} = 1;
126
127 # Creating some orders
128 my $booksellerid = C4::Bookseller::AddBookseller(
129     {
130         name         => "my vendor",
131         address1     => "bookseller's address",
132         phone        => "0123456",
133         active       => 1,
134         deliverytime => 5,
135     }
136 );
137
138 my $booksellerinfo = Koha::Acquisition::Bookseller->fetch({ id => $booksellerid });
139
140 is( $booksellerinfo->{deliverytime},
141     5, 'set deliverytime when creating vendor (Bug 10556)' );
142
143 my ( $basket, $basketno );
144 ok(
145     $basketno = NewBasket( $booksellerid, 1 ),
146     "NewBasket(  $booksellerid , 1  ) returns $basketno"
147 );
148 ok( $basket = GetBasket($basketno), "GetBasket($basketno) returns $basket" );
149
150 my $bpid=AddBudgetPeriod({
151         budget_period_startdate => '2008-01-01'
152         , budget_period_enddate => '2008-12-31'
153         , budget_period_active  => 1
154         , budget_period_description    => "MAPERI"
155 });
156
157 my $budgetid = C4::Budgets::AddBudget(
158     {
159         budget_code => "budget_code_test_getordersbybib",
160         budget_name => "budget_name_test_getordersbybib",
161         budget_period_id => $bpid,
162     }
163 );
164 my $budget = C4::Budgets::GetBudget($budgetid);
165
166 my @ordernumbers;
167 my ( $biblionumber1, $biblioitemnumber1 ) = AddBiblio( MARC::Record->new, '' );
168 my ( $biblionumber2, $biblioitemnumber2 ) = AddBiblio( MARC::Record->new, '' );
169 my ( $biblionumber3, $biblioitemnumber3 ) = AddBiblio( MARC::Record->new, '' );
170 my ( $biblionumber4, $biblioitemnumber4 ) = AddBiblio( MARC::Record->new, '' );
171
172 # Prepare 5 orders, and make distinction beween fields to be tested with eq and with ==
173 # Ex : a price of 50.1 will be stored internally as 5.100000
174
175 my @order_content = (
176     {
177         str => {
178             basketno       => $basketno,
179             biblionumber   => $biblionumber1,
180             budget_id      => $budget->{budget_id},
181             uncertainprice => 0,
182             order_internalnote => "internal note",
183             order_vendornote   => "vendor note",
184             ordernumber => '',
185         },
186         num => {
187             quantity  => 24,
188             listprice => 50.121111,
189             ecost     => 38.15,
190             rrp       => 40.15,
191             discount  => 5.1111,
192             tax_rate   => 0.0515
193         }
194     },
195     {
196         str => {
197             basketno     => $basketno,
198             biblionumber => $biblionumber2,
199             budget_id    => $budget->{budget_id}
200         },
201         num => { quantity => 42 }
202     },
203     {
204         str => {
205             basketno       => $basketno,
206             biblionumber   => $biblionumber2,
207             budget_id      => $budget->{budget_id},
208             uncertainprice => 0,
209             order_internalnote => "internal note",
210             order_vendornote   => "vendor note"
211         },
212         num => {
213             quantity  => 4,
214             ecost     => 42.1,
215             rrp       => 42.1,
216             listprice => 10.1,
217             ecost     => 38.1,
218             rrp       => 11.0,
219             discount  => 5.1,
220             tax_rate   => 0.1
221         }
222     },
223     {
224         str => {
225             basketno     => $basketno,
226             biblionumber => $biblionumber3,
227             budget_id    => $budget->{budget_id},
228             order_internalnote => "internal note",
229             order_vendornote   => "vendor note"
230         },
231         num => {
232             quantity       => 4,
233             ecost          => 40,
234             rrp            => 42,
235             listprice      => 10,
236             ecost          => 38.15,
237             rrp            => 11.00,
238             discount       => 0,
239             uncertainprice => 0,
240             tax_rate        => 0
241         }
242     },
243     {
244         str => {
245             basketno     => $basketno,
246             biblionumber => $biblionumber4,
247             budget_id    => $budget->{budget_id},
248             order_internalnote => "internal note",
249             order_vendornote   => "vendor note"
250         },
251         num => {
252             quantity       => 1,
253             ecost          => 10,
254             rrp            => 10,
255             listprice      => 10,
256             ecost          => 10,
257             rrp            => 10,
258             discount       => 0,
259             uncertainprice => 0,
260             tax_rate        => 0
261         }
262     }
263 );
264
265 # Create 4 orders in database
266 for ( 0 .. 4 ) {
267     my %ocontent;
268     @ocontent{ keys %{ $order_content[$_]->{num} } } =
269       values %{ $order_content[$_]->{num} };
270     @ocontent{ keys %{ $order_content[$_]->{str} } } =
271       values %{ $order_content[$_]->{str} };
272     $ordernumbers[$_] = Koha::Acquisition::Order->new( \%ocontent )->insert->{ordernumber};
273     $order_content[$_]->{str}->{ordernumber} = $ordernumbers[$_];
274 }
275
276 DelOrder( $order_content[3]->{str}->{biblionumber}, $ordernumbers[3] );
277
278 my $invoiceid = AddInvoice(
279     invoicenumber => 'invoice',
280     booksellerid  => $booksellerid,
281     unknown       => "unknown"
282 );
283
284 my $invoice = GetInvoice( $invoiceid );
285
286 my ($datereceived, $new_ordernumber) = ModReceiveOrder(
287     {
288         biblionumber      => $biblionumber4,
289         order             => GetOrder( $ordernumbers[4] ),
290         quantityreceived  => 1,
291         invoice           => $invoice,
292         budget_id          => $order_content[4]->{str}->{budget_id},
293     }
294 );
295
296 my $search_orders = SearchOrders({
297     booksellerid => $booksellerid,
298     basketno     => $basketno
299 });
300 isa_ok( $search_orders, 'ARRAY' );
301 ok(
302     (
303         ( scalar @$search_orders == 4 )
304           and !grep ( $_->{ordernumber} eq $ordernumbers[3], @$search_orders )
305     ),
306     "SearchOrders only gets non-cancelled orders"
307 );
308
309 $search_orders = SearchOrders({
310     booksellerid => $booksellerid,
311     basketno     => $basketno,
312     pending      => 1
313 });
314 ok(
315     (
316         ( scalar @$search_orders == 3 ) and !grep ( (
317                      ( $_->{ordernumber} eq $ordernumbers[3] )
318                   or ( $_->{ordernumber} eq $ordernumbers[4] )
319             ),
320             @$search_orders )
321     ),
322     "SearchOrders with pending params gets only pending orders (bug 10723)"
323 );
324
325 $search_orders = SearchOrders({
326     booksellerid => $booksellerid,
327     basketno     => $basketno,
328     pending      => 1,
329     ordered      => 1,
330 });
331 is( scalar (@$search_orders), 0, "SearchOrders with pending and ordered params gets only pending ordered orders (bug 11170)" );
332
333 $search_orders = SearchOrders({
334     ordernumber => $ordernumbers[4]
335 });
336 is( scalar (@$search_orders), 1, "SearchOrders takes into account the ordernumber filter" );
337
338 $search_orders = SearchOrders({
339     biblionumber => $biblionumber4
340 });
341 is( scalar (@$search_orders), 1, "SearchOrders takes into account the biblionumber filter" );
342
343 $search_orders = SearchOrders({
344     biblionumber => $biblionumber4,
345     pending      => 1
346 });
347 is( scalar (@$search_orders), 0, "SearchOrders takes into account the biblionumber and pending filters" );
348
349 #
350 # Test GetBudgetByOrderNumber
351 #
352 ok( GetBudgetByOrderNumber( $ordernumbers[0] )->{'budget_id'} eq $budgetid,
353     "GetBudgetByOrderNumber returns expected budget" );
354
355 my @lateorders = GetLateOrders(0);
356 is( scalar grep ( $_->{basketno} eq $basketno, @lateorders ),
357     0, "GetLateOrders does not get orders from opened baskets" );
358 C4::Acquisition::CloseBasket($basketno);
359 @lateorders = GetLateOrders(0);
360 isnt( scalar grep ( $_->{basketno} eq $basketno, @lateorders ),
361     0, "GetLateOrders gets orders from closed baskets" );
362 ok( !grep ( $_->{ordernumber} eq $ordernumbers[3], @lateorders ),
363     "GetLateOrders does not gets cancelled orders" );
364 ok( !grep ( $_->{ordernumber} eq $ordernumbers[4], @lateorders ),
365     "GetLateOrders does not gets reveived orders" );
366
367 $search_orders = SearchOrders({
368     booksellerid => $booksellerid,
369     basketno     => $basketno,
370     pending      => 1,
371     ordered      => 1,
372 });
373 is( scalar (@$search_orders), 3, "SearchOrders with pending and ordered params gets only pending ordered orders. After closing the basket, orders are marked as 'ordered' (bug 11170)" );
374
375 #
376 # Test AddClaim
377 #
378
379 my $order = $lateorders[0];
380 AddClaim( $order->{ordernumber} );
381 my $neworder = GetOrder( $order->{ordernumber} );
382 is(
383     $neworder->{claimed_date},
384     strftime( "%Y-%m-%d", localtime(time) ),
385     "AddClaim : Check claimed_date"
386 );
387
388 my $order2 = GetOrder( $ordernumbers[1] );
389 $order2->{order_internalnote} = "my notes";
390 ( $datereceived, $new_ordernumber ) = ModReceiveOrder(
391     {
392         biblionumber     => $biblionumber2,
393         order            => $order2,
394         quantityreceived => 2,
395         invoice          => $invoice,
396     }
397 )
398 ;
399 $order2 = GetOrder( $ordernumbers[1] );
400 is( $order2->{'quantityreceived'},
401     0, 'Splitting up order did not receive any on original order' );
402 is( $order2->{'quantity'}, 40, '40 items on original order' );
403 is( $order2->{'budget_id'}, $budgetid,
404     'Budget on original order is unchanged' );
405 is( $order2->{order_internalnote}, "my notes",
406     'ModReceiveOrder and GetOrder deal with internal notes' );
407
408 $neworder = GetOrder($new_ordernumber);
409 is( $neworder->{'quantity'}, 2, '2 items on new order' );
410 is( $neworder->{'quantityreceived'},
411     2, 'Splitting up order received items on new order' );
412 is( $neworder->{'budget_id'}, $budgetid, 'Budget on new order is unchanged' );
413
414 is( $neworder->{ordernumber}, $new_ordernumber, 'Split: test ordernumber' );
415 is( $neworder->{parent_ordernumber}, $ordernumbers[1], 'Split: test parent_ordernumber' );
416
417 my $orders = GetHistory( ordernumber => $ordernumbers[1] );
418 is( scalar( @$orders ), 1, 'GetHistory with a given ordernumber returns 1 order' );
419 $orders = GetHistory( ordernumber => $ordernumbers[1], search_children_too => 1 );
420 is( scalar( @$orders ), 2, 'GetHistory with a given ordernumber and search_children_too set returns 2 orders' );
421
422 my $budgetid2 = C4::Budgets::AddBudget(
423     {
424         budget_code => "budget_code_test_modrecv",
425         budget_name => "budget_name_test_modrecv",
426     }
427 );
428
429 my $order3 = GetOrder( $ordernumbers[2] );
430 $order3->{order_internalnote} = "my other notes";
431 ( $datereceived, $new_ordernumber ) = ModReceiveOrder(
432     {
433         biblionumber     => $biblionumber2,
434         order            => $order3,
435         quantityreceived => 2,
436         invoice          => $invoice,
437         budget_id        => $budgetid2,
438     }
439 );
440
441 $order3 = GetOrder( $ordernumbers[2] );
442 is( $order3->{'quantityreceived'},
443     0, 'Splitting up order did not receive any on original order' );
444 is( $order3->{'quantity'}, 2, '2 items on original order' );
445 is( $order3->{'budget_id'}, $budgetid,
446     'Budget on original order is unchanged' );
447 is( $order3->{order_internalnote}, "my other notes",
448     'ModReceiveOrder and GetOrder deal with notes' );
449
450 $neworder = GetOrder($new_ordernumber);
451 is( $neworder->{'quantity'}, 2, '2 items on new order' );
452 is( $neworder->{'quantityreceived'},
453     2, 'Splitting up order received items on new order' );
454 is( $neworder->{'budget_id'}, $budgetid2, 'Budget on new order is changed' );
455
456 $order3 = GetOrder( $ordernumbers[2] );
457 $order3->{order_internalnote} = "my third notes";
458 ( $datereceived, $new_ordernumber ) = ModReceiveOrder(
459     {
460         biblionumber     => $biblionumber2,
461         order            => $order3,
462         quantityreceived => 2,
463         invoice          => $invoice,
464         budget_id        => $budgetid2,
465     }
466 );
467
468 $order3 = GetOrder( $ordernumbers[2] );
469 is( $order3->{'quantityreceived'}, 2,          'Order not split up' );
470 is( $order3->{'quantity'},         2,          '2 items on order' );
471 is( $order3->{'budget_id'},        $budgetid2, 'Budget has changed' );
472 is( $order3->{order_internalnote}, "my third notes", 'ModReceiveOrder and GetOrder deal with notes' );
473
474 my $nonexistent_order = GetOrder();
475 is( $nonexistent_order, undef, 'GetOrder returns undef if no ordernumber is given' );
476 $nonexistent_order = GetOrder( 424242424242 );
477 is( $nonexistent_order, undef, 'GetOrder returns undef if a nonexistent ordernumber is given' );
478
479 # Tests for DelOrder
480 my $order1 = GetOrder($ordernumbers[0]);
481 my $error = DelOrder($order1->{biblionumber}, $order1->{ordernumber});
482 ok((not defined $error), "DelOrder does not fail");
483 $order1 = GetOrder($order1->{ordernumber});
484 ok((defined $order1->{datecancellationprinted}), "order is cancelled");
485 ok((not defined $order1->{cancellationreason}), "order has no cancellation reason");
486 ok((defined GetBiblio($order1->{biblionumber})), "biblio still exists");
487
488 $order2 = GetOrder($ordernumbers[1]);
489 $error = DelOrder($order2->{biblionumber}, $order2->{ordernumber}, 1);
490 ok((not defined $error), "DelOrder does not fail");
491 $order2 = GetOrder($order2->{ordernumber});
492 ok((defined $order2->{datecancellationprinted}), "order is cancelled");
493 ok((not defined $order2->{cancellationreason}), "order has no cancellation reason");
494 ok((not defined GetBiblio($order2->{biblionumber})), "biblio does not exist anymore");
495
496 my $order4 = GetOrder($ordernumbers[3]);
497 $error = DelOrder($order4->{biblionumber}, $order4->{ordernumber}, 1, "foobar");
498 ok((not defined $error), "DelOrder does not fail");
499 $order4 = GetOrder($order4->{ordernumber});
500 ok((defined $order4->{datecancellationprinted}), "order is cancelled");
501 ok(($order4->{cancellationreason} eq "foobar"), "order has cancellation reason \"foobar\"");
502 ok((not defined GetBiblio($order4->{biblionumber})), "biblio does not exist anymore");
503 # End of tests for DelOrder
504
505 subtest 'ModOrder' => sub {
506     plan tests => 1;
507     ModOrder( { ordernumber => $order1->{ordernumber}, unitprice => 42 } );
508     my $order = GetOrder( $order1->{ordernumber} );
509     is( int($order->{unitprice}), 42, 'ModOrder should work even if biblionumber if not passed');
510 };
511
512 # Budget reports
513 my $all_count = scalar GetBudgetsReport();
514 ok($all_count >= 1, "GetBudgetReport OK");
515
516 my $active_count = scalar GetBudgetsReport(1);
517 ok($active_count >= 1 , "GetBudgetsReport(1) OK");
518
519 is($all_count, scalar GetBudgetsReport(), "GetBudgetReport returns inactive budget period acquisitions.");
520 ok($active_count >= scalar GetBudgetsReport(1), "GetBudgetReport doesn't return inactive budget period acquisitions.");
521
522 $schema->storage->txn_rollback();