Le 30/10/2012 09:13, David DURIEUX a écrit :
Nouveau patch upstream


Salut,

Je viens de faire un premier passage très rapide sur ton patch.
Déjà merci pour une telle contribution.

Quelques soucis assez gênants déjà remontés en direct :
- gestion des meta recherches HS
- manque de commentaires pour bien comprendre les mécanismes en place (fonctions + interne au code) - des erreurs du genre Undefined index: nosort in /home/dombre/httpd/glpi-test/inc/search.class.php at line 1286
- quelques petits soucis d'indentation du code.
- ligne 4448 la gestion des liens est plutôt étrange... ne tiens pas compte des datatypes. - Gestion des unités (ex clôture auto sur les entités -> -2day / -1day ) (sûrement des modifs à faire dans CommonDBTM aussi

Voilà pour mes premières remarques rapides, je n'ai malheureusement pas plus de temps pour tester.

++

Julien




_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Reply via email to