[Catalyst] Trailing undef in uri_for()

Erik Broes erikbroes at ripe.net
Tue Jun 5 09:54:41 GMT 2007


> I think for '' the behaviour's correct.
>
> Not sure about undef, it doesn't strike me as a valid value. Thoughts,
> anyone?

I think we can safely delete any trailing undef values. I agree for '' it 
should append /s.

And it's as easy as:

    ### Remove trailing undef arguments.
    pop @args while not defined $args[-1];

and editing the tests.

-- 
Erik Broes
Database Group
RIPE NCC



More information about the Catalyst mailing list