Limit Sonos module to only show a specific speaker

[strawberry 3.141]

@dinkybluebug245 you forgot to put the mentioned line in the defaults object, this will cause a problem for those who want all rooms displayed, if you add this line and make a commit your pull request will get updated automatically, and i suggest to put also an information aubout this option into the readme file

Edit to your new post, as i said i have no clue about the sonos system and can’t test it I think you will find a way to get around this issue

[dinkybluebug245]

@strawberry-3.141 Thanks for all your help.

Really appreciate it.

Marc

[strawberry 3.141]

@dinkybluebug245 is this what you mean by nested rooms?

$.each(item.members, function (j, member) {
					room += member.roomName + ', ';
});

[dinkybluebug245]

@strawberry-3.141 Yes I think so.

Excluding all rooms but ‘Kitchen’,

Nested rooms "Living rioom, Kitchen’ - shows nothing
Nested rooms "Kitchen, Living room’ shows info

[strawberry 3.141]

@dinkybluebug245 can you give this a try?

$.each(data, (i, item) => {
    if(this.config.exclude.indexOf(item.coordinator.roomName) === -1 || item.members.length > 1){
        var room = item.coordinator.roomName;
        var state = item.coordinator.state.zoneState;
        var artist = item.coordinator.state.currentTrack.artist;
        var track = item.coordinator.state.currentTrack.title;
        var cover = item.coordinator.state.currentTrack.absoluteAlbumArtURI;
        var streamInfo = item.coordinator.state.currentTrack.streamInfo;
        if(item.members.length > 1){
            room = '';
            $.each(item.members, (j, member) => {
                if(this.config.exclude.indexOf(member.roomName) === -1){
                    room += member.roomName + ', ';
                }
            });
            room = room.slice(0, -2);
        }
        text += this.renderRoom(state, artist, track, cover, room);
    }
}.bind(this));

[dinkybluebug245]

@strawberry-3.141 shouldnt the first line have ‘function’ in it?

	$.each(data, function (i, item) {

[strawberry 3.141]

@dinkybluebug245 then you will change the scope of this, that’s why I prefer to use big arrow functions () => {} is the same as function(){} but you keep the scope

[dinkybluebug245]

@strawberry-3.141 I love how you think I know what that means. Ha! I’ll take your word on that and give it a go

[strawberry 3.141]

a lot of people are doing this by saving the scope into a variable like

var self = this;
setTimeout(function(){
        doSomething();
        self.sendSocketNotification('NOTIFICATION');
}, 1000);

but I like this cleaner way

setTimeout(() => {
        doSomething();
        this.sendSocketNotification('NOTIFICATION');
}, 1000);

[dinkybluebug245]

@strawberry-3.141 That doesnt work. Kitchen is the only room NOT excluded in my config.js

Kitchen, then group in Living room - shows info.

Living room, then grouping in Kitchen - doesnt show.

[strawberry 3.141]

@dinkybluebug245 sry then you have to figure it out on your own or ask some person with sonos speakers for help

[dinkybluebug245]

This post is deleted!

[dinkybluebug245]

Ive had a go at trying to fix this myself and cant work it out - but then again my own knowledge of js is non existant. It works when rooms are isolated on their own, or when the speaker you want to display is listed first in a group of grouped speakers - the solutions above dont work when the speaker is not listed first in the group.

Can anyone else help?

Thanks again to @strawberry-3-141 for having a stab at this.

[Snille]

Hi all,
I don’t know if this is the right place, but here it goes.
I forked and made some changes to the Sonos module.
Can be found here: https://github.com/Snille/MMM-Sonos
Let me know is I should present it in some other thread. :)

Best regards Snille