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

Compteur de membres erroné dans la liste des membres #2324

Closed
firm1 opened this issue Feb 13, 2015 · 4 comments

Comments

@firm1
Copy link
Contributor

commented Feb 13, 2015

Actuellement sur la 1.6 RC1 en allant sur le lien : http://beta.zestedesavoir.com/membres/

On voit que le nombre de membre est égal à 100 et pourtant il y'en a bien plus.

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Feb 13, 2015

L'API n'est pas concerné ici.

@gustavi gustavi added this to the "Futur proche" (v1.x) milestone Feb 13, 2015

@gustavi gustavi self-assigned this Feb 14, 2015

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 24, 2015

L'API n'est pas concerné ici.

Effectivement, c'est plus au niveau du refactoring que ca cloche. Ce qui se passe c'est que l'on renvoie toujours xx membres par page du paginator et du coup comme on affiche "members.count" on se retrouve à afficher xx. Il faudrait j'imagine retourner une variable supplémentaire "total" mais avec le nouveau "format" du module je galère un peu à voir comment faire

GerardPaligot added a commit to GerardPaligot/zds-site that referenced this issue Feb 24, 2015
fix(member): Displays number of members in the list of members page.
Uses paginator's property count to get the total number of members,
across all pages.

Closes zestedesavoir#2324
@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Feb 24, 2015

Comme j'avais retiré le tag API, j'avais complètement oublié cette issue. PR faite du coup.

@Eskimon : En fait, la solution que tu as proposé m'a fait peur. Fallait absolument que je propose la solution propre. ^^

@Eskimon Eskimon modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 24, 2015

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 24, 2015

Voila ca c'est fait rapidement et proprement :)

@Eskimon Eskimon closed this Feb 24, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.