Subversion Repositories eFlore/Projets.eflore-projets

Rev

Go to most recent revision | Show changed files | Details | Compare with Previous | Blame | RSS feed

Filtering Options

Rev Age Author Path Log message Diff
845 3955 d 8 h raphael /trunk/services/bibliotheque/ lib: Utilisateurs: clarification de la classe (en attendant que le RestClient soit un toolkit "stateless"  
841 3955 d 9 h raphael /trunk/services/modules/0.1/eflore/ eflore: Support des requêtes ontologies multiplexées.
Le code des ontologies communes aux référentiels (bdnt) et celui des
ontologies baseflor était à l'origine *quasi-similaire*.

Ainsi, moyennant l'application manuelle du hunk #1, le patch représentant
l'historique du passage au multiplexage pour les ontologies bdnt s'applique
à eflore:
$ svn diff -r300:HEAD services/modules/0.1/commun/Ontologies.php|patch -p4 -F3

Les ontologies eflore sont nécessaires à de nombreux endroits, à commencer
par les fiches d'eflore consultation mais aussi à d'autres webservices
(par exemple meta-donnees)

Avec ce commit il est désormais possible de multiplexer les critères:
/service:eflore:0.1/eflore/ontologies/contactType:o,contactType:p,datum:wgs84
Les valeurs sont retournées dans un tableau indexé par critère, eg:
{ "datum:wgs84": { ... }, "contactType:o": { ... } }

commits connexes: 732,733,832,836
TODO: tests phpunit
 
839 3956 d 1 h raphael /trunk/ revert r838 (commit problem)  
838 3956 d 1 h raphael /trunk/ misc: ajout Changelog  
837 3956 d 2 h raphael /trunk/services/modules/0.1/commun/ meta-données: ajout d'un cache pour les ontologies redondantes  
836 3956 d 5 h raphael /trunk/services/modules/0.1/baseflor/ baseflor: Support des requêtes ontologies multiplexées.
Le code des ontologies communes aux référentiels (bdnt) et celui des
ontologies baseflor était à l'origine *quasi-similaire*.

Ainsi à l'aide d'un peu de fuzzing et quelques tweaks, le patch représentant
l'historique du passage au multiplexage pour les ontologies bdnt s'applique
à baseflor:
$ svn diff -r300:HEAD services/modules/0.1/commun/Ontologies.php|patch -p4 -F3

Un peu de contexte: les ontologies baseflor sont nécessaires:
* lors de l'affichage de l'onglet "écologie" d'eflore consultation
* plus précisément dans les graphiques
* plus précisément encore dans les légendes

Ainsi ce n'était pas moins de 94 requêtes HTTP (moyenne) qui étaient
effectuées sur le modèle suivant:
/service:eflore:0.1/baseflor/ontologies/VES:8
/service:eflore:0.1/baseflor/ontologies/VEC:6
...

Avec ce commit il est désormais possible de multiplexer les critères:
/service:eflore:0.1/baseflor/ontologies/VEL:1,VEL:2,VEL:3,VEL:4,VEL:5,VEL:6,VEL:7,VEL:8 ...
Les valeurs sont retournées dans un tableau indexé par critère, eg:
{ "VEHE:7": { ... }, "VER:7": { ... } }

Le côté client sera implémenté dans:
eflore consultation: metier/api_0.1/Graphiques.php

TODO: tests phpunit
 
835 3957 d 0 h raphael /trunk/services/tests/0.1/nvjfl/ tests: oublie d'un fichier (suite r834)  
834 3957 d 0 h raphael /trunk/services/tests/0.1/ tests: ajout de plusieurs tests relatifs aux ontologies:
* relatifs aux ontologies bdnt (ontologies communes aux ontologies de référentiel)
* relatifs aux ontologies bdnt multiplexés (plusieurs paramètres séparés par des virgules)
* relatifs aux ontologies nvjfl multiplexées (cf r832 eflore/projet)
+ bugfix de tests existants.

==> 8 failures (api 0.1 uniquement)
 
833 3957 d 0 h raphael /trunk/services/tests/0.1/ tests: optionnalité du passage des paramètres dans l'utilisation de l'API de test  
832 3957 d 0 h raphael /trunk/services/modules/0.1/nvjfl/ nom-vernaculaires: multiplexage des requêtes ontologiques amélioré [ontologie "genre" pris en compte]

Les requêtes multi-critères sont désormais utilisées pour plusieurs ontologies différentes, du moment
que le webservice distant est le même. Ainsi les performances sont augmentées significativement
lors plusieurs retour.champs appartement à un même webservice d'ontologies sont demandés.

Exemple, pour le taxon nt=1382, la langue français lg=fra et les retour.champs=conseil_emploi,genre:
(/nvjfl/noms-vernaculaires/attributions?masque.nt=1382&masque.lg=fra&retour.champs=conseil_emploi,genre&navigation.limite=4)

La requête d'ontologies interne est désormais:
* /nvjfl/ontologies/numStatut:1,genreNombre:11,genreNombre:10

Alors qu'auparavant elle aurait induit:
* /nvjfl/ontologies/numStatut:1 # multiplage actif, mais une seule valeur pour cette ontologie
* /nvjfl/ontologies/genreNombre:11 # ontologie différente (non-multiplexé par ailleurs)
* /nvjfl/ontologies/genreNombre:10 # idem
 
831 3957 d 2 h raphael /trunk/services/modules/0.1/nvjfl/ nvjfl: ontologies, 2 bugfixes
1) l'attribut _result_ontologies@nvjfl était présent dans le JSON renvoyé à cause
de l'utilisation de isset() sur une valeur de tableau existante mais NULL.
array_key_exists() devait être utilisé

