Re: Not played since filter: really working?

This is a multi-part message in MIME format.
--------------U5AW7sQ7Sofv1uCRF8REb0y2
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

The following patch does the trick. Since I am not a lisp programmer, it 
is maybe not very elegant, but you'll get the idea.

Best,

On 11/26/25 01:21, Erica Qi wrote:
> Yes of course.
> 
> I responded this morning but I must have been on the wrong email account.
> I'll take a look.
> Erica.
> 
> Sent from Proton Mail for Android.
> 
> -------- Original Message --------
> On Tuesday, 11/25/25 at 13:14 Yoni Rabkin <[email protected]> wrote:
> Igor Sosa Mayor <[email protected]> writes:
> 
>> Hi emms-users,
>>
>> I have already a small problem. In this case with the filter factory
>> defined as "Not played since". If I filter my tracks with "Played
>> since" and "Not played since" 30 days, I get exactly the same
>> results... Either it is a problem with the way my tracks are storing
>> the time or a problem with the code.
>>
>> More precisely my question is: is this code working at expected?
>>
>> (defun emms-filters-make-filter-not-played-within (days)
>>    "Make a not played since DAYS filter."
>>    (lambda (track)
>>      (funcall (emms-filters-make-filter-played-within days) track)))
>>
>> Thanks in advance!
> 
> Erica, can you please have a look into this when you have time?
> 
> --
>     "Cut your own wood and it will warm you twice"
> 

--------------U5AW7sQ7Sofv1uCRF8REb0y2
Content-Type: text/x-patch; charset=UTF-8; name="0001-Not-played-since.patch"
Content-Disposition: attachment; filename="0001-Not-played-since.patch"
Content-Transfer-Encoding: base64

