-
Notifications
You must be signed in to change notification settings - Fork 17
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
Petit cleanup du code des verif_sorties #412
Conversation
<td><?= $d['last_hero_timestamp'] !== $d['timestamp'] ? $users[$d['id_last_hero']]['mail'] : '' ?></td> | ||
<td><?= $d['last_hero_timestamp'] !== $d['timestamp'] ? $d['last_hero_timestamp'] : '' ?></td> | ||
<td><a style="appearance: button" class="btn btn-warning btn-sm" href="../ifaces/modification_<?= $props['endpoint'] ?>.php?id=<?= $d['id'] ?>">Modifier</a></td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Décalé le bouton a la fin parce-que sinon dès que on a une date de motif + éditeur c'est tout décalé, çà limite la casse.
$req = $bdd->prepare("SELECT | ||
$sortiesSQL, | ||
sorties.id_type_sortie id_type, | ||
type_sortie.nom, | ||
-- localites.nom localisation, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Juste plus d'actualité
@@ -226,14 +220,12 @@ function VerifSortiesTable(array $props) { | |||
VerifSortiesTable(array_merge($base, [ | |||
'h3' => 'Dons simples :', | |||
'data' => $sortiesDon, | |||
'classe' => '' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cette clef sert plus en fait :|
])); | ||
?> | ||
|
||
<div class="panel panel-info"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Un peu beaucoup de repetition entre les deux trucs je sais pas pourquoi on l'avais pas déjà nettoyé surement avant un truc coté serveur devait bloqué et fesait qu'on devait avoir DEUX codes différents d'UI pour les arguments d'url du <a>
On se répétais un peu et y'avais du code mort allez hop hop.