2) Lorsque la recherche d'ontologies est nécessaire, mais que 1 seule l'est, le
JSON d'ontologies renvoyé n'a pas la même structure (il n'est pas indexé par critère
de requête). Ce cas n'était pas traité correctement ce qui amenait à retourner un
"conseil_emploi" NULL.
exemple d'URL: /nvjfl//noms-vernaculaires/attributions?masque.nt=1382&masque.lg=fra&retour.champs=conseil_emploi&navigation.limite=2
 
830 3957 d 6 h raphael /trunk/services/modules/0.1/commun/ aide: en cas d'erreur, ne pas générer d'E_NOTICE de variable non-existente ce qui ne pourrait qu'amplifier le problème  
826 3959 d 5 h raphael /trunk/services/tests/0.1/cel/ tests: cel: ajout d'un test sur une date antérieure à 1901 (php 64 bits) associée à une image Cel  
825 3959 d 5 h raphael /trunk/services/tests/0.1/baseflor/ tests: baseflor: ajout d'un test sur /baseflor/informations/bdtfx.nn:XXX/relations/catminat  
815 3959 d 9 h raphael /trunk/services/modules/0.1/ trivial syntax error  
814 3959 d 9 h raphael /trunk/services/modules/0.1/ class-name conflict: "moissonnage" vs "moissonnnage"...  
813 3959 d 9 h raphael /trunk/services/configurations/ config: correction du répertoire de cache en -test avec "make test"  
812 3959 d 9 h raphael /trunk/services/configurations/ config: suppression du sous-répertoire de cache /eflore/ dans {ref:chemin_cache}  
811 3962 d 1 h raphael /trunk/services/modules/0.1/ projets/tests: fix class-name conflict sur "Cartes"  
810 3962 d 2 h raphael /trunk/services/configurations/ config: changements des valeurs par défaut ("bdd_nom" pour cel et "chemin_cache" en test)  
809 3962 d 3 h raphael /trunk/services/modules/0.1/nvjfl/ nvjfl: utilisation de la bonne URL de webservice (prod-prod, test-test)  
808 3962 d 3 h raphael /trunk/services/modules/0.1/baseflor/ r807 follow-up: conflit de nom de méthode pour convertirEnPNGAvecRsvg|indexerFichierPng dans CommunGraphiques.php  
807 3962 d 4 h raphael /trunk/services/modules/0.1/ chorodep|bdtxa/cartes: factorisation du code de génération du fichier de carte PNG, dans Commun  
806 3962 d 9 h raphael /trunk/services/configurations/ config: fix la génération automatique de bdd_nom pour prendre en compte tb_cel[_test]  
805 3962 d 9 h raphael /trunk/services/modules/0.1/chorodep/cartes/ chorodep/cartes: test du retour de exec(rsvg-convert)  
804 3962 d 9 h raphael /trunk/services/modules/0.1/chorodep/cartes/ chorodep/cartes: test présence répertoire de cache lors de la génération des vignettes png  
803 3963 d 4 h raphael /trunk/services/modules/0.1/nvjfl/ nvjfl: compat PHP-5.2, utilisation de "self::m" au lieu de "static::m"  
802 3963 d 4 h raphael /trunk/services/configurations/ config: amélioration du Makefile et suppression de double-quote problèmatiques pour parser les .ini  
801 3963 d 4 h raphael /trunk/services/ cache: fix de l'obtention de la durée du cache auprès de Commun  
800 3963 d 5 h raphael /trunk/services/configurations/ config: correction de la directive "donnees" pour coste.ini  
799 3963 d 5 h raphael /trunk/services/configurations/ config: ajout d'un Makefile pour la génération des fichiers de configuration en -test et en -prod  
798 3963 d 7 h raphael /trunk/services/configurations/ config: quelques modifications/améliorations issues de l'état du serveur + saut de ligne en fin de fichier  
797 3963 d 7 h raphael /trunk/services/ config: substitution des directives de configuration `cache.*` par `cache_*`

