Skip Menu |

This queue is for tickets about the HTML-TurboForm CPAN distribution.

Report information
The Basics
Id: 90489
Status: resolved
Worked: 5 min
Priority: 0/
Queue: HTML-TurboForm

People
Owner: CAMELCASE [...] cpan.org
Requestors: adrian [...] gt.net
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Missing semi-colon in try
Date: Mon, 18 Nov 2013 12:59:46 -0800
To: bug-HTML-TurboForm [...] rt.cpan.org
From: Adrian Yee <adrian [...] gt.net>
HTML::TurboForm->get_value uses Try::Tiny's try, but is missing a semi-colon resulting in the whole block to fail silently. This in turn causes HTML::TurboForm to always save empty forms. perl v5.10.1 Try::Tiny 0.18 I also noticed that the package includes some files that probably shouldn't be in there: lib/.project lib/HTML/.project lib/HTML/TurboForm.komodoproject lib/HTML/TurboForm.kpf lib/HTML/TurboForm/.project lib/TurboForm.komodoproject lib/TurboForm.kpf Here's the patch for the missing semi-colon: --- HTML/TurboForm.pm.orig 2013-11-18 12:53:35.867959989 -0800 +++ HTML/TurboForm.pm 2013-11-18 12:53:48.374627409 -0800 @@ -524,7 +524,7 @@ my $id=''; try{ $id=$self->{element_index}->{$self->{prefix}.$name}->{index}; - } + }; $result=$self->{element}[$id]->get_value() if ($id);
resolved in Versions 0.73. thx for pointing it out