Now that we have node queue machine names it becomes much easier to allow for deployment of node queue contents to a remote site. It would be awesome to have the services side of things (i.e. the saving of the subqueue contents on the remote) handled by nodequeue_service module rather than having to implement this service in a custom module.
Here's a patch that adds this functionality.

CommentFileSizeAuthor
#2 nodequeue_service.1045678.2.patch3.92 KBkatbailey
#1 nodequeue_service_save_subqueue.patch3.21 KBkatbailey
nodequeue_service_save_subqueue.patch3.42 KBkatbailey

Comments

katbailey’s picture

katbailey commented
StatusFileSize
new nodequeue_service_save_subqueue.patch3.21 KB

Just realised I had some extra stuff in there to do with grabbing the title of the queue, which is not needed...

katbailey’s picture

katbailey commented
StatusFileSize
new nodequeue_service.1045678.2.patch3.92 KB

Ugh, just noticed I wasn't handling subqueue references properly - another new patch attached...

amateescu’s picture

amateescu commented
Status: Needs review » Patch (to be ported)

Commited #2 to 6.x. This patch should be ported after #817558: Machine names gets into 7.x.

http://drupalcode.org/project/nodequeue.git/commit/a45f9ee

amateescu’s picture

amateescu commented

We now have machine names in 7.x. Any takers for a re-roll?

katbailey’s picture

katbailey commented

Heh, well seeing as I'm the only other person on this thread I guess I should have a crack at it ;-) Unfortunately I won't know how to test it though seeing as my use for it in D6 was for deployment of subqueue contents. Hmm, will see what I can do, but if anyone else has another use case for this or would like to take a stab at it, please have at it!

amateescu’s picture

amateescu commented

@katbailey, what do you think about the patch from #1349590: Nodequeue services 3.x integration. My first impression is that breaking the existing functionality is a no-go for 6.x-2.x and 7.x-2.x, but I'm not really sure how many people actually use this feature..

katbailey’s picture

katbailey commented

Oh wow - just read my comment in #5, I had completely forgotten I'd offered to help with this :-/
As for the patch in the other issue, my first concern is that it works off qids instead of machine names, which wouldn't work at all for any use case I'd have in mind. Also, I'm pretty sure dixon_ is working on the area of nodequeue subqueue deployment, which of course would have a services component - might be worth seeing where he's at wtih that and seeing if it could be tied in with patrickd's work.