Bugzilla – Attachment 39899 Details for
Bug 14338
Unable to delete patron images
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14338 - Unable to delete patron images
Bug-14338---Unable-to-delete-patron-images.patch (text/plain), 2.29 KB, created by
Kyle M Hall (khall)
on 2015-06-05 12:07:36 UTC
(
hide
)
Description:
Bug 14338 - Unable to delete patron images
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2015-06-05 12:07:36 UTC
Size:
2.29 KB
patch
obsolete
>From 5ba3673b7dcaedacc8f8b667773129b1a8f779a3 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Fri, 5 Jun 2015 08:06:29 -0400 >Subject: [PATCH] Bug 14338 - Unable to delete patron images > >The call to RmPatronImage is still passing cardnumber as its parameter >instead of borrowernumber. > >Test Plan: >1) Upload a patron image >2) Attempt to delete patron image >3) Image will remain >4) Apply this patch >5) Attempt to delete patron image >6) Image will be removed >--- > .../prog/en/modules/members/moremember.tt | 2 +- > tools/picture-upload.pl | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >index 8ddb526..e3ca832 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >@@ -280,7 +280,7 @@ function validate1(date) { > <fieldset class="action"> > <input type="submit" value="Upload" class="submit" /> > <input name="op" type="hidden" value="Upload" /> >- [% IF ( picture ) %]<a id="delpicture" href="/cgi-bin/koha/tools/picture-upload.pl?op=Delete&cardnumber=[% cardnumber %]&borrowernumber=[% borrowernumber %]" class="delete">Delete</a>[% END %] >+ [% IF ( picture ) %]<a id="delpicture" href="/cgi-bin/koha/tools/picture-upload.pl?op=Delete&borrowernumber=[% borrowernumber %]" class="delete">Delete</a>[% END %] > </fieldset> > </fieldset> > </form> >diff --git a/tools/picture-upload.pl b/tools/picture-upload.pl >index 3ae4f62..0d753d8 100755 >--- a/tools/picture-upload.pl >+++ b/tools/picture-upload.pl >@@ -157,8 +157,8 @@ elsif ( ( $op eq 'Upload' ) && !$uploadfile ) { > $template->param( filetype => $filetype ); > } > elsif ( $op eq 'Delete' ) { >- my $dberror = RmPatronImage($cardnumber); >- $debug and warn "Patron image deleted for $cardnumber"; >+ my $dberror = RmPatronImage($borrowernumber); >+ $debug and warn "Patron image deleted for $borrowernumber"; > warn "Database returned $dberror" if $dberror; > } > if ( $borrowernumber && !%errors && !$template->param('ERRORS') ) { >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 14338
:
39899
|
39904
|
39947
|
41186