Summary: | Length of filtered playlist is not calculated - total playlist length displayed | ||
---|---|---|---|
Product: | [Applications] amarok | Reporter: | eugen.dahm |
Component: | Playlist | Assignee: | Amarok Developers <amarok-bugs-dist> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | cuentanumerouno, kiwiiii, matej, nhn, oskar.jauch, robert, shlomif, teo |
Priority: | LO | ||
Version: | 2.8.0 | ||
Target Milestone: | --- | ||
Platform: | Fedora RPMs | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: | screenshot, showing the faulty behaviour |
Description
eugen.dahm
2011-09-01 00:44:02 UTC
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. *** |