Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pagination to results #1

Open
coudot opened this issue Dec 6, 2016 · 6 comments · Fixed by #32
Open

Add pagination to results #1

coudot opened this issue Dec 6, 2016 · 6 comments · Fixed by #32
Assignees
Milestone

Comments

@coudot
Copy link
Member

coudot commented Dec 6, 2016

For the moment all results are displayed in a single page

We should have pagination (different configuration between standard results and gallery)

The must would be to map the pagination to LDAP SSSVLV control to only query visible entries, with http://php.net/manual/en/function.ldap-control-paged-result.php

@coudot coudot added this to the 0.2 milestone Dec 6, 2016
@coudot coudot self-assigned this Dec 6, 2016
@coudot
Copy link
Member Author

coudot commented Dec 19, 2017

Seems not easy to map web pagination to LDAP pagination, as the LDAP pagination relies on an opaque cookie. So there are two different features:

  • Pagination of displayed results
  • Paged Result in LDAP search

@nqb
Copy link
Contributor

nqb commented Jan 22, 2018

+1

@coudot
Copy link
Member Author

coudot commented Feb 9, 2018

Seems interesting to look into SmartyPaginate: http://www.phpinsider.com/php/code/SmartyPaginate/

See also https://www.smarty.net/forums/viewtopic.php?p=82579

@coudot
Copy link
Member Author

coudot commented Mar 5, 2018

Reopen to see how create pagination for gallery and boxes view

@artlog
Copy link

artlog commented Jan 3, 2023

taking a look to this issue

@artlog
Copy link

artlog commented Jan 3, 2023

stopping research on this issue.
notes;
server side ldap paging is currently he missing part.
server side page fetching from client side browsing might be one difficult part.
current smarty pagination assumes whole data is already fetched at server side before browsing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants