From 9e6f9bba0b83572042031303baa7f7b0fc644196 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ma=C3=AFeul=20Rouquette?= Date: Sun, 10 Sep 2023 14:29:44 +0200 Subject: [PATCH] style: phpcbf --- action/dissocier_encart.php | 25 ++++---- action/supprimer_encart.php | 6 +- base/encarts.php | 61 ++++++++++--------- encarts_administrations.php | 22 +++---- encarts_autorisations.php | 19 +++--- encarts_options.php | 1 + encarts_pipelines.php | 26 ++++---- formulaires/editer_encart.php | 43 +++++++------ inc/marquer_doublons_encart.php | 30 ++++----- phpcs.xml.dist | 16 +++++ .../liste/articles_encart_fonctions.php | 1 + .../liste/encarts_associer_fonctions.php | 3 +- prive/objets/liste/encarts_lies_fonctions.php | 6 +- 13 files changed, 139 insertions(+), 120 deletions(-) create mode 100644 phpcs.xml.dist diff --git a/action/dissocier_encart.php b/action/dissocier_encart.php index 421b138..39b9e3f 100644 --- a/action/dissocier_encart.php +++ b/action/dissocier_encart.php @@ -1,4 +1,5 @@ '*'), array($objet => $id_objet)))) { + if (is_array($liens = objet_trouver_liens(['encart' => '*'], [$objet => $id_objet]))) { foreach ($liens as $lien) { if ($lien['vu'] == 'non') { $ids_encarts[] = $lien['id_encart']; } } - objet_dissocier(array('encart' => $ids_encarts), array($objet => $id_objet)); + objet_dissocier(['encart' => $ids_encarts], [$objet => $id_objet]); } break; - default : + default: if ( $id_encart = intval($encart) - AND autoriser('dissocier', 'encart', $id_encart, '', array( + and autoriser('dissocier', 'encart', $id_encart, '', [ 'objet' => $objet, 'id_objet' => $id_objet - )) + ]) ) { - objet_dissocier(array('encart' => $id_encart), array($objet => $id_objet)); + objet_dissocier(['encart' => $id_encart], [$objet => $id_objet]); } break; } } } - -?> diff --git a/action/supprimer_encart.php b/action/supprimer_encart.php index 3277f71..7a9a13b 100644 --- a/action/supprimer_encart.php +++ b/action/supprimer_encart.php @@ -1,4 +1,5 @@ 'encart', - 'principale' => "oui", - 'field' => array( - "id_encart" => "bigint(21) NOT NULL", - "titre" => "tinytext NOT NULL", - "texte" => "text NOT NULL", - "date" => "datetime NOT NULL DEFAULT '0000-00-00 00:00:00'", - "maj" => "TIMESTAMP" - ), - 'key' => array( - "PRIMARY KEY" => "id_encart", - ), + 'principale' => 'oui', + 'field' => [ + 'id_encart' => 'bigint(21) NOT NULL', + 'titre' => 'tinytext NOT NULL', + 'texte' => 'text NOT NULL', + 'date' => "datetime NOT NULL DEFAULT '0000-00-00 00:00:00'", + 'maj' => 'TIMESTAMP' + ], + 'key' => [ + 'PRIMARY KEY' => 'id_encart', + ], 'titre' => "titre AS titre, '' AS lang", - 'date' => "date", + 'date' => 'date', 'page' => false, - 'champs_editables' => array('titre', 'texte'), - 'champs_versionnes' => array(), - 'rechercher_champs' => array(), - 'tables_jointures' => array('spip_encarts_liens'), + 'champs_editables' => ['titre', 'texte'], + 'champs_versionnes' => [], + 'rechercher_champs' => [], + 'tables_jointures' => ['spip_encarts_liens'], - ); + ]; // jointures sur les encarts pour tous les objets $tables[]['tables_jointures'][] = 'encarts_liens'; @@ -85,18 +86,18 @@ function encarts_declarer_tables_objets_sql($tables) { */ function encarts_declarer_tables_auxiliaires($tables) { - $tables['spip_encarts_liens'] = array( - 'field' => array( - "id_encart" => "bigint(21) DEFAULT '0' NOT NULL", - "id_objet" => "bigint(21) DEFAULT '0' NOT NULL", - "objet" => "VARCHAR(25) DEFAULT '' NOT NULL", - "vu" => "VARCHAR(6) DEFAULT 'non' NOT NULL" - ), - 'key' => array( - "PRIMARY KEY" => "id_encart,id_objet,objet", - "KEY id_encart" => "id_encart" - ) - ); + $tables['spip_encarts_liens'] = [ + 'field' => [ + 'id_encart' => "bigint(21) DEFAULT '0' NOT NULL", + 'id_objet' => "bigint(21) DEFAULT '0' NOT NULL", + 'objet' => "VARCHAR(25) DEFAULT '' NOT NULL", + 'vu' => "VARCHAR(6) DEFAULT 'non' NOT NULL" + ], + 'key' => [ + 'PRIMARY KEY' => 'id_encart,id_objet,objet', + 'KEY id_encart' => 'id_encart' + ] + ]; return $tables; } diff --git a/encarts_administrations.php b/encarts_administrations.php index 215a021..a31d5ff 100644 --- a/encarts_administrations.php +++ b/encarts_administrations.php @@ -1,4 +1,5 @@ 'encarts', 'objet' => $objet, 'id_objet' => $id_objet, - )); + ]); } if ($texte) { - if ($p = strpos($flux['data'], "")) { + if ($p = strpos($flux['data'], '')) { $flux['data'] = substr_replace($flux['data'], $texte, $p, 0); } else { $flux['data'] .= $texte; @@ -92,7 +93,7 @@ function encarts_affiche_milieu($flux) { */ function encarts_optimiser_base_disparus($flux) { include_spip('action/editer_liens'); - $flux['data'] += objet_optimiser_liens(array('encart' => '*'), '*'); + $flux['data'] += objet_optimiser_liens(['encart' => '*'], '*'); return $flux; } @@ -111,9 +112,9 @@ function encarts_pre_propre($texte) { foreach ($regs as $reg) { $css = 'encart'; if ($reg[1] != 'encart') { - $css .= " " . $reg[1]; + $css .= ' ' . $reg[1]; } - $texte = str_replace($reg[0], "" . $reg[2] . "", $texte); + $texte = str_replace($reg[0], "" . $reg[2] . '', $texte); } } } @@ -130,10 +131,11 @@ function encarts_post_edition($flux) { include_spip('inc/utils'); include_spip('base/objets'); - if (!empty($flux['args']['action']) - and $flux['args']['action'] === 'modifier' - and !empty($flux['args']['table']) - and !in_array($flux['args']['table'], ['spip_forum', 'spip_signatures']) + if ( + !empty($flux['args']['action']) + and $flux['args']['action'] === 'modifier' + and !empty($flux['args']['table']) + and !in_array($flux['args']['table'], ['spip_forum', 'spip_signatures']) ) { $serveur = $flux['args']['serveur'] ?? ''; $table_sql = $flux['args']['table']; diff --git a/formulaires/editer_encart.php b/formulaires/editer_encart.php index ad7d263..f1c2161 100644 --- a/formulaires/editer_encart.php +++ b/formulaires/editer_encart.php @@ -1,4 +1,5 @@ 'input', - 'options' => array( + 'options' => [ 'nom' => 'titre', 'label' => _T('encart:label_titre'), 'defaut' => '', 'class' => 'multilang', - ) - ), - array( + ] + ], + [ 'saisie' => 'textarea', - 'options' => array( + 'options' => [ 'nom' => 'texte', 'label' => _T('encart:label_texte'), 'defaut' => '', 'rows' => 5, 'class' => 'multilang', - ) - ) - ); + ] + ] + ]; return $saisies; } @@ -70,8 +71,8 @@ function formulaires_editer_encart_saisies_dist($id_encart = 'new', $retour = '' * @return string * Hash du formulaire */ -function formulaires_editer_encart_identifier_dist($id_encart = 'new', $retour = '', $associer_objet = '', $lier_trad = 0, $config_fonc = '', $row = array(), $hidden = '') { - return serialize(array(intval($id_encart), $associer_objet)); +function formulaires_editer_encart_identifier_dist($id_encart = 'new', $retour = '', $associer_objet = '', $lier_trad = 0, $config_fonc = '', $row = [], $hidden = '') { + return serialize([intval($id_encart), $associer_objet]); } /** @@ -99,7 +100,7 @@ function formulaires_editer_encart_identifier_dist($id_encart = 'new', $retour = * @return array * Environnement du formulaire */ -function formulaires_editer_encart_charger_dist($id_encart = 'new', $retour = '', $associer_objet = '', $lier_trad = 0, $config_fonc = '', $row = array(), $hidden = '') { +function formulaires_editer_encart_charger_dist($id_encart = 'new', $retour = '', $associer_objet = '', $lier_trad = 0, $config_fonc = '', $row = [], $hidden = '') { $valeurs = formulaires_editer_objet_charger('encart', $id_encart, '', $lier_trad, $retour, $config_fonc, $row, $hidden); return $valeurs; @@ -130,7 +131,7 @@ function formulaires_editer_encart_charger_dist($id_encart = 'new', $retour = '' * @return array * Tableau des erreurs */ -function formulaires_editer_encart_verifier_dist($id_encart = 'new', $retour = '', $associer_objet = '', $lier_trad = 0, $config_fonc = '', $row = array(), $hidden = '') { +function formulaires_editer_encart_verifier_dist($id_encart = 'new', $retour = '', $associer_objet = '', $lier_trad = 0, $config_fonc = '', $row = [], $hidden = '') { $erreurs = formulaires_editer_objet_verifier('encart', $id_encart); return $erreurs; @@ -161,23 +162,21 @@ function formulaires_editer_encart_verifier_dist($id_encart = 'new', $retour = ' * @return array * Retours des traitements */ -function formulaires_editer_encart_traiter_dist($id_encart = 'new', $retour = '', $associer_objet = '', $lier_trad = 0, $config_fonc = '', $row = array(), $hidden = '') { +function formulaires_editer_encart_traiter_dist($id_encart = 'new', $retour = '', $associer_objet = '', $lier_trad = 0, $config_fonc = '', $row = [], $hidden = '') { $res = formulaires_editer_objet_traiter('encart', $id_encart, '', $lier_trad, $retour, $config_fonc, $row, $hidden); // Un lien a prendre en compte ? - if ($associer_objet AND $id_encart = $res['id_encart']) { + if ($associer_objet and $id_encart = $res['id_encart']) { list($objet, $id_objet) = explode('|', $associer_objet); - if ($objet AND $id_objet AND autoriser('modifier', $objet, $id_objet)) { + if ($objet and $id_objet and autoriser('modifier', $objet, $id_objet)) { include_spip('action/editer_liens'); - objet_associer(array('encart' => $id_encart), array($objet => $id_objet)); + objet_associer(['encart' => $id_encart], [$objet => $id_objet]); if (isset($res['redirect'])) { - $res['redirect'] = parametre_url($res['redirect'], "id_lien_ajoute", $id_encart, '&'); + $res['redirect'] = parametre_url($res['redirect'], 'id_lien_ajoute', $id_encart, '&'); } } } return $res; - } - diff --git a/inc/marquer_doublons_encart.php b/inc/marquer_doublons_encart.php index 5f8c5d6..8e4bbb7 100644 --- a/inc/marquer_doublons_encart.php +++ b/inc/marquer_doublons_encart.php @@ -1,4 +1,5 @@ array('encart'))); // detecter les doublons - sql_updateq("spip_encarts_liens", array("vu" => 'non'), "id_objet=".intval($id_objet)." AND objet=" . sql_quote($objet)); + $GLOBALS['doublons_encarts_inclus'] = []; + traiter_modeles($champs_a_traiter, ['encarts' => ['encart']]); // detecter les doublons + sql_updateq('spip_encarts_liens', ['vu' => 'non'], 'id_objet=' . intval($id_objet) . ' AND objet=' . sql_quote($objet)); if (count($GLOBALS['doublons_encarts_inclus'])) { // on repasse par une requete sur spip_encarts pour verifier que les encarts existent bien ! - $in_liste = sql_in('id_encart', - $GLOBALS['doublons_encarts_inclus']); - $res = sql_select("id_encart", "spip_encarts", $in_liste); + $in_liste = sql_in( + 'id_encart', + $GLOBALS['doublons_encarts_inclus'] + ); + $res = sql_select('id_encart', 'spip_encarts', $in_liste); while ($row = sql_fetch($res)) { // Creer le lien s'il n'existe pas deja - sql_insertq("spip_encarts_liens", array( + sql_insertq('spip_encarts_liens', [ 'id_objet' => $id_objet, 'objet' => $objet, 'id_encart' => $row['id_encart'], 'vu' => 'oui' - )); - sql_updateq("spip_encarts_liens", array("vu" => 'oui'), "id_objet=".intval($id_objet)." AND objet=" . sql_quote($objet) . " AND id_encart=" . $row['id_encart']); + ]); + sql_updateq('spip_encarts_liens', ['vu' => 'oui'], 'id_objet=' . intval($id_objet) . ' AND objet=' . sql_quote($objet) . ' AND id_encart=' . $row['id_encart']); } } } - diff --git a/phpcs.xml.dist b/phpcs.xml.dist new file mode 100644 index 0000000..344df64 --- /dev/null +++ b/phpcs.xml.dist @@ -0,0 +1,16 @@ + + + . + vendor/* + lang/* + tests/* + + + + + + + + + + diff --git a/prive/objets/liste/articles_encart_fonctions.php b/prive/objets/liste/articles_encart_fonctions.php index bf0a183..2ed6e11 100644 --- a/prive/objets/liste/articles_encart_fonctions.php +++ b/prive/objets/liste/articles_encart_fonctions.php @@ -1,4 +1,5 @@ diff --git a/prive/objets/liste/encarts_lies_fonctions.php b/prive/objets/liste/encarts_lies_fonctions.php index 50e81f3..f2c3546 100644 --- a/prive/objets/liste/encarts_lies_fonctions.php +++ b/prive/objets/liste/encarts_lies_fonctions.php @@ -1,4 +1,5 @@