Multi Player Support

[Groovylein]

Ground work for multiple players.

Basis to solve Spotify issues or other streaming services on future3, see #1815 and #2116

[s-martin]

For reference this PR is the base for #2116

[pabera]

To reduce repetition in your class MPDBackend, where you frequently update self.player_status, you could introduce a decorator that handles the status updates. This way, you won't have to manually call self.player_status.update(playing=True/False) in every method where this is needed. This also keeps your method implementations cleaner and focused on their primary responsibilities.

Decorator Definition (not sure it works directly, but should be close)

def update_player_status(playing=None):
    def decorator(func):
        def wrapper(self, *args, **kwargs):
            result = func(self, *args, **kwargs)
            if playing is not None:
                self.player_status.update(playing=playing)
            return result
        return wrapper
    return decorator

Use the Decorator in Your Class

Apply this decorator to the methods of your MPDBackend class. For methods that set the player to play, pass playing=True, and for those that pause or stop, pass playing=False. You can skip the parameter for methods that don't change the playing status.

class MPDBackend(BackendPlayer): 
    @plugin.tag
    @update_player_status(playing=True)
    def play(self):
        self._active.play()
        
    ...
    @plugin.tag
    @update_player_status()
    def toggle(self):
        self._active.toggle()
        # Dynamically determines playing status based on current state
        current_playing = self.player_status.get_value('playing')
        self.player_status.update(playing=not current_playing)
    
    @plugin.tag
    @update_player_status(playing=False)
    def stop(self):
        self._save_state()
        self._active.stop()

[pabera]

I like this! The Timer Classes are pretty well done and can be used for such use cases nicely!

[pabera]

Looks rather annoying to add self.player_status.update(playing=True/False) everywhere. You could use a decorator approach, maybe even in the BackendPlayer class (to be verified).

We already use decorators with @plugin.tag.

[maldag]

Is there any progress on this? How is it being resolved and merged? Last commit is 5 months old

[Groovylein]

Currently no time to develop sorry.
The status is, that the player needs to work with the web app.

The problem is especially the status report to the web app so that the player can be displayed properly.

Feel free to continue ☺️

[Pippo8291]

What is the open task? The part from above with Using decorators? Or anything else? Hardcoded a fast mopidy-spotify Version within my box with some drawbacks. Would be nice to have this instead.

[Groovylein]

As already mentioned, developing a status handler which can be then used in the Web app. Additionally all Web app functionality needs to be revised.

[Pippo8291]

Thanks for answering first of all.. sounds like there is still much work to do?!

[bursaar]

Is the expectation here to be able to point phoniebox at a subsonic server?