Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.8.1

Bug #17430 Multiple Net_IDNA class declarations
Submitted: 2010-05-24 13:31 UTC
From: tokul Assigned: doconnor
Status: Closed Package: Net_IDNA (version 0.7.3)
PHP Version: 4.4.9 OS:
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 25 - 11 = ?

 
 [2010-05-24 13:31 UTC] tokul (Tomas Kuliavas)
Description: ------------ Package has one Net_IDNA class in IDNA.php and other Net_IDNA class in Net/IDNA/php4.php and tries to load them both. fatal error is suppressed in IDNA.php '@include_once $classfile;' Cannot redeclare class net_idna in /some-path/php/lib/php/Net/IDNA/php4.php on line 2973 Current SVN (rev. 292676) has same problem. Haven't tested it, but looked at the source in svn web browser. Test script: --------------- include('Net/IDNA.php'); $idn_instance = Net_IDNA::getInstance(); echo 'done'; Expected result: ---------------- done Actual result: -------------- no output.

Comments

 [2010-05-25 08:04 UTC] doconnor (Daniel O'Connor)
How very annoying. One needs to be Net_IDNA_Factory; which is a BC break. For now, I'd suggest you make use of require_once 'Net/IDNA/php4.php'; and skip all of the getInstance() stuff. Out of curiousity; what's keeping you on PHP4? (You are a rare one these days!)
 [2010-05-25 08:51 UTC] tokul (Tomas Kuliavas)
> Out of curiousity; what's keeping you on PHP4? (You are a rare one these days!) Bug report No. 14426 was about PHP4. So I've tested package on last PHP4 version. I use php5, but I have legacy systems that use older PHP versions and I don't like unnecessarily dependencies that increase PHP version requirements. I am planning to bundle net_idna with my package in order to provide fallback idn decoding and encoding support and I think I'll remove all PHP5 dependencies (exceptions and public/private declarations, maybe others) from it.
 [2010-06-03 06:00 UTC] doconnor (Daniel O'Connor)
I've fixed this in SVN (There's now a Net_IDNA2); but releases appear upset at the moment.
 [2010-06-03 06:03 UTC] doconnor (Daniel O'Connor)
-Status: Open +Status: Closed -Assigned To: +Assigned To: doconnor
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.