Attachment 54396 Details for Bug 16996 – Bug 16996: Do not explode if mandatory fields are missing

[patch] Bug 16996: Do not explode if mandatory fields are missing

Bug-16996-Do-not-explode-if-mandatory-fields-are-m.patch (text/plain), 2.01 KB, created by Nick Clemens (kidclamp) on 2016-08-12 18:16:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2016-08-12 18:16:42 UTC

Size: 2.01 KB

patch

obsolete

>From 376475c5e6f9cb96270a447cfa5f7ff742e37b2d Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 3 Aug 2016 16:28:24 +0100
>Subject: [PATCH] Bug 16996: Do not explode if mandatory fields are missing
>
>At the OPAC, if a patron modify his/her information and at least 1
>mandatory field is missing, Koha will crash with
>  Template process failed: undef error - Can't call method "description"
>
>It is raised by Koha::Template::Plugin::Categories::GetName called with
>an undefined categorycode.
>The problem is that the values sent originaly are not sent back to the
>template if the user missed something.
>This patch makes that all info are resent to the template in order to
>show the same form to the user.
>
>Test plan:
>0. Apply patch
>
>1. Make sure that the categorycode is not hidden in the OPAC in:
>     PatronSelfRegistrationBorrowerUnwantedField or PatronSelfModificationBorrowerUnwantedField
>
>2. Add a required field in PatronSelfRegistrationBorrowerMandatoryField, for example email.
>
>3. Go to Your Personal Details in the OPAC (opac-memberentry.pl) and leave blank the required field of point 2
>
>4. Press "Submit update request"
>
>There should be no software error.
>
>NOTE: The warns will be floody on the logs.
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>
>Signed-off-by: Marc <veron@veron.ch>
>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> opac/opac-memberentry.pl | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/opac/opac-memberentry.pl b/opac/opac-memberentry.pl
>index 592c99b..c864c19 100755
>--- a/opac/opac-memberentry.pl
>+++ b/opac/opac-memberentry.pl
>@@ -196,6 +196,9 @@ elsif ( $action eq 'update' ) {
>       CheckMandatoryFields( \%borrower_changes, $action );
>     my $invalidformfields = CheckForInvalidFields(\%borrower);
> 
>+    # Send back the data to the template
>+    %borrower = ( %$borrower, %borrower );
>+
>     if (@empty_mandatory_fields || @$invalidformfields) {
>         $template->param(
>             empty_mandatory_fields => \@empty_mandatory_fields,
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 16996: 53959 | 54011 | 54148 | 54167 | 54175 | 54176 | 54396 | 54397 | 54947