Attachment 15351 Details for Bug 9586 – Bug 9586 - Remove $ENV{DEBUG} info from Member Template

[patch] Bug 9586 - Remove $ENV{DEBUG} info from Member Template

Bug-9586---Remove-ENVDEBUG-info-from-Member-Templa.patch (text/plain), 2.31 KB, created by Kyle M Hall (khall) on 2013-02-15 12:22:17 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2013-02-15 12:22:17 UTC

Size: 2.31 KB

patch

obsolete

>From b929850158bbeeadb2f3c1be9dfc3f63bfcd42de Mon Sep 17 00:00:00 2001
>From: David Cook <dcook@prosentient.com.au>
>Date: Mon, 11 Feb 2013 16:42:56 +1100
>Subject: [PATCH] Bug 9586 - Remove $ENV{DEBUG} info from Member Template
>
>Currently, if $ENV{DEBUG} is set (for example to 1), the debug level will be printed on the Add/Edit Patron pages (memberentrygen.tt) in the form of: "Debug is on (level 1)".
>
>While many if not most folks might not set $ENV{DEBUG} on a production system, there is no reason to have this information printing on the member template(s).
>
>No where else in Koha is there a template that prints $ENV{DEBUG} information, with the exception of calendar.inc, although that doesn't activate until at least debug level 2 or 5, so it might be worth holding on to. It is certainly more complicated than simply printing the debug level. I don't do development on the calendars, so I'm not going to address that one.
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../prog/en/modules/members/memberentrygen.tt      |    5 -----
> members/memberentry.pl                             |    1 -
> 2 files changed, 0 insertions(+), 6 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>index edabba0..68d6a30 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>@@ -141,11 +141,6 @@
> 			</div>
> 	[% END %]
> 
>-	[% IF ( debug ) %]
>-		<div id="debug">
>-				<div>Debug is on (level [% debug %])</div>
>-		</div>
>-	[% END %]
> 	[% IF ( nok ) %]
> 		<div class="dialog alert">
> 			<p>The following fields are wrong. Please fix them.</p>
>diff --git a/members/memberentry.pl b/members/memberentry.pl
>index f6d3e5e..7fc27c0 100755
>--- a/members/memberentry.pl
>+++ b/members/memberentry.pl
>@@ -699,7 +699,6 @@ $debug and warn "memberentry step: $step";
> $template->param(%data);
> $template->param( "step_$step"  => 1) if $step;	# associate with step to know where u are
> $template->param(  step  => $step   ) if $step;	# associate with step to know where u are
>-$template->param( debug  => $debug  ) if $debug;
> 
> $template->param(
>   BorrowerMandatoryField => C4::Context->preference("BorrowerMandatoryField"),#field to test with javascript
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 9586: 15216 | 15351 | 15894