Adding prev to cursors about fractal

[philsturgeon]

I do wonder how necessary it is. Do people walk through data in both directions in this sort of scenario?

I mean, I guess so? We can blindly copy Facebook and do it?
—
Sent from Mailbox for iPhone

On Tue, Apr 8, 2014 at 3:57 AM, Michele Massari [email protected]
wrote:

Hi Phil!
Will it make sense in your opinion to add a "prev" parameter to the cursor so the output can be like this?
"cursor": {
"current": "MTA=",
"prev": "LTEw",
"next": "MTA=",
"count": 10
}
I made some slight changes to the CursorInterface, Cursor, Scope and Collection because I needed the functionality for a personal project.

If you want I can make a pull request to let you review it.

Reply to this email directly or view it on GitHub:
#40

[mikemassari]

It was requested by the mobile app developer to implement a "prev" button on the list pagination.
Of course if you don't think it's useful there's no need to implement it. :)

[philsturgeon]

I was just curious but that is an obvious use case. Go for it! :)—
Sent from Mailbox for iPhone

On Tue, Apr 8, 2014 at 1:27 PM, Michele Massari [email protected]
wrote:

It was requested by the mobile app developer to implement a "prev" button on the list pagination.

Of course if you don't think it's useful there's no need to implement it. :)

Reply to this email directly or view it on GitHub:
#40 (comment)

[ipalaus]

Yeah, I think it would be a nice addition too. What if you're not using it, it will just be always null? How would you handle that?

[mediaholding]

I think it is very important to update Cursor docs at http://fractal.thephpleague.com/pagination/ because sample code now isn't work because of additional parameter prev.

This line:

$cursor = new Cursor($currentCursorStr, $newCursorStr, $books->count());

should be now something like that:

$cursor = new Cursor($currentCursorStr, $prevCursorStr, $newCursorStr, $books->count());

[philsturgeon]

I'm not sure how to do that with the current ID-based example. If you use cursors and use offsets would you be so kind as to change it to use offsets and send a PR?