chart code incompatibility with new PEAR module

[dmorton]

The PEAR module changed API on us somehow, and the Legend text does not appear in newer versions. Works at least up to Image_Graph 0.7.1 alpha, and is reported broken in 0.7.2

[rjl]

It's been confirmed that downgrading to Image_Graph 0.7.1 solves the problem, which would seem to confirm that something significant changed in 0.7.2. That said, 0.7.2 should probably become the minimum version requirement going forward, since it offers a number of new features we'll almost certainly want to take advantage of, including built-in support for grouping items with values below a specified threshold into a catch-all "others" group. It's just a matter of adapting the current chart code to work with 0.7.2 and increasing the minimum version requirement in configtest.php.

[rjl]

An update from Michael Enkelis, who points out that the behaviour in Image_Graph 0.7.2 is a known bug in the Pie.php file, and has been fixed in the CVS version. Evidently using this CVS version of Pie.php as a drop-in replacement for the version shipped with 0.7.2 solves the problem.

[dmorton]

We could add a check to configtest.php to warn about this version...

[rjl]

Have configtest.php direct users to the one-line patch attached to this ticket, which fixes the problem with 0.7.2. For the other reasons I mentioned earlier, I still think 0.7.2 needs to be considered the new minimum--the catch-all grouping of items with values below a certain threshold is one of the features we were going to have to implement ourselves, and it's now standard in 0.7.2. Once 0.7.3 becomes available we can avoid this problem altogether by upping the minimum version requirement to 0.7.3.

[dmorton]

[1127] added warning to configtest.php

[anonymous]

How do you apply this patch?

-Thanks

[anonymous]

Okay, so what is the attachment (file name?) and where does it go?

[anonymous]

The attachment "03/21/07" is the complete Image/Graph/Plot/Pie.php

modified.

[rjl]

Patch for Graph/Plot/Pie.php (context diff)

[dmorton]

Ticket should have been closed, probably lost in the server crash.