Fix: Speakers appear multiple times #114

[nraamat]

Added functionality that keeps tracks of Sonos devices that already been created AirPlay servers for. Skip creating a new AirPlay server when a discovered Sonos device is a duplicate

[nraamat]

This closes #114

[jeffreyameyer]

This fix removed the duplicate entry problem for me on OSX 10.10.2 and node 0.12.1.

[vkoi81]

Kudos to this fix. It removed my duped devices.

[jeeftor]

Do you take into account ping time? For example I have devices listed twice, however, they have different "ping" times it would appear. +0ms +20ms for the 1st listing for example and +3ms +14ms for the 2nd listing. In my small case they happen to be in order so taking the 1st device and ignoring the 2nd entry would work - but i don't know if this is alway the case.

[sjbiss]

How do I install this update?

[timrettop]

I've not been able to get this change to resolve my doubled devices. I'm a bit new to node/npm, so I could be doing something wrong.

I had first installed airsonos from npm, but after finding this issue, I downloaded the source and ran npm install -g in the source directory.

I then made the changes to the main.js (leaving that commented section commented) and then re-ran the npm install -g in the source directory.

It still shows my livingroom and bedroom play1's twice.

I tried removing the airsonos (running npm remove airsonos -g) and then re-built the source, and it still shows them twice (after restarting avahi).

Should this PR make the software only print each once in its output... because I get that twice:

Searching for Sonos devices on network...
Setting up AirSonos for Master Bedroom {10.1.50.138:1400}
Setting up AirSonos for Living Room {10.1.50.120:1400}
Setting up AirSonos for Master Bedroom {10.1.50.138:1400}
Setting up AirSonos for Living Room {10.1.50.120:1400}

[JBlackCat]

@sjbiss until the pull request is merged into master, you will need to go to @nraamat page and clone their airsonos fork to your machine and run airsonos from that repository to test. Hope that helps.

[lirenzhu]

Fix for this issue should be in node-sonos, referencing stephen/node-sonos#1 here.

[leachbj]

@lirenzhu your fix works for me.

[stephen]

Merged stephen/node-sonos#1 - reinstallation should solve this - thanks @lirenzhu!