[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#688121: marked as done (unblock: libconfig-model-tkui-perl/1.337-2)



Your message dated Wed, 19 Sep 2012 21:48:49 +0100
with message-id <1348087729.26713.18.camel@jacala.jungle.funky-badger.org>
and subject line Re: Bug#688121: unblock: libconfig-model-tkui-perl/1.337-2
has caused the Debian Bug report #688121,
regarding unblock: libconfig-model-tkui-perl/1.337-2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
688121: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=688121
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package libconfig-model-tkui-perl

The current package has a confusing bug: the GUI shows configuration
values as modified even though they are not specified in the configuration
files. This patch also fixes a bug where wrong values or values with
warnings can be hidden from the user.

This patch was taken from upstream. 

Thanks

unblock libconfig-model-tkui-perl/1.337-2

-- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (990, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.2.0-3-amd64 (SMP w/4 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
diff -Nru libconfig-model-tkui-perl-1.337/debian/changelog libconfig-model-tkui-perl-1.337/debian/changelog
--- libconfig-model-tkui-perl-1.337/debian/changelog	2012-06-29 14:26:50.000000000 +0200
+++ libconfig-model-tkui-perl-1.337/debian/changelog	2012-09-19 14:16:29.000000000 +0200
@@ -1,3 +1,9 @@
+libconfig-model-tkui-perl (1.337-2) unstable; urgency=low
+
+  * added upstream patch to fix green arrow display (Closes: #688097)
+
+ -- Dominique Dumont <dod@debian.org>  Wed, 19 Sep 2012 14:14:17 +0200
+
 libconfig-model-tkui-perl (1.337-1) unstable; urgency=low
 
   * Imported Upstream version 1.337:
diff -Nru libconfig-model-tkui-perl-1.337/debian/patches/fix-green-arrow-and-show-hide.patch libconfig-model-tkui-perl-1.337/debian/patches/fix-green-arrow-and-show-hide.patch
--- libconfig-model-tkui-perl-1.337/debian/patches/fix-green-arrow-and-show-hide.patch	1970-01-01 01:00:00.000000000 +0100
+++ libconfig-model-tkui-perl-1.337/debian/patches/fix-green-arrow-and-show-hide.patch	2012-09-19 14:16:29.000000000 +0200
@@ -0,0 +1,40 @@
+Description: Fix green arrow and show hide.patch
+ This patch fixes the logic used to display the
+ green arrow. 
+ Also don't hide values which have errors or warnings.
+ when 
+ the 'show only custom values' checkbox is set.
+Bug-Debian: 688097
+Forwarded: yes
+Author: dod
+--- a/lib/Config/Model/TkUI.pm
++++ b/lib/Config/Model/TkUI.pm
+@@ -835,13 +835,15 @@
+     my $value = $leaf_object->fetch(check => 'no', silent => 1) ;
+     my $tkt = $cw->{tktree} ;
+ 
+-    my ($is_customised, $img) ;
++    my ($is_customised, $img,$has_error,$has_warning) ;
+     {
+ 	no warnings qw/uninitialized/ ;
+-        $is_customised = ! ( $std_v eq $value)  ;
++        $is_customised = !! ( defined $value and ( $std_v ne $value ))  ;
+ 	$img = $cust_img if $is_customised ;
+-	$img = $warn_img if $leaf_object->warning_msg ;
+-	$img = $error_img if $leaf_object->error_msg;
++	$has_warning = !! $leaf_object->warning_msg ;
++	$img = $warn_img if $has_warning ;
++	$has_error = !! $leaf_object->error_msg;
++	$img = $error_img if $has_error;
+     }
+ 
+     if (defined $img) {
+@@ -859,7 +861,7 @@
+ 
+     $tkt->itemCreate($path,3, -text => $cw->trim_value($std_v)) ;
+     
+-    my $meth = ($cw->{show_only_custom} and not $is_customised) ? 'hide' : 'show' ;
++    my $meth = ($cw->{show_only_custom} and not ($is_customised or $has_error or $has_warning)) ? 'hide' : 'show' ;
+     $tkt->$meth(entry => $path) ; 
+ }
+ 
diff -Nru libconfig-model-tkui-perl-1.337/debian/patches/series libconfig-model-tkui-perl-1.337/debian/patches/series
--- libconfig-model-tkui-perl-1.337/debian/patches/series	2012-06-29 14:26:50.000000000 +0200
+++ libconfig-model-tkui-perl-1.337/debian/patches/series	2012-09-19 14:16:29.000000000 +0200
@@ -0,0 +1 @@
+fix-green-arrow-and-show-hide.patch

--- End Message ---
--- Begin Message ---
On Wed, 2012-09-19 at 17:25 +0200, Dominique Dumont wrote:
> Please unblock package libconfig-model-tkui-perl
> 
> The current package has a confusing bug: the GUI shows configuration
> values as modified even though they are not specified in the configuration
> files. This patch also fixes a bug where wrong values or values with
> warnings can be hidden from the user.

Unblocked; thanks.

Regards,

Adam

--- End Message ---

Reply to: