Jaca wrote: 
> Thank you!  
> 
> Had to change line 207 to " @destBuddies " you probably been patching
> this little too fast :p 

Oops! Thanks for fixing!

Jaca wrote: 
> 
> 
> *- Move queue is now compatible with "Groups Plugin"* ( even in case
> when source is part of dest group) 
> I had to let Group Plugin to handle syncing so if dest player is group
> it will power off (to dissasseble "default wrong" group) and power on
> (to assemble group correctly) .  btw I added 2 sec delay there , not
> sure if its down to LMS host performance but it didnt't work without
> it..

Not sure I like the sleeps - LMS is single threaded. But if source is
synced you power off/on - and then again if its a group. I'm guessing
both are not needed - as in it should be if "source is synced OR its a
goup" power off/on?

Jaca wrote: 
> 
> - COMMENTED \"CLEAR PLAYLIST LINE\"  
> 
> not sure why source player should stay with nothing to play.. makes no
> sense.. I would rather have smth there then nothing.

Because it is -moving- the queue, not -copying-. So clearing the source
-does- make sense.



Jaca wrote: 
> 
> *-occasionally had sync problem if destination was synced group so added
> Turn off and on again trick ( seems to help) *

That seems more like a hack, and the group plugin should handle this (if
it can).



*Material debug:* 1. Launch via http: //SERVER:9000/material/?debug=json
2. Open browser's developer tools 3. Open console tab in developer tools
4. REQ/RESP messages sent to/from LMS will be logged here.
------------------------------------------------------------------------
cpd73's Profile: http://forums.slimdevices.com/member.php?userid=66686
View this thread: http://forums.slimdevices.com/showthread.php?t=109624

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

Reply via email to