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

fix(paginator): #2556 Fixes paging page with previous item in list. #2558

Merged
merged 1 commit into from
Apr 15, 2015
Merged

fix(paginator): #2556 Fixes paging page with previous item in list. #2558

merged 1 commit into from
Apr 15, 2015

Conversation

GerardPaligot
Copy link
Member

Q R
Correction de bugs ? Oui
Nouvelle Fonctionnalité ? Non
Tickets (issues) concernés #2556 #2557

Corrige la pagination avec l'item précédent dans la liste. Cette correction est visible dans la liste des MPs (sans item précédent) et dans la liste des messages d'un MP (avec un item précédent).

Corrige #2556 et #2557 par effet de bord (positif).

QA :

  • Vérifier que la pagination s'affiche que lorsqu'une 2ème page est nécessaire.
  • Vérifier que la pagination affiche les bons messages sur chaque page.

@@ -40,11 +40,12 @@ def get_context_data(self, **kwargs):
context.update(kwargs)
return super(MultipleObjectMixin, self).get_context_data(**context)

def build_list(self):
def build_list_with_previous_item(self, queryset):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Y'a pas de risques d'effets de bord hors des MP avec ceci ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ZdSPagingListView est utilisé uniquement dans les vues CVB (donc dans les vues refactorées) et la liste des messages d'un MP est la première à devoir récupérer le dernier message de la page précédente. La méthode (définie dans ZdSPagingListView) est donc utilisée nulle part ailleurs.

@SpaceFox
Copy link
Contributor

QA : OK.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 1d5428f on GerardPaligot:hotfix-fix_2556 into 72db708 on zestedesavoir:prod.

@SpaceFox SpaceFox merged commit 1d5428f into zestedesavoir:prod Apr 15, 2015
@GerardPaligot GerardPaligot deleted the hotfix-fix_2556 branch April 15, 2015 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants