Skip Menu |

This queue is for tickets about the Rcs-Parser CPAN distribution.

Report information
The Basics
Id: 92643
Status: resolved
Estimated: 5 min
Worked: 5 min
Priority: 0/
Queue: Rcs-Parser
People
Owner: bennie [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in:
  • 0.03
  • 0.01
  • 0.02
Fixed in: 0.06

History Show all quoted text
  Sat Feb 01 16:33:30 2014 SREZIC [...] cpan.org - Ticket created
Subject: all_versions() is not OO
all_versions() caches the result of the sorted version array in an "our" variable, and effectively prevents using this method on multiple objects. Either store the result within $self (e.g. $self->{all_versions}), or don't cache the result at all. Regards, Slaven
  Wed Feb 05 19:35:34 2014 bennie [...] cpan.org - TimeEstimated changed from (no value) to '5'
  Wed Feb 05 19:35:34 2014 bennie [...] cpan.org - TimeWorked changed from (no value) to '5'
  Wed Feb 05 19:35:35 2014 bennie [...] cpan.org - Taken
  Wed Feb 05 19:35:35 2014 bennie [...] cpan.org - Broken in 0.01 added
  Wed Feb 05 19:35:35 2014 bennie [...] cpan.org - Broken in 0.02 added
  Wed Feb 05 19:36:47 2014 bennie [...] cpan.org - Correspondence added
Good catch on "our" bypassing multiple objects. Dumb mistake. Will go with your suggestion of binding it to $self.
  Wed Feb 05 19:36:47 2014 The RT System itself - Status changed from 'new' to 'open'
  Thu Feb 20 17:52:04 2014 bennie [...] cpan.org - Status changed from 'open' to 'patched'
  Sat Feb 22 16:20:13 2014 bennie [...] cpan.org - Status changed from 'patched' to 'resolved'
  Sat Feb 22 16:20:13 2014 bennie [...] cpan.org - Fixed in 0.06 added