Menage #173
Merged
Menage #173
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
darnuria
force-pushed
the
menage
branch
2 times, most recently
from
eb39093
to
bb0cb98
Compare
|
Bon comme expliquer dans le message de commit de bb0cb98 l'historique est affreux. C'est le soucis du gros ménage... Ça vire des doublons. J'ai fait le choix d'avoir le formulaire de login en page statique avec seulement du JS et sans appel de php pour la générer pour eviter de se prendre la tête avec |
This was referenced Mar 30, 2017
mart1ver
requested changes
Mar 30, 2017
ifaces/tete.php
Outdated
| </li | ||
| --> | ||
| <li> | ||
| <a href="gains_recycleurs.php">Gains recycleurs</a> |
There was a problem hiding this comment.
yep je les vire de dev ce matin d'ailleurs.
ifaces/ventes.php
Outdated
| } | ||
| 'use strict'; | ||
| const force_pes_vente = <?= json_encode($_SESSION['force_pes_vente']) ?>; | ||
| const tva_active = <?= json_encode($_SESSION['tva_active'] === 'oui') ?>; |
There was a problem hiding this comment.
tva_active doit déjà être un booléen!!
Donc l'historique est carrement affreux mais voila ce que ça fait: - Supprime tete_ventes.php en utilisant tete.php à la place - Quelques modifications dans tete.php pour etre plus générique - Ça répare le switch des ventes qui était cassé depuis le début du ménage - Vire enfin jquery 2.0.3 - Yay - - Repares quelques bug dans jours.php - Simplifie un peu le JS inline de ventes.php - Ajout des balises script pour les ventes vu qu'elles ne sont plus dans tete.php - Le formulaire de login est maintenant une simple page HTML(Failles XSS, problemes de sessions ect) - Suppression de pied_bilan on utilise pied.php a la place. - Jquery et bootstrap.js sont dans tete.php vu que ce sont des dependances obligatoires... - Nettoyage de la visualisation de la caisse. Attention étiquetes.php et pesees_stats sont du coup cassees mais ces pages etants suggestes à deletion/retrait en attendant de mieux cernees les besoins les concernants. On va dire que du coup c'est acceptable de plus elles ne sont plus accessibles depuis la GUI. Ajout de fonctions pour manipuler la session, quelques parametres de la session sont a present des booleens pour le cote client (force_pes_vente, pes_vente, viz_caisse, lot_caisse). De plus pour tout les soucis d'ordre d'execution des script voir la doc MDN de l'attribut defer: https://developer.mozilla.org/en/docs/Web/HTML/Element/script
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Le premier commit supprime
tete_venteset fait quelques améliorations dans les ventes.C'est principalement du ménage vu comme l'historique est affreux je peux venir la relire avec toi en physique.
Le second lui supprime jquery 2.0.3 corriges quelques soucis.
Pour plus de détails voir les messages de commit associées.
Aussi je rends inaccessible a partir de ce commit pesée_stats et étiquettes on va les bouger dans une branche a part afin de les peaufiner tranquillement.