php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #56202 Stats-per-release graph broken
Submitted: 2004-09-30 09:51 UTC Modified: 2005-06-01 10:14 UTC
From: eru@php.net Assigned: danielc (profile)
Status: Closed Package: PECL website (PECL)
PHP Version: Irrelevant OS: Irrelevant
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: eru@php.net
New email:
PHP Version: OS:

 

 [2004-09-30 09:51 UTC] eru@php.net 
Description:
------------
On modifying the statistics graph for a package (adding a release at "Show graph of" with its own color and pressing "Update graph") no picture is being shown. Reason is multiple errors in the picture-generating script...








Reproduce code:
---------------
Try to modify a download statistic or browse to
http://pear.php.net/package-stats-graph.php?pid=14&releases=1092_339900


Expected result:
----------------
A nice graph.

Actual result:
--------------
Warning: Invalid argument supplied for foreach() in /usr/local/www/pearweb/public_html/package-stats-graph.php on line 63

Warning: array_values(): The argument should be an array in /usr/local/www/pearweb/public_html/package-stats-graph.php on line 103

Warning: Cannot modify header information - headers already sent by (output started at /usr/local/www/pearweb/public_html/package-stats-graph.php:63) in /usr/local/www/pearweb/public_html/package-stats-graph.php on line 116

Warning: Cannot modify header information - headers already sent by (output started at /usr/local/www/pearweb/public_html/package-stats-graph.php:63) in /usr/local/www/pearweb/public_html/package-stats-graph.php on line 117

Warning: Cannot modify header information - headers already sent by (output started at /usr/local/www/pearweb/public_html/package-stats-graph.php:63) in /usr/local/www/pearweb/public_html/package-stats-graph.php on line 118

Warning: Cannot modify header information - headers already sent by (output started at /usr/local/www/pearweb/public_html/package-stats-graph.php:63) in /usr/local/www/pearweb/public_html/package-stats-graph.php on line 119
JpGraph Error: Graph::Add() You tried to add a null plot to the graph.

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2005-06-01 06:45 UTC] klaus at capitalfocus dot org 
It seems that the whole problem is the ampersand as an HTML entity. If it is changed to a simple ampersand, it works just fine. Of course, it takes forever for PEAR to generate a complete graph. But on the statistics page it shows up properly all the same.

This should be a simple fix. Unfortunately I don't have the pearweb module checked out else I would submit a patch. (The problem is in the "updateGraph" javascript function: line 250 of /pearweb/public_html/package-stats.php.)
 [2005-06-01 10:14 UTC] danielc at analysisandsolutions dot com 
Fixed in CVS.  Thanks Klaus.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Apr 25 22:01:29 2024 UTC