Charles K. Clarkson wrote:
# Silently add site level run modes here.
sub run_modes {
    my $self = shift;
    my $run_modes = shift;
    return $self->SUPER::run_modes( [ 'login', @$run_modes ] );
}

    Is there any gotcha I need to worry about or is this an
acceptable practice? (Note: I'm using C::A 4.0_2)

There's one gotcha with this, and that is the fact that you can pass a hash to run_modes:

        $app->run_modes(
                screen1 => 'sub1',
                screen2 => 'sub2',
                );

This will not work, because you assume $run_modes is an arrayref in your 
overridden run_modes().

But, as I wrote a bit earlier, you don't need to override run_modes() in the 
first place :-)

Rhesa

---------------------------------------------------------------------
Web Archive:  http://www.mail-archive.com/[email protected]/
             http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to