Re: [Perl-workflow-devel] [solved] Re: Bug found in Workflow::Action::init

[Jonas B. N. <jo...@gm...>]

Hi Alejandro,

Sorry for not responding, I have flagged your mail for follow-up, but  
never really got around to it.

Please keep using the list, I am at least reading everything :-)

jonasbn

On 21/09/2009, at 21.14, Alejandro Imass wrote:

> Ok, my bad, sorry folks.
>
> I was overriding init() but did not realize it was later called by
> Base::new(), so I was calling it twice and with the wrong params.
> Oops.
>
> I am now overriding new() and everything seems to be working fine.
>
> BTW, seems like I'm talking to myself here but what the heck, better
> to resolve the thread for future reference....
>
> Thanks,
> Alejandro Imass
>
> On Sun, Sep 20, 2009 at 11:07 AM, Alejandro Imass <ai...@p2...>  
> wrote:
>> I can almost confirm there is a bug in Workflow::Action::init
>>
>> Workflow::Action::init currently reads:
>>
>> sub init {
>>   my ( $self, $wf, $params ) = @_;
>>
>> When in fact, I think it should read:
>>
>> sub init {
>>   my ( $self, $params ) = @_;
>>
>> Just like the init() in Condition, State, Validator and so on.
>>
>> In fact Base::new() calls $self->init(@params); sow where the heck is
>> $wf supposed to come from?
>> If all this is true, then it's a fact that the Action accessors are
>> currently broken and $self->param() of Action is being clobbered by
>> the wrong object properties.
>>
>> Can _anyone_ , _please_, _pretty pleae_ confirm this?
>> I have sent 3 mails regarding this issue and it's hard to believe I'm
>> the only one that has problems using the Action accessors!!!
>>
>> Thanks,
>> Alejandro Imass
>>
>
> ------------------------------------------------------------------------------
> Come build with us! The BlackBerry&reg; Developer Conference in SF, CA
> is the only developer event you need to attend this year. Jumpstart  
> your
> developing skills, take BlackBerry mobile applications to market and  
> stay
> ahead of the curve. Join us from November 9&#45;12, 2009. Register  
> now&#33;
> http://p.sf.net/sfu/devconf
> _______________________________________________
> Perl-workflow-devel mailing list
> Per...@li...
> https://lists.sourceforge.net/lists/listinfo/perl-workflow-devel