mherger wrote: 
> But:
> 
> > $params->{"update:$plugin"}-' becomes irrelevant. I assume that this
> 
> This one actually is set. So... checking the existence only for 
> "install", but the actual value for "update" seems to do the trick. 
> Please give it a try. A new build is on its way. Thanks for the pointer!

Your recent fix
(https://github.com/Logitech/slimserver/commit/dc39e5b5b06ee9efc2c0edabe1117e1b60819045)
doesn't do the trick here.

I find that: '-exists $params->{"install:$plugin"}-' always evaluates to
true (1) when the plugin update is available. So the update proceeds,
regardless of the state of the "update" checkbox.

Removing that second, pesky, '-exists-' will "correct" matters when it
comes to updates, but now interferes with the first plugin installation.
In this scenario, when first installing, the plugin is not actually
installed but does get shown in the "Updated plugins available" section.
Which, if then checked, will then install. But this is not how it's
meant to work.

I haven't sat down and worked out the logic of what's going on.


------------------------------------------------------------------------
mrw's Profile: http://forums.slimdevices.com/member.php?userid=38299
View this thread: http://forums.slimdevices.com/showthread.php?t=116987

_______________________________________________
Squeezecenter mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/squeezecenter

Reply via email to