@mattab opened this issue on November 11th 2012

It should be sorted by nb_hits but it's not:

http://demo.piwik.org/index.php?idSite=25&date=today&hideMetricsDoc=1&flat=1&filter_sort_column=nb_hits&apiModule=Actions&apiAction=getPageUrls&period=year&filter_limit=30&module=API&token_auth=x&format=json&method=API.getProcessedReport&language=en&

@mattab commented on November 11th 2012

(In [7432]) it looks like it fixes #3532 without side effect? Please test on demo

@BeezyT commented on November 11th 2012

If I remember correctly, I wrote this code and there was a reason for the order. You might want to test extra carefully...

Btw, we really need to have the unit tests back!

@mattab commented on November 13th 2012

Yes I thought the same, but I run the integration tests and they look good - surely this would have failed the tests?? ;-)

Yes we badly need tests back and Stefan & Anthon are onto it hopefully in the next days.. I cant wait myself it's so frustrating without it!

This issue was closed on November 13th 2012
Powered by GitHub Issue Mirror