Commande effectuée:
$ sed -i -r \
-e 's/cache\.(stockageChemin|miseEnCache|dureeDeVie)/cache_\1/' \
-e "s/\$this->config\['cache'\]\['(stockageChemin|miseEnCache|dureeDeVie)'\]/\$this->config['cache_\1']/" \
$(egrep -rl 'cache.*(miseEnCache|dureeDeVie|stockageChemin)')
 
796 3963 d 8 h raphael /trunk/services/tests/0.1/baseflor/ config: substitution de cache.stockageChemin par cache_stockageChemin  
795 3963 d 8 h raphael /trunk/services/ config: substitution de cache.stockageChemin par cache_stockageChemin car
framework::Config() ne sait pas traiter une variable dotée d'un point dont
la partie gauche ne représente pas une section d'un fichier .ini.

Commande effectuée:
$ sed -i \
-e 's/cache.stockageChemin/cache_stockageChemin/' \
-e "s/\$this->config['cache']['stockageChemin']/\$this->config['cache_stockageChemin']/" \
configurations/* modules/0.1/*/*/*.php modules/0.1/eflore/Cartes.php tests/0.1/baseflor/GraphiquesBaseflorTest.php
 
794 3964 d 1 h raphael /trunk/services/modules/0.1/ projets/tests: correction d'une régression de r791: les segments d'URL supplémentaires n'étaient pas pris en compte  
793 3964 d 2 h raphael /trunk/services/tests/0.1/commun/ tests: fix CommunMetaDonneesTest (projet "commun" => "eflore")  
792 3964 d 2 h raphael /trunk/services/modules/0.1/ typo: francisation  
791 3964 d 3 h raphael /trunk/services/modules/0.1/ projets/tests: workaround autour du fait qu'on cherche à utiliser parfois simultanément
plusieurs classes distinctes utilisant le même nom sans namespace (PHP-5.2)... no comment
testsuite "services": 13 échecs
 
790 3965 d 1 h raphael /trunk/services/tests/0.1/ webservices/tests: consulterBrut() et creerUrl() statiques et passage de \$service et \$projet à consulterJson().
En effet, on peut observer que certains tests, tels que EfloreOntologiesTest::testRecherche(), se comportent
différemment selon que la suite est lancée en intégralité ou non.
Le service requêté peut par exemple s'avérer finalement être "baseflor" au
lieu initialement "eflore". L'usage extensif d'attribut de classe pourrait en être
la cause (dans le cas du lancement de tests en parallèles par exemple).
En tout état de cause, et pour faciliter la compréhension du code au détriment d'un léger
accroissement de la verbosité, consulterBrut(), creerUrl() et consulterJson() nécessitent
désormais l'intégralité des paramètres dont ils font usage.
[ le nombre de tests en échec ne change pas: 20 ]