Bug #50164 for HTML-Table-FromDatabase: -override

Fri Oct 02 07:40:48 2009 ipluta [...] wp.pl - Ticket created

Subject:	-override_headers does not work
Date:	Fri, 02 Oct 2009 13:40:03 +0200
To:	bug-HTML-Table-FromDatabase [...] rt.cpan.org.
From:	Ireneusz Pluta <ipluta [...] wp.pl>

version: 0.05.

A trivial bug:
To see why and how to make work the option work, simply see the patch:

--- FromDatabase.pm.orig        Fri Oct 2 13:26:40 2009
+++ FromDatabase.pm     Fri Oct 2 13:27:05 2009
@@ -161,7 +161,7 @@
         @heading_names = @$override_headers;
     }

-    $self->addSectionRow('thead', 0, @columns);
+    $self->addSectionRow('thead', 0, @heading_names);
     $self->setSectionRowHead('thead', 0, 1);

     # Add all the rows:

Sat Oct 03 12:51:11 2009 davidp [...] preshweb.co.uk - Correspondence added

On Fri Oct 02 07:40:48 2009, ipluta@wp.pl wrote: Show quoted text

> version: 0.05. > > A trivial bug: > To see why and how to make work the option work, simply see the patch:

Heh, a very simple and silly bug indeed. Serves me right for not exercising those features with the test suite. Fixed, tests improved, and version 0.06 should be making its way to a CPAN mirror near you soon. Thanks for your contribution!

Sat Oct 03 12:51:12 2009 The RT System itself - Status changed from 'new' to 'open'

Sat Oct 03 12:51:13 2009 davidp [...] preshweb.co.uk - Status changed from 'open' to 'resolved'

Sat Oct 03 14:00:29 2009 ipluta [...] wp.pl - Correspondence added

Subject:	Re: [rt.cpan.org #50164] -override_headers does not work
Date:	Sat, 03 Oct 2009 19:59:25 +0200
To:	bug-HTML-Table-FromDatabase [...] rt.cpan.org
From:	Ireneusz Pluta <ipluta [...] wp.pl>

Thanks for mentioning me in changelog!

David Precious via RT pisze: Show quoted text

<URL: https://rt.cpan.org/Ticket/Display.html?id=50164 >

On Fri Oct 02 07:40:48 2009, ipluta@wp.pl wrote:

version: 0.05.

A trivial bug:
To see why and how to make work the option work, simply see the patch:

Heh, a very simple and silly bug indeed.  Serves me right for not
exercising those features with the test suite.

Fixed, tests improved, and version 0.06 should be making its way to a
CPAN mirror near you soon.

Thanks for your contribution!

Sat Oct 03 14:00:30 2009 The RT System itself - Status changed from 'resolved' to 'open'

Mon Oct 05 11:04:29 2009 davidp [...] preshweb.co.uk - Correspondence added

On Sat Oct 03 14:00:29 2009, ipluta@wp.pl wrote: Show quoted text

> Thanks for mentioning me in changelog!

You're very welcome, it's only fair :)

Mon Oct 05 11:04:30 2009 davidp [...] preshweb.co.uk - Status changed from 'open' to 'resolved'

Bug #50164 for HTML-Table-FromDatabase: -override_headers does not work

Preferred bug tracker