Amélioration des notifications

inc/getid3_verifier_binaires.php

@@ -23,11 +23,9 @@ if (!defined("_ECRIRE_INC_VERSION")) return;
  * -* 1 en cas d'échec (l'application est là mais retourne une erreur)
  * -* 127 en cas d'absence de l'application
  * 
- * @param unknown_type $valeurs
  * @param boolean $notif : On notifie ou pas?
  */
 function inc_getid3_verifier_binaires_dist($notif=false){
-	spip_log('Verification des binaires','getid3');
 	$erreurs = array();
 	
 	$tags_write = array('mp3','mpc','ogg','flac');
@@ -36,7 +34,7 @@ function inc_getid3_verifier_binaires_dist($notif=false){
 	/**
 	 * Tester vorbiscomment
 	 */
-	exec('vorbiscomment --help',$retour,$retour_int);	
+	exec('vorbiscomment --help',$retour,$retour_int);
 	if($retour_int != 0){
 		ecrire_config('getid3_vorbiscomment_casse', 'oui');
 		$erreurs[] = 'vorbiscomment';
@@ -66,10 +64,8 @@ function inc_getid3_verifier_binaires_dist($notif=false){
 	$tags_write = array_diff($tags_write,$tags_impossible);
 	ecrire_config('getid3_write',serialize($tags_write));
 	
-	if($notif){
+	if((count($erreurs) > 0) && $notif){
 		if ($notifications = charger_fonction('notifications', 'inc')) {
-			spip_log('notifications verifier_binaires','getid3');
-			spip_log($erreurs,'getid3');
 			$notifications('getid3_verifier_binaires', 1,
 				array(
 					'erreurs' => $erreurs

notifications/getid3_verifier_binaires.html

@@ -10,12 +10,10 @@
 	Squelette de notification lorsque des binaires nécessaires sont absents
 
 ][(#HTTP_HEADER{Content-type: text/plain[; charset=(#CHARSET)]})]#CACHE{0}
-[[(#NOM_SITE_SPIP|filtrer_entites|nettoyer_titre_email|trim)]] - <:getid3:info_erreurs:> : [(#ENV{nb}|affiche_un_ou_plusieurs{<:getid3:message_titre_binaire_manquant|nettoyer_titre_email:>,<:getid3:message_titre_binaires_manquant|nettoyer_titre_email:>})]
-[(#ENV{nb}|>{1}|non)
-<:getid3:message_texte_binaire_manquant:>][(#ENV{nb}|>{1}|oui)
-<:getid3:message_texte_binaires_manquant:>]
+[[(#NOM_SITE_SPIP|nettoyer_titre_email|trim)]] - <:getid3:info_erreurs:> : [(#ENV{nb}|singulier_ou_pluriel{<:getid3:message_titre_binaire_manquant|nettoyer_titre_email:>,<:getid3:message_titre_binaires_manquant|nettoyer_titre_email:>})]
+[(#ENV{nb}|singulier_ou_pluriel{<:getid3:message_texte_binaire_manquant:>,<:getid3:message_texte_binaires_manquant:>})]
 <BOUCLE_binaires(POUR){tableau #ENV*{erreurs}}>
-- #VALEUR
+-* #VALEUR
 </BOUCLE_binaires>
 <:getid3:message_texte_binaires_informer:>
-#FILTRE{supprimer_tags|trim}
+#FILTRE{supprimer_tags|filtrer_entites|trim}

notifications/getid3_verifier_binaires.php

@@ -30,8 +30,7 @@ function notifications_getid3_verifier_binaires($quoi, $id, $options){
 		}
 		$destinataires = pipeline('notifications_destinataires',
 			array(
-				'args'=>array('quoi'=>$quoi,'id'=>$id,'options'=>$options)
-			,
+				'args'=>array('quoi'=>$quoi,'id'=>$id,'options'=>$options),
 				'data'=>$tous)
 		);
 		$msg_mail = recuperer_fond('notifications/getid3_verifier_binaires',array('erreurs'=>$options['erreurs'],'nb' => $nb));
@@ -40,7 +39,9 @@ function notifications_getid3_verifier_binaires($quoi, $id, $options){
 		 * et la validité des emails
 		 */
 		notifications_nettoyer_emails($destinataires);
-		notifications_envoyer_mails($destinataires, $msg_mail,$sujet_mail);
+		foreach ($destinataires as $email) {
+			notifications_envoyer_mails($email, $msg_mail);
+		}
 	}
 }
 ?>