RnJvbSAwNWUwNjc4YTcyNWNkYzIzN2VlZjNlZGIwYmNhY2JiZDliZjYxOGM1IE1vbiBTZXAg
MTcgMDA6MDA6MDAgMjAwMQpGcm9tOiBJZ29yIFNvc2EgTWF5b3IgPGpvc2VsZW9wb2xkbzE3
OTJAZ21haWwuY29tPgpEYXRlOiBXZWQsIDI2IE5vdiAyMDI1IDA4OjEyOjU3ICswMTAwClN1
YmplY3Q6IFtQQVRDSF0gTm90IHBsYXllZCBzaW5jZS4KCi0tLQogZW1tcy1maWx0ZXJzLmVs
IHwgMTAgKysrKysrLS0tLQogMSBmaWxlIGNoYW5nZWQsIDYgaW5zZXJ0aW9ucygrKSwgNCBk
ZWxldGlvbnMoLSkKCmRpZmYgLS1naXQgYS9lbW1zLWZpbHRlcnMuZWwgYi9lbW1zLWZpbHRl
cnMuZWwKaW5kZXggMTI0ODVhZi4uMGQxYWJkMiAxMDA2NDQKLS0tIGEvZW1tcy1maWx0ZXJz
LmVsCisrKyBiL2VtbXMtZmlsdGVycy5lbApAQCAtODYwLDcgKzg2MCw3IEBAIFVzZXMgYSBy
ZWdleCBhbmNob3JpbmcgZGlybmFtZSB0byB0aGUgYmVnaW5uaW5nIG9mIHRoZSBleHBhbmRl
ZCBwYXRoLiIKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgJygoIkRpcmVjdG9yeTog
IiAoOnN0cmluZyAuIG5pbCkpKSkKIAogOzsgc2Vjb25kcyBpbiBhIGRheSAoKiA2MCA2MCAy
NCkgPSA4NjQwMAotKGRlZnVuIGVtbXMtZmlsdGVycy1tYWtlLWZpbHRlci1wbGF5ZWQtd2l0
aGluIChkYXlzKQorKGRlZnVuIGVtbXMtZmlsdGVycy1tYWtlLWZpbHRlci1wbGF5ZWQtd2l0
aGluLW9yLW5vdC13aXRoaW4gKGRheXMgJm9wdGlvbmFsIG5vdC13aXRoaW4pCiAgICJTaG93
IG9ubHkgdHJhY2tzIHBsYXllZCB3aXRoaW4gdGhlIGxhc3QgbnVtYmVyIG9mIERBWVMuIgog
ICAobGV0ICgoc2Vjb25kcy10by10aW1lIChzZWNvbmRzLXRvLXRpbWUgKCogZGF5cyA4NjQw
MCkpKSkKICAgICAjJyhsYW1iZGEgKHRyYWNrKQpAQCAtODcwLDE2ICs4NzAsMTggQEAgVXNl
cyBhIHJlZ2V4IGFuY2hvcmluZyBkaXJuYW1lIHRvIHRoZSBiZWdpbm5pbmcgb2YgdGhlIGV4
cGFuZGVkIHBhdGguIgogICAgICAgICAgICAgICBsYXN0LXBsYXllZCkKICAgICAgICAgICAo
YW5kIChzZXRxIGxhc3QtcGxheWVkCiAgICAgICAgICAgICAgICAgICAgICAoZW1tcy10cmFj
ay1nZXQgdHJhY2sgJ2xhc3QtcGxheWVkIG5pbCkpCi0gICAgICAgICAgICAgICAodGltZS1s
ZXNzLXAgbWluLWRhdGUgbGFzdC1wbGF5ZWQpKSkpKSkKKyAgICAgICAgICAgICAgIChpZiBu
b3Qtd2l0aGluCisgICAgICAgICAgICAgICAgICAgKHRpbWUtbGVzcy1wIGxhc3QtcGxheWVk
IG1pbi1kYXRlKQorICAgICAgICAgICAgICAgICAodGltZS1sZXNzLXAgbWluLWRhdGUgbGFz
dC1wbGF5ZWQpKSkpKSkpCiAKIChlbW1zLWZpbHRlcnMtcmVnaXN0ZXItZmlsdGVyLWZhY3Rv
cnkgIlBsYXllZCBzaW5jZSIKLSAgICAgICAgICAgICAgICAgICAgICAgICAgICAgJ2VtbXMt
ZmlsdGVycy1tYWtlLWZpbHRlci1wbGF5ZWQtd2l0aGluCisgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICdlbW1zLWZpbHRlcnMtbWFrZS1maWx0ZXItcGxheWVkLXdpdGhpbi1vci1u
b3R3aXRoaWhpbgogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAnKCgiRGF5czogIiAo
Om51bWJlciAuIG5pbCkpKSkKIAogKGRlZnVuIGVtbXMtZmlsdGVycy1tYWtlLWZpbHRlci1u
b3QtcGxheWVkLXdpdGhpbiAoZGF5cykKICAgIk1ha2UgYSBub3QgcGxheWVkIHNpbmNlIERB
WVMgZmlsdGVyLiIKICAgKGxhbWJkYSAodHJhY2spCi0gICAgKGZ1bmNhbGwgKGVtbXMtZmls
dGVycy1tYWtlLWZpbHRlci1wbGF5ZWQtd2l0aGluIGRheXMpIHRyYWNrKSkpCisgICAgKGZ1
bmNhbGwgKGVtbXMtZmlsdGVycy1tYWtlLWZpbHRlci1wbGF5ZWQtd2l0aGluLW9yLW5vdHdp
dGhpaGluIGRheXMgdCkgdHJhY2spKSkKIAogKGVtbXMtZmlsdGVycy1yZWdpc3Rlci1maWx0
ZXItZmFjdG9yeSAiTm90IHBsYXllZCBzaW5jZSIKICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgJ2VtbXMtZmlsdGVycy1tYWtlLWZpbHRlci1ub3QtcGxheWVkLXdpdGhpbgotLSAK
Mi41Mi4wCgo=

--------------U5AW7sQ7Sofv1uCRF8REb0y2--