FireFly Media Server (formerly mt-daapd) › Firefly Media Server Forums › Firefly Media Server › Add-on Software › FireStats – Top 40 php script (now with added charts) › Reply To: FireStats – Top 40 php script (now with added charts)
Wow this is indeed a lot faster.
2 sec render time now.
Their engine is very speedy, I suppose it has to scale to support all the requests they get per minute.
BUT there are 2 issues I could see:
1) The Decades Grafik doesnt work. It doesnt autoscale. If all values are over 100 then you get a chart where all columns are max.
The API isn’t very clever on scaling, I will see what I can do with this.
2) More seriously, you use a hardcoded exec to /opt/bin/wget. Most php’s run with exec() disabled as it’s a security hazard. Can you maneuver around using an exec call and do it in php itself? One should defenitly not need to use wget in php, as php has plenty of functions to access images on the web and load em.
This is my second or third week with php, I will find an alternative approach. I first had the charts render just using the google url, but I need the image copy to reference in the summary page. Now that the speed is ok, I suppose the summary page could always get fresh data – but I don’t like that.
3) If google is for whatever reason down, then it simply wont show the grafics. Can you put in a fallback to the standard libchart method if googles api doesnt answer withing 10 secs?
Not sure, worth considering though 🙂