http://pear.php.net/bugs/3641 [Closed] Translation2 is not cloneable en-us pear-webmaster@lists.php.net pear-webmaster@lists.php.net hourly 1 2000-01-01T12:00+00:00 http://pear.php.net/bugs/3641 Translation2 Bug Reported by ojai@... 2005-02-28T18:36:21-00:00 PHP: 5.0.2 OS: Linux Debian Package Version: Description: ------------ Translation2 version : current CVS Translation2 doesn't make use of a __clone() method to clone its internal object references. Here's a patch that corrects this issue : http://samalyse.com/ln/t2-clone.diff I think you should add a __clone() method to the Decorator as well. Reproduce code: --------------- require_once 'PEAR.php'; require_once 'Translation2.php'; $t2 =& Translation2::factory ('XML', array('filename' => 'i18n.xml')); $t2->setLang ('en'); $t2_clone = clone ($t2); $t2_clone->setLang('fr'); // This shouldn't change $t2 echo $t2->get('invoice','documents'); // But it does... Expected result: ---------------- Invoice Actual result: -------------- Facture]]> Translation2 Bug Reported by ojai@... 2005-02-28T18:36:21-00:00 PHP: 5.0.2 OS: Linux Debian Package Version: Description: ------------ Translation2 version : current CVS Translation2 doesn't make use of a __clone() method to clone its internal object references. Here's a patch that corrects this issue : http://samalyse.com/ln/t2-clone.diff I think you should add a __clone() method to the Decorator as well. Reproduce code: --------------- require_once 'PEAR.php'; require_once 'Translation2.php'; $t2 =& Translation2::factory ('XML', array('filename' => 'i18n.xml')); $t2->setLang ('en'); $t2_clone = clone ($t2); $t2_clone->setLang('fr'); // This shouldn't change $t2 echo $t2->get('invoice','documents'); // But it does... Expected result: ---------------- Invoice Actual result: -------------- Facture]]> 2005-02-28T18:36:21-00:00 http://pear.php.net/bugs/3641#1109867846 This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. -- patch applied, many thanks. I also added it to Translation2_Decorator, is it working as expected?]]> This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. -- patch applied, many thanks. I also added it to Translation2_Decorator, is it working as expected?]]> 2005-03-03T16:37:26-00:00