Kevin,
I supplied a unit test with my patch.
However, as noted in the bug report, the fix exposes lots of Xincide code that has not been executed/tested yet. This afternoon I uncovered a bug in some of that untested function evaluation code. A fix for that is underway.
Please hold off on applying the patch for 23468 until further notice. I will attach at least one more patch to the bug to fix the new problem that has come to light.
-Terry
Kevin O'Neill wrote:
Thanks Terry and Mario for your recent patches. I haven't applied them as yet because I want to ensure they work by adding a unit test that exposes the problem. This is especially where the changes are deep within the core engine. I'm working on these and as soon as I have managed to create a the unit test I'll apply, verify and hopefully commit the patches.
-k.
