Opened 17 years ago

Closed 11 years ago

#5742 closed Patch - Feature (Won't Fix)

MythWeb Music last play filter

Reported by: otto at kolsi dot fi Owned by: Rob Smith
Priority: minor Milestone: unknown
Component: Plugin - MythWeb Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Attached patch adds Last Play filter to MythWeb Music Random Mix Maker. Together with Rating filter, this can be used to e.g. find "good songs not played recently".

I first used "<" and ">" symbols in UI but then changed these to "earlier than" and "later than" texts which feel more natural.

Patch has couple of new translations and English language file modifications are included in the patch. Not sure if this is needed or not.

Attachments (2)

mythweb-music-lastplay.patch​ (5.3 KB ) - added by otto at kolsi dot fi 17 years ago.
mythweb-music-lastplay-v2.patch​ (16.3 KB ) - added by otto at kolsi dot fi 17 years ago.

Download all attachments as: .zip

Change History (19)

by otto at kolsi dot fi, 17 years ago

Attachment: mythweb-music-lastplay.patch​ added

comment:1 by otto at kolsi dot fi, 17 years ago

If I remember correctly, there was a bug in music scan/import which did set the lastplay column in music_songs table to the timestamp of import (when it should be NULL).

If anyone is interested, following SQL statement can be used to fix this small issue in DB: 

update music_songs set lastplay = NULL where numplays = 0;

comment:2 by Rob Smith, 17 years ago

Status: new → infoneeded_new

Patch doesn't apply cleanly to current trunk, care to update?

comment:3 by otto at kolsi dot fi, 17 years ago

Yes, I can update the patch but that'll take some time since I'll probably have to move my "production" from fixes to trunk. So if this ticket could stay open a bit longer than normally...

by otto at kolsi dot fi, 17 years ago

Attachment: mythweb-music-lastplay-v2.patch​ added

comment:4 by otto at kolsi dot fi, 17 years ago

Attached new patch which applies to mythweb trunk.

comment:5 by Rob Smith, 17 years ago

Status: infoneeded_new → new

comment:6 by Rob Smith, 17 years ago

Owner: changed from xris to Rob Smith
Status: new → accepted

comment:7 by Rob Smith, 16 years ago

Component: mythweb → Plugin - MythWeb
Owner: Rob Smith removed
Status: accepted → new

comment:8 by Rob Smith, 16 years ago

Milestone: unknown → 0.23

comment:9 by robertm, 16 years ago

Pushing to .24 since we're in freeze.

comment:10 by robertm, 16 years ago

Milestone: 0.23 → 0.24

comment:11 by robertm, 15 years ago

Milestone: 0.24 → unknown
Owner: set to paulh
Status: new → assigned

comment:12 by paulh, 15 years ago

Owner: changed from paulh to Rob Smith

comment:13 by Rob Smith, 15 years ago

Milestone: unknown → 0.25

comment:14 by stuartm, 15 years ago

Milestone: 0.25

Milestone 0.25 deleted

comment:15 by Rob Smith, 15 years ago

Milestone: 0.25 → unknown
Version: head → Trunk Head

We're planning a full rewrite of the main mythweb template. This is going to be postponed until after that is complete.

comment:16 by stuartm, 14 years ago

Type: patch → Patch - Feature

Kormoc, same question as #8970. is that planned re-write now postponed? Can this patch not be committed now in time for 0.25?

comment:17 by Rob Smith, 11 years ago

Resolution: → Won't Fix
Status: assigned → closed

Given Mythweb is EOL, I'm closing new features. This should be done via the backend webserver now.

Note: See TracTickets for help on using tickets.