Bugzilla – Attachment 39947 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]
[PASSED QA] Bug 14338 - Unable to delete patron images
PASSED-QA-Bug-14338---Unable-to-delete-patron-imag.patch (text/plain), 2.52 KB, created by
Katrin Fischer
on 2015-06-06 17:32:16 UTC
(
hide
)
Description:
[PASSED QA] Bug 14338 - Unable to delete patron images
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2015-06-06 17:32:16 UTC
Size:
2.52 KB
patch
obsolete
>From 6cc4ad6256f75aba7a7b77474eada5c01e3c3794 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] [PASSED QA] 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) Ensure the card number is not the same as the borrower number >3) Attempt to delete patron image > -- Image will remain >4) Apply this patch >5) Attempt to delete patron image > -- Image will be removed >6) run koha qa test tools > >Signed-off-by: Mark Tompsett <mtompset@hotmail.com> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > koha-tmpl/intranet-tmpl/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.9.1
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