Etienne Pallier / labinvent

08 Jul, 2020

1 commit

  • c9ad82a7   Filtre "age" FINALISÉ sur page index des matériels (v3.7.9.58) ... Browse File » 
    - Affichage des matériels par tranche d'âge (0-5 ans, 5-10, ...)
- Grosse refactorisation du javascript dans la page materiels/index pour
utiliser le meme code pour les 2 filtres "age" et "nb lines"
- Bugfix url rewriting dans la page materiels/index pour supprimer le
paramètre "page" car ça n'est pas pertinent et ça provoque plantages
- Nombreux petits bugfixes sur les css
    Etienne Pallier
     

07 Jul, 2020

2 commits

06 Jul, 2020

3 commits

04 Jul, 2020

1 commit

03 Jul, 2020

1 commit

  • 13481544   Bugfix Emprunts calcul sur les dates pourri... (v3.7.9.53) ... Browse File » 
    - Emprunts : on utilise maintenant FrozenDate et FrozenTime de cakephp
au lieu de DateTime de php
- Changement stratégie find matériels, liste des Propriétaire : ce n'est
plus la liste des gens qui ont au moins 1 materiel, mais la liste de
tous les utilisateurs (tout le ldap)
- Bugfix suivis : type suivi doit être obligatoire !!!
- Désactivé tests en php5 sur SurCategories à cause erreur Template file
"Surcategories/index.ctp" is missing (General.php line 1101)
    Etienne Pallier
     

30 Jun, 2020

1 commit

  • 08573950   - Bugfix et refactor des actions documents /admission, /sortie, /fiche ... Browse File » 
    (v3.7.9.52)

=> désormais, on passe l'id du matos associé et non pas son numero
d'inventaire !!! (mauvais raccourci)
=> ca permet de tester les conditions d'accès de ces actions en fonction
du matériel concerné !!

- Bugfix test de ces mêmes actions (mal testées avant)

- L'ajout (/add) d'un suivi, emprunt, ou doc doit toujours se faire avec
l'id de l'entité parente associée (en général materiel id)
=> (/add/id) sinon refus

- ...
    Etienne Pallier
     

29 Jun, 2020

2 commits

26 Jun, 2020

3 commits

  • 4caeda09   - TEST3 : Ajout (terminé) d'un 3ème (et DERNIER) gros test (v3.7.9.49) ... Browse File » 
    - Test générique des vues index et view de Materiels (/materiels/index
et /view) qui teste le CONTENU des vues pour TOUS les materiels de la BD
de test (fixture), aussi bien dans la vue index globale que dans CHAQUE
vue détaillée de chaque matériel (et surtout en vérifiant aussi la liste
des entités associées (suivis,emprunts,documents)
- bugfix autorisations d'affichage des boutons d'action sur documents
dans materiels/view/
- Nombreuses refactorisations
- tous les tests passent
- ...

Je vais enfin pouvoir passer à AUTRE CHOSE QUE LES TESTS !!
genre, des choses plus constructives !
    Etienne Pallier
     
  • c194ff64   TEST3 : Ajout en cours (presque fini) d'un 3ème (et DERNIER) gros test ... Browse File » 
    (v3.7.9.48)

Test générique des vues index et view de Materiels (/materiels/index et
/view) qui teste le CONTENU des vues pour TOUS les materiels de la BD de
test (fixture), aussi bien dans la vue index globale que dans CHAQUE vue
détaillée de chaque matériel (et surtout en vérifiant aussi la liste des
entités associées (suivis,emprunts,documents).
    Etienne Pallier
     
  • c2e147cb   TEST3 : Ajout (en cours) d'un 3ème (et DERNIER) gros test (v3.7.9.47) ... Browse File » 
    Test générique des vues index et view de Materiels (/materiels/index et
/view) qui testeRA le CONTENU des vues pour TOUS les materiels de la BD
de test (fixture), aussi bien dans la vue index globale que dans CHAQUE
vue détaillée de chaque matériel (en vérifiant aussi la liste des suivis
et emprunts associés)
    Etienne Pallier
     

25 Jun, 2020

3 commits

24 Jun, 2020

1 commit

23 Jun, 2020

1 commit

22 Jun, 2020

3 commits

19 Jun, 2020

1 commit

  • 5faf8198   Tests génériques pour les controleurs principaux (v3.7.9.39) ... Browse File » 
    (Materiels, Suivis, et Emprunts) mais aussi le controleur "quelconque"
SurCategories (pour vérifier que ça marche aussi !!!)

- Dans ces tests génériques, on teste non seulement qu'une action
s'exécute seulement quand elle y est autorisée, mais aussi qu'elle
s'exécute correctement (au moins pour les actions CRUD :
add,edit,delete...)
- Le nombre total de tests (en ajoutant ces tests automatiques) monte à
200 (pile poil) !!!
- ajout champ 'nom' à la table emprunts => ajout d'un fichier sql pour
la mise à jour
- ...
    Etienne Pallier
     

17 Jun, 2020

1 commit

  • 4775e899   Tests génériques (auto) pour les controleurs principaux (v3.7.9.38) ... Browse File » 
    (Materiels, Suivis, Emprunts)

- Dans ces tests génériques, on teste non seulement qu'une action
s'exécute seulement quand elle y est autorisée, mais aussi qu'elle
s'exécute correctement (au moins pour les actions CRUD :
add,edit,delete...)
- L'ajout d'un suivi, emprunt, ou document n'est plus autorisé SANS
préciser l'id du matériel concerné, ce qui est logique
- ajout champ 'nom' à la table emprunts
- ...
    Etienne Pallier
     

