diff --git a/action/editer_breve.php b/action/editer_breve.php index 1c5d186..cb12d17 100644 --- a/action/editer_breve.php +++ b/action/editer_breve.php @@ -45,12 +45,12 @@ function action_editer_breve_dist($arg = null) { } if (!$id_breve) { - return array(0, ''); + return [0, '']; } // erreur $err = breve_modifier($id_breve); - return array($id_breve, $err); + return [$id_breve, $err]; } @@ -77,17 +77,17 @@ function breve_inserer($id_rubrique, $set = null) { } // La langue a la creation : c'est la langue de la rubrique - $row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique='.intval($id_rubrique)); + $row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique)); $lang = $row['lang']; $id_rubrique = $row['id_secteur']; // garantir la racine - $champs = array( + $champs = [ 'id_rubrique' => $id_rubrique, 'statut' => 'prop', 'date_heure' => date('Y-m-d H:i:s'), 'lang' => $lang, 'langue_choisie' => 'non' - ); + ]; if ($set) { $champs = array_merge($champs, $set); @@ -96,23 +96,23 @@ function breve_inserer($id_rubrique, $set = null) { // Envoyer aux plugins $champs = pipeline( 'pre_insertion', - array( - 'args' => array( + [ + 'args' => [ 'table' => 'spip_breves', - ), + ], 'data' => $champs - ) + ] ); $id_breve = sql_insertq('spip_breves', $champs); pipeline( 'post_insertion', - array( - 'args' => array( + [ + 'args' => [ 'table' => 'spip_breves', 'id_objet' => $id_breve - ), + ], 'data' => $champs - ) + ] ); return $id_breve; @@ -141,7 +141,7 @@ function breve_modifier($id_breve, $set = null) { // white list objet_info('breve', 'champs_editables'), // black list - array('id_parent', 'statut'), + ['id_parent', 'statut'], // donnees eventuellement fournies $set ); @@ -150,27 +150,29 @@ function breve_modifier($id_breve, $set = null) { $indexation = false; // Si la breve est publiee, invalider les caches et demander sa reindexation - $t = sql_getfetsel('statut', 'spip_breves', 'id_breve='.intval($id_breve)); + $t = sql_getfetsel('statut', 'spip_breves', 'id_breve=' . intval($id_breve)); if ($t == 'publie') { $invalideur = "id='breve/$id_breve'"; $indexation = true; } - if ($err = objet_modifier_champs( - 'breve', - $id_breve, - array( + if ( + $err = objet_modifier_champs( + 'breve', + $id_breve, + [ 'data' => $set, - 'nonvide' => array('titre' => _T('breves:titre_nouvelle_breve') . ' ' . _T('info_numero_abbreviation') . $id_breve), + 'nonvide' => ['titre' => _T('breves:titre_nouvelle_breve') . ' ' . _T('info_numero_abbreviation') . $id_breve], 'invalideur' => $invalideur, 'indexation' => $indexation - ), - $c - )) { + ], + $c + ) + ) { return $err; } - $c = collecter_requests(array('id_parent', 'statut'), array(), $set); + $c = collecter_requests(['id_parent', 'statut'], [], $set); $err = breve_instituer($id_breve, $c); return $err; @@ -191,7 +193,7 @@ function breve_modifier($id_breve, $set = null) { * Null si aucun champ à modifier, chaîne vide sinon. */ function breve_instituer($id_breve, $c) { - $champs = array(); + $champs = []; // Changer le statut de la breve ? $row = sql_fetsel('statut, id_rubrique,lang, langue_choisie', 'spip_breves', 'id_breve=' . intval($id_breve)); @@ -201,7 +203,8 @@ function breve_instituer($id_breve, $c) { $langue_old = $row['lang']; $langue_choisie_old = $row['langue_choisie']; - if (isset($c['statut']) + if ( + isset($c['statut']) and $c['statut'] and $c['statut'] != $statut and autoriser('publierdans', 'rubrique', $id_rubrique) @@ -242,15 +245,15 @@ function breve_instituer($id_breve, $c) { // Envoyer aux plugins $champs = pipeline( 'pre_edition', - array( - 'args' => array( + [ + 'args' => [ 'table' => 'spip_breves', 'id_objet' => $id_breve, 'action' => 'instituer', 'statut_ancien' => $statut_ancien, - ), + ], 'data' => $champs - ) + ] ); if (!$champs) { @@ -274,22 +277,24 @@ function breve_instituer($id_breve, $c) { // Pipeline pipeline( 'post_edition', - array( - 'args' => array( + [ + 'args' => [ 'table' => 'spip_breves', 'id_objet' => $id_breve, 'action' => 'instituer', 'statut_ancien' => $statut_ancien, - ), + ], 'data' => $champs - ) + ] ); // Notifications if ($notifications = charger_fonction('notifications', 'inc')) { - $notifications('instituerbreve', $id_breve, - array('statut' => $statut, 'statut_ancien' => $statut_ancien) + $notifications( + 'instituerbreve', + $id_breve, + ['statut' => $statut, 'statut_ancien' => $statut_ancien] ); } diff --git a/base/breves.php b/base/breves.php index b5ac88f..3c88cf0 100644 --- a/base/breves.php +++ b/base/breves.php @@ -53,7 +53,7 @@ function breves_declarer_tables_interfaces($interfaces) { * Description complétée des tables */ function breves_declarer_tables_objets_sql($tables) { - $tables['spip_breves'] = array( + $tables['spip_breves'] = [ 'texte_retour' => 'icone_retour', 'texte_objets' => 'breves:breves', 'texte_objet' => 'breves:breve', @@ -68,7 +68,7 @@ function breves_declarer_tables_objets_sql($tables) { 'date' => 'date_heure', 'principale' => 'oui', 'introduction_longueur' => '300', - 'field' => array( + 'field' => [ 'id_breve' => 'bigint(21) NOT NULL', 'date_heure' => "datetime DEFAULT '0000-00-00 00:00:00' NOT NULL", 'titre' => "text DEFAULT '' NOT NULL", @@ -80,61 +80,61 @@ function breves_declarer_tables_objets_sql($tables) { 'lang' => "VARCHAR(10) DEFAULT '' NOT NULL", 'langue_choisie' => "VARCHAR(3) DEFAULT 'non'", 'maj' => 'TIMESTAMP' - ), - 'key' => array( + ], + 'key' => [ 'PRIMARY KEY' => 'id_breve', 'KEY id_rubrique' => 'id_rubrique', - ), - 'parent' => array('type' => 'rubrique', 'champ' => 'id_rubrique'), - 'join' => array( + ], + 'parent' => ['type' => 'rubrique', 'champ' => 'id_rubrique'], + 'join' => [ 'id_breve' => 'id_breve', 'id_rubrique' => 'id_rubrique' - ), - 'statut' => array( - array( + ], + 'statut' => [ + [ 'champ' => 'statut', 'publie' => 'publie', 'previsu' => 'publie,prop', 'exception' => 'statut' - ) - ), + ] + ], 'texte_changer_statut' => 'breves:entree_breve_publiee', 'aide_changer_statut' => 'brevesstatut', - 'statut_titres' => array( + 'statut_titres' => [ 'prop' => 'breves:titre_breve_proposee', 'publie' => 'breves:titre_breve_publiee', 'refuse' => 'breves:titre_breve_refusee', - ), - 'statut_textes_instituer' => array( + ], + 'statut_textes_instituer' => [ 'prop' => 'breves:item_breve_proposee', //_T('texte_statut_propose_evaluation') 'publie' => 'breves:item_breve_validee', //_T('texte_statut_publie') 'refuse' => 'breves:item_breve_refusee', //_T('texte_statut_refuse') - ), + ], - 'rechercher_champs' => array( + 'rechercher_champs' => [ 'titre' => 8, 'texte' => 2, 'lien_titre' => 1, 'lien_url' => 1 - ), - 'rechercher_jointures' => array( - 'document' => array('titre' => 2, 'descriptif' => 1) - ), - 'champs_editables' => array( + ], + 'rechercher_jointures' => [ + 'document' => ['titre' => 2, 'descriptif' => 1] + ], + 'champs_editables' => [ 'id_rubrique', 'titre', 'lien_titre', 'lien_url', 'texte', - ), - 'champs_versionnes' => array( + ], + 'champs_versionnes' => [ 'id_rubrique', 'titre', 'lien_titre', 'lien_url', 'texte', - ), - ); + ], + ]; return $tables; } diff --git a/breves_administrations.php b/breves_administrations.php index 185d75a..e4492ad 100644 --- a/breves_administrations.php +++ b/breves_administrations.php @@ -36,7 +36,8 @@ function breves_upgrade($nom_meta_base_version, $version_cible) { // pour gerer l'historique des installations SPIP <=2.1 if (!isset($GLOBALS['meta'][$nom_meta_base_version])) { $trouver_table = charger_fonction('trouver_table', 'base'); - if ($desc = $trouver_table('spip_breves') + if ( + $desc = $trouver_table('spip_breves') and isset($desc['exist']) and $desc['exist'] ) { ecrire_meta($nom_meta_base_version, '1.0.0'); @@ -44,10 +45,10 @@ function breves_upgrade($nom_meta_base_version, $version_cible) { // si pas de table en base, on fera une simple creation de base } - $maj = array(); - $maj['create'] = array( - array('maj_tables', array('spip_breves')), - ); + $maj = []; + $maj['create'] = [ + ['maj_tables', ['spip_breves']], + ]; include_spip('base/upgrade'); maj_plugin($nom_meta_base_version, $version_cible, $maj); diff --git a/breves_autoriser.php b/breves_autoriser.php index 9d87a00..cc86deb 100644 --- a/breves_autoriser.php +++ b/breves_autoriser.php @@ -131,6 +131,6 @@ function autoriser_breve_modifier_dist($faire, $type, $id, $qui, $opt) { $r and ( ($r['statut'] == 'publie' or (isset($opt['statut']) and $opt['statut'] == 'publie')) ? autoriser('publierdans', 'rubrique', $r['id_rubrique'], $qui, $opt) - : in_array($qui['statut'], array('0minirezo', '1comite')) + : in_array($qui['statut'], ['0minirezo', '1comite']) ); } diff --git a/breves_pipelines.php b/breves_pipelines.php index 8505627..9bfbfd8 100644 --- a/breves_pipelines.php +++ b/breves_pipelines.php @@ -52,12 +52,12 @@ function breves_rubrique_encours($flux) { // // Les breves a valider // - $flux['data'] .= $lister_objets('breves', array( + $flux['data'] .= $lister_objets('breves', [ 'titre' => _T('breves:info_breves_valider'), - 'statut' => array('prepa', 'prop'), + 'statut' => ['prepa', 'prop'], 'id_rubrique' => $id_rubrique, 'par' => 'date_heure' - )); + ]); } return $flux; @@ -73,7 +73,8 @@ function breves_rubrique_encours($flux) { * @return array Données du pipeline **/ function breves_affiche_enfants($flux) { - if (isset($flux['args']['exec']) + if ( + isset($flux['args']['exec']) and $e = trouver_objet_exec($flux['args']['exec']) and $e['type'] == 'rubrique' and $e['edition'] == false @@ -84,7 +85,7 @@ function breves_affiche_enfants($flux) { $lister_objets = charger_fonction('lister_objets', 'inc'); $bouton_breves = ''; $id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . $id_rubrique); - if (autoriser('creerbrevedans', 'rubrique', $id_rubrique, null, array('id_parent' => $id_parent))) { + if (autoriser('creerbrevedans', 'rubrique', $id_rubrique, null, ['id_parent' => $id_parent])) { $bouton_breves .= icone_verticale( _T('breves:icone_nouvelle_breve'), generer_url_ecrire('breve_edit', "id_rubrique=$id_rubrique&new=oui"), @@ -95,12 +96,12 @@ function breves_affiche_enfants($flux) { . "
"; } - $flux['data'] .= $lister_objets('breves', array( + $flux['data'] .= $lister_objets('breves', [ 'titre' => _T('breves:icone_ecrire_nouvel_article'), 'where' => "statut != 'prop' AND statut != 'prepa'", 'id_rubrique' => $id_rubrique, 'par' => 'date_heure' - )); + ]); $flux['data'] .= $bouton_breves; } } @@ -126,8 +127,8 @@ function breves_accueil_informations($texte) { $q = sql_select('COUNT(*) AS cnt, statut', 'spip_breves', '', 'statut', '', '', 'COUNT(*)<>0'); - $cpt = array(); - $cpt2 = array(); + $cpt = []; + $cpt2 = []; $where = false; if ($GLOBALS['visiteur_session']['statut'] == '0minirezo') { $where = sql_allfetsel( @@ -179,7 +180,8 @@ function breves_accueil_informations($texte) { * @return array Données du pipeline */ function breves_objet_compte_enfants($flux) { - if ($flux['args']['objet'] == 'rubrique' + if ( + $flux['args']['objet'] == 'rubrique' and $id_rubrique = intval($flux['args']['id_objet']) ) { // juste les publies ? @@ -218,8 +220,8 @@ function breves_trig_calculer_langues_rubriques($flux) { $id_breve = $row['id_breve']; sql_updateq( 'spip_breves', - array('lang' => $row['lang'], 'langue_choisie' => 'non'), - 'id_breve='.intval($id_breve) + ['lang' => $row['lang'], 'langue_choisie' => 'non'], + 'id_breve=' . intval($id_breve) ); } @@ -245,7 +247,7 @@ function breves_calculer_rubriques($flux) { while ($row = sql_fetch($r)) { sql_updateq( 'spip_rubriques', - array('statut_tmp' => 'publie', 'date_tmp' => $row['date_h']), + ['statut_tmp' => 'publie', 'date_tmp' => $row['date_h']], 'id_rubrique=' . intval($row['id']) ); } @@ -265,11 +267,11 @@ function breves_calculer_rubriques($flux) { function breves_accueil_encours($flux) { $lister_objets = charger_fonction('lister_objets', 'inc'); - $flux .= $lister_objets('breves', array( + $flux .= $lister_objets('breves', [ 'titre' => afficher_plus_info(generer_url_ecrire('breves')) . _T('breves:info_breves_valider'), - 'statut' => array('prepa', 'prop'), + 'statut' => ['prepa', 'prop'], 'par' => 'date_heure' - )); + ]); return $flux; } @@ -342,7 +344,7 @@ function breves_boite_infos($flux) { */ function breves_affiche_milieu($flux) { if ($flux['args']['exec'] == 'configurer_contenu') { - $flux['data'] .= recuperer_fond('prive/squelettes/inclure/configurer', array('configurer' => 'configurer_breves')); + $flux['data'] .= recuperer_fond('prive/squelettes/inclure/configurer', ['configurer' => 'configurer_breves']); } return $flux; diff --git a/formulaires/configurer_breves.php b/formulaires/configurer_breves.php index a61776d..3d5e0e8 100644 --- a/formulaires/configurer_breves.php +++ b/formulaires/configurer_breves.php @@ -26,10 +26,12 @@ if (!defined('_ECRIRE_INC_VERSION')) { * Environnement du formulaire **/ function formulaires_configurer_breves_charger_dist() { - $valeurs = array(); - foreach (array( + $valeurs = []; + foreach ( + [ 'activer_breves', - ) as $m) { + ] as $m + ) { $valeurs[$m] = $GLOBALS['meta'][$m]; } @@ -43,10 +45,12 @@ function formulaires_configurer_breves_charger_dist() { * Retours du traitement **/ function formulaires_configurer_breves_traiter_dist() { - $res = array('editable' => true); - foreach (array( + $res = ['editable' => true]; + foreach ( + [ 'activer_breves', - ) as $m) { + ] as $m + ) { if (!is_null($v = _request($m))) { ecrire_meta($m, $v == 'oui' ? 'oui' : 'non'); } diff --git a/formulaires/editer_breve.php b/formulaires/editer_breve.php index 4ea59e5..7ce2996 100644 --- a/formulaires/editer_breve.php +++ b/formulaires/editer_breve.php @@ -50,7 +50,7 @@ function formulaires_editer_breve_charger_dist( $retour = '', $lier_trad = 0, $config_fonc = 'breves_edit_config', - $row = array(), + $row = [], $hidden = '' ) { $valeurs = formulaires_editer_objet_charger( @@ -109,10 +109,10 @@ function formulaires_editer_breve_identifier_dist( $retour = '', $lier_trad = 0, $config_fonc = 'breves_edit_config', - $row = array(), + $row = [], $hidden = '' ) { - return serialize(array(intval($id_breve), $lier_trad)); + return serialize([intval($id_breve), $lier_trad]); } @@ -164,13 +164,13 @@ function formulaires_editer_breve_verifier_dist( $retour = '', $lier_trad = 0, $config_fonc = 'breves_edit_config', - $row = array(), + $row = [], $hidden = '' ) { // auto-renseigner le titre si il n'existe pas - titre_automatique('titre', array('texte')); + titre_automatique('titre', ['texte']); // on ne demande pas le titre obligatoire : il sera rempli a la volee dans editer_article si vide - $erreurs = formulaires_editer_objet_verifier('breve', $id_breve, array('id_parent')); + $erreurs = formulaires_editer_objet_verifier('breve', $id_breve, ['id_parent']); return $erreurs; } @@ -203,7 +203,7 @@ function formulaires_editer_breve_traiter_dist( $retour = '', $lier_trad = 0, $config_fonc = 'breves_edit_config', - $row = array(), + $row = [], $hidden = '' ) { return formulaires_editer_objet_traiter( diff --git a/tests/unit/abstract_sql/99_sql_types_tables_id.php b/tests/unit/abstract_sql/99_sql_types_tables_id.php index c33599c..a39a171 100644 --- a/tests/unit/abstract_sql/99_sql_types_tables_id.php +++ b/tests/unit/abstract_sql/99_sql_types_tables_id.php @@ -3,41 +3,41 @@ // nom du test $test = 'sql/sql_types_tables_id'; - $remonte = "../"; - while (!is_dir($remonte."ecrire")) + $remonte = '../'; + while (!is_dir($remonte . 'ecrire')) { $remonte = "../$remonte"; - require $remonte.'tests/test.inc'; + } + require $remonte . 'tests/test.inc'; include_spip('base/connect_sql'); // Des tests -$essais['table_objet'] = array( -array('breves','breve'), -); +$essais['table_objet'] = [ +['breves','breve'], +]; -$essais['table_objet_sql'] = array( -array('spip_breves','breve'), -); +$essais['table_objet_sql'] = [ +['spip_breves','breve'], +]; -$essais['id_table_objet'] = array( -array('id_breve','breve'), -); +$essais['id_table_objet'] = [ +['id_breve','breve'], +]; -$essais['objet_type'] = array( -array('breve','breves'), -); +$essais['objet_type'] = [ +['breve','breves'], +]; // hop ! on y va - $err = array(); - foreach($essais as $f=>$essai) - $err = array_merge(tester_fun($f, $essai),$err); - + $err = []; + foreach ($essais as $f => $essai) { + $err = array_merge(tester_fun($f, $essai), $err); + } + // si le tableau $err est pas vide ca va pas if ($err) { echo ('
' . join('', $err) . '
'); } else { - echo "OK"; + echo 'OK'; } - - diff --git a/tests/unit/connect_sql/id_table_objet.php b/tests/unit/connect_sql/id_table_objet.php index a4a1d29..8221310 100644 --- a/tests/unit/connect_sql/id_table_objet.php +++ b/tests/unit/connect_sql/id_table_objet.php @@ -1,48 +1,49 @@ ' . join('', $err) . ''); + die('
' . join('', $err) . '
'); } - echo "OK"; - + echo 'OK'; - function essais_id_table_objet(){ - $essais = array ( - array ( - 0 => 'id_breve', - 1 => 'breves', - ), - array ( - 0 => 'id_breve', - 1 => 'spip_breves', - ), - array ( - 0 => 'id_breve', - 1 => 'id_breve', - ), -array('id_breve','breve'), -); + + function essais_id_table_objet() { + $essais = [ + [ + 0 => 'id_breve', + 1 => 'breves', + ], + [ + 0 => 'id_breve', + 1 => 'spip_breves', + ], + [ + 0 => 'id_breve', + 1 => 'id_breve', + ], +['id_breve','breve'], +]; return $essais; } - diff --git a/tests/unit/connect_sql/objet_type.php b/tests/unit/connect_sql/objet_type.php index 532656a..3580c35 100644 --- a/tests/unit/connect_sql/objet_type.php +++ b/tests/unit/connect_sql/objet_type.php @@ -1,48 +1,49 @@ ' . join('', $err) . ''); + die('
' . join('', $err) . '
'); } - echo "OK"; - + echo 'OK'; - function essais_objet_type(){ - $essais = array ( - array ( - 0 => 'breve', - 1 => 'breves', - ), - array ( - 0 => 'breve', - 1 => 'spip_breves', - ), - array ( - 0 => 'breve', - 1 => 'id_breve', - ), -array('breve','breves'), -); + + function essais_objet_type() { + $essais = [ + [ + 0 => 'breve', + 1 => 'breves', + ], + [ + 0 => 'breve', + 1 => 'spip_breves', + ], + [ + 0 => 'breve', + 1 => 'id_breve', + ], +['breve','breves'], +]; return $essais; } - diff --git a/tests/unit/connect_sql/table_objet.php b/tests/unit/connect_sql/table_objet.php index 3af9db1..cd12a66 100644 --- a/tests/unit/connect_sql/table_objet.php +++ b/tests/unit/connect_sql/table_objet.php @@ -1,47 +1,49 @@ ' . join('', $err) . ''); + die('
' . join('', $err) . '
'); } - echo "OK"; - + echo 'OK'; + - function essais_table_objet(){ - $essais = array ( - array ( - 0 => 'breves', - 1 => 'breve', - ), - array ( - 0 => 'breves', - 1 => 'spip_breve', - ), - array ( - 0 => 'breves', - 1 => 'id_breve', - ), -array('breves','breve'), -); + function essais_table_objet() { + $essais = [ + [ + 0 => 'breves', + 1 => 'breve', + ], + [ + 0 => 'breves', + 1 => 'spip_breve', + ], + [ + 0 => 'breves', + 1 => 'id_breve', + ], +['breves','breve'], +]; return $essais; } diff --git a/tests/unit/connect_sql/table_objet_sql.php b/tests/unit/connect_sql/table_objet_sql.php index fd5e61e..5be5a09 100644 --- a/tests/unit/connect_sql/table_objet_sql.php +++ b/tests/unit/connect_sql/table_objet_sql.php @@ -1,47 +1,49 @@ ' . join('', $err) . ''); + die('
' . join('', $err) . '
'); } - echo "OK"; - + echo 'OK'; + - function essais_table_objet_sql(){ - $essais = array ( - array ( - 0 => 'spip_breves', - 1 => 'breve', - ), - array ( - 0 => 'spip_breves', - 1 => 'spip_breve', - ), - array ( - 0 => 'spip_breves', - 1 => 'id_breve', - ), -array('spip_breves','breve'), -); + function essais_table_objet_sql() { + $essais = [ + [ + 0 => 'spip_breves', + 1 => 'breve', + ], + [ + 0 => 'spip_breves', + 1 => 'spip_breve', + ], + [ + 0 => 'spip_breves', + 1 => 'id_breve', + ], +['spip_breves','breve'], +]; return $essais; } diff --git a/tests/unit/propre/liens_classes.php b/tests/unit/propre/liens_classes.php index b715cfe..21b0263 100644 --- a/tests/unit/propre/liens_classes.php +++ b/tests/unit/propre/liens_classes.php @@ -1,27 +1,30 @@ "; + $id = sql_getfetsel('id_breve', 'spip_breves', "statut='publie'", '', '', '0,1'); + if (!$id) { echo 'NA Necessite une breve publiee
'; + } $p0 = "[->breve$id]"; - if (!$c = extraire_attribut(propre($p0), 'class') - OR strpos($c,'spip_in')===FALSE - OR strpos($c,'spip_out')!==FALSE) - $err[] = "Classe $c errone dans $p0 : ".PtoBR(propre($p0)); + if ( + !$c = extraire_attribut(propre($p0), 'class') + or strpos($c, 'spip_in') === false + or strpos($c, 'spip_out') !== false + ) { + $err[] = "Classe $c errone dans $p0 : " . PtoBR(propre($p0)); + } // si le tableau $err est pas vide ca va pas if (count($err)) { echo ('
' . join('
', $err) . '
'); } else { - echo "OK"; + echo 'OK'; } - -