search

ransome1 / sleek

todo.txt manager for Linux, Windows and MacOS, free and open-source (FOSS)
https://github.com/ransome1/sleek/wiki
MIT License
1.28k stars 99 forks source link

projects/contexts list length, addresses ransome1/sleek#616 #636

Closed zerodat closed 5 months ago

zerodat commented 5 months ago

This code change to FilterQuery.js will address the problem described in issue #616. The old code worked with the todoObject as it behaved in version 1.2.9 of sleek, but fails in 2.0.6. I think that the old version must have represented missing projects or contexts as undefined or nil, while the new version represents this case as an empty list. This PR adds an explicit check of the length of the lists.

ransome1 commented 5 months ago

@zerodat lovely, I just checked it out and it works like a charm. Thanks for supporting once again.