PSR et éviter de longues lignes (0e75ac01) · Validations · Roman Mkrtchian / spip

prive/formulaires/dater.php

+0 −1

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -343,7 +343,6 @@ function dater_recuperer_date_saisie($post, $quoi = 'date') {

		return array(date('Y', $t), date('m', $t), date('d', $t));
		}

		}

		/**

prive/formulaires/declarer_bases.php

+6 −5

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -21,10 +21,10 @@ function formulaires_declarer_bases_charger_dist() {

		$deja = bases_referencees(_FILE_CONNECT);
		// proposer un nom de connect si pas encore saisi
		$nom_connect = "";
		$nom_connect = '';
		if (defined('_DECLARER_choix_db')) {
		$nom_connect = _DECLARER_choix_db;
		$n = "";
		$n = '';
		while (in_array($nom_connect . $n, $deja)) {
		$n = ($n ? $n + 1 : 1);
		}
		@@ -120,7 +120,7 @@ function formulaires_declarer_bases_verifier_1_dist() {
		}
		}

		$login_db = $pass_db = "";
		$login_db = $pass_db = '';
		if (!preg_match(',^sqlite,i', $serveur_db)) {
		if (!$login_db = _request('login_db')) {
		if (defined('_INSTALL_USER_DB')) {
		@@ -221,11 +221,12 @@ function formulaires_declarer_bases_traiter_dist() {
		_DECLARER_serveur_db,
		'',
		'',
		'');
		''
		);

		install_fichier_connexion(_DIR_CONNECT . _DECLARER_nom_connect . '.php', $conn);

		return array(
		'message_ok' => _T('install_connect_ok', array('connect' => "<strong>" . _DECLARER_nom_connect . "</strong>"))
		'message_ok' => _T('install_connect_ok', array('connect' => '<strong>' . _DECLARER_nom_connect . '</strong>'))
		);
		}

prive/formulaires/editer_article.php

+21 −5

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -54,8 +54,16 @@ function formulaires_editer_article_charger_dist(
		$row = array(),
		$hidden = ''
		) {
		$valeurs = formulaires_editer_objet_charger('article', $id_article, $id_rubrique, $lier_trad, $retour, $config_fonc,
		$row, $hidden);
		$valeurs = formulaires_editer_objet_charger(
		'article',
		$id_article,
		$id_rubrique,
		$lier_trad,
		$retour,
		$config_fonc,
		$row,
		$hidden
		);
		// il faut enlever l'id_rubrique car la saisie se fait sur id_parent
		// et id_rubrique peut etre passe dans l'url comme rubrique parent initiale
		// et sera perdue si elle est supposee saisie
		@@ -197,8 +205,16 @@ function formulaires_editer_article_traiter_dist(
		// ici on ignore changer_lang qui est poste en cas de trad,
		// car l'heuristique du choix de la langue est pris en charge par article_inserer
		// en fonction de la config du site et de la rubrique choisie
		set_request("changer_lang");
		set_request('changer_lang');

		return formulaires_editer_objet_traiter('article', $id_article, $id_rubrique, $lier_trad, $retour, $config_fonc, $row,
		$hidden);
		return formulaires_editer_objet_traiter(
		'article',
		$id_article,
		$id_rubrique,
		$lier_trad,
		$retour,
		$config_fonc,
		$row,
		$hidden
		);
		}

prive/formulaires/editer_auteur.php

+26 −13

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -161,7 +161,7 @@ function formulaires_editer_auteur_verifier_dist(
		include_spip('inc/auth');

		if (!nom_acceptable(_request('nom'))) {
		$erreurs['nom'] = _T("info_nom_pas_conforme");
		$erreurs['nom'] = _T('info_nom_pas_conforme');
		}

		if ($email = _request('email')) {
		@@ -186,15 +186,19 @@ function formulaires_editer_auteur_verifier_dist(
		#Nouvel auteur
		if (intval($id_auteur) == 0) {
		#Un auteur existe deja avec cette adresse ?
		if (sql_countsel("spip_auteurs", "email=" . sql_quote($email)) > 0) {
		if (sql_countsel('spip_auteurs', 'email=' . sql_quote($email)) > 0) {
		$erreurs['email'] = _T('erreur_email_deja_existant');
		}
		} else {
		#Un auteur existe deja avec cette adresse ? et n'est pas le user courant.
		if ((sql_countsel("spip_auteurs",
		"email=" . sql_quote($email)) > 0) and ($id_auteur != ($id_auteur_ancien = sql_getfetsel('id_auteur',
		'spip_auteurs', "email=" . sql_quote($email))))
		) {
		if ((sql_countsel(
		'spip_auteurs',
		'email=' . sql_quote($email)
		) > 0) and ($id_auteur != ($id_auteur_ancien = sql_getfetsel(
		'id_auteur',
		'spip_auteurs',
		'email=' . sql_quote($email)
		)))) {
		$erreurs['email'] = _T('erreur_email_deja_existant');
		}
		}
		@@ -276,8 +280,10 @@ function formulaires_editer_auteur_traiter_dist(
		}
		$retour = parametre_url($retour, 'email_confirm', '');

		set_request('email',
		email_valide(_request('email'))); // eviter d'enregistrer les cas qui sont acceptés par email_valide dans le verifier :
		set_request(
		'email',
		email_valide(_request('email'))
		); // eviter d'enregistrer les cas qui sont acceptés par email_valide dans le verifier :
		// "Marie@toto.com " ou encore "Marie Toto <Marie@toto.com>"

		include_spip('inc/autoriser');
		@@ -289,13 +295,20 @@ function formulaires_editer_auteur_traiter_dist(
		// son clic sur l'url du message permettre de confirmer le changement
		// et de revenir sur son profil
		if ($GLOBALS['visiteur_session']['id_auteur'] == $id_auteur
		and $email_nouveau != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur=' . intval($id_auteur)))
		and $email_nouveau !=
		($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur=' . intval($id_auteur)))
		) {
		$envoyer_mail = charger_fonction('envoyer_mail', 'inc');
		$texte = _T('form_auteur_mail_confirmation',
		$texte = _T(
		'form_auteur_mail_confirmation',
		array(
		'url' => generer_action_auteur('confirmer_email', $email_nouveau, parametre_url($retour, 'email_modif', 'ok'))
		));
		'url' => generer_action_auteur(
		'confirmer_email',
		$email_nouveau,
		parametre_url($retour, 'email_modif', 'ok')
		)
		)
		);
		$envoyer_mail($email_nouveau, _T('form_auteur_confirmation'), $texte);
		set_request('email_confirm', $email_nouveau);
		if ($email_ancien) {
		@@ -321,7 +334,7 @@ function formulaires_editer_auteur_traiter_dist(
		include_spip('action/editer_auteur');
		auteur_associer($id_auteur, array($objet => $id_objet));
		if (isset($res['redirect'])) {
		$res['redirect'] = parametre_url($res['redirect'], "id_lien_ajoute", $id_auteur, '&');
		$res['redirect'] = parametre_url($res['redirect'], 'id_lien_ajoute', $id_auteur, '&');
		}
		}
		}

prive/formulaires/editer_liens.php

+30 −24

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -104,26 +104,27 @@ function formulaires_editer_liens_charger_dist($a, $b, $c, $options = array()) {

		// L'éditabilité :) est définie par un test permanent (par exemple "associermots") ET le 4ème argument
		include_spip('inc/autoriser');
		$editable = ($editable and autoriser('associer' . $table_source, $objet, $id_objet) and autoriser('modifier', $objet,
		$id_objet));
		$editable = ($editable and autoriser('associer' . $table_source, $objet, $id_objet)
		and autoriser('modifier', $objet, $id_objet));

		if (!$editable and !count(objet_trouver_liens(array($objet_lien => '*'),
		array(($objet_lien == $objet_source ? $objet : $objet_source) => $id_objet)))
		) {
		if (!$editable and !count(objet_trouver_liens(
		array($objet_lien => '*'),
		array(($objet_lien == $objet_source ? $objet : $objet_source) => $id_objet)
		))) {
		return false;
		}

		// squelettes de vue et de d'association
		// ils sont différents si des rôles sont définis.
		$skel_vue = $table_source . "_lies";
		$skel_ajout = $table_source . "_associer";
		$skel_vue = $table_source . '_lies';
		$skel_ajout = $table_source . '_associer';

		// description des roles
		include_spip('inc/roles');
		if ($roles = roles_presents($objet_source, $objet)) {
		// on demande de nouveaux squelettes en conséquence
		$skel_vue = $table_source . "_roles_lies";
		$skel_ajout = $table_source . "_roles_associer";
		$skel_vue = $table_source . '_roles_lies';
		$skel_ajout = $table_source . '_roles_associer';
		}

		$valeurs = array(
		@@ -213,7 +214,7 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = array()) {
		and $oups = _request('_oups')
		and $oups = unserialize($oups)
		) {
		if ($oups_objets = charger_fonction("editer_liens_oups_{$table_source}_{$objet}_{$objet_lien}", "action", true)) {
		if ($oups_objets = charger_fonction("editer_liens_oups_{$table_source}_{$objet}_{$objet_lien}", 'action', true)) {
		$oups_objets($oups);
		} else {
		$objet_source = objet_type($table_source);
		@@ -249,9 +250,11 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = array()) {
		}

		if ($supprimer) {
		if ($supprimer_objets = charger_fonction("editer_liens_supprimer_{$table_source}_{$objet}_{$objet_lien}",
		"action", true)
		) {
		if ($supprimer_objets = charger_fonction(
		"editer_liens_supprimer_{$table_source}_{$objet}_{$objet_lien}",
		'action',
		true
		)) {
		$oups = $supprimer_objets($supprimer);
		} else {
		include_spip('action/editer_liens');
		@@ -259,17 +262,21 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = array()) {

		foreach ($supprimer as $k => $v) {
		if ($lien = lien_verifier_action($k, $v)) {
		$lien = explode("-", $lien);
		$lien = explode('-', $lien);
		list($objet_source, $ids, $objet_lie, $idl, $role) = $lien;
		// appliquer une condition sur le rôle si défini ('*' pour tous les roles)
		$cond = (!is_null($role) ? array('role' => $role) : array());
		if ($objet_lien == $objet_source) {
		$oups = array_merge($oups,
		objet_trouver_liens(array($objet_source => $ids), array($objet_lie => $idl), $cond));
		$oups = array_merge(
		$oups,
		objet_trouver_liens(array($objet_source => $ids), array($objet_lie => $idl), $cond)
		);
		objet_dissocier(array($objet_source => $ids), array($objet_lie => $idl), $cond);
		} else {
		$oups = array_merge($oups,
		objet_trouver_liens(array($objet_lie => $idl), array($objet_source => $ids), $cond));
		$oups = array_merge(
		$oups,
		objet_trouver_liens(array($objet_lie => $idl), array($objet_source => $ids), $cond)
		);
		objet_dissocier(array($objet_lie => $idl), array($objet_source => $ids), $cond);
		}
		}
		@@ -279,8 +286,7 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = array()) {
		}

		if ($ajouter) {
		if ($ajouter_objets = charger_fonction("editer_liens_ajouter_{$table_source}_{$objet}_{$objet_lien}", "action",
		true)
		if ($ajouter_objets = charger_fonction("editer_liens_ajouter_{$table_source}_{$objet}_{$objet_lien}", 'action', true)
		) {
		$ajout_ok = $ajouter_objets($ajouter);
		} else {
		@@ -289,7 +295,7 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = array()) {
		foreach ($ajouter as $k => $v) {
		if ($lien = lien_verifier_action($k, $v)) {
		$ajout_ok = true;
		list($objet1, $ids, $objet2, $idl) = explode("-", $lien);
		list($objet1, $ids, $objet2, $idl) = explode('-', $lien);
		$qualifs = lien_retrouver_qualif($objet_lien, $lien);
		if ($objet_lien == $objet1) {
		lien_ajouter_liaisons($objet1, $ids, $objet2, $idl, $qualifs);
		@@ -334,10 +340,10 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = array()) {
		*/
		function lien_verifier_action($k, $v) {
		$action = '';
		if (preg_match(",^\w+-[\w]+-[\w]+-[\w]+(-[\w])?,", $k)) {
		if (preg_match(',^\w+-[\w]+-[\w]+-[\w]+(-[\w])?,', $k)) {
		$action = $k;
		}
		if (preg_match(",^\w+-[\w]+-[\w]+-[\w]+(-[\w])?,", $v)) {
		if (preg_match(',^\w+-[\w]+-[\w]+-[\w]+(-[\w])?,', $v)) {
		if (is_numeric($k)) {
		$action = $v;
		}
		@@ -346,7 +352,7 @@ function lien_verifier_action($k, $v) {
		}
		}
		// ajout un role null fictif (plus pratique) si pas défini
		if ($action and count(explode("-", $action)) == 4) {
		if ($action and count(explode('-', $action)) == 4) {
		$action .= '-';
		}