Opened 17 years ago
Closed 17 years ago
#3085 closed defect (fixed)
Smart Playlist: search criteria specifying 'date imported' fails
Reported by: | briand <turbo at talstar dot com> | Owned by: | stuartm |
---|---|---|---|
Priority: | minor | Milestone: | 0.21 |
Component: | mythmusic | Version: | head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
Cannot create a smart playlist using 'Date Imported'; no match against music library is ever found, regardless of date(s) entered.
The code in smartplaylist.cpp is specifying music_songs.date_added, but the actual column in the database is music_songs.date_entered
Patch attached.
Attachments (1)
Change History (6)
Changed 17 years ago by
Attachment: | fix_smartplaylist.patch added |
---|
comment:1 Changed 17 years ago by
Milestone: | unknown → 0.21 |
---|---|
Owner: | changed from Isaac Richards to stuartm |
comment:2 Changed 17 years ago by
The currently attached patch does not completely fix the problem, only corrects the column specified in the Smart Playlist Editor. Even after this change, the 'Date Imported' logic does not find any files. Additionally, the 'Last Played' choice is non-functional. I suspect incorrect/incomplete voodoo in the evaluateDateValue function. (I changed the type on those two choices from ftDate to ftString, and the functions "sort of" work [in as much as dates are not strings]) -- at least, they then start to return qualifying rows and enable the [Save] and [Show Results] buttons.
comment:3 Changed 17 years ago by
comment:4 Changed 17 years ago by
Status: | new → assigned |
---|
I don't use the smart playlist function so I'm not very familiar with it. However with the column name fix in place it seems to work as expected for me. Could you give an example of a rule which isn't working for you?
patch to correct smart playlist