Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#12348 closed Bug Report - General (fixed)

Smart playlist date ranges are incorrectly handled

Reported by: stuartm Owned by: paulh
Priority: minor Milestone: 0.28
Component: Plugin - MythMusic Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description (last modified by stuartm)

This one is a matter of interpretation, and it could be argued that the current behaviour is correct but for the sake of simplicity for the average user I think it needs changing.

If you specify a date range with the 'is between' criteria, it only returns matches which fall within, but not including those dates.

e.g. is between "$DATE - 7 days and $DATE" the results returned exclude 'today'. If you want to include 'today' you have to change the date to point to the future i.e. 'tomorrow' $DATE + 1.

I think it's more intuitive if the ranges specified are inclusive.

Change History (3)

comment:1 Changed 6 years ago by stuartm

Description: modified (diff)

comment:2 Changed 6 years ago by Paul Harrison <pharrison@…>

Resolution: fixed
Status: newclosed

In 5f9c48074f1c42ece0302876adceb607de232fcc/mythtv:

Revert "MythMusic: Fix smart playlists uses lastplay or date_entered."

This reverts commit 428609b782c5644b2e885d861f19dbb354f344c0.

Those changes are required to remove the time part of the lastplay
and date_entered fields so smart playlists using them work correctly.

Fixes #12348.

comment:3 Changed 6 years ago by paulh

Milestone: unknown0.28
Version: UnspecifiedMaster Head
Note: See TracTickets for help on using tickets.