0019109: Playlists can't be imported thru context menu - MantisBT

ID	Project	Category	View Status	Date Submitted	Last Update

0019109	MMW 5	Playlists	public	2022-05-23 06:45	2022-09-16 13:34

Reporter	peke	Assigned To
Priority	urgent	Severity	minor	Reproducibility	always
Status	closed	Resolution	duplicate
Product Version	5.0
Target Version	5.0.4

Summary	0019109: Playlists can't be imported thru context menu
Description	Playlists can't be imported thru context menu either thru tree or thru track browser. i) Tree example: 1. Right click on playlist in Tree when seen in folder browser 2. There is no Import Playlist, just add files folder which add playlist to scan list ii) Track browser example: 1. Enable Grid (Folders) 2. Right click on playlist listed in Folders Grid 3. There is no Import Playlist, just add files folder which add playlist to scan list Expected that: a) either instead of Add files/folder to library there is Import playlist that will add playlist as sub playlist of imported playlists tree node (easiest solution) b) Create context tree where user will be able to select what will happen with that playlist eg. copy context function and menu structure exactly the same as if user use send to -> Playlist and select playlist https://ibb.co/TRBW7kQ NOTE: If Folder with Only playlists is added into library then all files from imported playlists are automatically added into library.
Additional Information	https://www.mediamonkey.com/forum/viewtopic.php?t=101962 Originating partially from https://www.mediamonkey.com/forum/viewtopic.php?t=101946
Tags	No tags attached.
Attached Files	image.png (15,581 bytes) image.png (15,581 bytes) image-2.png (28,187 bytes) image-2.png (28,187 bytes)

Fixed in build

Ludek 2022-05-23 14:57 developer ~0068206	Sounds to me as duplicate of 0018709 where actually the expected is to add both the files and the playlist to the library.

peke 2022-05-23 21:29 developer ~0068211 Last edited: 2022-05-23 21:48	These are two different issues. This bug as picture shows is how to use Context menu to directly import and control playlist import with one click see b) and accompany picture. While 0018709 refers to using File -> Add/Files to library dialog. which is completely different step to import playlists and involves multiple clicks and diaogs. As explained in forum User can already Add non library files to MM playlist using Send To in https://www.mediamonkey.com/forum/viewtopic.php?p=497806#p497806 same image as b). Also Right clicking on Playlist -> "Add/Remove files to library" is incorrect as desired "Import playlist and its content to library" like Erwin explained in https://www.mediamonkey.com/forum/viewtopic.php?t=101962#p497770 That is why I put two one click expected behavior of Import playlist function where a) is very basic one that put import playlist subplaylist of "Imported Playlists" while b) gives user full control where playlist will be imported. I prefer b) but a) is satisfactory and should be easy to fix.

Ludek 2022-05-24 12:51 developer ~0068213	It's already fixed in 5.0.4.2650 in course of 0018709. i.e. right-click M3U as on your screenshots > Add/Rescan files add/re-scans just the single M3U and the tracks included within the M3U Assigned to Rusty to review (in 5.0.4.2650) whether a change in terminology is needed and/or skipping the Add/Rescan dialog (in case of single playlist/folder adding/re-scanning).

rusty 2022-09-15 19:56 administrator ~0069303	My only comment re. wording would be that: - 'Add/Rescan files to the library...' is generally used for scanning locations (directories and subdirectories) in which the user doesn't know what's being added, and the scan results dialog subsequently appears. - 'Add to Library' seems more appropriate when the user chooses a specific file/files, and so there shouldn't be a need for the scan results dialog (unless there's an error to report). In the case of a playlist, I think that the former approach matches the usecase best, and iiuc, that's what has been implemented--in which case there's nothing left to change. Re-assigning back to Ludek for corroboration / resolution.

Ludek 2022-09-16 13:34 developer ~0069317 Last edited: 2022-09-16 13:34	Agreed, so resolving/closing..