Browse Source

Meilleure compatibilité avec PSR-2 et nos règles d'écriture, en appliquant

différents fix avec php-cs-fixers. Fixers appliqués ici :

	'encoding',                // utf8
	'eof_ending',              // un saut de ligne en fin de fichier
	'elseif',                  // elseif plutôt que else if
	'function_call_space',     // espaces sur fonctions
	'function_declaration',    // espaces sur fonctions
	'function_typehint_space', // espaces sur fonctions
	'linefeed',                // sauts de ligne \n uniquement
	'lowercase_constants',     // true, false, null en munuscule
	'lowercase_keywords',      // mots clés PHP en lowercase
	'method_argument_space',   // espaces sur appels de fonctions
	'multiple_use',            // use unique sur fonctions anonymes
	'newline_after_open_tag',  // ouverture de php… et c'est tout sur cette ligne
	'operators_spaces',        // espaces de part et d'autres des opérateurs binaires
	'parenthesis',             // pas d'espace juste après parenthèse ouvrante, ou avant parenthèse fermante
	'php_closing_tag',         // pas de fermeture de php
	'short_tag',               // tag PHP corrects
	'trailing_spaces',         // pas d'espace qui traîne en fin de ligne
	'visibility',              // déclarer 'public / private / protected' sur les méthodes
spip-3.1
marcimat@rezo.net 6 years ago
parent
commit
0048ccb6c4
  1. 4
      action/actionner.php
  2. 2
      action/actualiser_depot.php
  3. 2
      action/editer_depot.php
  4. 2
      action/supprimer_depot.php
  5. 2
      base/svp_declarer.php
  6. 14
      exec/admin_plugin.php
  7. 9
      formulaires/admin_plugin.php
  8. 2
      formulaires/ajouter_depot.php
  9. 5
      formulaires/charger_plugin.php
  10. 3
      formulaires/charger_plugin_archive.php
  11. 2
      formulaires/editer_depot.php
  12. 2
      formulaires/inc-admin_plugin_fonctions.php
  13. 2
      genie/svp_actualiser_depots.php
  14. 4
      genie/svp_taches_generales_cron.php
  15. 6977
      inc/pcltar.php
  16. 118
      inc/svp_actionner.php
  17. 100
      inc/svp_decider.php
  18. 15
      inc/svp_depoter_distant.php
  19. 8
      inc/svp_depoter_local.php
  20. 22
      inc/svp_outiller.php
  21. 12
      inc/svp_phraser.php
  22. 18
      inc/svp_rechercher.php
  23. 2
      inc/where_compatible_spip.php
  24. 2
      svp_administrations.php
  25. 20
      svp_fonctions.php
  26. 64
      svp_ieconfig.php
  27. 6
      svp_pipelines.php
  28. 2
      teleporter/git.php
  29. 16
      teleporter/http.php
  30. 2
      teleporter/http_deballe_tgz.php
  31. 5
      teleporter/http_deballe_zip.php
  32. 2
      teleporter/svn.php

4
action/actionner.php

@ -67,7 +67,7 @@ function action_actionner_dist() {
foreach ($actionneur->done as $done) {
if ($done['todo'] == 'on') {
if ($voir = session_get('svp_admin_plugin_voir')
AND $voir == 'inactif'
and $voir == 'inactif'
) {
session_set('svp_admin_plugin_voir', 'actif');
}
@ -85,5 +85,3 @@ function action_actionner_dist() {
}
}
?>

2
action/actualiser_depot.php

@ -46,5 +46,3 @@ function action_actualiser_depot_dist() {
}
}
}
?>

2
action/editer_depot.php

@ -99,5 +99,3 @@ function depots_set($id_depot) {
return $err;
}
?>

2
action/supprimer_depot.php

@ -33,5 +33,3 @@ function action_supprimer_depot_dist() {
spip_log("ACTION SUPPRIMER DEPOT (manuel) : id_depot = " . $id_depot, 'svp_actions.' . _LOG_INFO);
}
}
?>

2
base/svp_declarer.php

@ -290,5 +290,3 @@ function svp_declarer_tables_interfaces($interface) {
return $interface;
}
?>

14
exec/admin_plugin.php

