Attachment #71487 for bug #18789

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

Lines 609-615 if ( $patron ) { Link Here

(-)a/circ/circulation.pl (-1 lines)
609	categoryname => $patron->category->description,	609	categoryname => $patron->category->description,
610	expiry => $patron->dateexpiry,	610	expiry => $patron->dateexpiry,
611	is_child => ( $patron->category->category_type eq 'C' ),	611	is_child => ( $patron->category->category_type eq 'C' ),
612	picture => ( $patron->image ? 1 : 0 ),
613	);	612	);
614	}	613	}
615		614

Lines 7-13 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc (-2 / +2 lines)
7	<![endif]-->	7	<![endif]-->
8	<ul class="patronbriefinfo">	8	<ul class="patronbriefinfo">
9	[% IF ( patronimages ) %]	9	[% IF ( patronimages ) %]
10	[% IF ( picture ) %]	10	[% IF ( patron.image ) %]
11	<li>	11	<li>
12	<img src="/cgi-bin/koha/members/patronimage.pl?borrowernumber=[% patron.borrowernumber \| uri %]" id="patronimage" alt="[% patron.firstname \| html %] [% patron.surname \| html %] ([% patron.cardnumber \| html %])" />	12	<img src="/cgi-bin/koha/members/patronimage.pl?borrowernumber=[% patron.borrowernumber \| uri %]" id="patronimage" alt="[% patron.firstname \| html %] [% patron.surname \| html %] ([% patron.cardnumber \| html %])" />
13	</li>	13	</li>
Lines 97-103 Link Here
97	[% IF ( logview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/tools/viewlog.pl?do_it=1&modules=MEMBERS&modules=circulation&object=[% patron.borrowernumber %]&src=circ">Modification log</a></li>	97	[% IF ( logview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/tools/viewlog.pl?do_it=1&modules=MEMBERS&modules=circulation&object=[% patron.borrowernumber %]&src=circ">Modification log</a></li>
98	[% END %]	98	[% END %]
99	[% IF CAN_user_borrowers_edit_borrowers %]	99	[% IF CAN_user_borrowers_edit_borrowers %]
100	[% IF ( sentnotices ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/notices.pl?borrowernumber=[% patron.borrowernumber %]">Notices</a></li>	100	[% IF ( sentnotices ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/notices.pl?borrowernumber=[% patron.borrowernumber %]">Notices</a></li>
101	[% END %]	101	[% END %]
102	[% IF CAN_user_borrowers_edit_borrowers %]	102	[% IF CAN_user_borrowers_edit_borrowers %]
103	[% IF ( statisticsview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/statistics.pl?borrowernumber=[% patron.borrowernumber %]">Statistics</a></li>	103	[% IF ( statisticsview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/statistics.pl?borrowernumber=[% patron.borrowernumber %]">Statistics</a></li>




        ],
        "bPaginate": false
    }));
    [% IF ( patron.image ) %]
    // new YAHOO.widget.Button("delpicture");   // FIXME: formatting mismatch between YUI and normal button
	$('#delpicture').click(function(){
		 return confirm(_("Are you sure you want to delete this patron image? This cannot be undone."));

    [% IF ( CAN_user_tools_batch_upload_patron_images ) %]
        <form method="post" action="/cgi-bin/koha/tools/picture-upload.pl" enctype="multipart/form-data">
            <fieldset class="brief">
            [% IF ( patron.image ) %]
                <legend>Manage patron image</legend>
                <div class="hint">To update the image for [% patron.title | html %] [% patron.surname | html %], select a new image file and click 'Upload.' <br />Click the 'Delete' button to remove the current image.
            [% ELSE %]

                    <input type="hidden" name="csrf_token" value="[% csrf_token %]" />
                    <input type="submit" value="Upload" class="submit" />
                    <input name="op" type="hidden" value="Upload" />
                    [% IF ( patron.image ) %]<a id="delpicture" href="/cgi-bin/koha/tools/picture-upload.pl?op=Delete&amp;borrowernumber=[% patron.borrowernumber %]&amp;csrf_token=[% csrf_token %]" class="delete">Delete</a>[% END %]
                </fieldset>
            </fieldset>
        </form>

    <li><span class="label">Borrowernumber: </span> [% patron.borrowernumber %]</li>
    <li><span class="label">Category: </span>[% patron.category.description %] ([% patron.categorycode %])</li>
    <li><span class="label">Registration date: </span>[% patron.dateenrolled | $KohaDates %]</li>

    <li><span class="label">Expiration date: </span>
    [% IF ( was_renewed ) %]
            <strong class="reregistrinfo">[% patron.dateexpiry | $KohaDates %]</strong>

    <li><span class="label">First name: </span>[% patron.altcontactfirstname | html %]</li>
    <li><span class="label">Address: </span>[% patron.altcontactaddress1 | html %]</li>
    <li><span class="label">Address 2: </span>[% patron.altcontactaddress2 | html %]</li>
    <li><span class="label">City: </span>[% patron.altcontactaddress3 | html %]</li>
    [% IF ( patron.altcontactstate ) %]<li><span class="label">State: </span>[% patron.altcontactstate | html %]</li>[% END %]
    <li><span class="label">ZIP/Postal code: </span>[% patron.altcontactzipcode | html %]</li>
    [% IF ( patron.altcontactcountry ) %]<li><span class="label">Country: </span>[% patron.altcontactcountry | html %]</li>[% END %]
    [% IF ( patron.altcontactphone ) %]<li><span class="label">Phone: </span><a href="tel:[% patron.altcontactphone | url %]">[% patron.altcontactphone | html %]</a></li>[% END %]
    </ol></div>
</div>

Lines 99-106 foreach my $accountline ( @{$accts}) { Link Here

(-)a/members/boraccount.pl (-2 lines)
99		99
100	$template->param( adultborrower => 1 ) if ( $patron->category->category_type =~ /^(A\|I)$/ );	100	$template->param( adultborrower => 1 ) if ( $patron->category->category_type =~ /^(A\|I)$/ );
101		101
102	$template->param( picture => 1 ) if $patron->image;
103
104	if (C4::Context->preference('ExtendedPatronAttributes')) {	102	if (C4::Context->preference('ExtendedPatronAttributes')) {
105	my $attributes = GetBorrowerAttributes($borrowernumber);	103	my $attributes = GetBorrowerAttributes($borrowernumber);
106	$template->param(	104	$template->param(

Lines 109-115 my $op = $input->param('op') \|\| 'delete_confirm'; Link Here

(-)a/members/deletemem.pl (-1 lines)
109	my $dbh = C4::Context->dbh;	109	my $dbh = C4::Context->dbh;
110	my $is_guarantor = $dbh->selectrow_array("SELECT COUNT(*) FROM borrowers WHERE guarantorid=?", undef, $member);	110	my $is_guarantor = $dbh->selectrow_array("SELECT COUNT(*) FROM borrowers WHERE guarantorid=?", undef, $member);
111	if ( $op eq 'delete_confirm' or $countissues > 0 or $flags->{'CHARGES'} or $is_guarantor or $deletelocal == 0) {	111	if ( $op eq 'delete_confirm' or $countissues > 0 or $flags->{'CHARGES'} or $is_guarantor or $deletelocal == 0) {
112	$template->param( picture => 1 ) if $patron->image;
113		112
114	$template->param( adultborrower => 1 ) if $patron->category->category_type =~ /^(A\|I)$/;	113	$template->param( adultborrower => 1 ) if $patron->category->category_type =~ /^(A\|I)$/;
115		114

Lines 103-110 my @validated_discharges = Koha::Patron::Discharge::get_validated({ Link Here

(-)a/members/discharge.pl (-2 lines)
103	borrowernumber => $borrowernumber,	103	borrowernumber => $borrowernumber,
104	});	104	});
105		105
106	$template->param( picture => 1 ) if $patron->image;
107
108	$template->param(	106	$template->param(
109	patron => $patron,	107	patron => $patron,
110	can_be_discharged => $can_be_discharged,	108	can_be_discharged => $can_be_discharged,

Lines 115-121 else { Link Here

(-)a/members/files.pl (-1 lines)
115	);	115	);
116	}	116	}
117		117
118	$template->param( picture => 1 ) if $patron->image;
119		118
120	$template->param( adultborrower => 1 )	119	$template->param( adultborrower => 1 )
121	if ( $patron_category->category_type eq 'A' \|\| $patron_category->category_type eq 'I' );	120	if ( $patron_category->category_type eq 'A' \|\| $patron_category->category_type eq 'I' );




output_and_exit_if_error( $input, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );

# Get supporting cast
my ( $branch, $category, $houseboundprofile, $visit );
if ( $patron ) { # FIXME This test is not needed - output_and_exit_if_error handles it
    $patron_image = $patron->image;
    $category = Koha::Patron::Categories->new->find($patron->categorycode);
    $houseboundprofile = $patron->housebound_profile;
}


$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' || $category->category_type eq 'I' );
$template->param(
    picture            => $patron_image,
    housebound_profile => $houseboundprofile,
    visit              => $houseboundvisit,
    messages           => \@messages,

Lines 87-93 if ($add){ Link Here

(-)a/members/mancredit.pl (-1 lines)
87	}	87	}
88		88
89	$template->param( adultborrower => 1 ) if ( $patron->category->category_type =~ /^(A\|I)$/ );	89	$template->param( adultborrower => 1 ) if ( $patron->category->category_type =~ /^(A\|I)$/ );
90	$template->param( picture => 1 ) if $patron->image;
91		90
92	if (C4::Context->preference('ExtendedPatronAttributes')) {	91	if (C4::Context->preference('ExtendedPatronAttributes')) {
93	my $attributes = GetBorrowerAttributes($borrowernumber);	92	my $attributes = GetBorrowerAttributes($borrowernumber);

Lines 115-121 if ($add){ Link Here

(-)a/members/maninvoice.pl (-1 lines)
115	}	115	}
116		116
117	$template->param( adultborrower => 1 ) if ( $patron->category->category_type =~ /^(A\|I)$/ );	117	$template->param( adultborrower => 1 ) if ( $patron->category->category_type =~ /^(A\|I)$/ );
118	$template->param( picture => 1 ) if $patron->image;
119		118
120	if (C4::Context->preference('ExtendedPatronAttributes')) {	119	if (C4::Context->preference('ExtendedPatronAttributes')) {
121	my $attributes = GetBorrowerAttributes($borrowernumber);	120	my $attributes = GetBorrowerAttributes($borrowernumber);

Lines 19-25 use Koha::Patron::Categories; Link Here

(-)a/members/member-flags.pl (-2 lines)
19	use Koha::Patrons;	19	use Koha::Patrons;
20		20
21	use C4::Output;	21	use C4::Output;
22	use Koha::Patron::Images;
23	use Koha::Token;	22	use Koha::Token;
24		23
25	my $input = new CGI;	24	my $input = new CGI;
Lines 190-196 if ($input->param('newflags')) { Link Here
190	}	189	}
191		190
192	$template->param( adultborrower => 1 ) if ( $category_type =~ /^(A\|I)$/ );	191	$template->param( adultborrower => 1 ) if ( $category_type =~ /^(A\|I)$/ );
193	$template->param( picture => 1 ) if $patron->image;
194		192
195	if (C4::Context->preference('ExtendedPatronAttributes')) {	193	if (C4::Context->preference('ExtendedPatronAttributes')) {
196	my $attributes = GetBorrowerAttributes($bor->{'borrowernumber'});	194	my $attributes = GetBorrowerAttributes($bor->{'borrowernumber'});

Lines 104-111 if ( $category_type eq 'C') { Link Here

(-)a/members/member-password.pl (-2 lines)
104		104
105	$template->param( adultborrower => 1 ) if ( $category_type =~ /^(A\|I)$/ );	105	$template->param( adultborrower => 1 ) if ( $category_type =~ /^(A\|I)$/ );
106		106
107	$template->param( picture => 1 ) if $patron->image;
108
109	if ( C4::Context->preference('ExtendedPatronAttributes') ) {	107	if ( C4::Context->preference('ExtendedPatronAttributes') ) {
110	my $attributes = GetBorrowerAttributes( $bor->{'borrowernumber'} );	108	my $attributes = GetBorrowerAttributes( $bor->{'borrowernumber'} );
111	$template->param(	109	$template->param(




use Koha::AuthorisedValues;
use Koha::CsvProfiles;
use Koha::Patron::Debarments qw(GetDebarments);
use Koha::Patron::Images;
use Module::Load;
if ( C4::Context->preference('NorwegianPatronDBEnable') && C4::Context->preference('NorwegianPatronDBEnable') == 1 ) {
    load Koha::NorwegianPatronDB, qw( NLGetSyncDataFromBorrowernumber );

    }
}

# check to see if patron's image exists in the database
# basically this gives us a template var to condition the display of
# patronimage related interface on
$template->param( picture => 1 ) if $patron->image;
# Generate CSRF token for upload and delete image buttons
$template->param(
    csrf_token => Koha::Token->new->generate_csrf({ session_id => $input->cookie('CGISESSID'),}),

Lines 52-59 my ($template, $loggedinuser, $cookie) Link Here

(-)a/members/notices.pl (-2 lines)
52	my $logged_in_user = Koha::Patrons->find( $loggedinuser ) or die "Not logged in";	52	my $logged_in_user = Koha::Patrons->find( $loggedinuser ) or die "Not logged in";
53	output_and_exit_if_error( $input, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );	53	output_and_exit_if_error( $input, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );
54		54
55	$template->param( picture => 1 ) if $patron->image;
56
57	# Allow resending of messages in Notices tab	55	# Allow resending of messages in Notices tab
58	my $op = $input->param('op') \|\| q{};	56	my $op = $input->param('op') \|\| q{};
59	if ( $op eq 'resend_notice' ) {	57	if ( $op eq 'resend_notice' ) {

Lines 41-47 use C4::Koha; Link Here

(-)a/members/pay.pl (-3 lines)
41	use C4::Overdues;	41	use C4::Overdues;
42	use C4::Members::Attributes qw(GetBorrowerAttributes);	42	use C4::Members::Attributes qw(GetBorrowerAttributes);
43	use Koha::Patrons;	43	use Koha::Patrons;
44	use Koha::Patron::Images;
45		44
46	use Koha::Patron::Categories;	45	use Koha::Patron::Categories;
47	use URI::Escape;	46	use URI::Escape;
Lines 141-148 sub add_accounts_to_template { Link Here
141	}	140	}
142	borrower_add_additional_fields($patron->unblessed);	141	borrower_add_additional_fields($patron->unblessed);
143		142
144	my $patron_image = Koha::Patron::Images->find($borrower->{borrowernumber});
145	$template->param( picture => 1 ) if $patron_image;
146	$template->param(	143	$template->param(
147	patron => $patron,	144	patron => $patron,
148	accounts => \@accounts,	145	accounts => \@accounts,




use C4::Members::Attributes qw(GetBorrowerAttributes);
use C4::Accounts;
use C4::Koha;
use Koha::Patron::Images;
use Koha::Patrons;
use Koha::Account;
use Koha::Token;

        $b_ref->{adultborrower} = 1;
    }

    my $patron_image = Koha::Patron::Images->find($b_ref->{borrowernumber});
    $template->param( picture => 1 ) if $patron_image;

    if (C4::Context->preference('ExtendedPatronAttributes')) {
        $b_ref->{extendedattributes} = GetBorrowerAttributes($b_ref->{borrowernumber});
    }

Lines 117-124 for (my $i=0;$i<$numaccts;$i++){ Link Here

(-)a/members/printfeercpt.pl (-2 lines)
117		117
118	$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' \|\| $category->category_type eq 'I' );	118	$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' \|\| $category->category_type eq 'I' );
119		119
120	$template->param( picture => 1 ) if $patron->image;
121
122	$template->param(	120	$template->param(
123	patron => $patron,	121	patron => $patron,
124	finesview => 1,	122	finesview => 1,

Lines 117-124 for ( my $i = 0 ; $i < $numaccts ; $i++ ) { Link Here

(-)a/members/printinvoice.pl (-2 lines)
117		117
118	$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' \|\| $category->category_type eq 'I' );	118	$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' \|\| $category->category_type eq 'I' );
119		119
120	$template->param( picture => 1 ) if $patron->image;
121
122	$template->param(	120	$template->param(
123	patron => $patron,	121	patron => $patron,
124	finesview => 1,	122	finesview => 1,

Lines 60-67 if (C4::Context->preference('ExtendedPatronAttributes')) { Link Here

(-)a/members/purchase-suggestions.pl (-2 lines)
60	);	60	);
61	}	61	}
62		62
63	$template->param( picture => 1 ) if $patron->image;
64
65	my $suggestions = SearchSuggestion( { suggestedby => $borrowernumber } );	63	my $suggestions = SearchSuggestion( { suggestedby => $borrowernumber } );
66		64
67	$template->param( suggestions => $suggestions );	65	$template->param( suggestions => $suggestions );

Lines 105-112 if (! $limit){ Link Here

(-)a/members/readingrec.pl (-2 lines)
105	$limit = 'full';	105	$limit = 'full';
106	}	106	}
107		107
108	$template->param( picture => 1 ) if $patron->image;
109
110	if (C4::Context->preference('ExtendedPatronAttributes')) {	108	if (C4::Context->preference('ExtendedPatronAttributes')) {
111	my $attributes = GetBorrowerAttributes($patron->borrowernumber);	109	my $attributes = GetBorrowerAttributes($patron->borrowernumber);
112	$template->param(	110	$template->param(

Lines 89-94 if (C4::Context->preference('ExtendedPatronAttributes')) { Link Here

(-)a/members/routing-lists.pl (-2 lines)
89	);	89	);
90	}	90	}
91		91
92	$template->param( picture => 1 ) if $patron and $patron->image;
93
94	output_html_with_http_headers $query, $cookie, $template->output;	92	output_html_with_http_headers $query, $cookie, $template->output;

Lines 84-91 if (C4::Context->preference('ExtendedPatronAttributes')) { Link Here

(-)a/members/statistics.pl (-3 lines)
84	);	84	);
85	}	85	}
86		86
87	$template->param( picture => 1 ) if $patron->image;
88
89	$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' \|\| $category->category_type eq 'I' );	87	$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' \|\| $category->category_type eq 'I' );
90		88
91	$template->param(	89	$template->param(
92	-

Return to bug 18789

Lines 28-34 use C4::Members; Link Here

(-)a/members/paycollect.pl (-4 lines)
28	use C4::Members::Attributes qw(GetBorrowerAttributes);	28	use C4::Members::Attributes qw(GetBorrowerAttributes);
29	use C4::Accounts;	29	use C4::Accounts;
30	use C4::Koha;	30	use C4::Koha;
31	use Koha::Patron::Images;
32	use Koha::Patrons;	31	use Koha::Patrons;
33	use Koha::Account;	32	use Koha::Account;
34	use Koha::Token;	33	use Koha::Token;
Lines 201-209 sub borrower_add_additional_fields { Link Here
201	$b_ref->{adultborrower} = 1;	200	$b_ref->{adultborrower} = 1;
202	}	201	}
203		202
204	my $patron_image = Koha::Patron::Images->find($b_ref->{borrowernumber});
205	$template->param( picture => 1 ) if $patron_image;
206
207	if (C4::Context->preference('ExtendedPatronAttributes')) {	203	if (C4::Context->preference('ExtendedPatronAttributes')) {
208	$b_ref->{extendedattributes} = GetBorrowerAttributes($b_ref->{borrowernumber});	204	$b_ref->{extendedattributes} = GetBorrowerAttributes($b_ref->{borrowernumber});
209	}	205	}

Lines 73-79 $(document).ready(function() { Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt (-6 / +6 lines)
73	],	73	],
74	"bPaginate": false	74	"bPaginate": false
75	}));	75	}));
76	[% IF ( picture ) %]	76	[% IF ( patron.image ) %]
77	// new YAHOO.widget.Button("delpicture"); // FIXME: formatting mismatch between YUI and normal button	77	// new YAHOO.widget.Button("delpicture"); // FIXME: formatting mismatch between YUI and normal button
78	$('#delpicture').click(function(){	78	$('#delpicture').click(function(){
79	return confirm(_("Are you sure you want to delete this patron image? This cannot be undone."));	79	return confirm(_("Are you sure you want to delete this patron image? This cannot be undone."));
Lines 282-288 function validate1(date) { Link Here
282	[% IF ( CAN_user_tools_batch_upload_patron_images ) %]	282	[% IF ( CAN_user_tools_batch_upload_patron_images ) %]
283	<form method="post" action="/cgi-bin/koha/tools/picture-upload.pl" enctype="multipart/form-data">	283	<form method="post" action="/cgi-bin/koha/tools/picture-upload.pl" enctype="multipart/form-data">
284	<fieldset class="brief">	284	<fieldset class="brief">
285	[% IF ( picture ) %]	285	[% IF ( patron.image ) %]
286	<legend>Manage patron image</legend>	286	<legend>Manage patron image</legend>
287	<div class="hint">To update the image for [% patron.title \| html %] [% patron.surname \| html %], select a new image file and click 'Upload.' <br />Click the 'Delete' button to remove the current image.	287	<div class="hint">To update the image for [% patron.title \| html %] [% patron.surname \| html %], select a new image file and click 'Upload.' <br />Click the 'Delete' button to remove the current image.
288	[% ELSE %]	288	[% ELSE %]
Lines 303-309 function validate1(date) { Link Here
303	<input type="hidden" name="csrf_token" value="[% csrf_token %]" />	303	<input type="hidden" name="csrf_token" value="[% csrf_token %]" />
304	<input type="submit" value="Upload" class="submit" />	304	<input type="submit" value="Upload" class="submit" />
305	<input name="op" type="hidden" value="Upload" />	305	<input name="op" type="hidden" value="Upload" />
306	[% IF ( picture ) %]<a id="delpicture" href="/cgi-bin/koha/tools/picture-upload.pl?op=Delete&borrowernumber=[% patron.borrowernumber %]&csrf_token=[% csrf_token %]" class="delete">Delete</a>[% END %]	306	[% IF ( patron.image ) %]<a id="delpicture" href="/cgi-bin/koha/tools/picture-upload.pl?op=Delete&borrowernumber=[% patron.borrowernumber %]&csrf_token=[% csrf_token %]" class="delete">Delete</a>[% END %]
307	</fieldset>	307	</fieldset>
308	</fieldset>	308	</fieldset>
309	</form>	309	</form>
Lines 393-399 function validate1(date) { Link Here
393	<li><span class="label">Borrowernumber: </span> [% patron.borrowernumber %]</li>	393	<li><span class="label">Borrowernumber: </span> [% patron.borrowernumber %]</li>
394	<li><span class="label">Category: </span>[% patron.category.description %] ([% patron.categorycode %])</li>	394	<li><span class="label">Category: </span>[% patron.category.description %] ([% patron.categorycode %])</li>
395	<li><span class="label">Registration date: </span>[% patron.dateenrolled \| $KohaDates %]</li>	395	<li><span class="label">Registration date: </span>[% patron.dateenrolled \| $KohaDates %]</li>
396		396
397	<li><span class="label">Expiration date: </span>	397	<li><span class="label">Expiration date: </span>
398	[% IF ( was_renewed ) %]	398	[% IF ( was_renewed ) %]
399	<strong class="reregistrinfo">[% patron.dateexpiry \| $KohaDates %]</strong>	399	<strong class="reregistrinfo">[% patron.dateexpiry \| $KohaDates %]</strong>
Lines 486-495 function validate1(date) { Link Here
486	<li><span class="label">First name: </span>[% patron.altcontactfirstname \| html %]</li>	486	<li><span class="label">First name: </span>[% patron.altcontactfirstname \| html %]</li>
487	<li><span class="label">Address: </span>[% patron.altcontactaddress1 \| html %]</li>	487	<li><span class="label">Address: </span>[% patron.altcontactaddress1 \| html %]</li>
488	<li><span class="label">Address 2: </span>[% patron.altcontactaddress2 \| html %]</li>	488	<li><span class="label">Address 2: </span>[% patron.altcontactaddress2 \| html %]</li>
489	<li><span class="label">City: </span>[% patron.altcontactaddress3 \| html %]</li>	489	<li><span class="label">City: </span>[% patron.altcontactaddress3 \| html %]</li>
490	[% IF ( patron.altcontactstate ) %]<li><span class="label">State: </span>[% patron.altcontactstate \| html %]</li>[% END %]	490	[% IF ( patron.altcontactstate ) %]<li><span class="label">State: </span>[% patron.altcontactstate \| html %]</li>[% END %]
491	<li><span class="label">ZIP/Postal code: </span>[% patron.altcontactzipcode \| html %]</li>	491	<li><span class="label">ZIP/Postal code: </span>[% patron.altcontactzipcode \| html %]</li>
492	[% IF ( patron.altcontactcountry ) %]<li><span class="label">Country: </span>[% patron.altcontactcountry \| html %]</li>[% END %]	492	[% IF ( patron.altcontactcountry ) %]<li><span class="label">Country: </span>[% patron.altcontactcountry \| html %]</li>[% END %]
493	[% IF ( patron.altcontactphone ) %]<li><span class="label">Phone: </span><a href="tel:[% patron.altcontactphone \| url %]">[% patron.altcontactphone \| html %]</a></li>[% END %]	493	[% IF ( patron.altcontactphone ) %]<li><span class="label">Phone: </span><a href="tel:[% patron.altcontactphone \| url %]">[% patron.altcontactphone \| html %]</a></li>[% END %]
494	</ol></div>	494	</ol></div>
495	</div>	495	</div>

Lines 62-70 my $patron = Koha::Patrons->find($borrowernumber); Link Here

(-)a/members/housebound.pl (-3 / +1 lines)
62	output_and_exit_if_error( $input, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );	62	output_and_exit_if_error( $input, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );
63		63
64	# Get supporting cast	64	# Get supporting cast
65	my ( $branch, $category, $houseboundprofile, $visit, $patron_image );	65	my ( $branch, $category, $houseboundprofile, $visit );
66	if ( $patron ) { # FIXME This test is not needed - output_and_exit_if_error handles it	66	if ( $patron ) { # FIXME This test is not needed - output_and_exit_if_error handles it
67	$patron_image = $patron->image;
68	$category = Koha::Patron::Categories->new->find($patron->categorycode);	67	$category = Koha::Patron::Categories->new->find($patron->categorycode);
69	$houseboundprofile = $patron->housebound_profile;	68	$houseboundprofile = $patron->housebound_profile;
70	}	69	}
Lines 170-176 if ( C4::Context->preference('ExtendedPatronAttributes') and $patron ) { Link Here
170		169
171	$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' \|\| $category->category_type eq 'I' );	170	$template->param( adultborrower => 1 ) if ( $category->category_type eq 'A' \|\| $category->category_type eq 'I' );
172	$template->param(	171	$template->param(
173	picture => $patron_image,
174	housebound_profile => $houseboundprofile,	172	housebound_profile => $houseboundprofile,
175	visit => $houseboundvisit,	173	visit => $houseboundvisit,
176	messages => \@messages,	174	messages => \@messages,

Lines 54-60 use C4::Members::Attributes qw(GetBorrowerAttributes); Link Here

(-)a/members/moremember.pl (-5 lines)
54	use Koha::AuthorisedValues;	54	use Koha::AuthorisedValues;
55	use Koha::CsvProfiles;	55	use Koha::CsvProfiles;
56	use Koha::Patron::Debarments qw(GetDebarments);	56	use Koha::Patron::Debarments qw(GetDebarments);
57	use Koha::Patron::Images;
58	use Module::Load;	57	use Module::Load;
59	if ( C4::Context->preference('NorwegianPatronDBEnable') && C4::Context->preference('NorwegianPatronDBEnable') == 1 ) {	58	if ( C4::Context->preference('NorwegianPatronDBEnable') && C4::Context->preference('NorwegianPatronDBEnable') == 1 ) {
60	load Koha::NorwegianPatronDB, qw( NLGetSyncDataFromBorrowernumber );	59	load Koha::NorwegianPatronDB, qw( NLGetSyncDataFromBorrowernumber );
Lines 275-284 if ( C4::Context->preference('NorwegianPatronDBEnable') && C4::Context->preferen Link Here
275	}	274	}
276	}	275	}
277		276
278	# check to see if patron's image exists in the database
279	# basically this gives us a template var to condition the display of
280	# patronimage related interface on
281	$template->param( picture => 1 ) if $patron->image;
282	# Generate CSRF token for upload and delete image buttons	277	# Generate CSRF token for upload and delete image buttons
283	$template->param(	278	$template->param(
284	csrf_token => Koha::Token->new->generate_csrf({ session_id => $input->cookie('CGISESSID'),}),	279	csrf_token => Koha::Token->new->generate_csrf({ session_id => $input->cookie('CGISESSID'),}),