12 Jun, 2020

1 commit

  • bcc25a64   Tests génériques automatiques des Materiels => OK (v3.7.9.37) ... Browse File » 
    - Tests génériques automatiques pour (presque) toutes les actions du
controleur Materiels => OK

- Amélioré l'action materiels/print_label_ruban (impression etiquette) :
un peu plus conforme au standard, un peu moins bidouille qu'avant, et
surtout plus besoin de la finir par un "sale" "exit" => donc on peut
maintenant la tester, et c'est ce qui a été fait ! (elle est pas encore
total nickel mais bon c'est mieux qu'avant)

- modifié Fixtures pour supprimer les 'id' explicites

- bugfix action materiels/statusToBeArchived (ou statusToBearchived) =>
remplacé partout par statusTobearchived
    Etienne Pallier
     

10 Jun, 2020

3 commits

05 Jun, 2020

2 commits

04 Jun, 2020

1 commit

  • 718dc8d1   ADAPTATION DE TOUS LES CONTROLEURS au nouveau système d'autorisations ... Browse File » 
    (v 3.7.9.31)

	- SIMPLIFICATION DES AUTORISATIONS POUR LA PLUPART DES CONTROLEURS
SIMPLES
	- AUTORISATIONS PAR DÉFAUT POUR TOUT NOUVEAU CONTROLEUR BASIQUE
	- AMELIORATIONS DE LA PAGE AFFICHANT LES AUTORISATIONS DYNAMIQUEMENT
(/pages/acls ou /pages/acls/#nom_du_controleur) => On peut ainsi
vérifier ce qui est appliqué à l'instant T (juste après la modif d'une
règle par exemple)
    Etienne Pallier
     

03 Jun, 2020

2 commits

  • a74f7710   NOUVELLE PAGE AFFICHANT LES AUTORISATIONS DYNAMIQUEMENT (v3.7.9.30) ... Browse File » 
    => /pages/acls ou /pages/acls/#nom_du_controleur
=> On peut ainsi vérifier ce qui est appliqué à l'instant T (juste après
la modif d'une règle par exemple)
    Etienne Pallier
     
  • aaeefcef   NOUVEAU SYSTEME D'AUTORISATION DES ACTIONS CENTRALISÉ => OK (v3.7.9.29) ... Browse File » 
    => TOUS les tests passent sauf certains qui ne passent plus car les
nouvelles conditions sont plus strictes
		(ils ont été temporairement désactivés en attendant réécriture plus
générique qui viendra très vite)

