Initial contribution of the Sonos 2.0 binding

Signed-off-by: Karel Goderis <karel.goderis@me.com> (github: kgoderis)
You are in turbo mode with the review ;-)

InstanceID seems to be a constant, as far as the Sonos implementation goes today, but that could easily change in the future

> On 24 Oct 2014, at 17:39, Kai Kreuzer <notifications@github.com> wrote:
> 
> In addons/binding/org.openhab.binding.sonos/src/main/java/org/openhab/binding/sonos/handler/ZonePlayerHandler.java:
> 
> > +			if (pollingJob == null || pollingJob.isCancelled()) {
> > +				Configuration config = getThing().getConfiguration();
> > +				// use default if not specified
> > +				int refreshInterval = DEFAULT_REFRESH_INTERVAL;
> > +				Object refreshConfig = config.get("refresh");
> > +				if (refreshConfig != null) {
> > +					refreshInterval = Integer.parseInt((String)refreshConfig);
> > +				}
> > +				pollingJob = scheduler.scheduleAtFixedRate(pollingRunnable, 0, refreshInterval, TimeUnit.SECONDS);
> > +			}
> > +		}
> > +	}
> > +
> > +	protected void updateMediaInfo() {
> > +		Map<String, String> inputs = new HashMap<String,String>();
> > +		inputs.put("InstanceID", "0");
> Shouldn't you define InstanceID better somewhere as a constant?
> 
> —
> Reply to this email directly or view it on GitHub <https://github.com/openhab/openhab2/pull/20/files#r19346178>.
>