From 001375b9a18dfa4c10dc68d12d9ada318a9bc7f1 Mon Sep 17 00:00:00 2001 From: "marcimat@rezo.net" <> Date: Mon, 15 May 2017 19:24:48 +0000 Subject: [PATCH] Des notices PHP en moins (xdjuj) --- action/duplicator.php | 5 +++-- inc/duplicator.php | 30 ++++++++++++++++++++---------- paquet.xml | 2 +- 3 files changed, 24 insertions(+), 13 deletions(-) diff --git a/action/duplicator.php b/action/duplicator.php index f2bd418..337d3b7 100755 --- a/action/duplicator.php +++ b/action/duplicator.php @@ -19,14 +19,15 @@ function action_duplicator_dist($args=null) { include_spip('inc/duplicator'); - list($objet,$id_objet,$articles) = explode(':',$args); + list($objet, $id_objet, $articles) = array_pad(explode(':', $args), 3, null); if ( ($objet=="rubrique") && ($id=intval($id_objet)) ){ // On duplique la rubrique spip_log("Duplication de la rubrique : $id.",'duplicator'); $dup_articles = true; - if($articles) + if ($articles) { $dup_articles = false; + } $nouvelle_rubrique = dupliquer_rubrique($id,null,' (cible)',$dup_articles); spip_log("Nouvelle rubrique créée : id_rubrique $nouvelle_rubrique.",'duplicator'); include_spip('inc/headers'); diff --git a/inc/duplicator.php b/inc/duplicator.php index 0d1b8c4..ce2b568 100755 --- a/inc/duplicator.php +++ b/inc/duplicator.php @@ -79,15 +79,20 @@ function dupliquer_article($id_article,$rubrique){ "type='article'", ); $infos_url = sql_fetsel('*', 'spip_urls', $where, 'date', 'date DESC'); - - $infos_url['id_objet'] = $id_article; - $url = $infos_url['url']; - //$infos_url['url'] - $u = preg_replace('/(.*)(-|,)\d*$/', '$1', $url, -1, $c); // supprimer le numéro à la fin - if ($c == 0) $infos_url['url'] = $url.'-'.$id_article; // Ajoute le numéro de l'article - else $infos_url['url'] = $u.'-'.$id_article; - sql_insertq('spip_urls', $infos_url); - + if ($infos_url) { + $infos_url['id_objet'] = $id_article; + $url = $infos_url['url']; + //$infos_url['url'] + $u = preg_replace('/(.*)(-|,)\d*$/', '$1', $url, -1, $c); // supprimer le numéro à la fin + if ($c == 0) { + $infos_url['url'] = $url.'-'.$id_article; // Ajoute le numéro de l'article + } else { + $infos_url['url'] = $u.'-'.$id_article; + } + + sql_insertq('spip_urls', $infos_url); + } + return $id_article; } @@ -243,8 +248,13 @@ function dupliquer_logo($id_source, $id_destination, $type='article', $bsurvol = $chercher_logo = charger_fonction('chercher_logo', 'inc'); $logo_source = $chercher_logo($id_source, 'id_'.$type, $logo_type ); + if (!$logo_source) { + return false; + } $logo_source = $logo_source[0]; - if ( !file_exists($logo_source) ) return false; + if ( !file_exists($logo_source) ) { + return false; + } $size = @getimagesize($logo_source); $mime = !$size ? '': $size['mime']; diff --git a/paquet.xml b/paquet.xml index 5894c1d..9c74598 100755 --- a/paquet.xml +++ b/paquet.xml @@ -1,7 +1,7 @@