Skip Menu |
 

This queue is for tickets about the Jifty-DBI CPAN distribution.

Report information
The Basics
Id: 91088
Status: resolved
Priority: 0/
Queue: Jifty-DBI

People
Owner: Nobody in particular
Requestors: fraserbn [...] gmail.com
Cc:
AdminCc:

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



Subject: Fix for perl 5.19.4+
A bit of background. Under 5.19.4, @{ foo {"bar"} } is always parsed as @foo{bar} -- in previous version of perl, it was parsed as either that or @{ ( foo{"bar"} ) }, depending on whenever there was a newline after the '@{', and on whenever you were inside of an eval STRING or not. Jifty::DBI::Collection uses '@{\neval {} ...}', which now parses as a syntax error. To get things working again, you'll have to explicitly mark the inside of @{} as an expression, which can be achieved by changing this: return map lc( $_->[0] ), @{ eval { $dbh->column_info( '', '', $table, '' )->fetchall_arrayref( [3] ); } || $dbh->selectall_arrayref("DESCRIBE $table;") || $dbh->selectall_arrayref("DESCRIBE \u$table;") || [] }; Into this: return map lc( $_->[0] ), @{( eval { $dbh->column_info( '', '', $table, '' )->fetchall_arrayref( [3] ); } || $dbh->selectall_arrayref("DESCRIBE $table;") || $dbh->selectall_arrayref("DESCRIBE \u$table;") || [] )};
Subject: Re: [rt.cpan.org #91088] Fix for perl 5.19.4+
Date: Sun, 01 Dec 2013 12:40:45 -0500
To: bug-Jifty-DBI [...] rt.cpan.org
From: Alex Vandiver <alex [...] chmrr.net>
Download (untitled) / with headers
text/plain 133b
On Sun, 2013-12-01 at 02:11 -0500, Brian Fraser via RT wrote: Show quoted text
> A bit of background. [snip]
Thanks -- applied as 44c54f3. - Alex


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.