=> On peut voir dynamiquement l'état actuel des autorisations via la
page web "/autorisations"

=> Autorisations définies dans chaque controleur par setAuthorizations()
qui initialise un tableau UNIQUE de AppController
($this->is_authorized_action)

=> La méthode isAuthorized() de chaque controleur appelle
isAuthorizedAction() (de AppController) qui autorise ou pas l'action en
fonction de ce tableau unique

=> Quand on veut changer une autorisation, il suffit de le faire UNE
SEULE FOIS dans la méthode setAuthorizations()
		   Cette autorisation sera alors prise en compte PARTOUT dans le code
(code des actions, des vues, etc...)

=> L'initialisation des autorisations est faite (par la méthode
setAuthorizationForAction() de AppController) dans un format standard,
		quelque soit l'autorisation, donc c'est très LISIBLE et facile à
modifier
	- ajout de 2 très gros tests génériques pour les actions et les vues
matos/index et matos/view en cours... (ce qui m'évitera plein de tests à
la main)
    Etienne Pallier
     

15 May, 2020

1 commit

  • bc6fe47e   Version: 3.7.9.28 ... Browse File » 
    	- Renforcement tests suivis et emprunts (materiel validated only) et
refactorisations
	- ajout de 2 très gros tests qui tuent des vues matos/index et
