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.

Sign up for GitHub

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

Jump to bottom

$A = $A #45

Open
benjaminplanche opened this issue Feb 14, 2013 · 2 comments
Open

$A = $A #45

benjaminplanche opened this issue Feb 14, 2013 · 2 comments
Assignees
@blackheaven
Labels
duplicate invalid

Comments

@benjaminplanche
Copy link
Member

Contexte

Cela concerne les modifications apportées par la PR #42 côté PHP, et a été relevé par @BenjBouv (issue créée pour pas que ce soit oublié).

Comportement

Du code inutile, avec beaucoup de lignes du style $A = $A un peu partout
Ex: https://github.com/blackheaven/S.I/commit/b1bfa30d2e6d876afba922484d123109a9ff1905#L3R59

Piste

Problème un CTRL+H un peu trop hâtif ... Juste besoin de bien relire le code et corriger proprement.
@ghost ghost assigned blackheaven Feb 14, 2013
@blackheaven
Copy link
Contributor

ça inclus les "alias" ? (comme $nom_entreprise = $_POST['name'];)

@benjaminplanche
Copy link
Member Author

Je te laisse évaluer ou non de leur utilité ... ;)

@blackheaven blackheaven mentioned this issue Feb 14, 2013
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@blackheaven blackheaven

Labels
duplicate invalid
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@benjaminplanche @blackheaven