Attachment #72333 for bug #19532

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





=head1 API

=head2 Internal Methods

=cut


    return $self->{_item};
}

=head3 patron

Returns the related Koha::Patron object for this recall

=cut

sub patron {

    my ($self) = @_;

    $self->{_patron} ||= Koha::Patrons->find( $self->borrowernumber() );

    return $self->{_patron};
}

=head3 library

Returns the related Koha::Library object for this recall

=cut

sub library {
    my ($self) = @_;

    $self->{_library} ||= Koha::Libraries->find( $self->branchcode() );

    return $self->{_library};
}

=head3 checkout

Lines 30-36 Koha::Recalls - Koha Recalls Object Set class Link Here

(-)a/Koha/Recalls.pm (-1 / +1 lines)
30		30
31	=head1 API	31	=head1 API
32		32
33	=head2 Class Methods	33	=head2 Internal Methods
34		34
35	=cut	35	=cut
36		36

Lines 190-196 if ( $query->param('reserve_id') ) { Link Here

(-)a/circ/returns.pl (-1 / +1 lines)
190		190
191	if ( $query->param('recall_id') ){	191	if ( $query->param('recall_id') ){
192	my $recall = Koha::Recalls->find(scalar $query->param('recall_id'));	192	my $recall = Koha::Recalls->find(scalar $query->param('recall_id'));
193	my $recall_borrower = $recall->borrower;	193	my $recall_borrower = $recall->patron;
194	my $item = Koha::Items->find($recall->itemnumber);	194	my $item = Koha::Items->find($recall->itemnumber);
195	my $biblio = Koha::Biblios->find($item->biblionumber);	195	my $biblio = Koha::Biblios->find($item->biblionumber);
196		196

Lines 26-32 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/includes/recalls-reports.inc (-2 / +2 lines)
26	</td>	26	</td>
27	<td class="recall-borrower">	27	<td class="recall-borrower">
28	<a class="recall-borrower" href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% recall.borrowernumber %]">	28	<a class="recall-borrower" href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% recall.borrowernumber %]">
29	[% recall.borrower.firstname %] [% recall.borrower.surname %] ([% recall.borrowernumber %])	29	[% recall.patron.firstname %] [% recall.patron.surname %] ([% recall.borrowernumber %])
30	</a>	30	</a>
31	</td>	31	</td>
32	<td class="recall-title">	32	<td class="recall-title">
Lines 59-65 Link Here
59	</td>	59	</td>
60		60
61	<td class="recall-branch">	61	<td class="recall-branch">
62	[% recall.branch.branchname %]	62	[% recall.library.branchname %]
63	</td>	63	</td>
64		64
65	<td class="recall-status">	65	<td class="recall-status">

Lines 45-51 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/includes/recalls.inc (-1 / +1 lines)
45	</td>	45	</td>
46		46
47	<td class="recall-branch">	47	<td class="recall-branch">
48	[% recall.branch.branchname %]	48	[% recall.library.branchname %]
49	</td>	49	</td>
50		50
51	<td class="recall-status">	51	<td class="recall-status">

Lines 58-64 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/recalls_overdue.tt (-2 / +2 lines)
58	</td>	58	</td>
59	<td class="recall-borrower">	59	<td class="recall-borrower">
60	<a class="recall-borrower" href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% recall.borrowernumber %]">	60	<a class="recall-borrower" href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% recall.borrowernumber %]">
61	[% recall.borrower.firstname %] [% recall.borrower.surname %] ([% recall.borrowernumber %])	61	[% recall.patron.firstname %] [% recall.patron.surname %] ([% recall.borrowernumber %])
62	</a>	62	</a>
63	</td>	63	</td>
64	<td class="recall-title">	64	<td class="recall-title">
Lines 83-89 Link Here
83	</a>	83	</a>
84	</td>	84	</td>
85	<td class="recall-branch">	85	<td class="recall-branch">
86	[% recall.branch.branchname %]	86	[% recall.library.branchname %]
87	</td>	87	</td>
88	<td class="recall-cancel">	88	<td class="recall-cancel">
89	<a class="btn btn-xs btn-default" id="cancel_recall" data-id="[% recall.recall_id %]"><i class="fa fa-times"></i> Cancel</a>	89	<a class="btn btn-xs btn-default" id="cancel_recall" data-id="[% recall.recall_id %]"><i class="fa fa-times"></i> Cancel</a>




                                        </td>
                                        <td>
                                            <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% recall.borrowernumber %]">[% recall.borrower.firstname %] [% recall.borrower.surname %]</a>
                                            [% IF ( recall.patron.phone ) %]<br />[% recall.patron.phone %][% END %]
                                            [% IF ( recall.patron.email ) %]<br /><a href="mailto:[% recall.patron.email %]?subject=Recall waiting: [% recall.biblio.title %]">[% recall.patron.email %]</a>[% END %]
                                        </td>
                                        <td>[% recall.library.branchname %]</td>
                                        <td>
                                            <form action="/cgi-bin/koha/circ/recalls_waiting.pl" method="post">
                                                <input type="hidden" name="recall_id" value="[% recall.recall_id %]">

                                            <br><i>Barcode: [% recall.item.barcode %]</i>
                                        </td>
                                        <td>
                                            <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% recall.borrowernumber %]">[% recall.patron.firstname %] [% recall.patron.surname %]</a>
                                            [% IF ( recall.patron.phone ) %]<br />[% recall.patron.phone %][% END %]
                                            [% IF ( recall.patron.email ) %]<br /><a href="mailto:[% recall.patron.email %]?subject=Recall waiting: [% recall.biblio.title %]">[% recall.patron.email %]</a>[% END %]
                                        </td>
                                        <td>[% recall.library.branchname %]</td>
                                        <td>
                                            <form action="/cgi-bin/koha/circ/recalls_waiting.pl" method="post">
                                                <input type="hidden" name="recall_id" value="[% recall.recall_id %]">

