WebSVN – eFlore/Applications.del – Diff – /trunk/services/modules/0.1/images/ListeImages2.php

  * Par contre, le tri par moyenne des votes, sous-entend "pour un protocole donné".
  * Dès lors le choix d'un protocole doit avoir été fait afin de régler le JOIN et ainsi l'ORDER BY.
  * (cf requestFilterParams())
+ *
  * Histoire: auparavant (pré-r142x) un AVG + GROUP BY étaient utilisés pour générer on-the-fly les valeurs
- * utilsées ensuite pour l'ORDER BY. La situation à base de del_image_stat
+ * utilisées ensuite pour l'ORDER BY. La situation à base de del_image_stat
- * est déjà bien meilleur sans être pour autant optimale. cf commentaire de sqlAddConstraint()
+ * est déjà bien meilleure sans être pour autant optimale. cf commentaire de sqlAddConstraint()
+ *
+ *
  * Tags:
  * Le comportement habituel dans le masque *général*: les mots sont séparés par des espaces,
  * implod()ed par des AND (tous les mots doivent matcher).
  * - subqueries dans le FROM pour les critère WHERE portant directement sur v_del_image
  * plutôt que dans WHERE (qui nécessite dès lors un FULL-JOIN)
  * (http://www.mysqlperformanceblog.com/2007/04/06/using-delayed-join-to-optimize-count-and-limit-queries/)
  * - éviter de dépendre d'une jointure systématique sur `cel_obs`, uniquement pour `(date_)transmission
  * (cf VIEW del_image)
- * - réorganiser les méthodes statiques parmis Observation, ListeObservations et ListImages2
+ * - poursuivre la réorganisation des méthodes statiques parmis Observation, ListeObservations et ListImages2
  * - *peut-être*: passer requestFilterParams() en méthode de classe
+ *
+ *
  * MySQL sux:
  * EXPLAIN SELECT  id_image FROM v_del_image vdi WHERE vdi.id_image IN (SELECT ce_image FROM del_image_tag WHERE actif = 1 LIMIT 1);
  *	5.1: DEPENDENT SUBQUERY	del_image_tag	index_subquery	ce_image ce_image 8 func 1 Using where
  * FORCE INDEX/IGNORE INDEX semble incapable de résoudre le problème de l'optimiseur MySQL
+ *
  */
 require_once(dirname(__FILE__) . '/../DelTk.php');
 require_once(dirname(__FILE__) . '/../observations/ListeObservations.php');
 require_once(dirname(__FILE__) . '/../observations/Observation.php');
 restore_error_handler();
 restore_exception_handler();
 	// TODO: PHP-x.y, ces variables devrait être des "const"
 	static $format_image_possible = array('O','CRX2S','CRS','CXS','CS','XS','S','M','L','XL','X2L','X3L');
 	static $tri_possible = array('date_transmission', 'date_observation', 'votes', 'tags');
-	// en plus de ceux dans ListeObservations
+	// en plus de ceux dans DelTk
 	static $parametres_autorises = array('protocole', 'masque.tag_cel', 'masque.tag_pictoflora', 'masque.milieu');
 		// filtrage de l'INPUT général, on réutilise 90% de identiplante en terme de paramètres autorisés
 		// ($parametres_autorises) sauf... masque.type qui fait des modif' de WHERE sur les mots-clefs.
 		// Évitons ce genre de chose pour PictoFlora et les risques de conflits avec masque.tag
 		// même si ceux-ci sont improbables (pas d'<input> pour cela).
-		$params_ip = ListeObservations::requestFilterParams($parametres,
+		$params_ip = DelTk::requestFilterParams($parametres,
-        array_diff(ListeObservations::$parametres_autorises,
+        array_diff(DelTk::$parametres_autorises,
         array('masque.type')),
         $this->conteneur);
 		// notre propre filtrage sur l'INPUT
 		$params_pf = self::requestFilterParams($parametres,
-        array_merge(ListeObservations::$parametres_autorises,
+        array_merge(DelTk::$parametres_autorises,
         self::$parametres_autorises));
 		/* filtrage des tags + sémantique des valeurs multiples:
 		   Lorsqu'on utilise masque.tag* pour chercher des tags, ils sont
+		   postulés comme séparés par des virgule, et l'un au moins des tags doit matcher. */
-		   postulés comme séparés par des virgule, et l'un au moins des tags doit matcher. */
+		$params_pf['masque.tag_cel'] = DelTk::buildTagsAST(@$parametres['masque.tag_cel'], 'OR', ',');
-		$params_pf['masque.tag_cel'] = self::buildTagsAST(@$parametres['masque.tag_cel'], 'OR', ',');
+		$params_pf['masque.tag_pictoflora'] = DelTk::buildTagsAST(@$parametres['masque.tag_pictoflora'], 'OR', ',');
-		$params_pf['masque.tag_pictoflora'] = self::buildTagsAST(@$parametres['masque.tag_pictoflora'], 'OR', ',');
+		$params = array_merge(
-		$params = array_merge(ListeObservations::$default_params, // paramètre par défaut Identiplante
+            DelTk::$default_params, // paramètre par défaut Identiplante
-        self::$default_params, // paramètres par défaut PictoFlora
+            self::$default_params, // paramètres par défaut PictoFlora
-        $params_ip, // les paramètres passés, traités par Identiplante
+            $params_ip, // les paramètres passés, traités par Identiplante
-        $params_pf); // les paramètres passés, traités par PictoFlora
+            $params_pf); // les paramètres passés, traités par PictoFlora
+		// XXX: temp tweak
 		/* $this->conteneur->setParametre('url_images', sprintf($this->conteneur->getParametre('url_images'),
-		// XXX: temp tweak
+		   "%09d", $params['format']));*/
 		/* $this->conteneur->setParametre('url_images', sprintf($this->conteneur->getParametre('url_images'),
-		   "%09d", $params['format']));*/
+		// création des contraintes (génériques de DelTk)
 		DelTk::sqlAddConstraint($params, $db, $req);
 		$idobs_tab = self::getIdImages($params, $req, $db);
 		// Ce n'est pas la peine de continuer s'il n'y a pas eu de résultats dans la table del_obs_images
 		if(!$idobs_tab) {
 			$resultat = new ResultatService();
-			$resultat->corps = array('entete' => ListeObservations::makeJSONHeader(0, $params, Config::get('url_service')),
+			$resultat->corps = array('entete' => DelTk::makeJSONHeader(0, $params, Config::get('url_service')),
             'resultats' => array());
 			return $resultat;
 			/*
               header('HTTP/1.0 404 Not Found');
         $o = new Observation($this->conteneur);
         foreach($idobs as $i) {
         $images[$i] = $o->consulter(array($i), array('justthrow' => 1));
+        }
 		*/
-		list($images, $images_keyed_by_id_image) = ListeObservations::reformateImagesDoubleIndex(
+		list($images, $images_keyed_by_id_image) = self::reformateImagesDoubleIndex(
 			$liaisons,
 			$this->conteneur->getParametre('url_images'),
 			$params['format']);
 		// les deux masques de tags sont transformés en AST dans le processus de construction de la requête.
 		// Reprenous les paramètres originaux non-nettoyés (ils sont valables car le nettoyage est déterministe)
 		$params_header = array_merge($params, array_filter(array('masque.tag_cel' => @$parametres['masque.tag_cel'],
         'masque.tag_pictoflora' => @$parametres['masque.tag_pictoflora'])));
 		$resultat = new ResultatService();
-		$resultat->corps = array('entete' => ListeObservations::makeJSONHeader($total, $params_header, Config::get('url_service')),
+		$resultat->corps = array('entete' => DelTk::makeJSONHeader($total, $params_header, Config::get('url_service')),
         'resultats' => $images);
 		return $resultat;
+	}
 	 * équivalent à:
 	 * (split sur " ", "OR" entre chaque condition, "AND" pour chaque valeur de tag)
+	 *
 	 */
 	static function sqlAddConstraint($p, $db, &$req, Conteneur $c = NULL) {
-		// TODO implement dans ListeObservations ?
+		// TODO implement dans DelTk ?
 		if(!empty($p['masque.milieu'])) {
 			$req['where'][] = 'vdi.milieu LIKE '.$db->proteger('%' . $p['masque.milieu'].'%');
+		}
 			$p['navigation.depart'], $p['navigation.limite'], __FILE__ . ':' . __LINE__));
+	}
 	static function chargerImages($db, $idImg) {
-		$obs_fields = Observation::sqlFieldsToAlias(self::$mappings['observations'], NULL);
+		$obs_fields = DelTk::sqlFieldsToAlias(self::$mappings['observations'], NULL);
-		$image_fields = Observation::sqlFieldsToAlias(self::$mappings['images'], NULL);
+		$image_fields = DelTk::sqlFieldsToAlias(self::$mappings['images'], NULL);
 		return $db->recupererTous(sprintf('SELECT '.
             // tri est aussi nécessaire car affecte les contraintes de JOIN
             'tri' => $p['tri'],
             'ordre' => $p['ordre']);
 			$or_masque = array_merge(
-				ListeObservations::requestFilterParams($or_params, NULL, $c /* pour masque.departement */),
+				DelTk::requestFilterParams($or_params, NULL, $c /* pour masque.departement */),
 				self::requestFilterParams($or_params));
 			/* Lorsqu'on utilise le masque général pour chercher des tags, ils sont
 			   postulés comme séparés par des espaces, et doivent être tous matchés. */
-			$or_masque['masque.tag_cel'] = self::buildTagsAST($p['masque'], 'AND', ' ');
+			$or_masque['masque.tag_cel'] = DelTk::buildTagsAST($p['masque'], 'AND', ' ');
-			$or_masque['masque.tag_pictoflora'] = self::buildTagsAST($p['masque'], 'AND', ' ');
+			$or_masque['masque.tag_pictoflora'] = DelTk::buildTagsAST($p['masque'], 'AND', ' ');
 			// pas de select, groupby & co ici: uniquement 'join' et 'where'
-			// pas de select, groupby & co ici: uniquement 'join' et 'where'
+			$or_req = array('join' => array(), 'where' => array());
-			$or_req = array('join' => array(), 'where' => array());
+			DelTk::sqlAddConstraint($or_masque, $db, $or_req);
 				$req['join'] = array_unique(array_merge($req['join'], $or_req['join']));
+			}
+		}
+	}
+	// cf Observation::reformateObservationSimpleIndex() et ListeObservations::reformateObservation()
+    // (trop de variétés de formatage, à unifier côté client pour unifier côté backend ...)
+	static function reformateImagesDoubleIndex($obs, $url_pattern = '', $image_format = 'XL') {
+		// XXX: cf Observation.php::consulter(), nous pourriouns ici
+		// conserver les valeurs vides (pour les phptests notamment, ou non)
+		// $obs = array_map('array_filter', $obs);
+		$obs_merged = $obs_keyed_by_id_image = array();
+		foreach($obs as $o) {
+			// ceci nous complique la tâche pour le reste du processing...
+			$id = $o['jsonindex'];
+			// ainsi nous utilisons deux tableaux: le final, indexé par couple d'id(image-obs)
+			// et celui indexé par simple id_image qui est fort utile pour mapVotesToImages()
+			// mais tout deux partage leur référence à "protocole"
+			$image = array(
+				'id_image' => $o['id_image'],
+				'binaire.href' => sprintf($url_pattern, $o['id_image'], $image_format),
+				'mots_cles_texte' => @$o['i_mots_cles_texte'], // @, peut avoir été filtré par array_map() ci-dessus
+			);
+			unset($o['id_image'], $o['i_mots_cles_texte'], $o['jsonindex']);
+			if(!isset($obs_merged[$id])) $obs_merged[$id] = $image;
+			$obs_merged[$id]['observation'] = $o;
+			$obs_merged[$id]['protocoles_votes'] = array();
+			$obs_keyed_by_id_image[$image['id_image']]['protocoles_votes'] = &$obs_merged[$id]['protocoles_votes'];
+		}
+		return array($obs_merged,$obs_keyed_by_id_image);
+	}
-	// complete & override ListeObservations::requestFilterParams() (même usage)
+	// complete & override DelTk::requestFilterParams() (même usage)
 	static function requestFilterParams(Array $params, $parametres_autorises = NULL) {
 		if($parametres_autorises) { // filtrage de toute clef inconnue
 			$params = array_intersect_key($params, array_flip($parametres_autorises));
+		}
 		$p = array();
-		$p['tri'] = ListeObservations::unsetIfInvalid($params, 'tri', self::$tri_possible);
+		$p['tri'] = DelTk::unsetIfInvalid($params, 'tri', self::$tri_possible);
-		$p['format'] = ListeObservations::unsetIfInvalid($params, 'format', self::$format_image_possible);
+		$p['format'] = DelTk::unsetIfInvalid($params, 'format', self::$format_image_possible);
 		return array_filter($p, create_function('$a','return !in_array($a, array("",false,null),true);'));
-	}
-	/* Construit un (vulgaire) abstract syntax tree:
-	   "AND" => [ "tag1", "tag2" ]
-	   Idéalement (avec un parser simple comme proposé par http://hoa-project.net/Literature/Hack/Compiler.html#Langage_PP)
-	   nous aurions:
-	   "AND" => [ "tag1", "tag2", "OR" => [ "tag3", "tag4" ] ]
-	   Ici nous devons traiter les cas suivants:
-	   tags séparés par des "ET/AND OU/OR", séparés par des espaces ou des virgules.
-	   Mais la chaîne peut aussi avoir été issue du "masque général" (la barre de recherche générique).
-	   ce qui implique des comportement par défaut différents afin de préserver la compatibilité.
-	   Théorie:
-) tags passés par "champ tag":
-	   - support du ET/OU, et explode par virgule.
-	   - si pas d'opérande détectée: "OU"
-) tags passés par "recherche générale":
-	   - support du ET/OU, et explode par whitespace.
-	   - si pas d'opérande détectée: "ET"
-	   La présence de $additional_sep s'explique car ET/OU sous-entendent une séparation par des espaces.
-	   Mais ce n'est pas toujours pertinent car: 1) la compatibilité suggère de considérer parfois
-	   la virgule comme séparateur et 2) les tags *peuvent* contenir des espaces. Par conséquent:
-	   * a,b,c => "a" $default_op "b" $default_op "c"
-	   * a,b AND c => "a" AND "b" AND "c"
-	   * a OR b AND c,d => "a" AND "b" AND "c" AND "d"
-	   C'est à dire par ordre décroissant de priorité:
-) opérande contenu dans la chaîne
-) opérande par défaut
-) les séparateurs présents sont substitués par l'opérande déterminée par 1) ou 2)
-	   // TODO: support des parenthèses, imbrications & co: "(", ")"
-	   // http://codehackit.blogspot.fr/2011/08/expression-parser-in-php.html
-	   // http://blog.angeloff.name/post/2012/08/05/php-recursive-patterns/
-	   @param $str: la chaîne à "parser"
-	   @param $default_op: "AND" ou "OR"
-	   @param $additional_sep: séparateur de mots:
-	*/
-	static function buildTagsAST($str = NULL, $default_op, $additional_sep = ',') {
-		if(!$str) return;
-		$words = preg_split('/ (OR|AND|ET|OU) /', $str, -1, PREG_SPLIT_NO_EMPTY);
-		if(preg_match('/\b(ET|AND)\b/', $str)) $op = 'AND';
-		elseif(preg_match('/\b(OU|OR)\b/', $str)) $op = 'OR';
-		else $op = $default_op;
-		if($additional_sep) {
-			array_walk($words,
-            create_function('&$v, $k, $sep', '$v = preg_split("/".$sep."/", $v, -1, PREG_SPLIT_NO_EMPTY);'),
-            $additional_sep);
-		}
-		$words = self::array_flatten($words);
-		$words = array_map('trim', $words);
 		return array($op => array_filter($words));
+	}
 	static function revOrderBy($orderby) {
 		return $orderby == 'asc' ? 'desc' : 'asc';
-	}
-	static function array_flatten($arr) {
-		$arr = array_values($arr);
-		while (list($k,$v)=each($arr)) {
-			if (is_array($v)) {
-				array_splice($arr,$k,1,$v);
-				next($arr);
-			}
-		}
-		return $arr;
+	}

Subversion Repositories eFlore/Applications.del

(root)/trunk/services/modules/0.1/images/ListeImages2.php – Rev 1486 → 1490