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

  * Et le test effectué doit matcher sur:
  * ^(expression)% *OU* %(expression)% [cf getConditionsImages()]
+ *
  * Par défaut les tags sont comma-separated (OU logique).
  * Cependant pour conserver le comportement du masque général qui sous-entend un ET logique sur
  * des tags séparés par des espaces recherche
+ *
  * TODO:
  * -affiner la gestion de passage de mots-clefs dans le masque général.
  * - 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)
 // del/services/0.1/images?navigation.depart=0&navigation.limite=12&tri=votes&ordre=desc&protocole=3
 // del/services/0.1/images?navigation.depart=0&navigation.limite=12&tri=votes&ordre=desc&protocole=3&masque=plop
 class ListeImages {
-    // TODO: PHP-x.y, ces variables devrait être des "const"
+	// 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 $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', 'points');
-    // en plus de ceux dans DelTk
-    static $parametres_autorises = array('protocole', 'masque.tag_cel', 'masque.tag_pictoflora', 'masque.milieu');
-    static $default_params = array('navigation.depart' => 0, 'navigation.limite' => 10,
-				   'tri' => 'date_transmission', 'ordre' => 'desc',
-				   // spécifiques à PictoFlora:
-				   'format' => 'XL');
-    static $default_proto = 3; // proto par défaut: capitalisation d'img (utilisé uniquement pour tri=(tags|votes|points))
-    static $mappings = array(
-	'observations' => array( // v_del_image
-	    "id_observation" => 1,
-	    "date_observation" => 1,
-	    "date_transmission" => 1,
-	    "famille" => "determination.famille",
-	    "nom_sel" => "determination.ns",
-	    "nom_sel_nn" => "determination.nn",
-	    "nom_referentiel" => "determination.referentiel",
-	    "nt" => "determination.nt",
-	    "ce_zone_geo" => "id_zone_geo",
-	    "zone_geo" => 1,
-	    "lieudit" => 1,
-	    "station" => 1,
-	    "milieu" => 1,
-	    "mots_cles_texte" => "mots_cles_texte",
-	    "commentaire" => 1,
-	    "ce_utilisateur" => "auteur.id",
-	    "nom_utilisateur" => "auteur.nom",
-	    "prenom_utilisateur" => "auteur.prenom",
-	),
-	'images' => array( // v_del_image
-	    'id_image' => 1,
+	static $tri_possible = array('date_transmission', 'date_observation', 'votes', 'tags', 'points');
+	// en plus de ceux dans DelTk
+	static $parametres_autorises = array('protocole', 'masque.tag_cel', 'masque.tag_pictoflora', 'masque.milieu');
+	static $default_params = array('navigation.depart' => 0, 'navigation.limite' => 10,
+		'tri' => 'date_transmission', 'ordre' => 'desc',
+		// spécifiques à PictoFlora:
+		'format' => 'XL');
+	static $default_proto = 3; // proto par défaut: capitalisation d'img (utilisé uniquement pour tri=(tags|votes|points))
+	static $mappings = array(
+		'observations' => array( // v_del_image
+			"id_observation" => 1,
+			"date_observation" => 1,
+			"date_transmission" => 1,
+			"famille" => "determination.famille",
+			"nom_sel" => "determination.ns",
+			"nom_sel_nn" => "determination.nn",
+			"nom_referentiel" => "determination.referentiel",
+			"nt" => "determination.nt",
+			"ce_zone_geo" => "id_zone_geo",
+			"zone_geo" => 1,
+			"lieudit" => 1,
+			"station" => 1,
+			"milieu" => 1,
+			"mots_cles_texte" => "mots_cles_texte",
+			"commentaire" => 1,
+			"ce_utilisateur" => "auteur.id",
+			"nom_utilisateur" => "auteur.nom",
+			"prenom_utilisateur" => "auteur.prenom",),
+		'images' => array( // v_del_image
-	    // l'alias suivant est particulier: in-fine il doit s'appeler mots_cles_texte
+			'id_image' => 1,
-	    // mais nous afin d'éviter un conflit d'alias nous le renommons plus tard (reformateImagesDoubleIndex)
+			// l'alias suivant est particulier: in-fine il doit s'appeler mots_cles_texte
-	    'i_mots_cles_texte' => 1
+			// mais nous afin d'éviter un conflit d'alias nous le renommons plus tard (reformateImagesDoubleIndex)
-     * nécessaire ? tableau sprintf(key (tri) => value (ordre), key => value ...).
+	 * nécessaire ? tableau sprintf(key (tri) => value (ordre), key => value ...).
-     * Cependant il est impensable de joindre sur un AVG() des valeurs des votes pour
+	 * Cependant il est impensable de joindre sur un AVG() des valeurs des votes pour
-     * *chaque* couple (id_image, protocole) de la base afin de trouver les images
+	 * *chaque* couple (id_image, protocole) de la base afin de trouver les images
-     * les "mieux notées", ou bien les images ayant le "plus de tags" (COUNT())
+	 * les "mieux notées", ou bien les images ayant le "plus de tags" (COUNT())
-     */
+	 */
-    static function sqlOrderBy($p, $db, &$req) {
+	static function sqlOrderBy($p, $db, &$req) {
 		// parmi self::$tri_possible
-		if($p['tri'] == 'votes') { // LEFT JOIN sur "dis" ci-dessous
+		if ($p['tri'] == 'votes') { // LEFT JOIN sur "dis" ci-dessous
-		    $req['orderby'] = 'dis.moyenne ' . $p['ordre'] . ', dis.nb_votes ' . $p['ordre'];
+			$req['orderby'] = 'dis.moyenne ' . $p['ordre'] . ', dis.nb_votes ' . $p['ordre'];
-		    return;
+			return;
 		}
-		if($p['tri'] == 'points') { // LEFT JOIN sur "dis" ci-dessous
+		if ($p['tri'] == 'points') { // LEFT JOIN sur "dis" ci-dessous
-		    $req['orderby'] = 'dis.nb_points ' . $p['ordre'] . ', dis.moyenne ' . $p['ordre'] . ', dis.nb_votes ' . $p['ordre'];
+			$req['orderby'] = 'dis.nb_points ' . $p['ordre'] . ', dis.moyenne ' . $p['ordre'] . ', dis.nb_votes ' . $p['ordre'];
-		    return;
+			return;
 		}
-		if($p['tri'] == 'tags') { // LEFT JOIN sur "dis" ci-dessous
+		if ($p['tri'] == 'tags') { // LEFT JOIN sur "dis" ci-dessous
-		    $req['orderby'] = 'dis.nb_tags ' . $p['ordre'];
+			$req['orderby'] = 'dis.nb_tags ' . $p['ordre'];
-		    return;
+			return;
 		}
-		if($p['tri'] == 'date_observation') {
+		if ($p['tri'] == 'date_observation') {
-		    $req['orderby'] = 'date_observation ' . $p['ordre'] . ', id_observation ' . $p['ordre'];
+			$req['orderby'] = 'date_observation ' . $p['ordre'] . ', id_observation ' . $p['ordre'];
-		    return;
+			return;
 		}
 		// tri == 'date_transmission'
 		// avant cel:r1860, date_transmission pouvait être NULL
 		// or nous voulons de la cohérence (notamment pour phpunit)
 		$req['orderby'] = 'date_transmission ' . $p['ordre'] . ', id_observation ' . $p['ordre'];
-    }
+	}
-    /*
+	/*
-     * in $p: un tableau de paramètres, dont:
+	 * in $p: un tableau de paramètres, dont:
-     * - 'masque.tag_cel': *tableau* de mots-clefs à chercher parmi cel_image.mots_clefs_texte
+	 * - 'masque.tag_cel': *tableau* de mots-clefs à chercher parmi cel_image.mots_clefs_texte
-     * - 'masque.tag_pictoflora': *tableau* de mots-clefs à chercher parmi del_image_tag.tag_normalise
+	 * - 'masque.tag_pictoflora': *tableau* de mots-clefs à chercher parmi del_image_tag.tag_normalise
-     * - 'tag_explode_semantic': défini si les éléments sont tous recherchés ou NON
+	 * - 'tag_explode_semantic': défini si les éléments sont tous recherchés ou NON
-     *
+	 *
-     * in/ou: $req: un tableau de structure de requête MySQL
+	 * in/ou: $req: un tableau de structure de requête MySQL
-     *
+	 *
-     * Attention, le fait que nous cherchions masque.tag_cel OU/ET masque.tag_cel
+	 * Attention, le fait que nous cherchions masque.tag_cel OU/ET masque.tag_cel
-     * ne dépend pas de nous, mais du niveau supérieur de construction de la requête:
+	 * ne dépend pas de nous, mais du niveau supérieur de construction de la requête:
-     * Soit directement $this->consulter() si des masque.tag* sont passés
+	 * Soit directement $this->consulter() si des masque.tag* sont passés
-     * (split sur ",", "AND" entre chaque condition, "OR" pour chaque valeur de tag)
+	 * (split sur ",", "AND" entre chaque condition, "OR" pour chaque valeur de tag)
-     * Soit via sqlAddMasqueConstraint():
+	 * Soit via sqlAddMasqueConstraint():
-     * (pas de split, "OR" entre chaque condition) [ comportement historique ]
+	 * (pas de split, "OR" entre chaque condition) [ comportement historique ]
-     * équivalent à:
+	 * équivalent à:
-     * (split sur " ", "OR" entre chaque condition, "AND" pour chaque valeur de tag)
+	 * (split sur " ", "OR" entre chaque condition, "AND" pour chaque valeur de tag)
-     *
-     */
+	 *
-    static function sqlAddConstraint($p, $db, &$req, Conteneur $c = NULL) {
+	 */
-		// TODO implement dans DelTk ?
+	static function sqlAddConstraint($p, $db, &$req, Conteneur $c = NULL) {
-		if(!empty($p['masque.milieu'])) {
+		// TODO implement dans DelTk ?
-		    $req['where'][] = 'vdi.milieu LIKE '.$db->proteger('%' . $p['masque.milieu'].'%');
+		if (!empty($p['masque.milieu'])) {
-		}
+			$req['where'][] = 'vdi.milieu LIKE '.$db->proteger('%' . $p['masque.milieu'].'%');
+		}
 		/* Pour le tri par AVG() des votes nous avons toujours un protocole donné,
-		   celui-ci indique sur quels votes porte l'AVG.
+			celui-ci indique sur quels votes porte l'AVG.
-		   (c'est un *vote* qui porte sur un protocole et non l'image elle-même) */
+			(c'est un *vote* qui porte sur un protocole et non l'image elle-même) */
 		/* TODO: perf problème:
 ) SQL_CALC_FOUND_ROWS: fixable en:
-	           - dissociant le comptage de la récup d'id + javascript async
+				- dissociant le comptage de la récup d'id + javascript async
-	           - ou ne rafraîchir le total *que* pour les requête impliquant un changement de pagination
+				- ou ne rafraîchir le total *que* pour les requête impliquant un changement de pagination
-	           (paramètre booléen "with-total" par exemple)
+				(paramètre booléen "with-total" par exemple)
 ) jointure forcées: en utilisant `del_imagè`, nous forçons les 2 premiers
-	           JOIN sur cel_obs_images et cel_obs pour filtrer sur "transmission".
+				JOIN sur cel_obs_images et cel_obs pour filtrer sur "transmission".
-	           Dénormaliser cette valeur et l'intégrer à `cel_images` ferait économiser cette couteuse
+				Dénormaliser cette valeur et l'intégrer à `cel_images` ferait économiser cette couteuse
-	           jointure, ... lorsqu'aucun masque portant sur `cel_obs` n'est utilisé
+				jointure, ... lorsqu'aucun masque portant sur `cel_obs` n'est utilisé
 ) non-problème: l'ordre des joins est forcé par l'usage de la vue:
-	           (cel_images/cel_obs_images/cel_obs/del_image_stat)
+				(cel_images/cel_obs_images/cel_obs/del_image_stat)
-	           Cependant c'est à l'optimiseur de définir son ordre préféré. */
+				Cependant c'est à l'optimiseur de définir son ordre préféré. */
-		if($p['tri'] == 'votes' || $p['tri'] == 'points') {
+		if ($p['tri'] == 'votes' || $p['tri'] == 'points') {
-		    // $p['protocole'] *est* défini (cf requestFilterParams())
+			// $p['protocole'] *est* défini (cf requestFilterParams())
-		    // petite optimisation: INNER JOIN si ordre DESC car les 0 à la fin
+			// petite optimisation: INNER JOIN si ordre DESC car les 0 à la fin
-		    if($p['ordre'] == 'desc') {
+			if($p['ordre'] == 'desc') {
 				// pas de group by nécessaire pour cette jointure
 				// PRIMARY KEY (`ce_image`, `ce_protocole`)
 				$req['join']['dis'] = sprintf('INNER JOIN del_image_stat dis'.
-							 ' ON vdi.id_image = dis.ce_image'.
+					 ' ON vdi.id_image = dis.ce_image'.
-							 ' AND dis.ce_protocole = %d',
+					 ' AND dis.ce_protocole = %d',
-							 $p['protocole']);
+					 $p['protocole']);
-		    } else {
+			} else {
 				$req['join']['dis'] = sprintf('LEFT JOIN del_image_stat dis'.
-							 ' ON vdi.id_image = dis.ce_image'.
+					 ' ON vdi.id_image = dis.ce_image'.
-							 ' AND dis.ce_protocole = %d',
+					 ' AND dis.ce_protocole = %d',
-							 $p['protocole']);
+					 $p['protocole']);
 				// nécessaire (dup ce_image dans del_image_stat)
 				$req['groupby'][] = 'vdi.id_observation';
-		    }
+			}
+		}
-		if($p['tri'] == 'tags') {
+		if ($p['tri'] == 'tags') {
-		    $req['join'][] = sprintf('%s JOIN del_image_stat dis ON vdi.id_image = dis.ce_image',
+			$req['join'][] = sprintf('%s JOIN del_image_stat dis ON vdi.id_image = dis.ce_image',
+			' ORDER BY %s'.
+			' LIMIT %d, %d -- %s',
+			$req['join'] ? implode(' ', array_unique($req['join'])) : '',
+			$req['where'] ? implode(' AND ', $req['where']) : 'TRUE',
-	    $req['groupby'] ? ('GROUP BY ' . implode(', ', array_unique($req['groupby']))) : '',
+			$req['groupby'] ? ('GROUP BY ' . implode(', ', array_unique($req['groupby']))) : '',
-	    $req['orderby'],
+			$req['orderby'],
-	    $p['navigation.depart'], $p['navigation.limite'], __FILE__ . ':' . __LINE__));
+			$p['navigation.depart'], $p['navigation.limite'], __FILE__ . ':' . __LINE__));
-    }
+	}
-    static function chargerImages($db, $idImg) {
+	static function chargerImages($db, $idImg) {
 		$obs_fields = DelTk::sqlFieldsToAlias(self::$mappings['observations'], NULL);
 		$image_fields = DelTk::sqlFieldsToAlias(self::$mappings['images'], NULL);
 		return $db->recupererTous(sprintf('SELECT '.
-					  ' CONCAT(id_image, "-", id_observation) AS jsonindex,'.
+			' CONCAT(id_image, "-", id_observation) AS jsonindex,'.
-					  ' %1$s, %2$s FROM v_del_image '.
+			' %1$s, %2$s FROM v_del_image '.
-					  ' WHERE %3$s'.
+			' WHERE %3$s'.
-					  ' ORDER BY %4$s'. // important car MySQL ne conserve par l'ordre du IN()
+			' ORDER BY %4$s'. // important car MySQL ne conserve par l'ordre du IN()
-					  ' -- %5$s',
+			' -- %5$s',
-					  $obs_fields, $image_fields,
+			$obs_fields, $image_fields,
-					  sprintf('id_image IN (%s)', implode(',', $idImg)),
+			sprintf('id_image IN (%s)', implode(',', $idImg)),
-					  sprintf('FIELD(id_image, %s)', implode(',', $idImg)),
+			sprintf('FIELD(id_image, %s)', implode(',', $idImg)),
-					  __FILE__ . ':' . __LINE__));
+			__FILE__ . ':' . __LINE__));
-    }
+	}
-    /* "masque" ne fait jamais que faire une requête sur la plupart des champs, (presque) tous traités
+	/* "masque" ne fait jamais que faire une requête sur la plupart des champs, (presque) tous traités
-       de manière identique à la seule différence que:
+		de manière identique à la seule différence que:
 ) ils sont combinés par des "OU" logiques plutôt que des "ET".
 ) les tags sont traités différemment pour conserver la compatibilité avec l'utilisation historique:
-       Tous les mots-clefs doivent matcher et sont séparés par des espaces
+		Tous les mots-clefs doivent matcher et sont séparés par des espaces
-       (dit autrement, str_replace(" ", ".*", $mots-clefs-requête) =~ $mots-clefs-mysql)
+		(dit autrement, str_replace(" ", ".*", $mots-clefs-requête) =~ $mots-clefs-mysql)
-       Pour plus d'information: (ListeObservations|DelTk)::sqlAddMasqueConstraint() */
+		Pour plus d'information: (ListeObservations|DelTk)::sqlAddMasqueConstraint() */
-    static function sqlAddMasqueConstraint($p, $db, &$req, Conteneur $c = NULL) {
+	static function sqlAddMasqueConstraint($p, $db, &$req, Conteneur $c = NULL) {
-		if(!empty($p['masque'])) {
+		if (!empty($p['masque'])) {
-		    $or_params = array('masque.auteur' => $p['masque'],
+			$or_params = array('masque.auteur' => $p['masque'],
-				       'masque.departement' => $p['masque'],
+				'masque.departement' => $p['masque'],
-				       'masque.commune' => $p['masque'], // TODO/XXX ?
+				'masque.commune' => $p['masque'], // TODO/XXX ?
-				       'masque.id_zone_geo' => $p['masque'],
+				'masque.id_zone_geo' => $p['masque'],
-				       /* tous-deux remplacent masque.tag
+				/* tous-deux remplacent masque.tag
-					  mais sont traité séparément des requestFilterParams() */
+				mais sont traité séparément des requestFilterParams() */
-				       // 'masque.tag_cel' => $p['masque'],
+				// 'masque.tag_cel' => $p['masque'],
-				       // 'masque.tag_pictoflora' => $p['masque'],
+				// 'masque.tag_pictoflora' => $p['masque'],
-				       'masque.ns' => $p['masque'],
+				'masque.ns' => $p['masque'],
-				       'masque.famille' => $p['masque'],
+				'masque.famille' => $p['masque'],
-				       'masque.date' => $p['masque'],
+				'masque.date' => $p['masque'],
-				       'masque.genre' => $p['masque'],
+				'masque.genre' => $p['masque'],
-				       'masque.milieu' => $p['masque'],
+				'masque.milieu' => $p['masque'],
-		    		   'masque.tag_cel' => $p['masque'],
+				'masque.tag_cel' => $p['masque'],
-		    	     'masque.tag_pictoflora' => $p['masque'],
+				'masque.tag_pictoflora' => $p['masque'],
-				       // tri est aussi nécessaire car affecte les contraintes de JOIN
+				// tri est aussi nécessaire car affecte les contraintes de JOIN
-				       'tri' => $p['tri'],
+				'tri' => $p['tri'],
-				       'ordre' => $p['ordre']);
+				'ordre' => $p['ordre']);
-		    if (array_key_exists('protocole', $p)) {
+			if (array_key_exists('protocole', $p)) {
-		    	$or_params['protocole'] = $p['protocole'];
+				$or_params['protocole'] = $p['protocole'];
-		    }
+			}
-		    /* Cependant les champs spécifiques ont priorité sur le masque général.
+			/* Cependant les champs spécifiques ont priorité sur le masque général.
-		       Pour cette raison nous supprimons la génération de SQL du masque général sur les
+				Pour cette raison nous supprimons la génération de SQL du masque général sur les
-		       champ spécifiques qui feront l'objet d'un traitement avec une valeur propre. */
+				champ spécifiques qui feront l'objet d'un traitement avec une valeur propre. */
-		    if(isset($p['masque.auteur'])) unset($or_params['masque.auteur']);
+			if(isset($p['masque.auteur'])) unset($or_params['masque.auteur']);
-		    if(isset($p['masque.departement'])) unset($or_params['masque.departement']);
+			if(isset($p['masque.departement'])) unset($or_params['masque.departement']);
-		    if(isset($p['masque.commune'])) unset($or_params['masque.commune']);
+			if(isset($p['masque.commune'])) unset($or_params['masque.commune']);
-		    if(isset($p['masque.id_zone_geo'])) unset($or_params['masque.id_zone_geo']);
+			if(isset($p['masque.id_zone_geo'])) unset($or_params['masque.id_zone_geo']);
-		    if(isset($p['masque.ns'])) unset($or_params['masque.ns']);
+			if(isset($p['masque.ns'])) unset($or_params['masque.ns']);
-		    if(isset($p['masque.famille'])) unset($or_params['masque.famille']);
+			if(isset($p['masque.famille'])) unset($or_params['masque.famille']);
-		    if(isset($p['masque.date'])) unset($or_params['masque.date']);
+			if(isset($p['masque.date'])) unset($or_params['masque.date']);
-		    if(isset($p['masque.genre'])) unset($or_params['masque.genre']);
+			if(isset($p['masque.genre'])) unset($or_params['masque.genre']);
-		    if(isset($p['masque.milieu'])) unset($or_params['masque.milieu']);
+			if(isset($p['masque.milieu'])) unset($or_params['masque.milieu']);
-		    if(isset($p['masque.tag_cel'])) unset($or_params['masque.tag_cel']);
+			if(isset($p['masque.tag_cel'])) unset($or_params['masque.tag_cel']);
-		    if(isset($p['masque.tag_pictoflora'])) unset($or_params['masque.tag_pictoflora']);
+			if(isset($p['masque.tag_pictoflora'])) unset($or_params['masque.tag_pictoflora']);
+			$or_masque = array_merge(
-		    $or_masque = array_merge(
+				DelTk::requestFilterParams($or_params, NULL, $c /* pour masque.departement */),
-				DelTk::requestFilterParams($or_params, NULL, $c /* pour masque.departement */),
+				self::requestFilterParams($or_params)
-				self::requestFilterParams($or_params)
+			);
-		    );
+			/* Lorsqu'on utilise le masque général pour chercher des tags, ils sont
-		    /* 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. */
-		       postulés comme séparés par des espaces, et doivent être tous matchés. */
+			if (isset($or_params['masque.tag_cel'])) {
-		    if(isset($or_params['masque.tag_cel']))
+				$or_masque['masque.tag_cel'] = DelTk::buildTagsAST($p['masque'], 'AND', ' ');
-			$or_masque['masque.tag_cel'] = DelTk::buildTagsAST($p['masque'], 'AND', ' ');
+			}
-		    if(isset($or_params['masque.tag_pictoflora']))
+			if (isset($or_params['masque.tag_pictoflora'])) {
-			$or_masque['masque.tag_pictoflora'] = DelTk::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);
+			DelTk::sqlAddConstraint($or_masque, $db, $or_req);
-		    self::sqlAddConstraint($or_masque, $db, $or_req);
+			self::sqlAddConstraint($or_masque, $db, $or_req);
-		    if($or_req['where']) {
+			if ($or_req['where']) {
-			$req['where'][] = '(' . implode(' OR ', $or_req['where']) . ')';
+				$req['where'][] = '(' . implode(' OR ', $or_req['where']) . ')';
-			// utile au cas ou des jointures seraient rajoutées
+				// utile au cas ou des jointures seraient rajoutées
-			$req['join'] = array_unique(array_merge($req['join'], $or_req['join']));
+				$req['join'] = array_unique(array_merge($req['join'], $or_req['join']));
+			}
+		}
+	}
-    // cf Observation::reformateObservationSimpleIndex() et ListeObservations::reformateObservation()
+	// cf Observation::reformateObservationSimpleIndex() et ListeObservations::reformateObservation()
-    // (trop de variétés de formatage, à unifier côté client pour unifier côté backend ...)
+	// (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') {
+	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) {
+		foreach ($obs as $o) {
-		    // ceci nous complique la tâche pour le reste du processing...
+			// ceci nous complique la tâche pour le reste du processing...
-		    $id = $o['jsonindex'];
+			$id = $o['jsonindex'];
-		    // ainsi nous utilisons deux tableaux: le final, indexé par couple d'id(image-obs)
+			// 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()
+			// et celui indexé par simple id_image qui est fort utile pour mapVotesToImages()
-		    // mais tout deux partage leur référence à "protocole"
+			// 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
+			$image = array(
-		    );
+				'id_image' => $o['id_image'],
-		    unset($o['id_image'], $o['i_mots_cles_texte'], $o['jsonindex']);
+				'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;
+			if (!isset($obs_merged[$id])) {
-		    $obs_merged[$id]['observation'] = $o;
+				$obs_merged[$id] = $image;
-		    $obs_merged[$id]['protocoles_votes'] = array();
+			}
+			$obs_merged[$id]['observation'] = $o;
-		    $obs_keyed_by_id_image[$image['id_image']]['protocoles_votes'] = &$obs_merged[$id]['protocoles_votes'];
+			$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);
+		return array($obs_merged,$obs_keyed_by_id_image);
+	}
+	// complete & override DelTk::requestFilterParams() (même usage)
+	static function requestFilterParams(Array $params, $parametres_autorises = NULL) {
-    // complete & override DelTk::requestFilterParams() (même usage)
+		if ($parametres_autorises) { // filtrage de toute clef inconnue
-    static function requestFilterParams(Array $params, $parametres_autorises = NULL) {
+			$params = array_intersect_key($params, array_flip($parametres_autorises));
-		if($parametres_autorises) { // filtrage de toute clef inconnue
+		}
-		    $params = array_intersect_key($params, array_flip($parametres_autorises));
 		$p = array();
+		$p['tri'] = DelTk::unsetIfInvalid($params, 'tri', self::$tri_possible);
-		$p = array();
+		$p['format'] = DelTk::unsetIfInvalid($params, 'format', self::$format_image_possible);
-		$p['tri'] = DelTk::unsetIfInvalid($params, 'tri', self::$tri_possible);
-		$p['format'] = DelTk::unsetIfInvalid($params, 'format', self::$format_image_possible);
+		// "milieu" inutile pour IdentiPlantes ?
 		if (isset($params['masque.milieu'])) {
-		// "milieu" inutile pour IdentiPlantes ?
+			$p['masque.milieu'] = trim($params['masque.milieu']);
 		if(isset($params['masque.milieu'])) $p['masque.milieu'] = trim($params['masque.milieu']);
 		// compatibilité
-		if(isset($params['masque.tag'])) {
-		    $params['masque.tag_cel'] = $params['masque.tag_pictoflora'] = $params['masque.tag'];
-		}
+		if (isset($params['masque.tag'])) {
+			$params['masque.tag_cel'] = $params['masque.tag_pictoflora'] = $params['masque.tag'];
-		if($p['tri'] == 'votes' || $p['tri'] == 'tags' || $p['tri'] == 'points') {
-		    // ces critère de tri des image à privilégier ne s'applique qu'à un protocole donné
-		    if(!isset($params['protocole']) || !is_numeric($params['protocole'])) {
-				$p['protocole'] = self::$default_proto;
-		    } else {
-				$p['protocole'] = intval($params['protocole']);
-		    }
-		}
-		return array_filter($p, create_function('$a','return !in_array($a, array("",false,null),true);'));
-    }
-    // met à jour *toutes* les stats de nombre de tags et de moyenne des votes
-    static function _update_statistics($db) {
-	$db->requeter("TRUNCATE TABLE del_image_stat");
-	$db->requeter(<<<EOF
-INSERT INTO `del_image_stat` (
-	SELECT id_image, divo.ce_protocole, divo.moyenne, divo.nb_votes, dit.ctags
-	FROM `tb_cel`.`cel_images` ci
-	LEFT JOIN
-	( SELECT ce_image, ce_protocole, AVG(valeur) AS moyenne, COUNT(valeur) AS nb_votes FROM del_image_vote
 	  GROUP BY ce_image, ce_protocole ) AS divo
-	ON ci.id_image = divo.ce_image
+		if ($p['tri'] == 'votes' || $p['tri'] == 'tags' || $p['tri'] == 'points') {

Subversion Repositories eFlore/Applications.del

(root)/trunk/services/modules/0.1/images/ListeImages.php – Rev 1584 → 1604