@ -52,8 +52,8 @@ function exec_admin_plugin_dist() {
// pour etre sur que les bons fichiers seront charges lors de l'install
include_spip('inc/plugin');
$new = actualise_plugins_actifs();
if ($new AND _request('actualise') < 2) {
$url = parametre_url(self(), 'actualise', _request('actualise')+1, '&');
if ($new and _request('actualise') < 2) {
$url = parametre_url(self(), 'actualise', _request('actualise') + 1, '&');
include_spip('inc/headers');
echo redirige_formulaire($url);
exit;
@ -134,8 +134,8 @@ function exec_admin_plugin_dist() {
session_set('svp_admin_plugin_voir', $args['voir']);
}
if (!isset($args['voir'])
AND $voir = session_get('svp_admin_plugin_voir')
AND in_array($voir, array('tous', 'actif', 'inactif'))
and $voir = session_get('svp_admin_plugin_voir')
and in_array($voir, array('tous', 'actif', 'inactif'))
) {
$args['voir'] = $voir;
}
@ -150,8 +150,8 @@ function exec_admin_plugin_dist() {
session_set('svp_admin_plugin_verrouille', $args['verrouille']);
}
if (!isset($args['verrouille'])
AND $verrouille = session_get('svp_admin_plugin_verrouille')
AND in_array($verrouille, array('tous', 'oui', 'non'))
and $verrouille = session_get('svp_admin_plugin_verrouille')
and in_array($verrouille, array('tous', 'oui', 'non'))
) {
$args['verrouille'] = $verrouille;
}
@ -202,7 +202,7 @@ function svp_presenter_actions_realisees() {
$pres = $actionneur->presenter_actions($fin = true);
// s'il ne reste aucune action a faire
if (!$actionneur->est_verrouille() OR !count($actionneur->end)) {
if (!$actionneur->est_verrouille() or !count($actionneur->end)) {
$actionneur->nettoyer_actions();
}

9
formulaires/admin_plugin.php

@ -115,7 +115,7 @@ function formulaires_admin_plugin_verifier_dist($voir = 'actif', $verrouille = '
$a_actionner = array();
// actions globales...
if ($action_globale = _request('action_globale') AND _request('appliquer')) {
if ($action_globale = _request('action_globale') and _request('appliquer')) {
$ids_paquet = _request('ids_paquet');
if (!is_array($ids_paquet)) {
$erreurs['message_erreur'] = _T('svp:message_erreur_aucun_plugin_selectionne');
@ -151,14 +151,14 @@ function formulaires_admin_plugin_verifier_dist($voir = 'actif', $verrouille = '
$notices['decideur_warning'] = _T('svp:confirmer_desinstaller');
set_request('_notices', $notices);
} elseif (!isset($erreurs['decideur_erreurs'])
AND (!isset($erreurs['decideur_propositions']) OR !count($actions['decideur_propositions']))
and (!isset($erreurs['decideur_propositions']) or !count($actions['decideur_propositions']))
) {
set_request('valider_actions', true); // on fake la validation, non mais ho !
}
}
}
if (count($erreurs) AND !isset($erreurs['message_erreur'])) {
if (count($erreurs) and !isset($erreurs['message_erreur'])) {
$erreurs['message_erreur'] = '';
}
@ -206,6 +206,3 @@ function formulaires_admin_plugin_traiter_dist($voir = 'actif', $verrouille = 'n
return $retour;
}
?>

2
formulaires/ajouter_depot.php

@ -107,5 +107,3 @@ function svp_verifier_adresse_depot($url) {
return (copie_locale($url) ? true : false);
#return (!$xml = recuperer_page($url)) ? false : true;
}
?>

5
formulaires/charger_plugin.php

@ -114,7 +114,7 @@ function formulaires_charger_plugin_traiter_dist() {
$retour = array();
if (_request('rechercher') OR _request('annuler_actions')) {
if (_request('rechercher') or _request('annuler_actions')) {
} elseif (_request('valider_actions')) {
#refuser_traiter_formulaire_ajax();
@ -130,6 +130,3 @@ function formulaires_charger_plugin_traiter_dist() {
return $retour;
}
?>

3
formulaires/charger_plugin_archive.php

@ -96,6 +96,3 @@ function formulaires_charger_plugin_archive_traiter_dist() {
return $retour;
}
?>

2
formulaires/editer_depot.php

@ -78,5 +78,3 @@ function depots_edit_config($row) {
return $config;
}
?>

2
formulaires/inc-admin_plugin_fonctions.php

@ -27,4 +27,4 @@ include_spip("inc/plugin");
**/
function filtre_svp_nom_action($id_paquet, $action) {
return "actions[$action][$id_paquet]";
}
}

2
genie/svp_actualiser_depots.php

@ -37,5 +37,3 @@ function genie_svp_actualiser_depots_dist($last) {
return 1;
}
?>

4
genie/svp_taches_generales_cron.php

@ -48,10 +48,8 @@ function svp_taches_generales_cron($taches_generales) {
// Par defaut, toutes les 6h
// Conditionnee a la variable de configuration
if (_SVP_CRON_ACTUALISATION_DEPOTS) {
$taches_generales['svp_actualiser_depots'] = _SVP_PERIODE_ACTUALISATION_DEPOTS*3600;
$taches_generales['svp_actualiser_depots'] = _SVP_PERIODE_ACTUALISATION_DEPOTS * 3600;
}
return $taches_generales;
}
?>

6977
inc/pcltar.php
File diff suppressed because it is too large
View File

118
inc/svp_actionner.php

@ -26,7 +26,7 @@ class Actionneur {
*
* @var Decideur
*/
var $decideur;
public $decideur;
/**
* Loguer les différents éléments
@ -35,7 +35,7 @@ class Actionneur {
*
* @var bool
*/
var $log = false;
public $log = false;
/**
* Liste des actions à faire
@ -43,7 +43,7 @@ class Actionneur {
* @var array
* Tableau identifiant du paquet => type d'action
*/
var $start = array();
public $start = array();
/**
* Actions en cours d'analyse
@ -60,7 +60,7 @@ class Actionneur {
* Index 'on' : les paquets à activer (ordre des dépendances)
* Index 'neutre' : autres actions dont l'ordre a peu d'importance.
*/
var $middle = array(
public $middle = array(
'off' => array(),
'lib' => array(),
'on' => array(),
@ -76,7 +76,7 @@ class Actionneur {
*
* @var array
*/
var $end = array();
public $end = array();
/**
* Liste des actions faites
@ -84,7 +84,7 @@ class Actionneur {
*
* @var array
*/
var $done = array(); // faites
public $done = array(); // faites
/**
* Actions en cours
@ -92,14 +92,14 @@ class Actionneur {
*
* @var array
*/
var $work = array();
public $work = array();
/**
* Liste des erreurs
*
* @var array Liste des erreurs
*/
var $err = array();
public $err = array();
/**
* Verrou.
@ -108,7 +108,7 @@ class Actionneur {
* @var array
* Index 'id_auteur' : Identifiant de l'auteur ayant déclenché des actions
* Indix 'time' : timestamp de l'heure de déclenchement de l'action */
var $lock = array('id_auteur' => 0, 'time' => '');
public $lock = array('id_auteur' => 0, 'time' => '');
/**
* SVP (ce plugin) est-il à désactiver dans une des actions ?
@ -118,14 +118,14 @@ class Actionneur {
*
* @var bool
* false si SVP n'est pas à désactiver, true sinon */
var $svp_off = false;
public $svp_off = false;
/**
* Constructeur
*
* Détermine si les logs sont activés et instancie un décideur.
*/
function __construct() {
public function __construct() {
include_spip('inc/config');
$this->log = (lire_config('svp/mode_log_verbeux') == 'oui');
@ -146,7 +146,7 @@ class Actionneur {
* @param mixed $quoi
* La chose à logguer (souvent un texte)
**/
function log($quoi) {
public function log($quoi) {
if ($this->log) {
spip_log($quoi, 'actionneur');
}
@ -161,7 +161,7 @@ class Actionneur {
* @param string $erreur
* Le texte de l'erreur
**/
function err($erreur) {
public function err($erreur) {
if ($erreur) {
$this->err[] = $erreur;
}
@ -170,7 +170,7 @@ class Actionneur {
/**
* Remet à zéro les tableaux d'actions
*/
function clear() {
public function clear() {
$this->middle = array(
'off' => array(),
'lib' => array(),
@ -188,7 +188,7 @@ class Actionneur {
* @param array $todo
* Tableau des actions à faire (identifiant de paquet => type d'action)
**/
function ajouter_actions($todo) {
public function ajouter_actions($todo) {
if ($todo) {
foreach ($todo as $id => $action) {
$this->start[$id] = $action;
@ -208,7 +208,7 @@ class Actionneur {
* @param string $nom Nom de la librairie
* @param string $source URL pour obtenir la librairie
*/
function add_lib($nom, $source) {
public function add_lib($nom, $source) {
if (!$this->decideur->est_presente_lib($nom)) {
if (is_writable(_DIR_LIB)) {
$this->middle['lib'][$nom] = array(
@ -248,7 +248,7 @@ class Actionneur {
* - puis ce qui est a installer (à commencer par les librairies, puis paquets),
* - puis les actions neutres
*/
function ordonner_actions() {
public function ordonner_actions() {
// nettoyer le terrain
$this->clear();
@ -335,7 +335,7 @@ class Actionneur {
* Action à réaliser (on, upon)
* @return void
**/
function on($info, $action) {
public function on($info, $action) {
$info['todo'] = $action;
$p = $info['p'];
$this->log("ON: $p $action");
@ -349,7 +349,7 @@ class Actionneur {
$in[] = $dep['nom'];
}
// $info fourni ses procure
if (isset($info['procure']) AND $info['procure']) {
if (isset($info['procure']) and $info['procure']) {
$prov = array_keys($info['procure']);
}
// et se fournit lui meme evidemment
@ -368,7 +368,7 @@ class Actionneur {
// ie. ces plugins peuvent dependre de ce nouveau a activer.
foreach ($this->middle['on'] as $k => $inf) {
$out["$k:0"] = $inf['p'];
if (isset($inf['procure']) AND $inf['procure']) {
if (isset($inf['procure']) and $inf['procure']) {
$i = 1;
foreach ($inf['procure'] as $procure => $v) {
$out["$k:$i"] = $inf['p'];
@ -404,7 +404,7 @@ class Actionneur {
if ($key == count($this->middle['on'])) {
$this->middle['on'][] = $info;
} else {
array_splice($this->middle['on'], $key+1, 0, array($info));
array_splice($this->middle['on'], $key + 1, 0, array($info));
}
// intersection = plugin dependant de celui-ci
@ -452,7 +452,7 @@ class Actionneur {
* Action à réaliser (kill, get, up (sur plugin inactif))
* @return void
**/
function neutre($info, $action) {
public function neutre($info, $action) {
$info['todo'] = $action;
$this->log("NEUTRE: $info[p] $action");
$this->middle['neutre'][] = $info;
@ -476,7 +476,7 @@ class Actionneur {
* Action à réaliser (kill, get, up (sur plugin inactif))
* @return void
**/
function off($info, $action) {
public function off($info, $action) {
$info['todo'] = $action;
$p = $info['p'];
$this->log("OFF: $p $action");
@ -539,7 +539,7 @@ class Actionneur {
* @return string
* Bilan des actions au format HTML
**/
function presenter_actions($fin = false) {
public function presenter_actions($fin = false) {
$affiche = "";
include_spip('inc/filtres_boites');
@ -573,7 +573,7 @@ class Actionneur {
$texte .= " <span class='$ok_texte'>$i[done]</span>";
}
// si le plugin a ete active dans le meme lot, on remplace le message 'active' par le message 'installe'
if ($i['todo'] == 'install' AND $ok_texte == 'ok') {
if ($i['todo'] == 'install' and $ok_texte == 'ok') {
$cle_t = 'svp:message_action_finale_' . 'on' . '_' . $ok_texte;
$texte_on = _T($cle_t, array(
'plugin' => $i['n'],
@ -649,7 +649,7 @@ class Actionneur {
* @return bool
* true si un verrou est , false sinon
**/
function est_verrouille($id_auteur = '') {
public function est_verrouille($id_auteur = '') {
if ($id_auteur == '') {
return ($this->lock['id_auteur'] ? true : false);
}
@ -671,7 +671,7 @@ class Actionneur {
*
* @see Actionneur::sauver_actions()
**/
function verrouiller() {
public function verrouiller() {
$this->lock = array(
'id_auteur' => $GLOBALS['visiteur_session']['id_auteur'],
'time' => time(),
@ -681,7 +681,7 @@ class Actionneur {
/**
* Enlève le verrou
**/
function deverrouiller() {
public function deverrouiller() {
$this->lock = array(
'id_auteur' => 0,
'time' => '',
@ -699,7 +699,7 @@ class Actionneur {
*
* @see Actionneur::get_actions()
**/
function sauver_actions() {
public function sauver_actions() {
$contenu = serialize(array(
'todo' => $this->end,
'done' => $this->done,
@ -718,7 +718,7 @@ class Actionneur {
*
* @see Actionneur::sauver_actions()
**/
function get_actions() {
public function get_actions() {
lire_fichier(_DIR_TMP . 'stp_actions.txt', $contenu);
$infos = unserialize($contenu);
$this->end = $infos['todo'];
@ -733,7 +733,7 @@ class Actionneur {
*
* Remet tout à zéro pour pouvoir repartir d'un bon pied.
**/
function nettoyer_actions() {
public function nettoyer_actions() {
$this->todo = array();
$this->done = array();
$this->work = array();
@ -753,11 +753,11 @@ class Actionneur {
* False si aucune action à faire,
* sinon tableau de description courte du paquet + index 'todo' indiquant l'action
**/
function one_action() {
public function one_action() {
// s'il reste des actions, on en prend une, et on la fait
// de meme si une action est en cours mais pas terminee (timeout)
// on tente de la refaire...
if (count($this->end) OR $this->work) {
if (count($this->end) or $this->work) {
// on verrouille avec l'auteur en cours pour
// que seul lui puisse effectuer des actions a ce moment la
if (!$this->est_verrouille()) {
@ -809,7 +809,7 @@ class Actionneur {
* le résultat de l'action (un booléen indiquant si elle s'est bien
* déroulée).
**/
function do_action() {
public function do_action() {
if ($do = $this->work) {
$todo = 'do_' . $do['todo'];
lire_metas(); // avoir les metas a jour
@ -830,7 +830,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon.
*/
function do_geton($info) {
public function do_geton($info) {
if (!$this->tester_repertoire_plugins_auto()) {
return false;
}
@ -856,7 +856,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon.
*/
function do_on($info) {
public function do_on($info) {
$i = sql_fetsel('*', 'spip_paquets', 'id_paquet=' . sql_quote($info['i']));
// à télécharger ?
if (isset($i['id_zone']) and $i['id_zone'] > 0) {
@ -885,7 +885,7 @@ class Actionneur {
* false si erreur,
* description courte du nouveau plugin sinon.
*/
function do_up($info) {
public function do_up($info) {
// ecriture du nouveau
// suppression de l'ancien (si dans auto, et pas au meme endroit)
// OU suppression des anciens fichiers
@ -955,7 +955,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon
*/
function do_upon($info) {
public function do_upon($info) {
$i = sql_fetsel('*', 'spip_paquets', 'id_paquet=' . sql_quote($info['i']));
if ($dirs = $this->do_up($info)) {
$this->activer_plugin_dossier($dirs['dossier'], $i, $i['constante']);
@ -975,7 +975,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon
*/
function do_off($info) {
public function do_off($info) {
$i = sql_fetsel('*', 'spip_paquets', 'id_paquet=' . sql_quote($info['i']));
// il faudra prendre en compte les autres _DIR_xx
if (in_array($i['constante'], array('_DIR_PLUGINS', '_DIR_PLUGINS_SUPPL'))) {
@ -1002,7 +1002,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon
*/
function do_stop($info) {
public function do_stop($info) {
$i = sql_fetsel('*', 'spip_paquets', 'id_paquet=' . sql_quote($info['i']));
// il faudra prendre en compte les autres _DIR_xx
if (in_array($i['constante'], array('_DIR_PLUGINS', '_DIR_PLUGINS_SUPPL'))) {
@ -1015,7 +1015,7 @@ class Actionneur {
// - true : operation deja faite
// - tableau : operation faite ce tour ci.
$infos = $installer_plugins($dossier, 'uninstall', $i['constante']);
if (is_bool($infos) OR !$infos['install_test'][0]) {
if (is_bool($infos) or !$infos['install_test'][0]) {
include_spip('inc/plugin');
ecrire_plugin_actifs(array($dossier), false, 'enleve');
sql_updateq('spip_paquets', array('actif' => 'non', 'installe' => 'non'), 'id_paquet=' . sql_quote($info['i']));
@ -1040,7 +1040,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon
*/
function do_kill($info) {
public function do_kill($info) {
// on reverifie que c'est bien un plugin auto !
// il faudrait aussi faire tres attention sur un site mutualise
// cette option est encore plus delicate que les autres...
@ -1115,7 +1115,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon
*/
function do_getlib($info) {
public function do_getlib($info) {
if (!defined('_DIR_LIB') or !_DIR_LIB) {
$this->err(_T('svp:erreur_dir_dib_indefini'));
$this->log("/!\ Pas de _DIR_LIB defini !");
@ -1160,7 +1160,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon
*/
function do_get($info) {
public function do_get($info) {
if (!$this->tester_repertoire_plugins_auto()) {
return false;
}
@ -1185,7 +1185,7 @@ class Actionneur {
* @return bool
* false si erreur, true sinon
*/
function do_install($info) {
public function do_install($info) {
return $this->installer_plugin($info);
}
@ -1201,7 +1201,7 @@ class Actionneur {
* Constante indiquant le chemin de base du plugin (_DIR_PLUGINS, _DIR_PLUGINS_SUPPL, _DIR_PLUGINS_DIST)
* @return void
**/
function activer_plugin_dossier($dossier, $i, $constante = '_DIR_PLUGINS') {
public function activer_plugin_dossier($dossier, $i, $constante = '_DIR_PLUGINS') {
include_spip('inc/plugin');
$this->log("Demande d'activation de : " . $dossier);
@ -1253,7 +1253,7 @@ class Actionneur {
*
* Les plugins atteignant un score de zéro sont évacués ce la liste.
*/
function actualiser_plugin_interessants() {
public function actualiser_plugin_interessants() {
// Chaque fois que l'on valide des plugins,
// on memorise la liste de ces plugins comme etant "interessants",
// avec un score initial, qui sera decremente a chaque tour :
@ -1303,7 +1303,7 @@ class Actionneur {
* @param string $dir
* Chemin du répertoire du plugin
*/
function ajouter_plugin_interessants_meta($dir) {
public function ajouter_plugin_interessants_meta($dir) {
$plugins_interessants = @unserialize($GLOBALS['meta']['plugins_interessants']);
if (!is_array($plugins_interessants)) {
$plugins_interessants = array();
@ -1320,14 +1320,14 @@ class Actionneur {
* @return bool
* false si erreur, true sinon
*/
function installer_plugin($info) {
public function installer_plugin($info) {
// il faut info['dossier'] et info['constante'] pour installer
if ($plug = $info['dossier']) {
$installer_plugins = charger_fonction('installer', 'plugins');
$infos = $installer_plugins($plug, 'install', $info['constante']);
if ($infos) {
// en absence d'erreur, on met a jour la liste des plugins installes...
if (!is_array($infos) OR $infos['install_test'][0]) {
if (!is_array($infos) or $infos['install_test'][0]) {
$meta_plug_installes = @unserialize($GLOBALS['meta']['plugin_installes']);
if (!$meta_plug_installes) {
$meta_plug_installes = array();
@ -1371,7 +1371,7 @@ class Actionneur {
* - dir : Chemin du paquet téléchargé depuis la racine
* - dossier : Chemin du paquet téléchargé, depuis _DIR_PLUGINS
*/
function get_paquet_id($id_or_row, $dest_ancien = "") {
public function get_paquet_id($id_or_row, $dest_ancien = "") {
// on peut passer direct le row sql...
if (!is_array($id_or_row)) {
$i = sql_fetsel('*', 'spip_paquets', 'id_paquet=' . sql_quote($id_or_row));
@ -1448,7 +1448,7 @@ class Actionneur {
* @return bool
* True si on peut écrire dedans, false sinon
**/
function tester_repertoire_plugins_auto() {
public function tester_repertoire_plugins_auto() {
include_spip('inc/plugin'); // pour _DIR_PLUGINS_AUTO
if (!defined('_DIR_PLUGINS_AUTO') or !_DIR_PLUGINS_AUTO) {
$this->err(_T('svp:erreur_dir_plugins_auto_indefini'));
@ -1480,7 +1480,7 @@ class Actionneur {
* @return bool
* true si le répertoire est dans un ancien format
*/
function tester_repertoire_destination_ancien_format($dir_dans_auto) {
public function tester_repertoire_destination_ancien_format($dir_dans_auto) {
// si on tombe sur un auto/X ayant des fichiers (et pas uniquement des dossiers)
// ou un dossier qui ne commence pas par 'v'
// c'est que auto/X n'était pas chargé avec SVP
@ -1492,7 +1492,7 @@ class Actionneur {
$base_files = array_diff($base_files, array('.', '..'));
foreach ($base_files as $f) {
if (($f[0] != '.' and $f[0] != 'v') // commence pas par v
OR ($f[0] != '.' and !is_dir($dir_dans_auto . '/' . $f))
or ($f[0] != '.' and !is_dir($dir_dans_auto . '/' . $f))
) { // commence par v mais pas repertoire
return true;
}
@ -1513,7 +1513,7 @@ class Actionneur {
*
* @return string Nom du téléporteur à utiliser
**/
function choisir_teleporteur($teleporteur, $defaut = 'http') {
public function choisir_teleporteur($teleporteur, $defaut = 'http') {
// Utiliser un teleporteur vcs si possible si demandé
if (defined('SVP_PREFERER_TELECHARGEMENT_PAR_VCS') and SVP_PREFERER_TELECHARGEMENT_PAR_VCS) {
if ($teleporteur) {
@ -1543,11 +1543,11 @@ class Actionneur {
* @return bool
* true si SVP a été désactivé, false sinon
**/
function tester_si_svp_desactive() {
public function tester_si_svp_desactive() {
foreach ($this->done as $d) {
if ($d['p'] == 'SVP'
AND $d['done'] == true
AND in_array($d['todo'], array('off', 'stop'))
and $d['done'] == true
and in_array($d['todo'], array('off', 'stop'))
) {
return true;
}
@ -1581,5 +1581,3 @@ function svp_actionner_traiter_actions_demandees($actions, &$retour, $redirect =
set_request('_todo', '');
$retour['message_ok'] = _T("svp:action_patienter");
}
?>

100
inc/svp_decider.php

@ -33,7 +33,7 @@ class Decideur {
* Index 'i' : plugins triés par identifiant en base [i][32] = tableau de description
* Index 'p' : plugins triés par prefixe de plugin [p][MOTS] = tableau de description
*/
var $start = array(
public $start = array(
'i' => array(),
'p' => array(),
);
@ -45,7 +45,7 @@ class Decideur {
* Index 'i' : plugins triés par identifiant en base [i][32] = tableau de description
* Index 'p' : plugins triés par prefixe de plugin [p][MOTS] = tableau de description
*/
var $end = array(
public $end = array(
'i' => array(),
'p' => array(),
);
@ -56,7 +56,7 @@ class Decideur {
* @var array
* Tableau ('PREFIXE' => numéro de version)
*/
var $procure = array();
public $procure = array();
/**
* Toutes les actions à faire demandées
@ -65,7 +65,7 @@ class Decideur {
* @var array
* Tableau ('identifiant' => tableau de description)
*/
var $ask = array();
public $ask = array();
/**
* Toutes les actions à faire demandées et consécutives aux dépendances
@ -73,7 +73,7 @@ class Decideur {
* @var array
* Tableau ('identifiant' => tableau de description)
*/
var $todo = array();
public $todo = array();
/**
* Toutes les actions à faire consécutives aux dépendances
@ -83,7 +83,7 @@ class Decideur {
* @var array
* Tableau ('identifiant' => tableau de description)
*/
var $changes = array();
public $changes = array();
/**
* Tous les plugins à arrêter (désactiver ou désinstaller)
@ -91,7 +91,7 @@ class Decideur {
* @var array
* Tableau ('PREFIXE' => tableau de description)
*/
var $off = array();
public $off = array();
/**
* Tous les plugins invalidés (suite a des dependances introuvables, mauvaise version de SPIP...)
@ -99,7 +99,7 @@ class Decideur {
* @var array
* Tableau ('PREFIXE' => tableau de description)
*/
var $invalides = array();
public $invalides = array();
/**
* Liste des erreurs
@ -107,7 +107,7 @@ class Decideur {
* @var array
* Tableau ('identifiant' => liste des erreurs)
*/
var $err = array();
public $err = array();
/**
* État de santé (absence d'erreur)
@ -117,7 +117,7 @@ class Decideur {
*
* @var bool
*/
var $ok = true;
public $ok = true;
/**
* Loguer les différents éléments
@ -126,7 +126,7 @@ class Decideur {
*
* @var bool
*/
var $log = false;
public $log = false;
/**
* Générer une erreur si on demande une mise à jour d'un plugin
@ -134,14 +134,14 @@ class Decideur {
*
* @var bool
*/
var $erreur_sur_maj_introuvable = true;
public $erreur_sur_maj_introuvable = true;
/**
* Constructeur
*
* Initialise la propriété $log en fonction de la configuration
*/
function __construct() {
public function __construct() {
include_spip('inc/config');
$this->log = (lire_config('svp/mode_log_verbeux') == 'oui');
}
@ -154,7 +154,7 @@ class Decideur {
* Index 'i' : plugins triés par identifiant en base [i][32] = tableau de description
* Index 'p' : plugins triés par prefixe de plugin [p][MOTS] = tableau de description
*/
function liste_plugins_actifs() {
public function liste_plugins_actifs() {
return $this->infos_courtes(array('pa.actif=' . sql_quote('oui'), 'pa.attente=' . sql_quote('non')));
}
@ -175,7 +175,7 @@ class Decideur {
* @return bool
* Le plugin est-il en attente ?
*/
function est_attente_id($id) {
public function est_attente_id($id) {
static $attente = null;
if (is_null($attente)) {
$attente = $this->infos_courtes('pa.attente=' . sql_quote('oui'));
@ -192,7 +192,7 @@ class Decideur {
* @return array
* Tableau ('PREFIXE' => version)
*/
function liste_plugins_procure() {
public function liste_plugins_procure() {
$procure = array();
$get_infos = charger_fonction('get_infos', 'plugins');
$infos['_DIR_RESTREINT'][''] = $get_infos('./', false, _DIR_RESTREINT);
@ -213,7 +213,7 @@ class Decideur {
* @param mixed $quoi
* La chose à logguer (souvent un texte)
**/
function log($quoi) {
public function log($quoi) {
if ($this->log) {
spip_log($quoi, 'decideur');
}
@ -231,7 +231,7 @@ class Decideur {
* Index 'i' : plugins triés par identifiant en base [i][32] = tableau de description
* Index 'p' : plugins triés par prefixe de plugin [p][MOTS] = tableau de description
**/
function infos_courtes_id($id) {
public function infos_courtes_id($id) {
// on cache ceux la
static $plug = array();
if (!isset($plug[$id])) {
@ -272,7 +272,7 @@ class Decideur {
* Index 'p' : plugins triés par prefixe de plugin [p][MOTS] = tableau de description
* ou, avec $multiple=true : [p][MOTS][] = tableau de description
*/
function infos_courtes($condition, $multiple = false) {
public function infos_courtes($condition, $multiple = false) {
$plugs = array(
'i' => array(),
'p' => array()
@ -316,7 +316,7 @@ class Decideur {
}
unset($r['dependances']);
if (!$r['procure'] OR !$proc = unserialize($r['procure'])) {
if (!$r['procure'] or !$proc = unserialize($r['procure'])) {
$proc = array();
}
$r['procure'] = $proc;
@ -340,7 +340,7 @@ class Decideur {
// gerer les dependences autres que dans 0 (communs ou local) !!!!
// il peut exister des cles info[dn]["[version_spip_min;version_spip_max]"] de dependences
if (!isset($d[$cle][0]) OR count($d[$cle]) > 1) {
if (!isset($d[$cle][0]) or count($d[$cle]) > 1) {
$dep = array();
$dep[0] = isset($d[$cle][0]) ? $d[$cle][0] : array();
unset($d[$cle][0]);
@ -383,9 +383,9 @@ class Decideur {
* @param string $texte
* Texte de l'erreur
*/
function erreur($id, $texte = '') {
public function erreur($id, $texte = '') {
$this->log("erreur: $id -> $texte");
if (!isset($this->err[$id]) OR !is_array($this->err[$id])) {
if (!isset($this->err[$id]) or !is_array($this->err[$id])) {
$this->err[$id] = array();
}
$this->err[$id][] = $texte;
@ -400,7 +400,7 @@ class Decideur {
* @return bool|array
* false si pas d'erreur, tableau des erreurs sinon.
*/
function en_erreur($id) {
public function en_erreur($id) {
return isset($this->err[$id]) ? $this->err[$id] : false;
}
@ -416,7 +416,7 @@ class Decideur {
* false si pas de plugin plus récent trouvé
* tableau de description du paquet le plus récent sinon
*/
function chercher_plugin_recent($prefixe, $version) {
public function chercher_plugin_recent($prefixe, $version) {
$news = $this->infos_courtes(array(
'pl.prefixe=' . sql_quote($prefixe),
'pa.obsolete=' . sql_quote('non'),
@ -447,7 +447,7 @@ class Decideur {
* false si pas de plugin plus récent trouvé
* tableau de description du paquet le plus récent sinon
*/
function chercher_plugin_compatible($prefixe, $version) {
public function chercher_plugin_compatible($prefixe, $version) {
$plugin = array();
$v = '000.000.000';
@ -478,9 +478,9 @@ class Decideur {
), true);
foreach ($locaux_procure['i'] as $new) {
if (isset($new['procure'][$prefixe])
AND plugin_version_compatible($version, $new['procure'][$prefixe])
AND svp_verifier_compatibilite_spip($new['compatibilite_spip'])
AND spip_version_compare($new['procure'][$prefixe], $v, ">")
and plugin_version_compatible($version, $new['procure'][$prefixe])
and svp_verifier_compatibilite_spip($new['compatibilite_spip'])
and spip_version_compare($new['procure'][$prefixe], $v, ">")
) {
$plugin = $new;
$v = $new['v'];
@ -517,7 +517,7 @@ class Decideur {
* @param array $info
* Description du paquet
**/
function add($info) {
public function add($info) {
$this->end['i'][$info['i']] = $info;
$this->end['p'][$info['p']] = &$this->end['i'][$info['i']];
}
@ -530,7 +530,7 @@ class Decideur {
* @param bool $recur
* Passer à off les plugins qui en dépendent, de façon récursive ?
**/
function off($info, $recur = false) {
public function off($info, $recur = false) {
$this->log('- stopper ' . $info['p']);
$this->remove($info);
$this->off[$info['p']] = $info;
@ -559,7 +559,7 @@ class Decideur {
* @return bool
* Le paquet sera t'il off ?
**/
function sera_off($prefixe) {
public function sera_off($prefixe) {
return isset($this->off[$prefixe]) ? $this->off[$prefixe] : false;
}
@ -571,7 +571,7 @@ class Decideur {
* @return bool
* Le paquet sera t'il off ?
**/
function sera_off_id($id) {
public function sera_off_id($id) {
foreach ($this->off as $info) {
if ($info['i'] == $id) {
return $info;
@ -590,7 +590,7 @@ class Decideur {
* @return bool
* Le paquet sera t'il actif ?
**/
function sera_actif($prefixe) {
public function sera_actif($prefixe) {
if (isset($this->end['p'][$prefixe])) {
return $this->end['p'][$prefixe];
}
@ -599,7 +599,7 @@ class Decideur {
$plugin = false;
foreach ($this->end['p'] as $prefixe => $end) {
if (isset($end['procure'][$prefixe])
AND spip_version_compare($end['procure'][$prefixe], $v, ">")
and spip_version_compare($end['procure'][$prefixe], $v, ">")
) {
$v = $end['procure'][$prefixe];
$plugin = $this->end['p'][$prefixe];
@ -617,7 +617,7 @@ class Decideur {
* @return bool
* Le paquet sera t'il actif ?
**/
function sera_actif_id($id) {
public function sera_actif_id($id) {
return isset($this->end['i'][$id]) ? $this->end['i'][$id] : false;
}
@ -631,7 +631,7 @@ class Decideur {
* @param string $quoi
* Type d'action (on, off, kill, upon...)
*/
function ask($info, $quoi) {
public function ask($info, $quoi) {
$this->ask[$info['i']] = $info;
$this->ask[$info['i']]['todo'] = $quoi;
$this->todo($info, $quoi);
@ -648,7 +648,7 @@ class Decideur {
* @param string $quoi
* Type d'action (on, off, kill, upon...)
*/
function change($info, $quoi) {
public function change($info, $quoi) {
$this->changes[$info['i']] = $info;
$this->changes[$info['i']]['todo'] = $quoi;
$this->todo($info, $quoi);
@ -665,7 +665,7 @@ class Decideur {
* @param array $info
* Description du paquet concerné
*/
function annule_change($info) {
public function annule_change($info) {
unset($this->changes[$info['i']]);
}
@ -677,7 +677,7 @@ class Decideur {
* @param string $quoi
* Type d'action (on, off, kill, upon...)
*/
function todo($info, $quoi) {
public function todo($info, $quoi) {
$this->todo[$info['i']] = $info;
$this->todo[$info['i']]['todo'] = $quoi;
}
@ -688,7 +688,7 @@ class Decideur {
* @param array $info
* Description du paquet concerné
*/
function remove($info) {
public function remove($info) {
// aucazou ce ne soit pas les memes ids entre la demande et la bdd,
// on efface aussi avec l'id donne par le prefixe.
// Lorsqu'on desactive un plugin en "attente", il n'est pas actif !
@ -713,7 +713,7 @@ class Decideur {
* @param array $info
* Description du paquet concerné
*/
function invalider($info) {
public function invalider($info) {
$this->log("-> invalider $info[p]");
$this->remove($info); // suffisant ?
$this->invalides[$info['p']] = $info;
@ -729,7 +729,7 @@ class Decideur {
* @return bool
* Le paquet est t'il invalide ?
**/
function sera_invalide($p) {
public function sera_invalide($p) {
return isset($this->invalides[$p]) ? $this->invalides[$p] : false;
}
@ -741,7 +741,7 @@ class Decideur {
* @return bool
* La librairie est-elle présente ?
**/
function est_presente_lib($lib) {
public function est_presente_lib($lib) {
static $libs = false;
if ($libs === false) {
include_spip('inc/svp_outiller');
@ -773,7 +773,7 @@ class Decideur {
* @return bool
* False en cas d'erreur, true sinon
*/
function actionner($todo = null) {
public function actionner($todo = null) {
if (is_array($todo)) {
foreach ($todo as $id => $t) {
// plusieurs choses nous interessent... Sauf... le simple telechargement
@ -904,7 +904,7 @@ class Decideur {
* Les propriété $start et $end reçoivent la liste des plugins actifs
* $procure celle des plugins procurés par le Core
*/
function start() {
public function start() {
$this->start = $this->end = $this->liste_plugins_actifs();
$this->procure = $this->liste_plugins_procure();
}
@ -931,7 +931,7 @@ class Decideur {
* @return bool
* False en cas d'erreur, true sinon
*/
function verifier_dependances($todo = null) {
public function verifier_dependances($todo = null) {
$this->start();
@ -987,7 +987,7 @@ class Decideur {
* @return bool
* false si erreur (dépendance non résolue, incompatibilité...), true sinon
**/
function verifier_dependances_plugin($info, $prof = 0) {
public function verifier_dependances_plugin($info, $prof = 0) {
$this->log("- [$prof] verifier dependances " . $info['p']);
$id = $info['i'];
$err = false; // variable receptionnant parfois des erreurs
@ -1197,7 +1197,7 @@ class Decideur {
* @return array
* Liste des actions (joliement traduites et expliquées)
**/
function presenter_actions($quoi) {
public function presenter_actions($quoi) {
$res = array();
foreach ($this->$quoi as $id => $info) {
$trads = array(
@ -1257,5 +1257,3 @@ function svp_decider_verifier_actions_demandees($a_actionner, &$erreurs) {
return true;
}
?>

15
inc/svp_depoter_distant.php

@ -93,7 +93,7 @@ function svp_ajouter_depot($url, &$erreur = '') {
// Ajout des paquets dans spip_paquets et actualisation des plugins dans spip_plugins
$ok = svp_actualiser_paquets($id_depot, $infos['paquets'], $nb_paquets, $nb_plugins, $nb_autres);
if (!$ok OR ($nb_paquets == 0)) {
if (!$ok or ($nb_paquets == 0)) {
// Si une erreur s'est produite, on supprime le depot deja insere
sql_delete('spip_depots', 'id_depot=' . sql_quote($id_depot));
if (!$ok) {
@ -148,7 +148,7 @@ function svp_supprimer_depot($id) {
foreach ($resultats as $paquet) {
$id_plugin = $paquet['id_plugin'];
if (!isset($vmax[$id_plugin])
OR (spip_version_compare($vmax[$id_plugin], $paquet['version'], '<'))
or (spip_version_compare($vmax[$id_plugin], $paquet['version'], '<'))
) {
$vmax[$id_plugin] = $paquet['version'];
}
@ -371,7 +371,7 @@ function svp_actualiser_paquets($id_depot, $paquets, &$nb_paquets, &$nb_plugins,
$nb_autres = 0;
// Si aucun depot ou aucun paquet on renvoie une erreur
if ((!$id_depot) OR (!is_array($paquets))) {
if ((!$id_depot) or (!is_array($paquets))) {
return false;
}
@ -548,7 +548,7 @@ function svp_actualiser_paquets($id_depot, $paquets, &$nb_paquets, &$nb_plugins,
$plugin = sql_fetsel('id_plugin, vmax', 'spip_plugins',
array('prefixe=' . sql_quote($insert_plugin['prefixe'])));
if (!$plugin AND !array_key_exists($insert_plugin['prefixe'], $insert_plugins)) {
if (!$plugin and !array_key_exists($insert_plugin['prefixe'], $insert_plugins)) {
$insert_plugins[$insert_plugin['prefixe']] = array_merge($insert_plugin,
array('vmax' => $insert_paquet['version']));
} else {
@ -557,7 +557,7 @@ function svp_actualiser_paquets($id_depot, $paquets, &$nb_paquets, &$nb_plugins,
$prefixes[$insert_plugin['prefixe']] = $id_plugin;
}
if (array_key_exists($insert_plugin['prefixe'], $insert_plugins)
AND (spip_version_compare($insert_plugins[$insert_plugin['prefixe']]['vmax'], $insert_paquet['version'],
and (spip_version_compare($insert_plugins[$insert_plugin['prefixe']]['vmax'], $insert_paquet['version'],
'<='))
) {
// attribuer au plugin le nom et le slogan du paquet le plus à jour
@ -794,7 +794,7 @@ function svp_completer_plugins($ids_plugin) {
$complements['date_modif'] = $paquet['date_modif'];
}
if (($complements['date_crea'] === 0)
OR ($paquet['date_crea'] < $complements['date_crea'])
or ($paquet['date_crea'] < $complements['date_crea'])
) {
$complements['date_crea'] = $paquet['date_crea'];
}
@ -946,6 +946,3 @@ function svp_corriger_vmax_plugins($plugins) {
}
}
}
?>

8
inc/svp_depoter_local.php

@ -45,8 +45,8 @@ function svp_actualiser_paquets_locaux($force = false, &$erreurs_xml = array())
// un mode pour tout recalculer sans désinstaller le plugin... !
if ($force
OR _request('var_mode') == 'vider_paquets_locaux'
OR _request('var_mode') == 'recalcul'
or _request('var_mode') == 'vider_paquets_locaux'
or _request('var_mode') == 'recalcul'
) {
svp_base_supprimer_paquets_locaux();
svp_base_inserer_paquets_locaux($paquets);
@ -488,7 +488,7 @@ function svp_compiler_multis($prefixe, $dir_source) {
$nom = $slogan = $description = '';
foreach ($fichiers_langue as $_fichier_langue) {
$nom_fichier = basename($_fichier_langue, '.php');
$langue = substr($nom_fichier, strlen($module)+1-strlen($nom_fichier));
$langue = substr($nom_fichier, strlen($module) + 1 - strlen($nom_fichier));
// Si la langue est reconnue, on traite la liste des items de langue
if (isset($GLOBALS['codes_langues'][$langue])) {
$GLOBALS['idx_lang'] = $langue;
@ -748,5 +748,3 @@ function svp_actualiser_maj_version() {
}
}
}
?>

22
inc/svp_outiller.php

@ -191,7 +191,7 @@ function extraire_bornes($intervalle, $initialiser = false) {
}
if ($intervalle
AND preg_match(',^[\[\(\]]([0-9.a-zRC\s\-]*)[;]([0-9.a-zRC\s\-\*]*)[\]\)\[]$,Uis', $intervalle, $matches)
and preg_match(',^[\[\(\]]([0-9.a-zRC\s\-]*)[;]([0-9.a-zRC\s\-\*]*)[\]\)\[]$,Uis', $intervalle, $matches)
) {
if ($matches[1]) {
$bornes['min']['valeur'] = trim($matches[1]);
@ -249,7 +249,7 @@ function compiler_branches_spip($intervalle) {
// - vide
// - non vide mais avec les deux bornes vides
// Dans ces cas la compatibilite est totale, on renvoie toutes les branches
if (!$intervalle OR (!$bornes['min']['valeur'] AND !$bornes['max']['valeur'])) {
if (!$intervalle or (!$bornes['min']['valeur'] and !$bornes['max']['valeur'])) {
return implode(',', $liste_branches_spip);
}
@ -284,7 +284,7 @@ function co