phil65 / script.extendedinfo

script.extendedinfo
http://forum.xbmc.org/showthread.php?tid=160558
GNU General Public License v2.0
59 stars 79 forks source link

Top artists Listeners number #5

Closed Nessus85100 closed 10 years ago

Nessus85100 commented 10 years ago

In "TopArtists" (https://github.com/Nessus85100/script.extendedinfo/blob/master/default.py#L187) feature the 'Listeners' call returns a number that is not separated by thousands (eg 111310 instead of 111,310). Is it possible to add a separate by thousands in that number ?

Thanks Nessus

phil65 commented 10 years ago

yes, can do that.

phil65 commented 10 years ago

https://github.com/phil65/script.extendedinfo/commit/c369455a15e8aee8d8523ecead97e878fef3320e untested.

Nessus85100 commented 10 years ago

Is it something changed in TopArtists and NearEvents property's ?. I get no data in the skin. I've checked script's addon_data folder and the txt data files are there.

phil65 commented 10 years ago

NearEvents is workin here at least, didnt test TopArtists yet. Was it broken by the commit? Debug Log?

Nessus85100 commented 10 years ago

They were working with my fork. Here is the debug log : http://xbmclogs.com/show.php?id=276777 and here is how my static items are coded : http://xbmclogs.com/show.php?id=276783

phil65 commented 10 years ago

fixed now?

Nessus85100 commented 10 years ago

TopArtists yes (and the locale for listeners number), NearEvents still not working. Debug log again ?

phil65 commented 10 years ago

yes, download newest version and debug log again, please.

Nessus85100 commented 10 years ago

http://xbmclogs.com/show.php?id=276870

phil65 commented 10 years ago

how do you call the script exactly?

Nessus85100 commented 10 years ago

With an onload in Home window... onload condition="System.HasAddon(script.extendedinfo)">RunScript(script.extendedinfo,info=nearevents)</onload

Forgot to tell you that latest version gives me an error but the TopArtists is working

phil65 commented 10 years ago

new version pushed. try again, please. :)

Nessus85100 commented 10 years ago

Got it. All working fine now.

Thanks a lot

phil65 commented 10 years ago

no problem. :) keep in mind that it can break again temporarily because i´m restructuring some things (perhaps dividing it into an addon for the daemon and an addon for the info labels, i already moved out the googlemaps part to script.maps.browser )

Nessus85100 commented 10 years ago

Ok. No problem.