Skip Menu |

This queue is for tickets about the Data-Table CPAN distribution.

Report information
The Basics
Id: 78569
Status: resolved
Priority: 0/
Queue: Data-Table

People
Owner: Nobody in particular
Requestors: ra4720 [...] freescale.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.65
Fixed in: (no value)



Subject: Modifying the default behavior for group command option $keepRestCols in 1.64
Upon testing v1.65 (from v1.54), I noticed you have added an option $keepRestCols to the group command. I agree with adding this option as it now aligns with other commands (i.e. pivot); however, I recommend the default value should be 1 or TRUE to align with the assumptions in the previous versions of the module. I am seeing multiple scripts break due to this update. Where before I could assume that remaining columns would transfer, they are now no longer showing up. While the solution seems easy (add a 1 value to the end of all group commands), this is not practical given the large developer base currently utilizing the Data::Table module. I believe any update to a command should retain the original behavior but provide a means to change that behavior if the user provides a signal. Thank you for considering my request. I am not sure if it is really a bug, but it will prevent me from upgrading to v1.65. I'd really like to upgrade since you've provided a lot of great enhancements!
Thanks, set to 1 as default in 1.66. Just uploaded to CPAN. On Wed Jul 25 12:45:03 2012, kylehorton wrote: Show quoted text
> Upon testing v1.65 (from v1.54), I noticed you have added an option > $keepRestCols to the group command. I agree with adding this option
as Show quoted text
> it now aligns with other commands (i.e. pivot); however, I recommend
the Show quoted text
> default value should be 1 or TRUE to align with the assumptions in the > previous versions of the module. I am seeing multiple scripts break
due Show quoted text
> to this update. Where before I could assume that remaining columns > would transfer, they are now no longer showing up. While the solution > seems easy (add a 1 value to the end of all group commands), this is
not Show quoted text
> practical given the large developer base currently utilizing the > Data::Table module. I believe any update to a command should retain
the Show quoted text
> original behavior but provide a means to change that behavior if the > user provides a signal. > > Thank you for considering my request. I am not sure if it is really a > bug, but it will prevent me from upgrading to v1.65. I'd really like
to Show quoted text
> upgrade since you've provided a lot of great enhancements!