| 
      
            Lines 326-332
          ok(
      
      
        Link Here
      
     | 
  
        
          | 326 | 
          }  | 
          326 | 
          }  | 
        
        
          | 327 | 
           | 
          327 | 
           | 
        
        
          | 328 | 
          subtest 'AllowHoldsOnDamagedItems' => sub { | 
          328 | 
          subtest 'AllowHoldsOnDamagedItems' => sub { | 
        
          
            
              | 329 | 
                  plan tests => 5;  | 
              329 | 
                  plan tests => 7;  | 
            
        
          | 330 | 
           | 
          330 | 
           | 
        
        
          | 331 | 
              my $library    = $builder->build_object( { class => 'Koha::Libraries' } ); | 
          331 | 
              my $library    = $builder->build_object( { class => 'Koha::Libraries' } ); | 
        
        
          | 332 | 
              my $biblio     = $builder->build_sample_biblio;  | 
          332 | 
              my $biblio     = $builder->build_sample_biblio;  | 
        
  
    | 
      
            Lines 340-352
          subtest 'AllowHoldsOnDamagedItems' => sub {
      
      
        Link Here
      
     | 
  
        
          | 340 | 
                  }  | 
          340 | 
                  }  | 
        
        
          | 341 | 
              );  | 
          341 | 
              );  | 
        
        
          | 342 | 
           | 
          342 | 
           | 
        
          
            
              | 343 | 
                  my $itemnumber = $builder->build_sample_item(  | 
              343 | 
                  my $item = $builder->build_sample_item(  | 
            
        
          | 344 | 
                  { | 
          344 | 
                  { | 
        
        
          | 345 | 
                      library      => $library->branchcode,  | 
          345 | 
                      library      => $library->branchcode,  | 
        
        
          | 346 | 
                      biblionumber => $biblio->biblionumber,  | 
          346 | 
                      biblionumber => $biblio->biblionumber,  | 
        
        
          | 347 | 
                      damaged      => 1  | 
          347 | 
                      damaged      => 1  | 
        
        
          | 348 | 
                  }  | 
          348 | 
                  }  | 
        
          
            
              | 349 | 
                  )->itemnumber;  | 
              349 | 
                  );  | 
            
            
               | 
               | 
              350 | 
                  my $itemnumber = $item->itemnumber;  | 
            
        
          | 350 | 
              t::lib::Mocks::mock_preference( 'AllowHoldsOnDamagedItems', 1 );  | 
          351 | 
              t::lib::Mocks::mock_preference( 'AllowHoldsOnDamagedItems', 1 );  | 
        
        
          | 351 | 
              is(  | 
          352 | 
              is(  | 
        
        
          | 352 | 
                  CanItemBeReserved( $patron->borrowernumber, $itemnumber )->{status}, | 
          353 | 
                  CanItemBeReserved( $patron->borrowernumber, $itemnumber )->{status}, | 
        
  
    | 
      
            Lines 381-386
          subtest 'AllowHoldsOnDamagedItems' => sub {
      
      
        Link Here
      
     | 
  
        
          | 381 | 
                  !defined( ( CheckReserves($itemnumber) )[1] ), "Hold cannot be trapped for damaged item with AllowHoldsOnDamagedItems disabled"  | 
          382 | 
                  !defined( ( CheckReserves($itemnumber) )[1] ), "Hold cannot be trapped for damaged item with AllowHoldsOnDamagedItems disabled"  | 
        
        
          | 382 | 
              );  | 
          383 | 
              );  | 
        
        
          | 383 | 
           | 
          384 | 
           | 
        
            
               | 
               | 
              385 | 
               | 
            
            
              | 386 | 
               | 
            
            
              | 387 | 
                  t::lib::Mocks::mock_preference( 'AllowHoldsOnDamagedItems', 0 );  | 
            
            
              | 388 | 
                  $hold = Koha::Hold->new(  | 
            
            
              | 389 | 
                      { | 
            
            
              | 390 | 
                          borrowernumber => $patron->borrowernumber,  | 
            
            
              | 391 | 
                          itemnumber     => $itemnumber,  | 
            
            
              | 392 | 
                          biblionumber   => $biblio->biblionumber,  | 
            
            
              | 393 | 
                      }  | 
            
            
              | 394 | 
                  )->store();  | 
            
            
              | 395 | 
               | 
            
            
              | 396 | 
                  t::lib::Mocks::mock_userenv({ branchcode => $library->branchcode }); | 
            
            
              | 397 | 
                  my $checkout = AddIssue( $patron->unblessed, $item->barcode );  | 
            
            
              | 398 | 
                  ok( $checkout, 'AddIssue should have checkout the item' );  | 
            
            
              | 399 | 
                  is( Koha::Holds->find($hold->reserve_id), undef, 'AddIssue should have filled the hold' );  | 
            
            
              | 400 | 
               | 
            
        
          | 384 | 
          };  | 
          401 | 
          };  | 
        
        
          | 385 | 
           | 
          402 | 
           | 
        
        
          | 386 | 
          # Items that are not for loan, but holdable should not be trapped until they are available for loan  | 
          403 | 
          # Items that are not for loan, but holdable should not be trapped until they are available for loan  | 
        
            
              | 387 | 
              -   | 
               | 
               |