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

Répétition inutile dans le code #3394

Closed
gustavi opened this issue Feb 23, 2016 · 5 comments
Closed

Répétition inutile dans le code #3394

gustavi opened this issue Feb 23, 2016 · 5 comments
Labels
C-Back Concerne le back-end Django Facile Bon ticket pour débuter pour rejoindre le développement ! S-BUG Corrige un problème
Milestone

Comments

@gustavi
Copy link
Contributor

gustavi commented Feb 23, 2016

La ligne concernée https://github.com/zestedesavoir/zds-site/blob/dev/zds/member/views.py#L675

@gustavi gustavi added Facile Bon ticket pour débuter pour rejoindre le développement ! C-Back Concerne le back-end Django labels Feb 23, 2016
@GerardPaligot
Copy link
Member

Pourrais-tu expliquer la répétition inutile dans le code ?

@artragis
Copy link
Member

je pense qu'il faudrait modifier cette condition en :

form = LoginForm()
if next_page is not None:  # if the user was visiting some page before the registration
    form.helper.form_action += "?next=" + next_page

Ce genre de chose peut tout à fiat se faire dans une microPR sans passer par un ticket.

@gustavi
Copy link
Contributor Author

gustavi commented Feb 24, 2016

Exactement. Je sais, mais j'avais pas d'env de dev sous la main donc j'ai préféré poster ici pour ne pas l'oublier.

@orandin
Copy link
Contributor

orandin commented Feb 25, 2016

Oh le joli code dupliqué! :-)
Si personne ne s'en occupe, je proposerai une PR ce soir.

@gustavi
Copy link
Contributor Author

gustavi commented Feb 25, 2016

Tu peux grouper avec ça #3309 ;)

orandin pushed a commit to orandin/zds-site that referenced this issue Mar 4, 2016
@gustavi gustavi added this to the Version de développement milestone Mar 4, 2016
@gustavi gustavi closed this as completed Mar 4, 2016
Emeric54 pushed a commit to Emeric54/zds-site that referenced this issue Mar 20, 2016
@SpaceFox SpaceFox added the S-BUG Corrige un problème label Mar 31, 2016
Emeric54 pushed a commit to Emeric54/zds-site that referenced this issue Apr 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django Facile Bon ticket pour débuter pour rejoindre le développement ! S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

5 participants