Smart playlists with criteria involving timestamps do not work #19

Closed
slosd opened this Issue Dec 10, 2014 · 2 comments

Projects

None yet

3 participants

@slosd
Contributor
slosd commented Dec 10, 2014

The search string for criteria like "Last time played" isn't constructed correctly. Also, playlists like "all tracks not played in the last X weeks/days/..." don't contain any songs for which the __last_played field isn't set yet, although that is probably the expected behaviour.

Here is a patch with changes I made to fix it for me personally. Possibly that's not how it should be properly fixed. In particular, I'm not sure if the localization I removed is actually desired.

@virtuald virtuald added this to the Exaile 3.4.3 milestone Dec 10, 2014
@virtuald virtuald added the bug label Dec 10, 2014
@virtuald
Contributor

The localization was a bug, fixed in 708807e

@virtuald virtuald closed this in #20 Dec 15, 2014
@johannspies

3.4.0 and 3.4.5 on Debian. The smart play list using the following type of configuration, does not work. The gui resets the setting to seconds:

(dp1
S'custom_params'
p2
(lp3
sS'name'
p4
Vouer as 4 dae
p5
sS'track_count'
p6
I-1
sS'random_sort'
p7
I00
sS'search_params'
p8
(lp9
(lp10
S'__last_played'
p11
aS'<'
a(lp12
F4
aVdays
p13
aaasS'or_match'
p14
I00
s.%

With this setting, when , in the gui, I open the edit option on the setting, I get:

Last played in the past 4 seconds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment