nodequeue pager (pager through nodes in a queue) - code provided

litwol - May 16, 2009 - 06:47
Project:Nodequeue
Version:6.x-2.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs work
Description

So this may not be perfect but i think its a good start to get the ball rolling.

here is the code : http://drupalbin.com/9359

This is my first time using nodequeue so i am having a hard time providing a proper integration with nodequeue and views etc etc.

#1

ezra-g - May 16, 2009 - 15:36
Status:active» duplicate

Thanks for taking the time to write this code!

There has been some discussion about the potential complications of such a feature at #378160: Pageing for long nodeques which is worth reading. Please see the discussion there and feel free to jump in.

If this is not a duplicate, feel free to re-open this issue.

Thanks!

#2

merlinofchaos - May 16, 2009 - 16:16
Status:duplicate» active

This is not a dup. That issue talks about the admin view of a nodequeue.

This is a pager for an individual node, using next/prev to go through a nodequeue jumping from one node to the next. This is the kind of pager you would use on galleries, and it is one of the features that could make nodequeue an excellent tool for creating user gallery sets (think: flickr)

Though this use a paging object which is probably unnecessary...nodequeue's code isn't OO.

#3

ezra-g - May 19, 2009 - 17:58
Status:active» needs review

Thanks for the clarification. This would be an awesome addition, litwol! I agree with merlinofchaos that it would be more consistent with the rest of Nodequeue to have his feature not be OO. In any case, I hope to review soon.

#4

litwol - May 19, 2009 - 18:02

There is no technical reasons behing me doing this in OO, i just wanted to scratch my own itch. I do not know nodequeue well enough to provide proper inegration, but with some guidelines and points from you i'm sure i could do it *the right way*.

#5

ezra-g - July 24, 2009 - 20:15
Status:needs review» postponed (maintainer needs more info)

Any reason not to base this on http://drupal.org/project/custom_pagers ?

#6

ezra-g - October 19, 2009 - 04:14

#7

greggles - November 24, 2009 - 22:08
Status:postponed (maintainer needs more info)» closed

Needs more information for more than 5 weeks -> closed.

#8

litwol - November 30, 2009 - 02:21
Status:closed» needs work

It doesn't need more information. It needs work to port OO code to drupal/nodequeue standard.

#9

ezra-g - November 30, 2009 - 02:22
Status:needs work» postponed (maintainer needs more info)

Could you answer the question in #5?

#10

litwol - November 30, 2009 - 02:35
Status:postponed (maintainer needs more info)» needs work

It is different on a basis that it is more light weight, it is more an an API for developers than a self contained feature like custom_pagers. IMHO both are not mutually exclusive and have their own use cases. Best outcome would be to have both integrations. have to start somewhere right ?

 
 

Drupal is a registered trademark of Dries Buytaert.