View | Details | Raw Unified | Return to bug 23463
Collapse All | Expand All

(-)a/t/db_dependent/Circulation/IsItemIssued.t (+2 lines)
Lines 76-81 is( Link Here
76
AddReturn($item->barcode, $library->{branchcode});
76
AddReturn($item->barcode, $library->{branchcode});
77
is ( IsItemIssued( $item->itemnumber ), 0, "item has been returned" );
77
is ( IsItemIssued( $item->itemnumber ), 0, "item has been returned" );
78
78
79
$item->discard_changes; # FIXME We should not need that
80
                        # If we do not, $self->checkout in safe_to_delete will not know the item has been checked out
79
is(
81
is(
80
    $item->safe_delete,
82
    $item->safe_delete,
81
    1,
83
    1,
(-)a/t/db_dependent/Items_DelItemCheck.t (-1 / +1 lines)
Lines 117-122 AddReturn( $item->barcode, $branch->{branchcode} ); Link Here
117
t::lib::Mocks::mock_preference('IndependentBranches', 1);
117
t::lib::Mocks::mock_preference('IndependentBranches', 1);
118
$item->set( { homebranch => $branch2->{branchcode}, holdingbranch => $branch2->{branchcode} })->store;
118
$item->set( { homebranch => $branch2->{branchcode}, holdingbranch => $branch2->{branchcode} })->store;
119
119
120
$item->discard_changes;
120
is(
121
is(
121
    $item->safe_to_delete,
122
    $item->safe_to_delete,
122
    'not_same_branch',
123
    'not_same_branch',
123
- 

Return to bug 23463