Summary: | collection "edit filter" dialog should be "append" filter dialog. | ||
---|---|---|---|
Product: | [Applications] amarok | Reporter: | Alan Ezust <alan.ezust> |
Component: | Collection Browser | Assignee: | Amarok Developers <amarok-bugs-dist> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | 123kash, ralf-engels |
Priority: | NOR | Keywords: | regression |
Version: | 2.4-GIT | ||
Target Milestone: | --- | ||
Platform: | Unlisted Binaries | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 2.4.1 |
Description
Alan Ezust
2010-12-17 00:21:09 UTC
Please upgrade to the latest stable, Amarok 2.3.2 or try 2,4 beta. The filter works correctly here (besides the numbering being wrong, but that is another bug) Oops, I was completely wrong, it even is worse now in 2.4-git: Choosing a rating between 1 (half a star) and 7 (3.5 stars) produces this result: rating:>0.5 OR rating:<3.5 So it takes half the values and this is clearly a regression, the stars were always 1 star = 2 rating points "Semi-star" values seems not bug but feature (http://gitweb.kde.org/amarok.git/commit/c9966b25e259568de09688d32db7ee7ae6ef8795). Reporter's problem looks strange, need to look up more. Nope nothing wrong. Can't reproduce that. Probably you should really just try newer version.
> Choosing a rating between 1 (half a star) and 7 (3.5 stars) produces this
> result:
> rating:>0.5 OR rating:<3.5
First, my report was about INVERTED expressions and yours
is not inverted. But just looking at what you pasted, it is not correct!
I would expect to see something like this:
rating: >= 0.5 AND rating =< 3.5
Although between is ambiguous, so I would also accept
rating: > 0.5 AND rating < 3.5
What does between mean? Does it mean inclusive or non-inclusive?
Because an "inverted between" should NOT include those values, that is for sure.
Does that mean a "between" *does* include those values? I don't know!
I hate that word.
What expression does it show when you invert it?
PS: Ignore the other stuff I said about nothing being done unless I click "Append". I didn't realize that by clicking append, I see an expression immediately in the filter lineedit, that's cool! But clicking "ok" and not seeing anything at all is confusing. What is the difference between ok/cancel? The dialog title: "Edit Filter" is confusing. It is not editing my existing filter at all. It is just appending expressions to the existing filter field. So the dialog title should be "create/append filter", and the instructions which currently say "edit the filter for finding tracks" should say "click append to see your expression appended to the filter lineedit" or something. And then I would not expect to see my previous filter already populating the fields in the dialog form as I do now. Further, the "edit" button next to the filter field should perhaps be a "+" add filter button. And now that I've tried the latest git checkout, I see that the expressions are indeed translated correctly when you click "append" So I guess the only issue left on this ticket is the confusion between what "edit filter" means, when it should be "append", and the ok/cancel that should be merged into a single "close" button. Please do not remove the keywords until this is fixed. Also the filtering is still wrong, as it doesn't filter by full points but by half only. I don't see writing 3 stars as "3" as a bug. However I see a wish that the edit dialog would remember the last selected values. Also I see the problem with the append button since I fell into the same trap a couple of times myself. Well, initial problem already fixed, current could be solved by http://git.reviewboard.kde.org/r/100250/, but It couldn't be accepted before 2.4 release because of string freeze (at least). And It's impossible to rename Edit filter button before 2.4 also. So I think, this bug can not be release_blocker. Git commit 944febeac820d90b4f47e70f2a51500b0eb4f45f by Sergey Ivanov Pushed by ivanov into branch master Filter Editor restyling. (RR: 100250) BUG: 260333 M +1 -0 ChangeLog M +44 -40 src/core/meta/support/MetaConstants.cpp M +257 -132 src/dialogs/EditFilterDialog.cpp M +25 -11 src/dialogs/EditFilterDialog.h M +162 -259 src/dialogs/EditFilterDialog.ui M +22 -1 src/widgets/MetaQueryWidget.cpp M +22 -15 src/widgets/MetaQueryWidget.h M +3 -3 src/widgets/Token.cpp M +4 -4 src/widgets/Token.h M +3 -4 src/widgets/TokenDropTarget.cpp M +2 -1 src/widgets/TokenPool.cpp M +2 -2 src/widgets/TokenWithLayout.cpp M +2 -2 src/widgets/TokenWithLayout.h http://commits.kde.org/amarok/944febeac820d90b4f47e70f2a51500b0eb4f45f |