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

Bug #6426 load() should empty $this->strings
Submitted: 2006-01-06 04:44 UTC Modified: 2006-01-07 04:45 UTC
From: ivanwyc at gmail dot com Assigned: mike
Status: Closed Package: File_Gettext
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  


 [2006-01-06 04:44 UTC] ivanwyc at gmail dot com
Description: ------------ diff -purb File.orig/Gettext/MO.php File/Gettext/MO.php --- File.orig/Gettext/MO.php 2006-01-06 16:53:40.000000000 +0800 +++ File/Gettext/MO.php 2006-01-06 16:28:07.000000000 +0800 @@ -211,6 +211,7 @@ class File_Gettext_MO extends File_Gette ); } + $this->strings = array(); // read all for ($i = 0; $i < $count; $i++) { $this->strings[$this->_readStr($original[$i])] = diff -purb File.orig/Gettext/PO.php File/Gettext/PO.php --- File.orig/Gettext/PO.php 2006-01-06 16:53:45.000000000 +0800 +++ File/Gettext/PO.php 2006-01-06 16:27:46.000000000 +0800 @@ -74,6 +74,7 @@ class File_Gettext_PO extends File_Gette return parent::raiseError('No msgid/msgstr entries found'); } + $this->strings = array(); // get all msgids and msgtrs for ($i = 0; $i < $matched; $i++) { $msgid = substr(trim($matches[1][$i]), 1, -1);

Comments

 [2006-01-07 04:45 UTC] mike
This bug has been fixed in CVS. 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.