https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17403

--- Comment #6 from David Cook <[email protected]> ---
Comment on attachment 56067
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56067
Bug 17403: Internal Server Error while deleting patron

Review of attachment 56067:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=17403&attachment=56067)
-----------------------------------------------------------------

::: members/deletemem.pl
@@ -98,4 @@
>      if ( !C4::Context->IsSuperLibrarian() && $bor->{'branchcode'}){
>          unless ($userenv->{branch} eq $bor->{'branchcode'}){
>              print 
> $input->redirect("/cgi-bin/koha/members/moremember.pl?borrowernumber=$member&error=CANT_DELETE_OTHERLIBRARY");
> -            exit;

"exit" is the same as "exit 0", so this doesn't really matter, but I'm
certainly in favour of a consistent approach. 

Why use "exit 0" over "exit" though? Just to be more explicit?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to