Changeset 71850 in spip-zone

Timestamp:

Apr 9, 2013, 5:58:50 PM (6 years ago)

Author:

kent1@…

Message:

Un peu de phpdoc

Faire fonctionner correctement l'écriture des tags

Faire attention, quand on souhaite uniquement récupérer les infos et non pas les enregistrer en base

Un lien de documentation, un lien de développement, on incrémente en 1.3.2

Location:

_plugins_/getID3/trunk

Files:

• formulaires/editer_id3.php (modified) (3 diffs)
• genie/getid3_taches_generales.php (modified) (1 diff)
• inc/getid3_ecrire_infos.php (modified) (5 diffs)
• inc/getid3_recuperer_infos.php (modified) (3 diffs)
• paquet.xml (modified) (1 diff)

_plugins_/getID3/trunk/formulaires/editer_id3.php

@@ -6 +6 @@
  * Auteurs :
  * kent1 (http://www.kent1.info - kent1@arscenic.info), BoOz
- * 2008-2012 - Distribué sous licence GNU/GPL
+ * 2008-2013 - Distribué sous licence GNU/GPL
  * 
  * Formulaire d'édition des tags ID3 d'un fichier sonore
@@ -22 +22 @@
 function formulaires_editer_id3_charger($id,$retour=''){
         $valeurs = array();
-        $config_id3 = lire_config('getid3',array());
-        $infos_doc = sql_fetsel('*','spip_documents','id_document='.intval($id));
+        if(!function_exists('lire_config'))
+                $config_id3 = lire_config('getid3',array());
         
-        if(!in_array($infos_doc['extension'],lire_config('getid3_write',array('mp3')))){
+        $infos_doc = sql_fetsel('extension,distant,fichier','spip_documents','id_document='.intval($id));
+        if(!in_array($infos_doc['extension'],lire_config('getid3_write',array('mp3'))))
                 $valeurs['message_erreur'] = _T('getid3:message_extension_invalide_ecriture');
-        }else if($infos_doc['distant'] == 'oui'){
+        else if($infos_doc['distant'] == 'oui')
                 $valeurs['message_erreur'] = _T('getid3:message_erreur_document_distant_ecriture');
-        }
-        if(isset($valeurs['message_erreur'])){
+        
+        if(isset($valeurs['message_erreur']))
                 $valeurs['editable'] = false;
-        }else{
+        else{
                 /**
                  * Récupération des tags habituels:
@@ -43 +44 @@
                 include_spip('inc/documents');
                 $fichier = get_spip_doc($infos_doc['fichier']);
-                $recuperer_id3 = charger_fonction('recuperer_id3','inc');
-                $valeurs = $recuperer_id3($fichier);
+                $recuperer_id3 = charger_fonction('getid3_recuperer_infos','inc');
+                $valeurs = $recuperer_id3(false,$fichier,false,true);
+
                 foreach($valeurs as $valeur => $info){
                         if(preg_match('/cover/',$valeur)){
                                 $valeurs['covers'][] = $info;
                                 $valeurs['_hidden'] .= "<input type='hidden' name='old_cover' id='old_cover' value='$info' />"; 
-                        }else{
-                                $valeurs[$valeur] = filtrer_entites($info);
-                        }
+                        }else
+                                $valeurs[$valeur] = $info;
                 }
                 if(!count($valeurs['covers']) && (strlen($config_id3['cover_defaut']) > 0)){

_plugins_/getID3/trunk/genie/getid3_taches_generales.php

@@ -6 +6 @@
  * Auteurs :
  * kent1 (http://www.kent1.info - kent1@arscenic.info), BoOz
- * 2008-2012 - Distribué sous licence GNU/GPL
+ * 2008-2013 - Distribué sous licence GNU/GPL
  *
+ * @package SPIP\GetID3\Crons
  */
 

_plugins_/getID3/trunk/inc/getid3_ecrire_infos.php

@@ -6 +6 @@
  * Auteurs :
  * kent1 (http://www.kent1.info - kent1@arscenic.info), BoOz
- * 2008-2012 - Distribué sous licence GNU/GPL
+ * 2008-2013 - Distribué sous licence GNU/GPL
  *
+ * Ecriture des tags ID3 ou vorbis
  */
 
@@ -26 +27 @@
 
 function inc_getid3_ecrire_infos($id_document,$infos=array(),$images=null,$formats = array('id3v1', 'id3v2.3')){
-        if(!intval($id_document)){
-                return;
-        }
+        if(!intval($id_document))
+                return false;
         
         $document = sql_fetsel("fichier,distant,extension", "spip_documents","id_document=".intval($id_document));
@@ -42 +42 @@
                 }
                 
-                $err = array();
-                $TagData = array();
+                $err = $TagData = array();
                 
                 include_spip('inc/documents');
@@ -50 +49 @@
                 include_spip('getid3/getid3');
                 $getid3 = new getID3;
-                if(!$getid3){
+                if(!$getid3)
                         return false;
-                }
                 
                 include_spip('getid3/write');
@@ -133 +131 @@
                  * Les warnings
                  */
-                if (!empty($ecrire->warnings)) {
+                if (!empty($ecrire->warnings))
                 $err = array_merge($err,$ecrire->warnings);
-                }
         
                 /**
                  * Les erreurs
                  */
-                if (!empty($ecrire->errors)) {
+                if (!empty($ecrire->errors))
                         $err = array_merge($err,$ecrire->errors);
-                }
                 
                 /**

_plugins_/getID3/trunk/inc/getid3_recuperer_infos.php

@@ -106 +106 @@
          * pour que le point soit lié dans le pipeline post_edition
          */
-        if(defined('_DIR_PLUGIN_GIS') && is_numeric($infos['latitude']) && is_numeric($infos['longitude'])){
+        if(defined('_DIR_PLUGIN_GIS') && !$only_return && is_numeric($infos['latitude']) && is_numeric($infos['longitude'])){
                 $zoom = lire_config('gis/zoom',4);
                 $config = @unserialize($GLOBALS['meta']['gis']);
@@ -233 +233 @@
         }
         
-        if((count($covers) > 0)){
+        if((count($covers) > 0) && !$only_return){
                 if(intval($id_document))
                         $id_vignette = sql_getfetsel('id_vignette','spip_documents','id_document='.intval($id_document));
@@ -251 +251 @@
                                 $infos['id_vignette'] = $id_vignette;
                 }
-        }else if(strlen($cover_defaut = lire_config('getid3/cover_defaut','')) > 1){
+        }else if(!$only_return && strlen($cover_defaut = lire_config('getid3/cover_defaut','')) > 1){
                 /**
                  * Si on n'a pas de cover,

_plugins_/getID3/trunk/paquet.xml

@@ -2 +2 @@
         prefix="getid3"
         categorie="multimedia"
-        version="1.3.1"
+        version="1.3.2"
         etat="stable"
         compatibilite="[3.0.2;3.0.99]"
         logo="images/id3v2.png"
         schema="0.5.0"
+        documentation="http://www.mediaspip.net/?article32"
+        developpement="http://www.mediaspip.net/spip.php?page=tickets&composant=plugins%20-%20getID3"
 >