spip / SPIP

ecrire/action/redirect.php

Spacing +4 added lines, -4 removed lines

@@ -20,18 +20,18 @@
 	if (!preg_match('/^\w+$/', $type)) return;
 	if ($m = _request('var_mode')) {
 		// forcer la mise a jour de l'url de cet objet !
-		if (!defined('_VAR_URLS')) define('_VAR_URLS',true);
+		if (!defined('_VAR_URLS')) define('_VAR_URLS', true);
 		$m = 'var_mode='.urlencode($m);
 	}
 	$h = generer_url_entite_absolue(intval(_request('id')), $type, $m, '', true);
 	$status = '302';
-	if (_request('status') AND _request('status')=='301')
+	if (_request('status') AND _request('status') == '301')
 		$status = '301';
 
 	if ($h)
-		redirige_par_entete(str_replace('&', '&', $h),'',$status);
+		redirige_par_entete(str_replace('&', '&', $h), '', $status);
 	else
-		redirige_par_entete('/','',$status);
+		redirige_par_entete('/', '', $status);
 }
 
 ?>

ecrire/action/debloquer_edition.php

Spacing +2 added lines, -2 removed lines

@@ -24,8 +24,8 @@
 			debloquer_tous($GLOBALS['visiteur_session']['id_auteur']);
 		}
 		else {
-			$arg = explode("-",$arg);
-			list($objet,$id_objet) = $arg;
+			$arg = explode("-", $arg);
+			list($objet, $id_objet) = $arg;
 			debloquer_edition($GLOBALS['visiteur_session']['id_auteur'], $id_objet, $objet);
 		}
 	}

ecrire/action/supprimer_lien.php

Spacing +5 added lines, -5 removed lines

@@ -12,14 +12,14 @@
 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
