Arshavir Grigorian wrote:
>I am trying to access a DateTime::Duration method from within my TT
>template and getting an getting the following exception:
I don't see any problem. Here's my test script:
use strict;
use warnings;
use Template;
use DateTime::Duration;
my $template = Template->new();
my $duration = DateTime::Duration->new( years => 3 );
$template->process(\*DATA, { duration => $duration })
|| die $template->error();
__DATA__
[% IF duration.years -%]
* The duration is: [% duration.years %] years
[% ELSE -%]
* The duration is less than a year.
[% END -%]
I get the correct output and no exception. Can you "fix" my example to
demonstrate the problem?
> If we replace it with the following, the logic
> will short circuit and we will never have to compare apples to oranges.
>
> my $atroot = ((ref ($root) eq ref ($self)) && ($root eq $self));
>
> It should not break anything else. Right?
Who knows? That's what the test suite is for. :-)
If you have something that looks like a bug, you need to write a short test
like the above that clearly demonstrates the problem, add it to the test
suite, apply the fix, and then run the test suite to prove that the fix
fixes the bug and doesn't break anything else.
But I'm not in favour of adding a hack in TT to work around one particular
module's problems, especially when it's adding extra code that will be
executed every time every node of every dotted variable is evaluated in
every template in every TT installation across the World. That's an awful
lot of dotops. :-)
It would be much easier to fix the real problem in DateTime::Duration
which overloads <=> and cmp to _compare_overload() but then dies when
it actually gets called:
sub _compare_overload
{
die "DateTime::Duration does not overload comparison. See the
documentation on the compare() method for details.";
}
A
_______________________________________________
templates mailing list
[EMAIL PROTECTED]
http://lists.template-toolkit.org/mailman/listinfo/templates
