Skip Menu |
 

This queue is for tickets about the PHP-Include CPAN distribution.

Report information
The Basics
Id: 71352
Status: resolved
Worked: 20 min
Priority: 0/
Queue: PHP-Include

People
Owner: Nobody in particular
Requestors: nine [...] detonation.org
Cc:
AdminCc:

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



Subject: // in strings regarded as comment
Download (untitled) / with headers
text/plain 443b
PHP::Include does not allow // in strings anymore, since it thinks that those are comments and removes the rest of the line. Can be tested using the following PHP file and Perl test file: intern:~ # cat url.php <?php define ("URL", "http://metacpan.org"); ?> intern:~ # cat 015urls.t use strict; use warnings; use Test::More; use PHP::Include (DEBUG => 1); include_php_vars('url.php'); is(URL, 'http://metacpan.org'); done_testing;
Download (untitled) / with headers
text/plain 560b
Hopefully fixed on 0.34 ;) On Fri Sep 30 03:30:51 2011, NINE wrote: Show quoted text
> PHP::Include does not allow // in strings anymore, since it thinks > that those are > comments and removes the rest of the line. Can be tested using the > following PHP file > and Perl test file: > > intern:~ # cat url.php > <?php > define ("URL", "http://metacpan.org"); > ?> > > intern:~ # cat 015urls.t > use strict; > use warnings; > > use Test::More; > use PHP::Include (DEBUG => 1); > > include_php_vars('url.php'); > > is(URL, 'http://metacpan.org'); > > done_testing;


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.