-function action_supprimer_lien_dist($arg=null){
-	if (is_null($arg)){
-		$securiser_action = charger_fonction('securiser_action','inc');
+function action_supprimer_lien_dist($arg = null) {
+	if (is_null($arg)) {
+		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
 
-	$arg = explode("-",$arg);
-	list($objet_source,$ids,$objet_lie,$idl) = $arg;
+	$arg = explode("-", $arg);
+	list($objet_source, $ids, $objet_lie, $idl) = $arg;
 
 	include_spip('action/editer_liens');
 	objet_dissocier(array($objet_source=>$ids), array($objet_lie=>$idl));

ecrire/action/tester_taille.php

Spacing +10 added lines, -10 removed lines

@@ -16,7 +16,7 @@ 
 function action_tester_taille_error_handler($output)
 {
 	// on est ici, donc echec lors de la creation de l'image
-	if ($GLOBALS['redirect']){
+	if ($GLOBALS['redirect']) {
 		return redirige_formulaire($GLOBALS['redirect']);
 	}
 	return $output;
@@ -31,21 +31,21 @@ 
 		return;
 	
 	$taille = _request('arg');
-	$taille = explode('-',$taille);
+	$taille = explode('-', $taille);
 
 	$GLOBALS['taille_max'] = end($taille);
 	$GLOBALS['taille_min'] = 0;
-	if (count($taille)>1)
+	if (count($taille) > 1)
 		$GLOBALS['taille_min'] = reset($taille);
 
 	// si l'intervalle est assez petit, on garde la valeur min
-	if ($GLOBALS['taille_max']*$GLOBALS['taille_max']-$GLOBALS['taille_min']*$GLOBALS['taille_min']<50000){
-		ecrire_meta('max_taille_vignettes',$t=($GLOBALS['taille_min']*$GLOBALS['taille_min'])*0.9,'non');
-		echo round($t/1000000,3).' Mpx';
+	if ($GLOBALS['taille_max'] * $GLOBALS['taille_max'] - $GLOBALS['taille_min'] * $GLOBALS['taille_min'] < 50000) {
+		ecrire_meta('max_taille_vignettes', $t = ($GLOBALS['taille_min'] * $GLOBALS['taille_min']) * 0.9, 'non');
+		echo round($t / 1000000, 3).' Mpx';
 		die();
 	}
 
-	$taille = $GLOBALS['taille_test'] = round(($GLOBALS['taille_max']+$GLOBALS['taille_min'])/2);
+	$taille = $GLOBALS['taille_test'] = round(($GLOBALS['taille_max'] + $GLOBALS['taille_min']) / 2);
 
 	include_spip('inc/filtres');
 	// des inclusions representatives d'un hit prive et/ou public pour la conso memoire
@@ -67,16 +67,16 @@ 
 	include_spip('inc/header');
 	propre("<doc1>"); // charger propre avec le trairement d'un modele
 
-	$i = _request('i')+1;
+	$i = _request('i') + 1;
 	$image_source = chemin_image("test.png");
 	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=".$GLOBALS['taille_min']."-".$GLOBALS['taille_test']);
 
 	ob_start('action_tester_taille_error_handler');
-	filtrer('image_recadre',$image_source,$taille,$taille);
+	filtrer('image_recadre', $image_source, $taille, $taille);
 	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
 	// si la valeur intermediaire a reussi, on teste la valeur maxi qui est peut etre sous estimee
 	$taille = $GLOBALS['taille_max'];
-	filtrer('image_recadre',$image_source,$taille,$taille);
+	filtrer('image_recadre', $image_source, $taille, $taille);
 	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
 	ob_end_clean();
 

ecrire/action/etre_webmestre.php

Spacing +4 added lines, -4 removed lines

@@ -23,9 +23,9 @@ 
 	$securiser_action = charger_fonction('securiser_action', 'inc');
 	$time = $securiser_action();
 
-	if (time()-$time<15*60
-		AND $GLOBALS['visiteur_session']['statut']=='0minirezo'
-		AND $GLOBALS['visiteur_session']['webmestre']!=='oui') {
+	if (time() - $time < 15 * 60
+		AND $GLOBALS['visiteur_session']['statut'] == '0minirezo'
+		AND $GLOBALS['visiteur_session']['webmestre'] !== 'oui') {
 		$action = _T('info_admin_etre_webmestre');
 		$admin = charger_fonction('admin', 'inc');
 		// lance la verif par ftp et l'appel
@@ -39,7 +39,7 @@ 
 }
 
 function base_etre_webmestre_dist() {
-	if ($GLOBALS['visiteur_session']['statut']=='0minirezo' AND $GLOBALS['visiteur_session']['webmestre']!=='oui') {
+	if ($GLOBALS['visiteur_session']['statut'] == '0minirezo' AND $GLOBALS['visiteur_session']['webmestre'] !== 'oui') {
 		include_spip('action/editer_auteur');
 		instituer_auteur($GLOBALS['visiteur_session']['id_auteur'], array('webmestre'=>'oui'), true);
 	}

ecrire/action/tester.php

Spacing +11 added lines, -11 removed lines

@@ -31,7 +31,7 @@ 
 					$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK."test.gif");
 					if ($srcImage) {
 						$gd_formats_read_gif = ",gif";
-						ImageDestroy( $srcImage );
+						ImageDestroy($srcImage);
 					}
 				}
 			}
@@ -52,21 +52,21 @@ 
 				$srcImage = @ImageCreateFromJPEG(_ROOT_IMG_PACK."test.jpg");
 				if ($srcImage) {
 					$gd_formats[] = "jpg";
-					ImageDestroy( $srcImage );
+					ImageDestroy($srcImage);
 				}
 			}
 			if (function_exists('ImageCreateFromGIF')) {
 				$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK."test.gif");
 				if ($srcImage) {
 					$gd_formats[] = "gif";
-					ImageDestroy( $srcImage );
+					ImageDestroy($srcImage);
 				}
 			}
 			if (function_exists('ImageCreateFromPNG')) {
 				$srcImage = @ImageCreateFromPNG(_ROOT_IMG_PACK."test.png");
 				if ($srcImage) {
 					$gd_formats[] = "png";
-					ImageDestroy( $srcImage );
+					ImageDestroy($srcImage);
 				}
 			}
 		}
@@ -80,7 +80,7 @@ 
 	else if ($arg == "netpbm") {
 		define('_PNMSCALE_COMMAND', 'pnmscale'); // chemin a changer dans mes_options
 		if (_PNMSCALE_COMMAND == '') return;
-		$netpbm_formats= Array();
+		$netpbm_formats = Array();
 
 		$jpegtopnm_command = str_replace("pnmscale",
 			"jpegtopnm", _PNMSCALE_COMMAND);
@@ -88,7 +88,7 @@ 
 			"pnmtojpeg", _PNMSCALE_COMMAND);
 
 		$vignette = _ROOT_IMG_PACK."test.jpg";
-		$dest = _DIR_VAR . "test-jpg.jpg";
+		$dest = _DIR_VAR."test-jpg.jpg";
 		$commande = "$jpegtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
@@ -98,7 +98,7 @@ 
 		$giftopnm_command = str_replace("pnmscale", "giftopnm", _PNMSCALE_COMMAND);
 		$pnmtojpeg_command = str_replace("pnmscale", "pnmtojpeg", _PNMSCALE_COMMAND);
 		$vignette = _ROOT_IMG_PACK."test.gif";
-		$dest = _DIR_VAR . "test-gif.jpg";
+		$dest = _DIR_VAR."test-gif.jpg";
 		$commande = "$giftopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
@@ -108,7 +108,7 @@ 
 
 		$pngtopnm_command = str_replace("pnmscale", "pngtopnm", _PNMSCALE_COMMAND);
 		$vignette = _ROOT_IMG_PACK."test.png";
-		$dest = _DIR_VAR . "test-gif.jpg";
+		$dest = _DIR_VAR."test-gif.jpg";
 		$commande = "$pngtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
@@ -125,13 +125,13 @@ 
 	}
 
 	// et maintenant envoyer la vignette de tests
-	if (in_array($arg,array("gd1","gd2","imagick","convert","netpbm"))) {
+	if (in_array($arg, array("gd1", "gd2", "imagick", "convert", "netpbm"))) {
 		include_spip('inc/filtres');
 		include_spip('inc/filtres_images_mini');
 		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg',"reduire-$taille_preview-$taille_preview",'jpg');
+		$image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
 
-		$image['fichier_dest']=_DIR_VAR."test_$arg";
+		$image['fichier_dest'] = _DIR_VAR."test_$arg";
 		if ($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true)
 		AND ($preview['width'] * $preview['height'] > 0))
 			redirige_par_entete($preview['fichier']);

ecrire/action/supprimer_rubrique.php

Spacing +5 added lines, -5 removed lines

@@ -12,7 +12,7 @@ 
 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
-include_spip('inc/charsets');	# pour le nom de fichier
+include_spip('inc/charsets'); # pour le nom de fichier
 
 /**
  * Effacer une rubrique
@@ -22,14 +22,14 @@ 
  * @param null $id_rubrique
  * @return void
  */
-function action_supprimer_rubrique_dist($id_rubrique=null) {
+function action_supprimer_rubrique_dist($id_rubrique = null) {
 
-	if (is_null($id_rubrique)){
+	if (is_null($id_rubrique)) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$id_rubrique = $securiser_action();
 	}
 
-	if (intval($id_rubrique)){
+	if (intval($id_rubrique)) {
 
 		sql_delete("spip_rubriques", "id_rubrique=".intval($id_rubrique));
 		// Les admin restreints qui n'administraient que cette rubrique
@@ -44,7 +44,7 @@ 
 			$n = sql_countsel("spip_auteurs_liens", "objet='rubrique' AND id_objet!=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
 			if (!$n) {
 				include_spip('action/editer_auteur');
-				auteurs_set($id_auteur,array("statut" => '1comite'));
+				auteurs_set($id_auteur, array("statut" => '1comite'));
 			}
 			sql_delete("spip_auteurs_liens", "objet='rubrique' AND id_objet=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
 		}

ecrire/action/session.php

Spacing +2 added lines, -2 removed lines

@@ -21,9 +21,9 @@
 	if ($var = _request('var')
 	AND preg_match(',^[a-z_0-9-]+$,i', $var)
 	) {
-		if ($_SERVER['REQUEST_METHOD'] == 'POST'){
+		if ($_SERVER['REQUEST_METHOD'] == 'POST') {
 			include_spip('inc/session');
-			session_set('session_'.$var, $val=_request('val'));
+			session_set('session_'.$var, $val = _request('val'));
 			#spip_log("autosave:$var:$val",'autosave');
 		}
 	}

ecrire/action/editer_article.php

Spacing +33 added lines, -33 removed lines

@@ -13,10 +13,10 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 // http://doc.spip.org/@action_editer_article_dist
-function action_editer_article_dist($arg=null) {
+function action_editer_article_dist($arg = null) {
 	include_spip('inc/autoriser');
-	$err="";
-	if (is_null($arg)){
+	$err = "";
+	if (is_null($arg)) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
@@ -27,7 +27,7 @@ 
 		$id_parent = _request('id_parent');
 		if (!$id_parent)
 			$err = _L("creation interdite d'un article sans rubrique");
-		elseif(!autoriser('creerarticledans','rubrique',$id_parent))
+		elseif (!autoriser('creerarticledans', 'rubrique', $id_parent))
 			$err = _T("info_creerdansrubrique_non_autorise");
 		else
 			$id_article = article_inserer($id_parent);
@@ -37,9 +37,9 @@ 
 	if ($id_article > 0) $err = article_modifier($id_article);
 
 	if ($err)
-		spip_log("echec editeur article: $err",_LOG_ERREUR);
+		spip_log("echec editeur article: $err", _LOG_ERREUR);
 
-	return array($id_article,$err);
+	return array($id_article, $err);
 }
 
 /**
@@ -51,7 +51,7 @@ 
  * @param null $set
  * @return string
  */
-function article_modifier($id_article, $set=null) {
+function article_modifier($id_article, $set = null) {
 
 	// unifier $texte en cas de texte trop long
 	trop_longs_articles();
@@ -60,9 +60,9 @@ 
 	include_spip('inc/filtres');
 	$c = collecter_requests(
 		// white list
-		objet_info('article','champs_editables'),
+		objet_info('article', 'champs_editables'),
 		// black list
-		array('date','statut','id_parent'),
+		array('date', 'statut', 'id_parent'),
 		// donnees eventuellement fournies
 		$set
 	);
@@ -87,7 +87,7 @@ 
 		return $err;
 
 	// Modification de statut, changement de rubrique ?
-	$c = collecter_requests(array('date', 'statut', 'id_parent'),array(),$set);
+	$c = collecter_requests(array('date', 'statut', 'id_parent'), array(), $set);
 	$err = article_instituer($id_article, $c);
 
 	return $err;
@@ -106,7 +106,7 @@ 
 	// Si id_rubrique vaut 0 ou n'est pas definie, creer l'article
 	// dans la premiere rubrique racine
 	if (!$id_rubrique = intval($id_rubrique)) {
-		$row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0",'', '0+titre,titre', "1");
+		$row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0", '', '0+titre,titre', "1");
 		$id_rubrique = $row['id_rubrique'];
 	} else $row = sql_fetsel("lang, id_secteur", "spip_rubriques", "id_rubrique=$id_rubrique");
 
@@ -122,7 +122,7 @@ 
 	// dans les rubriques, on essaie avec la langue de l'auteur,
 	// ou a defaut celle de la rubrique
 	// Sinon c'est la langue de la rubrique qui est choisie + heritee
-	if (in_array('spip_articles',explode(',',$GLOBALS['meta']['multi_objets']))) {
+	if (in_array('spip_articles', explode(',', $GLOBALS['meta']['multi_objets']))) {
 		lang_select($GLOBALS['visiteur_session']['lang']);
 		if (in_array($GLOBALS['spip_lang'],
 		explode(',', $GLOBALS['meta']['langues_multilingue']))) {
@@ -167,11 +167,11 @@ 
 	);
 
 	// controler si le serveur n'a pas renvoye une erreur
-	if ($id_article > 0){
-		$id_auteur = ((is_null(_request('id_auteur')) AND isset($GLOBALS['visiteur_session']['id_auteur']))?
+	if ($id_article > 0) {
+		$id_auteur = ((is_null(_request('id_auteur')) AND isset($GLOBALS['visiteur_session']['id_auteur'])) ?
 			$GLOBALS['visiteur_session']['id_auteur']
 			:_request('id_auteur'));
-		if ($id_auteur){
+		if ($id_auteur) {
 			include_spip('action/editer_auteur');
 			auteur_associer($id_auteur, array('article'=>$id_article));
 		}
@@ -186,7 +186,7 @@ 
 // statut et rubrique sont lies, car un admin restreint peut deplacer
 // un article publie vers une rubrique qu'il n'administre pas
 // http://doc.spip.org/@instituer_article
-function article_instituer($id_article, $c, $calcul_rub=true) {
+function article_instituer($id_article, $c, $calcul_rub = true) {
 
 	include_spip('inc/autoriser');
 	include_spip('inc/rubriques');
@@ -198,8 +198,8 @@ 
 	$date_ancienne = $date = $row['date'];
 	$champs = array();
 
-	$d = isset($c['date'])?$c['date']:null;
-	$s = isset($c['statut'])?$c['statut']:$statut;
+	$d = isset($c['date']) ? $c['date'] : null;
+	$s = isset($c['statut']) ? $c['statut'] : $statut;
 
 	// cf autorisations dans inc/instituer_article
 	if ($s != $statut OR ($d AND $d != $date)) {
@@ -208,7 +208,7 @@ 
 		else if (autoriser('modifier', 'article', $id_article) AND $s != 'publie')
 			$statut = $champs['statut'] = $s;
 		else
-			spip_log("editer_article $id_article refus " . join(' ', $c));
+			spip_log("editer_article $id_article refus ".join(' ', $c));
 
 		// En cas de publication, fixer la date a "maintenant"
 		// sauf si $c commande autre chose
@@ -217,7 +217,7 @@ 
 		if ($champs['statut'] == 'publie'
 		 OR ($champs['statut'] == 'prop' AND ($d OR !in_array($statut_ancien, array('publie', 'prop'))))
 		) {
-			if ($d OR strtotime($d=$date)>time())
+			if ($d OR strtotime($d = $date) > time())
 				$champs['date'] = $date = $d;
 			else
 				$champs['date'] = $date = date('Y-m-d H:i:s');
@@ -298,7 +298,7 @@ 
 // fabrique la requete de modification de l'article, avec champs herites
 
 // http://doc.spip.org/@editer_article_heritage
-function editer_article_heritage($id_article, $id_rubrique, $statut, $champs, $cond=true) {
+function editer_article_heritage($id_article, $id_rubrique, $statut, $champs, $cond = true) {
 
 	// Si on deplace l'article
 	//  changer aussi son secteur et sa langue (si heritee)
@@ -308,7 +308,7 @@ 
 
 		$langue = $row_rub['lang'];
 		$champs['id_secteur'] = $row_rub['id_secteur'];
-		if (sql_fetsel('1', 'spip_articles', "id_article=$id_article AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue))) {
+		if (sql_fetsel('1', 'spip_articles', "id_article=$id_article AND langue_choisie<>'oui' AND lang<>".sql_quote($langue))) {
 			$champs['lang'] = $langue;
 		}
 	}
@@ -321,7 +321,7 @@ 
 
 	if ($cond) {
 		include_spip('inc/rubriques');
-		$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time()))?$champs['date']:false;
+		$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time())) ? $champs['date'] : false;
 		calculer_rubriques_if($id_rubrique, $champs, $statut, $postdate);
 	}
 }
@@ -334,27 +334,27 @@ 
 function trop_longs_articles() {
 	if (is_array($plus = _request('texte_plus'))) {
 		foreach ($plus as $n=>$t) {
-			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,","", $t);
+			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,", "", $t);
 		}
-		set_request('texte', join('',$plus) . _request('texte'));
+		set_request('texte', join('', $plus)._request('texte'));
 	}
 }
 
 
 // obsoletes
-function revisions_articles ($id_article, $c=false) {
-	return article_modifier($id_article,$c);
+function revisions_articles($id_article, $c = false) {
+	return article_modifier($id_article, $c);
 }
-function revision_article ($id_article, $c=false) {
-	return article_modifier($id_article,$c);
+function revision_article($id_article, $c = false) {
+	return article_modifier($id_article, $c);
 }
-function articles_set($id_article, $set=null) {
-	return article_modifier($id_article,$set);
+function articles_set($id_article, $set = null) {
+	return article_modifier($id_article, $set);
 }
 function insert_article($id_rubrique) {
 	return article_inserer($id_rubrique);
 }
-function instituer_article($id_article, $c, $calcul_rub=true) {
-	return article_instituer($id_article,$c,$calcul_rub);
+function instituer_article($id_article, $c, $calcul_rub = true) {
+	return article_instituer($id_article, $c, $calcul_rub);
 }
 ?>