View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009706 | MMW v4 | DLNA/UPnP | public | 2012-09-14 11:34 | 2013-02-04 11:51 |
| Reporter | Ludek | Assigned To | |||
| Priority | urgent | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 4.1 | ||||
| Target Version | 4.1 | Fixed in Version | 4.1 | ||
| Summary | 0009706: Allow control of MediaMonkey player should be available in Player options | ||||
| Description | Currently the option '[x] Allow control of MediaMonkey player via UPnP/DLNA' is quite hidden, as it is available on the Media Sharing tab only. I guess that the option should be available via right-click on player (similarly like the 'Level Playback Volume' option) or in the Options|Player. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 1620 | ||||
|
|
1) I think that this functionality (allow control of MM player via dlna) should probably remain on the DLNA config section because it will likely involve some of the security settings that appear on that panel (i.e. using MM as a renderer should only be possible for specific defined addresses that have been accepted--even though I don't believe that this is currently implemented). 2) We could add a shortcut as suggested to the player. e.g. right click on the player, and show: 'Play from: UPnP/DLNA source' > - source 1 - source 2 3) The wording for remote output should be changed to 'Play to:' and dlna devices should be described as such i.e. Play to: - Internal player - device 1 (UPnP/DLNA renderer) - device 2 (UPnP/DLNA renderer) 4) We could add controls for Play to: to the config menu, by adding the following to the Player menu: Send output to: [ ] Internal player [ ] device1 (UPnP/DLNA renderer) [ ] device2 (UPnP/DLNA renderer) what do you think? |
|
|
Re: 1) As you say, there is no other security check than enabling the checkbox (that is disabled by default). I think this is fine as also all other apps (e.g. WMP, Foobar, XBMC) behaves same way. i.e. I would suggest to add the option also to the player's right click menu as originaly suggested. The main reason is that it would be more visible to users. Alternativelly we could add it to main menu -> Play ? Re 2) I think that 'Play from' doesn't make sense and is not needed at all. It all depends from where user plays the song. If the song is played from MMW library then it is played using MMW library server, if it is played from Media Servers - > tVeristy node then it is played from tVersity server. Re 3) & 4) I believe that 3 and 4 are rather part of issue 0010320 that LL entered recently. Yes, we can change wording from "Select Player" -> "Play to" and we can also add it to options as suggested in 0010320, but we need to somehow integrate also with the Winamp player or deprecate this Winamp option. But this should be tracked in 0010320 |
|
|
1) OK--so let's assume that we won't add security to DLNA renderer for some time. Since the player already has a 'Setup player' shortcut, I think the best approach would be to move DLNA rendering to the Player menu. That way it's accessible from a location in the config menu that makes more sense, and via the player context menu. e.g. below Section for 'Default 'Double-click' Action' --DLNA Settings-- [ ] Accept input/control of MediaMonkey player via UPnP/DLNA Renderer name: _MediaMonkey Player_ Note: the change in the strings + inclusion of a default renderer name 2) OK--that suggestion was based on an assumption that we'd want some type of security for what can control the DLNA renderer, but we can see if that's needed in the future. 3/4) Tracked at 0010320 |
|
|
Fixed in 1620. |
|
|
Verified 1623 |
|
|
Verified 1623 |