matos/view en cours... (ce qui m'évitera plein de tests à la main)
	- Ajout de la méthode complexe isManageableByUser() dans
MaterielsController.php et dans Materiel.php
	- Ajout de la méthode getRelatedMateriel() dans AppController, très
optimisée
	- Vers une unicité toujours plus grande des tests d'autorisation
d'action (uniquement dans isAuthorized())
	- Utilisation systématique et UNIQUE de la méthode isAuthorized() de
CHAQUE controleur pour savoir si une action est autorisée ou pas
	- Utilisation systématique de variables "$CAN_xxx" dans les VUES pour
tester si un affichage est autorisé ou pas
		=> bcp plus centralisé qu'avant, et bcp plus clean et lisible
 		=> désormais TOUS les droits sont définis dans la SEULE méthode
isAuthorized() de CHAQUE controleur et NULLE PART AILLEURS
 		=> si on veut changer un droit d'accès quelconque, on ne le change
QUE là. Ouf, on respire, c'est bien plus cohérent et maintenable
qu'avant
 		=> FIN de la galère de la gestion des droits (acl) enfin, il aura
fallu quelques années pour y parvenir !!!
    Etienne Pallier
     

11 May, 2020

2 commits

  • 8479bba9   - Suppression des méthodes isAuthorizedAction() et ... Browse File » 
    isAuthorizedActionFor() car plus simple de ne garder QUE isAuthorized()
en la modifiant un peu... => du coup, c'est désormais cette SEULE
méthode (dans CHAQUE controleur) qui décide de l'accès ou non à une
action selon le profil utilisateur
    Etienne Pallier
     
  • f6e35b71   Version 3.7.9.27 ... Browse File » 
    - Ajout d'un LOG applicatif dans le fichier /logs/debug.log pour toutes
les actions importantes (add, edit, delete, ...) qui permet de savoir
kiafekoi
	- Suppression des méthodes isAuthorizedAction() et
isAuthorizedActionFor() car plus simple de ne garder QUE
isAuthorizedAction() en la modifiant un peu... => du coup, c'est
désormais cette SEULE méthode (dans CHAQUE controleur) qui décide de
l'accès ou non à une action selon le profil utilisateur
    Etienne Pallier
     

07 May, 2020

1 commit

  • 92acc9fe   Harmonisation (v3.7.9.26) ... Browse File » 
    	- shortcuts dans AppController (donc disponible pour TOUS les
controleurs) :
		- $this->u = user courant
		- $this->e = entité courante
		- $this->e_id = id de l'entité courante
		- $this->a = action en cours
		- $this->c = controleur en cours (?)
		- $this->getEntity($id=null) // id=null si matos courant
	- harmonisation des noms de méthodes pour MaterielsController :
		- $this->isCreated($id=null) // id=null si matos courant
		- $this->belongsToUser($username, $id=null) // id=null si matos
courant
		 	et $this->belongsToCurrentUser($id=null) // id=null si matos courant
		- $this->isSameGroupAsUser($userlogin, $id=null) // id=null si matos
courant
		 	et $this->isSameGroupAsCurrentUser($id=null) // // id=null si matos
courant
	- harmonisation des méthodes dans entité Materiel (Entity) :
		- is_created, is_validated, is_tobearchived, is_archived
		- belongsToUser($username) // owned or declared by user
		- isSameGroupAsUser($usergroup1, $usergroup2) // is same group as user
groups
		- ...
	- poursuite généralisation utilisation de la fonction
isAuthorizedAction() partout où on a besoin de connaitre les droits d'un
user (view, index, ...)
    Etienne Pallier
     

05 May, 2020

1 commit

  • da7f42f6   Diverses simplifications, refactorisations, optimisations (v3.7.9.25) ... Browse File » 
    - LOG : ajout de messages log sur actions importantes (add, edit,
delete, ...)
	- SIMPLIFICATION : Suppression des valeurs "N/A" (inutile et pose des
problèmes) dans 4 tables : groupes_thematiques, groupes_metiers, sites,
et type_documents => et remplacement des liens vers ces champs par NULL
dans tables materiels, suivis, emprunts, et documents !!!
	- Utilisation __toString() pour afficher une entité rapidement avec
echo $entity
	- bugfix ACLs isAuthorizedAction() de Materiels => début de
généralisation de l'utilisation de cette fonction partout où on a besoin
de connaitre les droits d'un user (view, index, ...)
    Etienne Pallier
     

04 May, 2020

1 commit

  • 2de8cefa   Diverses simplifications, refactorisations, optimisations (v3.7.9.24): ... Browse File » 
    	- Nouvelle variable $this->current_user_entity "globale" et héritée par
tous les controleurs (car définie dans AppController.beforeFilter())
		=> cette variable est un objet Entity et donc avec des propriétés
(virtuelles) pratiques permettant de faire par exemple :
			- $user->is_admin
			- $user->is_resp
			- $user->is_super
			mais aussi :
			- $user->is_admin_or_more()
			- $user->is_admin_or_less()

	- bugfix 3 TESTS !!!
    Etienne Pallier
     

30 Apr, 2020

1 commit

  • ded5e071   Diverses simplifications, refactorisations, optimisations (v3.7.9.23) ... Browse File » 
    	- ajout de l'option globale $this->SUPERADMIN_CAN_DO_EVERYTHING dans
AppController (false par défaut) pour donner rapidement TOUS les droits
à superadmin (juste passer à true)
	- regroupement des conditions d'accès des 3 actions ADD, EDIT, et
DELETE car très semblables => remplacé 60 lignes par 10 !!!
	- Grosse simplification (et bugfixes !!) des conditions d'accès à la
plupart des actions de MaterielsController => c'est bien plus clair à
lire
	- Nouvelles règles ACL (conditions d'accès) sous forme de tableau $acl
=> en cours...
		=> ce changement offre plusieurs avantages, et devrait notamment
permettre de :
		- clarifier les conditions d'accès,
		- les rendre plus facilement modifiables
		- les rendre copiables d'une action à l'autre (et donc facilement
réutilisables sans avoir à réécrire de code)
		- afficher une page web présentant automatiquement (dynamiquement) une
synthèse claire de toutes les ACL, action par action, et profil par
profil
			(ce qui permettra de se rendre compte plus facilement de l'état
actuel des conditions d'accès)
	- Utilisation de $this->action et $this->action_id (...) de
AppController.beforeFilter() un peu partout
	- Utilisation de $this->entity de AppController.beforeFilter() un peu
partout pour simplifier et surtout éviter (bcp) les accès BD redondants
	- Optimisations pour éviter trop de requetes en BD et simplification
des tests acl
    Etienne Pallier