Download & Extend

Add a service for saving the contents of a subqueue

Project:Nodequeue
Version:6.x-2.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:patch (to be ported)
Issue tags:services

Issue Summary

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.

AttachmentSizeStatusTest resultOperations
nodequeue_service_save_subqueue.patch3.42 KBTest request sentNoneView details

Comments

#1

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

AttachmentSizeStatusTest resultOperations
nodequeue_service_save_subqueue.patch3.21 KBTest request sentNoneView details

#2

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

AttachmentSizeStatusTest resultOperations
nodequeue_service.1045678.2.patch3.92 KBTest request sentNoneView details

#3

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

#4

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

#5

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!

#6

@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..

#7

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.