Thread: [Secureideas-base-devel] error with graphing
Brought to you by:
secureideas,
sinukas
From: Kevin J. <kjo...@se...> - 2004-09-18 20:29:19
|
Hi- I just noticed that when I click the "Graph alert data" link on the front page jpGraph gives an error... to fix it I needed to comment out the include of base_capabilities.php in base_state_query.inc.php I looked quickly if I could see why but don't see anything and was wondering if this is just a problem here. Kevin |
From: Chris S. <ch...@co...> - 2004-09-19 02:21:46
|
Kevin Johnson wrote: >Hi- > >I just noticed that when I click the "Graph alert data" link on the >front page jpGraph gives an error... to fix it I needed to comment out >the include of base_capabilities.php in base_state_query.inc.php > >I looked quickly if I could see why but don't see anything and was >wondering if this is just a problem here. > > This appears to be an issue with the charting library (I have the latest). It seems to me that it is picking up EVERY error, regardless of importance, and failing on it. It simply does not function as it is used in BASE in PHP 5. The "errors" it is reporting are E_NOTICE level, and really aren't worth halting on. It's reporting multiple session_start() use, use of var $var in class definitions as deprecated, etc., etc.. I'll do some looking into why it's reporting these as critical errors, it may be something in how BASE has been configured to use the graphing library. Commenting the capability check out will do it for the graphing page, but it will generate errors if you go to any other page that offers the mail capability links. My suggestion is give me a chance to fix it, and until do, leave the graphing library broken since it will be used in fewer pages. I also wrote the author of the PEAR::Image_Graph library about adding Pie charting support. He mentioned that they do not have support in current releases, but he is getting together with the authors of GraPHPite (http://graphpite.sourceforge.net) and they are merging that into PEAR now, so soon we will have a release of PEAR::Image_Graph that will feature all the features we need, and on surface testing appears to be much faster than jpgraph. Until then, I will create a feature request and assign it to myself in the bug tracker. -- Chris Shepherd Wise man say, chemist who falls into acid, absorbed in his work. |
From: Chris S. <ch...@co...> - 2004-09-19 03:32:21
|
Chris Shepherd wrote: > This appears to be an issue with the charting library (I have the > latest). It seems to me that it is picking up EVERY error, regardless > of importance, and failing on it. It simply does not function as it is > used in BASE in PHP 5. The "errors" it is reporting are E_NOTICE > level, and really aren't worth halting on. It's reporting multiple > session_start() use, use of var $var in class definitions as > deprecated, etc., etc.. I'll do some looking into why it's reporting > these as critical errors, it may be something in how BASE has been > configured to use the graphing library. Commenting the capability > check out will do it for the graphing page, but it will generate > errors if you go to any other page that offers the mail capability > links. My suggestion is give me a chance to fix it, and until do, > leave the graphing library broken since it will be used in fewer pages. As a followup, the error is generated by the forced checking of $GLOBALS['php_errormsg'] in the jpgraph.php file (lines 432 through 434). If you comment out that block it works fine. I can't imagine we're the first to encounter this error. If you could provide me with thorough details on what versions of PHP you're currently running, and what the error it's giving you is. -- Chris Shepherd Wise man say, chemist who falls into acid, absorbed in his work. |
From: Kevin J. <kjo...@se...> - 2004-09-19 03:43:51
|
On Sat, 2004-09-18 at 23:34, Chris Shepherd wrote: > Chris Shepherd wrote: >=20 > > This appears to be an issue with the charting library (I have the=20 > > latest). It seems to me that it is picking up EVERY error, regardless=20 > > of importance, and failing on it. It simply does not function as it is=20 > > used in BASE in PHP 5. The "errors" it is reporting are E_NOTICE=20 > > level, and really aren't worth halting on. It's reporting multiple=20 > > session_start() use, use of var $var in class definitions as=20 > > deprecated, etc., etc.. I'll do some looking into why it's reporting=20 > > these as critical errors, it may be something in how BASE has been=20 > > configured to use the graphing library. Commenting the capability=20 > > check out will do it for the graphing page, but it will generate=20 > > errors if you go to any other page that offers the mail capability=20 > > links. My suggestion is give me a chance to fix it, and until do,=20 > > leave the graphing library broken since it will be used in fewer pages. >=20 > As a followup, the error is generated by the forced checking of=20 > $GLOBALS['php_errormsg'] in the jpgraph.php file (lines 432 through=20 > 434). If you comment out that block it works fine. I can't imagine=20 > we're the first to encounter this error. If you could provide me with=20 > thorough details on what versions of PHP you're currently running, and=20 > what the error it's giving you is. The error is just saying General PHP Error: and then is blank.... version is 4.3.8 on Fedora Core 2 |
From: Chris S. <ch...@co...> - 2004-09-19 04:48:30
|
Kevin Johnson wrote: >On Sat, 2004-09-18 at 23:34, Chris Shepherd wrote: > > >>Chris Shepherd wrote: >> >> >> >>As a followup, the error is generated by the forced checking of >>$GLOBALS['php_errormsg'] in the jpgraph.php file (lines 432 through >>434). If you comment out that block it works fine. I can't imagine >>we're the first to encounter this error. If you could provide me with >>thorough details on what versions of PHP you're currently running, and >>what the error it's giving you is. >> >> > >The error is just saying General PHP Error: and then is blank.... >version is 4.3.8 on Fedora Core 2 > > Try commenting out the code block. The way they check in jpgraph is wrong anyway, they use isset() which returns true if $php_errormsg is an empty string. -- Chris Shepherd Wise man say, chemist who falls into acid, absorbed in his work. |