| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.1.1 |
| Bug #6699 | Text_Diff::reverse() wrong arument for clone | ||
|---|---|---|---|
| Submitted: | 2006-02-05 17:48 UTC | Modified: | 2006-12-17 11:11 UTC |
| From: | php at scarfhogg dot de | Assigned: | yunosh |
| Status: | Closed | Package: | Text_Diff (version 0.2.0) |
| PHP Version: | 5.1.2 | OS: | |
| Roadmaps: | (Not assigned) | ||
[2006-02-05 17:48 UTC] php at scarfhogg dot de
Description:
------------
Text_Diff::reverse() uses $obj instead of this when trying to clone the object when version_compare(zend_version(), '2', '>') evaluates to true
Test script:
---------------
$diff = new Text_Diff($lines1, $lines2);
$diff->reverse();
Expected result:
----------------
get reversed diff
Actual result:
--------------
Warning: __clone method called on non-object in ... Diff.php on line 77
Fatal error: Call to undefined method stdClass::getDiff() in ... Diff/Renderer.php on line 78
line 77 of Diff.php should be
$rev = clone $this;
Comments
[2006-02-05 19:23 UTC] yunosh
[2006-02-05 19:24 UTC] yunosh
Sorry, PEAR bug website is broken.
This has been fixed with the Text_Diff 0.2.1 release.