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

(-)a/Koha/Item.pm (-4 / +12 lines)
Lines 596-612 sub get_transfers { Link Here
596
596
597
=head3 last_returned_by
597
=head3 last_returned_by
598
598
599
Gets the last borrower to return an item.
599
Gets and sets the last patron to return an item.
600
600
601
$last_returned_by = $item->last_returned_by();
601
Accepts a patron's id (borrowernumber) and returns Koha::Patron objects
602
603
$item->last_returned_by( $borrowernumber );
604
605
my $patron = $item->last_returned_by();
602
606
603
=cut
607
=cut
604
608
605
sub last_returned_by {
609
sub last_returned_by {
606
    my ( $self ) = @_;
610
    my ( $self, $borrowernumber ) = @_;
611
    if ( $borrowernumber ) {
612
        $self->_result->update_or_create_related('last_returned_by',
613
            { borrowernumber => $borrowernumber, itemnumber => $self->itemnumber } );
614
    }
607
    my $rs = $self->_result->last_returned_by;
615
    my $rs = $self->_result->last_returned_by;
608
    return unless $rs;
616
    return unless $rs;
609
    return Koha::Patron->_new_from_dbic($rs);
617
    return Koha::Patron->_new_from_dbic($rs->borrowernumber);
610
}
618
}
611
619
612
=head3 can_article_request
620
=head3 can_article_request
(-)a/t/db_dependent/Circulation/StoreLastBorrower.t (-2 / +1 lines)
Lines 98-104 subtest 'Test StoreLastBorrower' => sub { Link Here
98
    );
98
    );
99
    $patron_object = Koha::Patrons->find( $patron->{borrowernumber} );
99
    $patron_object = Koha::Patrons->find( $patron->{borrowernumber} );
100
100
101
    $item->last_returned_by($patron_object);
101
    $item->last_returned_by($patron_object->borrowernumber);
102
    $item = $item->get_from_storage;
102
    $item = $item->get_from_storage;
103
    my $patron_object2 = $item->last_returned_by();
103
    my $patron_object2 = $item->last_returned_by();
104
    is( $patron_object->id, $patron_object2->id,
104
    is( $patron_object->id, $patron_object2->id,
105
- 

Return to bug 32894