Evolution #1979
Remplacer "minidom" par "etree" pour la lecture des dictionnaires
Description
Il y a de fortes chances pour qu'on y gagne en performance...
Associated revisions
see #1979 Can hide ticket category to post-only users (to be improved)
see #1979 remove TicketCategory::dropdown and add a condition parameter to dropdownValue
- creole/lxml_parser.py : nouveau parseur
- creole/dom_parser.py : ancien parseur (déprécié)
- creole/cfgparser.py : modification des imports
Ménage dans cfgparser.py (ref #1979 @30m)
correction du parsing lxml (separateurs sans texte) + retour read_string pour Zéphir (ref #1979 @30m)
lxml_parser : correction du calcul de l'attribut mode si pas de valeur dans le xml (ref #1979)
lxml_parser : encore une erreur sur le calcul des separators (ref #1979)
History
#1 Updated by Joël Cuissinat almost 12 years ago
- Assigned To set to Gwenael Remond
#2 Updated by Joël Cuissinat almost 12 years ago
- Target version changed from Mises à jour 2.3 - 02 RC to Mises à jour 2.2.3 - 03 RC
#3 Updated by Joël Cuissinat almost 12 years ago
- Target version changed from Mises à jour 2.2.3 - 03 RC to Mises à jour 2.3 - 02 RC
#4 Updated by Joël Cuissinat almost 12 years ago
- Target version changed from Mises à jour 2.3 - 02 RC to Mises à jour 2.3 - 02 Stable
#5 Updated by Joël Cuissinat almost 12 years ago
- Target version changed from Mises à jour 2.3 - 02 Stable to Mises à jour 2.3 - 03 RC
- Distribution set to EOLE 2.3
#6 Updated by Joël Cuissinat almost 12 years ago
- Target version changed from Mises à jour 2.3 - 03 RC to Mises à jour 2.3.4 RC
- Estimated time set to 10.00 h
#7 Updated by Joël Cuissinat over 11 years ago
- File lxml_parser.py View added
En remplaçant le dom_parser.py par le fichier lxml_parser.py ci-joint, la différence n'est pas évidente :(
=> à tester sur un AmonEcole
#8 Updated by Gwenael Remond over 11 years ago
pour que ça gagne du temps c'est moins le fait d'utiliser lxml par rapport à xml.etree, c'est surtout qu'il y a plein de findall() qu'il faut remplacer par du XPath, et là ça devrait gagner du temps.
#9 Updated by Joël Cuissinat over 11 years ago
- % Done changed from 0 to 20
Je comprends.
Mais c'est déjà un exploit en soi d'avoir réussi à changer le code et que ça marche toujours !
Enfin, pas tout à fait car le test unitaire : test_redefine.py plante :(
#10 Updated by Joël Cuissinat over 11 years ago
- File lxml_parser.py View added
#11 Updated by Joël Cuissinat over 11 years ago
- Status changed from Nouveau to Résolu
- % Done changed from 20 to 100
Appliqué par commit aabb787d958ca5683af6806beba7c111630dfe31.
#12 Updated by Fabrice Barconnière over 11 years ago
- Status changed from Résolu to Fermé