Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Workflow CPAN distribution.

Report information
The Basics
Id: 85380
Status: resolved
Priority: 0/
Queue: Workflow

People
Owner: jonasbn [...] cpan.org
Requestors: kaynehril [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.38



Subject: Possible bug in Workflow::Persister::DBI
Date: Fri, 17 May 2013 16:26:18 +0100
To: bug-Workflow [...] rt.cpan.org
From: Sérgio Alves <kaynehril [...] gmail.com>
Download (untitled) / with headers
text/plain 1010b
Greetings, I think I might have stumbled on a minor bug inside this module. When fetching the history, the DateTime created does not take into consideration the time zone of the defined workflow ( sub fetch_history ): my @history = (); while ( my $row = $sth->fetchrow_arrayref ) { my $hist = Workflow::History->new( { id => $row->[0], workflow_id => $row->[1], action => $row->[2], description => $row->[3], state => $row->[4], user => $row->[5], * date => $self->parser->parse_datetime( $row->[6] ),* } ); $log->is_debug && $log->debug("Fetched history object '$row->[0]'"); $hist->set_saved(); push @history, $hist; } The little patch I did to myself was to add the following line at the beginning of the sub: $self->parser()->time_zone($wf->time_zone); Best Regards, Sérgio Alves
Download (untitled) / with headers
text/plain 127b
Sorry for the delay, I will look into the issue and your proposed patch. Have a nice summer and thanks for reporting, jonasbn
The patch is included in release 1.38. Thanks for contributing. jonasbn
The patch has been applied and is in release 1.38 Thank you for the contribution. jonasbn


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.