Lines 499-505 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt (-1 / +1 lines)
499	<li class="error">Patron's address is in doubt</li>	499	<li class="error">Patron's address is in doubt</li>
500	[% END %]	500	[% END %]
501		501
502	<h4><strong>Wait for pickup at</strong> [% recall.branch.branchname %]</h4>	502	<h4><strong>Wait for pickup at</strong> [% recall.library.branchname %]</h4>
503		503
504	<input type="hidden" name="itemnumber" value="[% recall.itemnumber %]" />	504	<input type="hidden" name="itemnumber" value="[% recall.itemnumber %]" />
505	<input type="hidden" name="borrowernumber" value="[% patron.borrowernumber %]" />	505	<input type="hidden" name="borrowernumber" value="[% patron.borrowernumber %]" />

Lines 85-91 Link Here

(-)a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-recalls.tt (-1 / +1 lines)
85	</td>	85	</td>
86	<td class="branch">	86	<td class="branch">
87	<span class="tdlabel">Pick up location:</span>	87	<span class="tdlabel">Pick up location:</span>
88	[% RECALL.branch.branchname %]	88	[% RECALL.library.branchname %]
89	</td>	89	</td>
90	<td class="status">	90	<td class="status">
91	<span class="tdlabel">Status:</span>	91	<span class="tdlabel">Status:</span>

Lines 835-841 Using this account is not recommended because some parts of Koha will not functi Link Here

(-)a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt (-1 / +1 lines)
835	</td>	835	</td>
836	<td class="branch">	836	<td class="branch">
837	<span class="tdlabel">Pick up location:</span>	837	<span class="tdlabel">Pick up location:</span>
838	[% RECALL.branch.branchname %]	838	[% RECALL.library.branchname %]
839	</td>	839	</td>
840	<td class="status">	840	<td class="status">
841	<span class="tdlabel">Status:</span>	841	<span class="tdlabel">Status:</span>





is( $biblio->{biblionumber}, $recall->biblio->biblionumber, 'Can access biblio from recall' );

is( $patron->{borrowernumber}, $recall->patron->borrowernumber, 'Can access borrower from recall' );

is( $item->{itemnumber}, $recall->item->itemnumber, 'Can access item from recall' );

is( $library->{branchcode}, $recall->library->branchcode, 'Can access branch from recall' );

is( $checkout->{issue_id}, $recall->checkout->issue_id, 'Can access checkout from recall' );

- 

Return to bug 19532

Lines 67-77 is( $recall->has_expired, 1, 'Recall has expired' ); Link Here

(-)a/t/db_dependent/Koha/Recalls.t (-3 / +2 lines)
67		67
68	is( $biblio->{biblionumber}, $recall->biblio->biblionumber, 'Can access biblio from recall' );	68	is( $biblio->{biblionumber}, $recall->biblio->biblionumber, 'Can access biblio from recall' );
69		69
70	is( $patron->{borrowernumber}, $recall->borrower->borrowernumber, 'Can access borrower from recall' );	70	is( $patron->{borrowernumber}, $recall->patron->borrowernumber, 'Can access borrower from recall' );
71		71
72	is( $item->{itemnumber}, $recall->item->itemnumber, 'Can access item from recall' );	72	is( $item->{itemnumber}, $recall->item->itemnumber, 'Can access item from recall' );
73		73
74	is( $library->{branchcode}, $recall->branch->branchcode, 'Can access branch from recall' );	74	is( $library->{branchcode}, $recall->library->branchcode, 'Can access branch from recall' );
75		75
76	is( $checkout->{issue_id}, $recall->checkout->issue_id, 'Can access checkout from recall' );	76	is( $checkout->{issue_id}, $recall->checkout->issue_id, 'Can access checkout from recall' );
77		77
78	-

Lines 29-35 Koha::Recall - Koha Recall Object class Link Here

(-)a/Koha/Recall.pm (-9 / +9 lines)
29		29
30	=head1 API	30	=head1 API
31		31
32	=head2 Class Methods	32	=head2 Internal Methods
33		33
34	=cut	34	=cut
35		35
Lines 70-102 sub item { Link Here
70	return $self->{_item};	70	return $self->{_item};
71	}	71	}
72		72
73	=head3 borrower	73	=head3 patron
74		74
75	Returns the related Koha::Patron object for this recall	75	Returns the related Koha::Patron object for this recall
76		76
77	=cut	77	=cut
78		78
79	sub borrower {	79	sub patron {
80		80
81	my ($self) = @_;	81	my ($self) = @_;
82		82
83	$self->{_borrower} \|\|= Koha::Patrons->find( $self->borrowernumber() );	83	$self->{_patron} \|\|= Koha::Patrons->find( $self->borrowernumber() );
84		84
85	return $self->{_borrower};	85	return $self->{_patron};
86	}	86	}
87		87
88	=head3 branch	88	=head3 library
89		89
90	Returns the related Koha::Library object for this recall	90	Returns the related Koha::Library object for this recall
91		91
92	=cut	92	=cut
93		93
94	sub branch {	94	sub library {
95	my ($self) = @_;	95	my ($self) = @_;
96		96
97	$self->{_branch} \|\|= Koha::Libraries->find( $self->branchcode() );	97	$self->{_library} \|\|= Koha::Libraries->find( $self->branchcode() );
98		98
99	return $self->{_branch};	99	return $self->{_library};
100	}	100	}
101		101
102	=head3 checkout	102	=head3 checkout