Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#10710 closed Patch - Feature (fixed)

MythMusic immediately play songs from playlist editor

Reported by: trebor_s@… Owned by: paulh
Priority: minor Milestone: 0.27
Component: Plugin - MythMusic Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

I read about some usability complains on the mailing list. One issue is that the songs that were added from the playlist editor are not played immediately. The user had to go to the playlist, select the song and press play.

Attached is a patch that adds new functionality. When the user is in the playlist editor screens and navigates through his collection he can now press PLAY. This will also add the albums or tracks to the playlist (same as pressing SELECT) but it will start playing the first of the newly added songs immediately.

Robert S

Attachments (4)

mythmusic_play_tracks_from_playlisteditor_tree.patch (2.9 KB ) - added by trebor_s@… 14 years ago.
mythmusic_add_menu.patch (1.7 KB ) - added by trebor_s@… 14 years ago.
Adds a menu to add/replace files and play them directly
mythmusic_keybinding_for_play.patch (1.2 KB ) - added by trebor_s@… 14 years ago.
Use a keybinding for the functionality instead of play
patch.playbutton (4.1 KB ) - added by mayfields@… 14 years ago.
Includes additional cases preventing segfault eg playlists

Download all attachments as: .zip

Change History (11)

by trebor_s@…, 14 years ago

comment:1 by stuartm, 14 years ago

Milestone: unknown0.26
Owner: set to stuartm
Status: newaccepted
Version: UnspecifiedMaster Head

by trebor_s@…, 14 years ago

Attachment: mythmusic_add_menu.patch added

Adds a menu to add/replace files and play them directly

by trebor_s@…, 14 years ago

Use a keybinding for the functionality instead of play

comment:2 by mayfields@…, 14 years ago

Segfaults when pressing defined key on playlists, smart playlists. These nodes don't have metadata associated and need to be treated differently. Requires additional cases such as:

if (mnode->getAction() == "playlists") 

Example patch attached, provides similar (but not identical) functionality.

by mayfields@…, 14 years ago

Attachment: patch.playbutton added

Includes additional cases preventing segfault eg playlists

comment:3 by Kenni Lund [kenni a kelu dot dk], 13 years ago

Milestone: 0.260.27

comment:4 by Paul Harrison <mythtv@…>, 13 years ago

Resolution: fixed
Status: acceptedclosed

In fe13684b4d5d635a6fd930ab49358cdf9192c2de/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:5 by paulh, 13 years ago

Owner: changed from stuartm to paulh

comment:6 by warpme@…, 13 years ago

Paul, Quick Q to fe13684b4d5d635a6fd930ab49358cdf9192c2de commit: for me it works OK any higher node that single song (album, dir with albums, etc). It is not working for single song. It this planned behaviour or rather bug ?

comment:7 by Paul Harrison <mythtv@…>, 13 years ago

In 3860342e3452ecc6db7dc487a2924cbb2601b1cc/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.