Changeset 39368 in spip-zone


Ignore:
Timestamp:
Jul 19, 2010, 4:49:27 PM (10 years ago)
Author:
gilles.vincent@…
Message:
  • Il ne faut pas passer les paramètres par défaut en argument des appels de fonction (dans csvimport_importcsv())
  • un cas extrème dans lequel le foreach() plante : création d'un export sans se donner les droits
  • spip_fetch_array() est déprécié (sous 2.1)
  • indentation
Location:
_plugins_/csv_import
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/csv_import/csvimport_boutons.php

    r34095 r39368  
    2424        (!isset($GLOBALS['meta']["activer_csvimport"]) OR $GLOBALS['meta']["activer_csvimport"]!="non")) {
    2525
    26           // on voit le bouton dans la barre "naviguer"
     26                // on voit le bouton dans la barre "naviguer"
    2727                $boutons_admin['naviguer']->sousmenu["csvimport_tous"]= new Bouton(
    2828                        _DIR_PLUGIN_CSVIMPORT."img_pack/csvimport-24.gif",  // icone
  • _plugins_/csv_import/exec/csvimport_admin.php

    r29644 r39368  
    209209        $liste_des_tables_spip=array();
    210210        $liste_des_tables_autres=array();
    211         while ($row=spip_fetch_array($res,SPIP_NUM)){
    212                 $table = $row[0];
     211        while ($row=sql_fetch($res)){
     212                $table = array_shift($row);
    213213                // on ne retient que les tables prefixees par spip_
    214214                // evite les melanges sur une base avec plusieurs spip installes
  • _plugins_/csv_import/exec/csvimport_import.php

    r29644 r39368  
    1717function csvimport_visu_extrait($nombre,$import_mode,$table,$id_form){
    1818        // Extrait de la table en commençant par les dernieres maj
    19         if ($import_mode!='form')
     19        if ($import_mode!='form')
    2020                return csvimport_table_visu_extrait($table,$nombre);
    2121        else {
     
    3030        if ($mode=='table'){
    3131                $csvimport_tables_auth = csvimport_tables_auth();
    32                 if (isset($csvimport_tables_auth[$table]['field']))
     32                if (isset($csvimport_tables_auth[$table]['field']))
    3333                        $table_fields=$csvimport_tables_auth[$table]['field'];
    3434                else
     
    3838                        $table_fields[$key] = $key;
    3939                }
    40                 return $table_fields;
     40                return $table_fields;
    4141        }
    4242        if ($mode=='form' && $id_form){
     
    8181        if ($step==3){
    8282                if (($remplacer)&&(_request('annule_remplace')))
    83                   $step--;
     83                        $step--;
    8484                else if (($ajouter)&&(_request('annule_ajoute')))
    85                   $step--;
     85                        $step--;
    8686                else if ($apercu!=NULL)
    8787                        $step--;
     
    9090                else if (($ajouter)&&(!isset($csvimport_add_actif)))
    9191                        $step--;
    92   }
     92        }
    9393
    9494        if ($step==3){
     
    412412                        include_spip('inc/autoriser_compat');
    413413                $is_importable = autoriser('administrer','form',$id_form);
    414           $csvimport_replace_actif = true;
    415           $csvimport_add_actif = true;
     414                $csvimport_replace_actif = true;
     415                $csvimport_add_actif = true;
    416416        }
    417417        else {
     
    423423       
    424424                if (in_array('replaceall',$operations))
    425                   $csvimport_replace_actif = true;
     425                        $csvimport_replace_actif = true;
    426426                if (in_array('add',$operations))
    427                   $csvimport_add_actif = true;
     427                        $csvimport_add_actif = true;
    428428        }
    429429        $clean_link = $import_link;
  • _plugins_/csv_import/exec/csvimport_telecharger.php

    r29644 r39368  
    2929        $is_importable = csvimport_table_importable($table,$titre,$operations);
    3030        if (in_array('export',$operations))
    31           $csvimport_export_actif = true;
     31                $csvimport_export_actif = true;
    3232       
    3333       
     
    105105                        $ligne=array();
    106106                        foreach($tablefield as $key)
    107                           if (isset($row[$key]))
    108                             $ligne[]=$row[$key];
     107                                if (isset($row[$key]))
     108                                        $ligne[]=$row[$key];
    109109                                else
    110                                   $ligne[]="";
     110                                        $ligne[]="";
    111111                        $output .= csvimport_csv_ligne($ligne,$delim);
    112112                }
  • _plugins_/csv_import/inc/csvimport.php

    r29644 r39368  
    2727        $declared = false;
    2828        foreach($csvimport_tables_auth as $table=>$infos){
    29           if (strcmp($table,$nom_table)==0){
    30             $declared = true;
    31             if (isset($infos['statut'])){
     29                if (strcmp($table,$nom_table)==0){
     30                        $declared = true;
     31                        if (isset($infos['statut'])){
    3232                                if (!in_array($connect_statut,$infos['statut']))
    33                         $declared = false;
     33                                        $declared = false;
    3434                        }
    3535                        if ($declared){
     
    3737                                        $titre .= $infos['titre'];
    3838                                if (isset($infos['operations']))
    39                                   foreach($infos['operations'] as $op)
    40                                     $operations[]=$op;
     39                                        foreach($infos['operations'] as $op)
     40                                                $operations[]=$op;
    4141                        }
    4242                }
     
    4848        $csvimport_tables_auth = csvimport_tables_auth();
    4949        global $connect_statut;
     50        $vals = $table = array();
    5051
    5152        if (!$icone) $icone = "../"._DIR_PLUGIN_CSVIMPORT."img_pack/csvimport-24.png";
     
    6364                $compteur_liste = 0;
    6465                foreach($csvimport_tables_auth as $latable=>$info) {
    65             $declared = true;
    66             if (isset($info['statut'])){
     66                        $declared = true;
     67                        if (isset($info['statut'])){
    6768                                if (!in_array($connect_statut,$info['statut']))
    68                         $declared = false;
     69                                        $declared = false;
    6970                        }
    7071                        if ($declared) {
     
    196197function csvimport_importcsv($file, $head = 0, $delim = ",", $enclos = '"', $len = 10000) {
    197198        $importer_csv = charger_fonction('importer_csv','inc');
    198         return $importer_csv($file, $head = false, $delim = ",", $enclos = '"', $len = 10000);
     199        return $importer_csv($file, $head, $delim, $enclos, $len);
    199200}
    200201
     
    212213
    213214function csvimport_table_visu_extrait($nom_table,$nombre_lignes = 0){
    214         $maj_exist = true;
     215        $maj_exist = true;
    215216        $limit = "";
    216217        if ($nombre_lignes > 0)
    217           $limit = $nombre_lignes+1;
     218                $limit = $nombre_lignes+1;
    218219        $result = sql_select("*",$nom_table,"","","maj DESC",$limit);
    219220        if (!$result) {
     
    233234                                $ret .= "<tr>";
    234235                                foreach($row as $key=>$value){
    235                                   $ret .= "<th>" . $key . "</th>";
    236                                   $nb_col++;
     236                                        $ret .= "<th>" . $key . "</th>";
     237                                        $nb_col++;
    237238                                }
    238239                                $ret .= "</tr>\n";
     
    241242                        $ret .= "<tr>";
    242243                        foreach($row as $key=>$value)
    243                           $ret .= "<td>" . $value . "</td>";
     244                                $ret .= "<td>" . $value . "</td>";
    244245                        $ret .= "</tr>\n";
    245246                }
     
    251252        }
    252253        else
    253           $ret = "<p>"._T("csvimport:table_vide", array('table'=>$nom_table))."</p>";
     254                $ret = "<p>"._T("csvimport:table_vide", array('table'=>$nom_table))."</p>";
    254255         
    255256        return $ret;
     
    267268                                $output .= "<tr>";
    268269                                foreach($ligne as $key=>$value){
    269                                   $output .= "<th>" . $key . "</th>";
    270                                   $nb_col++;
     270                                        $output .= "<th>" . $key . "</th>";
     271                                        $nb_col++;
    271272                                }
    272273                                $output .= "</tr>\n";
     
    276277                                $output .= "<tr>";
    277278                                foreach($ligne as $value){
    278                                   $output .= "<td>" . $value . "</td>";
     279                                        $output .= "<td>" . $value . "</td>";
    279280                                }
    280281                                $output .= "</tr>\n";
    281282                        }
    282283                        if (($nombre_lignes>0)&&($data_count++>=$nombre_lignes))
    283                           break;
     284                                break;
    284285                }
    285286                if ($data_count>0)
    286                   $output .= '<tr><td style="border-top:1px dotted" colspan="'.$nb_col.'">'._T("csvimport:lignes_totales_csv",array("nb"=>count($data))).'</td></tr>';
     287                        $output .= '<tr><td style="border-top:1px dotted" colspan="'.$nb_col.'">'._T("csvimport:lignes_totales_csv",array("nb"=>count($data))).'</td></tr>';
    287288                $output .= "</table>\n";
    288289        }
    289290        if ($data_count==0)
    290           $output .= _T("csvimport:aucune_donnee");
     291                $output .= _T("csvimport:aucune_donnee");
    291292        return $output;
    292293}
     
    300301        $output .= "<tr>";
    301302        foreach($table_fields as $key=>$value){
    302           $output .= "<th>" . $value . "</th>";
     303                $output .= "<th>" . $value . "</th>";
    303304        }
    304305        $output .= "</tr>\n";
     
    322323                        $output .= "</tr>\n";
    323324                        if (($nombre_lignes>0)&&(++$data_count>=$nombre_lignes))
    324                           break;
     325                                break;
    325326                }
    326327        }
    327328        if ($data_count>0)
    328           $output .= '<tr><td style="border-top:1px dotted" colspan="'.$nb_col.'">'._T("csvimport:lignes_totales_csv",array("nb"=>count($data))).'</td></tr>';
     329                $output .= '<tr><td style="border-top:1px dotted" colspan="'.$nb_col.'">'._T("csvimport:lignes_totales_csv",array("nb"=>count($data))).'</td></tr>';
    329330        $output .= "</table>";
    330331        return $output;
     
    353354                $good_value = false;
    354355                if (isset($csvfield[$key])){
    355                   $good_key = true;
     356                        $good_key = true;
    356357                }
    357358                if ((isset($table_fields[$value]))||($value==-1)){
    358                   $good_value = true;
     359                        $good_value = true;
    359360                }
    360361                if (($good_key==false)||($good_value==false))
    361                   unset($assoc[$key]);
     362                        unset($assoc[$key]);
    362363                else{
    363364                        unset($csvfield[$key]);
     
    369370        foreach(array_keys($csvfield) as $csvkey){
    370371                foreach(array_keys($table_fields) as $tablekey)
    371                   if (strcasecmp($csvkey,$tablekey)==0){
     372                        if (strcasecmp($csvkey,$tablekey)==0){
    372373                                $assoc[$csvkey]=$tablekey;
    373374                                unset($csvfield[$csvkey]);
     
    399400                        $output .= "<option value='$tablekey'";
    400401                        if ($assoc[$csvkey]==$tablekey)
    401                           $output .= " selected='selected'";
     402                                $output .= " selected='selected'";
    402403                        $output .= ">$libelle</option>\n";
    403404                }
     
    443444        $test=array_flip($desc['field']);
    444445        if (isset($test['TIMESTAMP']))
    445           $stamp = $test['TIMESTAMP'];
     446                $stamp = $test['TIMESTAMP'];
    446447
    447448        if ($data!=false){
     
    463464                                $id_primary = sql_insertq($table, $what);
    464465                                if ($id_primary==0)
    465                                   $erreur[$count_lignes][] = "ajout impossible ::$what::$with::<br />";
     466                                        $erreur[$count_lignes][] = "ajout impossible ::$what::$with::<br />";
    466467                        }
    467468                        else
    468                           $erreur[$count_lignes][] = "rien &agrave; ajouter<br />";
     469                                $erreur[$count_lignes][] = "rien &agrave; ajouter<br />";
    469470                }
    470471        }
Note: See TracChangeset for help on using the changeset viewer.