Created attachment 63278 [details] screenshot, showing the faulty behaviour Version: 2.4.3 (using KDE 4.6.5) OS: Linux This is not about bug 273407 or 267380, it's different but also, maybe related. In earlier versions (don't know since when it's b0rked), the total playlist length was calculated with respect to the search filter. Since the last few versions, it always displays the total length of all songs in the playlist. ATM, I have 50 active songs because of the search filter, and the bar tells me, the total length is 17 days, 1h, 11min, which is more or less the total length of my collection. ps: number of songs is updated after selecting a song (not dynamically, while entering something in the search bar - this was the original behaviour in previous versions). Reproducible: Always Steps to Reproduce: - open a playlist with a fair number of songs - enter something into the search filter Actual Results: the total length of all songs(including the filtered) in the playlist is displayed. Expected Results: total length of the songs that match the filter
Can confirm here - Amarok 2.4.3 on KDE-4.7.0 on Mageia Linux 2 (cauldron).
Setting status to confirmed.
> Target: 2.5 Ok, 2.5 was released and it's NOT fixed. :(.
Thank you for the feedback.
Np :). This bug annoys the hell out of me, but unfortunately, the code base is too big to fix this for myself :/.
Again. Target 2.6 .. missed. Gaaawd. It's now 11!!! months since I submitted this bug report. Could someone pleeeeease be so kind to fix this damn bug. It's not like this should be a hard to track down bug. Would it change something if I donated some money? Does it always take over a year to fix a bug?
(In reply to comment #6) > Again. Target 2.6 .. missed. Gaaawd. It's now 11!!! months since I submitted > this bug report. Could someone pleeeeease be so kind to fix this damn bug. > It's not like this should be a hard to track down bug. Would it change > something if I donated some money? Does it always take over a year to fix a > bug? Please calm down, it's is just a matter of available resources, there were far bigger priorities to solve with the small group of developers we have and the limited time they have available for working on Free Software.
I was thinking about this and came to conclusion that the length and size should be calculated in the view, not in the model. Thanks to beginRemoveRows() we can use the same trick not to recount the length all the time. (this is rather a note to myself)
(In reply to comment #8) > I was thinking about this and came to conclusion that the length and size > should be calculated in the view, not in the model. Thanks to > beginRemoveRows() we can use the same trick not to recount the length all > the time. (this is rather a note to myself) Sorry, but this sounds rather ugly. I don't know if there is, but suppose there is a command line interface /ncurses / whatever alternative interface to amarok. To display the dinamically updated playlist length, you would have to reimplement this functionality in the alternative interface unnecessarily. i don't believe this was the original idea of MVC.
(In reply to comment #9) > Sorry, but this sounds rather ugly. I don't know if there is, but suppose > there is a command line interface /ncurses / whatever alternative interface > to amarok. To display the dinamically updated playlist length, you would > have to reimplement this functionality in the alternative interface > unnecessarily. i don't believe this was the original idea of MVC. The problem with MVC in Qt is that there is no MVC, there is MV, the functionality of Controller being merged to View. But since you're able to advise developers how to fix this, you'll be able to fix this yourself! Please read http://community.kde.org/Amarok/Development/Join on how to start with Amarok development, then submit a patch to http://reviewboard.kde.org/ Thanks!
Still reproducible with Amarok 2.8
I may have tested this script time ago and not sure if it's deployed in my website, but searching in my computer for scripts containing a common typo I use to write, I found your script having "lenght" instead of "length", in a line apparently counting the items in a playlist, in a file named mw.PlayList.js (part of mwEmbed library) https://github.com/kaltura/mwEmbed
*** Bug 313971 has been marked as a duplicate of this bug. ***