@@ -, +, @@ - members-toolbar.inc: Remove the category_type parameter passed to - memberentrygen.tt: Just remove the useless category_type parameter on editing a patron. Also remove the unused one passed to guarantor_search.pl. - tables/members_results.tt: the borrowernumber is enough to edit a patron. - memberentry.pl: check_categorytype is never used in the template, all the process to calculate/retrieve it is unnecessary. - members/nl-search.tt: The borrowernumber is enough to edit a patron. - the guarantor search still work - the form (memberentry) behave as before --- .../prog/en/includes/members-toolbar.inc | 22 +++++++--------------- .../prog/en/modules/members/memberentrygen.tt | 10 +++++++--- .../prog/en/modules/members/nl-search.tt | 8 +------- .../en/modules/members/tables/members_results.tt | 2 +- members/memberentry.pl | 9 --------- 5 files changed, 16 insertions(+), 35 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc +++ a/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc @@ -130,30 +130,22 @@ function searchToHold(){
[% IF ( CAN_user_borrowers ) %] - [% IF ( category_type ) %] - [% IF ( guarantor ) %] - - [% ELSE %] - - [% END %] - Edit - [% ELSE %] - [% IF ( categorycode ) %] - Edit - [% ELSE %] - Edit - [% END %] + [% IF ( guarantor ) %] + + [% ELSE %] + [% END %] + Edit [% END %] [% IF ( CAN_user_borrowers ) %] [% IF ( adultborrower AND activeBorrowerRelationship ) %] - Add child + Add child [% END %] [% IF ( CAN_user_borrowers ) %] Change password [% END %] - Duplicate + Duplicate [% END %]
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt @@ -184,7 +184,11 @@

Duplicate patron record?

View existing record

-
+
+ + + +
@@ -459,9 +463,9 @@
  •   [% IF ( guarantorid ) %] - + [% ELSE %] - + [% END %]
  • --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/nl-search.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/members/nl-search.tt @@ -32,15 +32,9 @@

    Existing patrons

      [% FOREACH patron IN local_result %] - [%# Find the category_type %] - [% FOREACH category IN categories %] - [% IF category.categorycode == patron.categorycode %] - [% patron.category_type = category.category_type %] - [% END %] - [% END %]
    • [% patron.firstname %] [% patron.surname %] [% patron.cardnumber %] | Details | - Edit | + Edit | Check out
    • [% END %] --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt @@ -27,7 +27,7 @@ "dt_borrowernotes": "[% data.borrowernotes.replace('\\\\' , '\\\\') |html |html_line_break |collapse %]", "dt_action": - "[% IF data.category_type %]Edit[% ELSE %][% IF data.categorycode %]Edit[% ELSE %]Edit[% END %][% END %]", + "Edit", "borrowernumber": "[% data.borrowernumber %]" }[% UNLESS loop.last %],[% END %] --- a/members/memberentry.pl +++ a/members/memberentry.pl @@ -90,8 +90,6 @@ my $guarantorinfo = $input->param('guarantorinfo'); my $step = $input->param('step') || 0; my @errors; my $default_city; -# $check_categorytype contains the value of duplicate borrowers category type to redirect in good template in step =2 -my $check_categorytype=$input->param('check_categorytype'); # NOTE: Alert for ethnicity and ethnotes fields, they are invalid in all borrowers form my $borrower_data; my $NoUpdateLogin; @@ -239,12 +237,6 @@ if ( ( $op eq 'insert' ) and !$nodouble ) { if ( !$check_member ) { $nodouble = 1; } - - # recover the category type if the borrowers is a doublon - if ($check_category) { - my $tmpborrowercategory = GetBorrowercategory($check_category); - $check_categorytype = $tmpborrowercategory->{'category_type'}; - } } #recover all data from guarantor address phone ,fax... @@ -713,7 +705,6 @@ $template->param( borrotitlepopup => $borrotitlepopup, guarantorinfo => $guarantorinfo, flagloop => \@flagdata, - check_categorytype =>$check_categorytype,#to recover the category type with checkcategorytype function category_type =>$category_type, modify => $modify, nok => $nok,#flag to konw if an error --