Skip Menu |

This queue is for tickets about the Spark-Form CPAN distribution.

Report information
The Basics
Id: 87232
Status: resolved
Priority: 0/
Queue: Spark-Form
People
Owner: Nobody in particular
Requestors: rurban [...] x-ray.at
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.2103-TRIAL

History Show all quoted text
  Tue Jul 23 18:26:32 2013 rurban [...] x-ray.at - Ticket created
Subject: [PATCH] wrong return precedence
returns binds stronger than or, so the expressions after or are ignored. See https://rt.perl.org/rt3/Public/Bug/Display.html?id=59802
Subject: Spark-Form-0.2102-returnor.patch
diff -bu Spark-Form-0.2102-jagdFj/lib/Spark/Form.pm~ Spark-Form-0.2102-jagdFj/lib/Spark/Form.pm --- Spark-Form-0.2102-jagdFj/lib/Spark/Form.pm~ 2009-09-13 16:50:37.000000000 -0500 +++ Spark-Form-0.2102-jagdFj/lib/Spark/Form.pm 2013-07-23 17:24:41.324085096 -0500 @@ -199,7 +199,7 @@ my ($self, $thing) = @_; return eval { $thing->isa('Spark::Form::Field') - } or 0; + } || 0; } sub _add_custom_field {
  Sun Dec 15 08:16:57 2013 ELPENGUIN [...] cpan.org - Correspondence added
On Tue Jul 23 18:26:32 2013, rurban@x-ray.at wrote: Show quoted text
> returns binds stronger than or, so the expressions after or are ignored. > See https://rt.perl.org/rt3/Public/Bug/Display.html?id=59802
Thanks. Sorry, it took me a while to notice this. The code you've patched no longer exists.
  Sun Dec 15 08:16:58 2013 The RT System itself - Status changed from 'new' to 'open'
  Sun Dec 15 08:16:58 2013 ELPENGUIN [...] cpan.org - Status changed from 'open' to 'resolved'
  Sun Dec 15 08:56:28 2013 KENTNL [...] cpan.org - Fixed in 0.2103-TRIAL added