spip / SPIP

ecrire/inc/nfslock.php

Indentation +72 added lines, -72 removed lines

@@ -92,86 +92,86 @@ 
 
 // http://doc.spip.org/@spip_nfslock
 function spip_nfslock($fichier,$max_age=0) {
-	$tries = 0;
+    $tries = 0;
 		
-	if (!$max_age) $max_age = _DEFAULT_LOCKTIME;
-	$lock_file = _DIR_TMP . _NAME_LOCK . "-" . substr(md5($fichier),0,8);
+    if (!$max_age) $max_age = _DEFAULT_LOCKTIME;
+    $lock_file = _DIR_TMP . _NAME_LOCK . "-" . substr(md5($fichier),0,8);
 
   
-	/*
+    /*
 	 * 1. create a tmp file with a psuedo random file name. we also make
 	 *    tpath which is a buffer to store the full pathname of the tmp file.
 	 */
 
-	$id = creer_uniqid();
-	$tpath = _DIR_TMP."slock.$id";
-	$tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
-	if (!$tmpfd) {	/* open failed */
-		@fclose($tmpfd);
-		spip_unlink($tpath);
-		return false; //NFSL_SYSF
-	}
+    $id = creer_uniqid();
+    $tpath = _DIR_TMP."slock.$id";
+    $tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
+    if (!$tmpfd) {	/* open failed */
+        @fclose($tmpfd);
+        spip_unlink($tpath);
+        return false; //NFSL_SYSF
+    }
   
-	/*
+    /*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file.
 	 *    then start looping trying to lock it
 	 */
 
-	while ($tries < 10) {
-		/*
+    while ($tries < 10) {
+        /*
 		 * 3. link tmp file to lock file.  if it goes, we win and we clean
 		 *    up and return the st_ctime of the lock file.
 		 */
 
-		if (link($tpath, $lock_file) == 1) {
-			spip_unlink($tpath); /* got it! */
-			@fclose($tmpfd);
-			if (($our_tmp = lstat($lock_file))==false) {	/* stat failed... shouldn't happen */
-				spip_unlink($lock_file);
-				return false; // (NFSL_SYSF);
-			}
-			return($our_tmp['ctime']);
-		}
+        if (link($tpath, $lock_file) == 1) {
+            spip_unlink($tpath); /* got it! */
+            @fclose($tmpfd);
+            if (($our_tmp = lstat($lock_file))==false) {	/* stat failed... shouldn't happen */
+                spip_unlink($lock_file);
+                return false; // (NFSL_SYSF);
+            }
+            return($our_tmp['ctime']);
+        }
 
-		/*
+        /*
 		 * 4. the lock failed.  check for a stale lock file, being mindful
 		 *    of NFS and the fact the time is set from the NFS server.  we
 		 *    do a write on the tmp file to update its time to the server's
 		 *    idea of "now."
 		 */
 
-		$old_stat = lstat($lock_file);
-		if (@fputs($tmpfd, "zz", 2)!=2 || !$our_tmp=fstat($tmpfd))
-			break; /* something bogus is going on */
+        $old_stat = lstat($lock_file);
+        if (@fputs($tmpfd, "zz", 2)!=2 || !$our_tmp=fstat($tmpfd))
+            break; /* something bogus is going on */
 
 
-		if ($old_stat!=false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
-			spip_unlink($lock_file); /* break the stale lock */
-			$tries++;
-			/* It is CRITICAL that we sleep after breaking
+        if ($old_stat!=false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
+            spip_unlink($lock_file); /* break the stale lock */
+            $tries++;
+            /* It is CRITICAL that we sleep after breaking
 			 * the lock. Otherwise, we could race with
 			 * another process and unlink it's newly-
 			 * created file.
 			 */
-			sleep(1+rand(0,4));
-			continue;
-		}
+            sleep(1+rand(0,4));
+            continue;
+        }
 
-		/*
+        /*
 		 * 5. try again
 		 */
 
     $tries++;
-		sleep(1+rand(0,4));
-  }
+        sleep(1+rand(0,4));
+    }
 
-	/*
+    /*
 	 * 6. give up, failure.
 	 */
 
-	spip_unlink($tpath);
-	@fclose($tmpfd);
-	return false; //(NFSL_LOCKED);
+    spip_unlink($tpath);
+    @fclose($tmpfd);
+    return false; //(NFSL_LOCKED);
 }
 
 /*
@@ -212,63 +212,63 @@ 
 
 // http://doc.spip.org/@spip_nfsunlock
 function spip_nfsunlock($fichier, $birth, $max_age=0, $test = false) {
-	$id = creer_uniqid();
-	if (!$max_age) $max_age = _DEFAULT_LOCKTIME;
+    $id = creer_uniqid();
+    if (!$max_age) $max_age = _DEFAULT_LOCKTIME;
 
-	/*
+    /*
 	 * 1. Build a temp file and stat that to get an idea of what the server
 	 *    thinks the current time is (our_tmp.st_ctime)..
 	 */
 
-	$tpath = _DIR_TMP."stime.$id";
-  $tmpfd = @fopen($tpath,'w');
-  if ((!$tmpfd)
+    $tpath = _DIR_TMP."stime.$id";
+    $tmpfd = @fopen($tpath,'w');
+    if ((!$tmpfd)
     OR (@fputs($tmpfd, "zz", 2) != 2)
-		OR !($our_tmp = fstat($tmpfd))) {
-		/* The open failed, or we can't write the file, or we can't stat it */
-		@fclose($tmpfd);
-		spip_unlink($tpath);
-		return false; //(NFSL_SYSF);
-	}
+        OR !($our_tmp = fstat($tmpfd))) {
+        /* The open failed, or we can't write the file, or we can't stat it */
+        @fclose($tmpfd);
+        spip_unlink($tpath);
+        return false; //(NFSL_SYSF);
+    }
 
-	@fclose($tmpfd);		/* We don't need this once we have our_tmp.st_ctime. */
-	spip_unlink($tpath);
+    @fclose($tmpfd);		/* We don't need this once we have our_tmp.st_ctime. */
+    spip_unlink($tpath);
 
-	/*
+    /*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file
 	 */
 
-	$lock_file = _DIR_TMP . _NAME_LOCK . "-" . substr(md5($fichier),0,8);
+    $lock_file = _DIR_TMP . _NAME_LOCK . "-" . substr(md5($fichier),0,8);
 
-	/*
+    /*
 	 * 3. If the ctime hasn't been modified, unlink the file and return. If the
 	 *    lock has expired, sleep the usual random interval before returning.
 	 *    If we didn't sleep, there could be a race if the caller immediately
 	 *    tries to relock the file.
 	 */
 
-  if ( ($old_stat=@lstat($lock_file))	/* stat succeeds so file is there */
-	  && ($old_stat['ctime'] == $birth)) {	/* hasn't been modified since birth */
-	  if (!$test)
-    	spip_unlink($lock_file);			/* so the lock is ours to remove */
+    if ( ($old_stat=@lstat($lock_file))	/* stat succeeds so file is there */
+      && ($old_stat['ctime'] == $birth)) {	/* hasn't been modified since birth */
+        if (!$test)
+        spip_unlink($lock_file);			/* so the lock is ours to remove */
     if ($our_tmp['ctime'] >= $birth + $max_age){	/* the lock has expired */
-    	if (!$test) return false; //(NFSL_LOST);
-      sleep(1+(random(0,4)));		/* so sleep a bit */
+        if (!$test) return false; //(NFSL_LOST);
+        sleep(1+(random(0,4)));		/* so sleep a bit */
+    }
+        return true;//(NFSL_OK);			/* success */
     }
-		return true;//(NFSL_OK);			/* success */
-  }
 
-	/*
+    /*
 	 * 4. Either ctime has been modified, or the entire lock file is missing.
 	 *    If the lock should still be ours, based on the ctime of the temp
 	 *    file, return with NFSL_STOLEN. If not, then our lock is expired and
 	 *    someone else has grabbed the file, so return NFSL_LOST.
 	 */
 
-	if ($our_tmp['ctime'] < $birth + $max_age)	/* lock was stolen */
-		return false; //(NFSL_STOLEN);
+    if ($our_tmp['ctime'] < $birth + $max_age)	/* lock was stolen */
+        return false; //(NFSL_STOLEN);
 
-	return false; //(NFSL_LOST);	/* The lock must have expired first. */
+    return false; //(NFSL_LOST);	/* The lock must have expired first. */
 }
 
 
@@ -291,7 +291,7 @@ 
 
 // http://doc.spip.org/@spip_nfslock_test
 function spip_nfslock_test($fichier, $birth, $max_age=0) {
-	return spip_nfsunlock($fichier, $birth, $max_age, true);
+    return spip_nfsunlock($fichier, $birth, $max_age, true);
 }
 
 ?>

Spacing +19 added lines, -19 removed lines

@@ -87,15 +87,15 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 include_spip('inc/acces');
-define('_DEFAULT_LOCKTIME',60);
-define('_NAME_LOCK','spip_nfs_lock');
+define('_DEFAULT_LOCKTIME', 60);
+define('_NAME_LOCK', 'spip_nfs_lock');
 
 // http://doc.spip.org/@spip_nfslock
-function spip_nfslock($fichier,$max_age=0) {
+function spip_nfslock($fichier, $max_age = 0) {
 	$tries = 0;
 		
 	if (!$max_age) $max_age = _DEFAULT_LOCKTIME;
-	$lock_file = _DIR_TMP . _NAME_LOCK . "-" . substr(md5($fichier),0,8);
+	$lock_file = _DIR_TMP._NAME_LOCK."-".substr(md5($fichier), 0, 8);
 
   
 	/*
@@ -126,7 +126,7 @@ 
 		if (link($tpath, $lock_file) == 1) {
 			spip_unlink($tpath); /* got it! */
 			@fclose($tmpfd);
-			if (($our_tmp = lstat($lock_file))==false) {	/* stat failed... shouldn't happen */
+			if (($our_tmp = lstat($lock_file)) == false) {	/* stat failed... shouldn't happen */
 				spip_unlink($lock_file);
 				return false; // (NFSL_SYSF);
 			}
@@ -141,11 +141,11 @@ 
 		 */
 
 		$old_stat = lstat($lock_file);
-		if (@fputs($tmpfd, "zz", 2)!=2 || !$our_tmp=fstat($tmpfd))
+		if (@fputs($tmpfd, "zz", 2) != 2 || !$our_tmp = fstat($tmpfd))
 			break; /* something bogus is going on */
 
 
-		if ($old_stat!=false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
+		if ($old_stat != false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
 			spip_unlink($lock_file); /* break the stale lock */
 			$tries++;
 			/* It is CRITICAL that we sleep after breaking
@@ -153,7 +153,7 @@ 
 			 * another process and unlink it's newly-
 			 * created file.
 			 */
-			sleep(1+rand(0,4));
+			sleep(1 + rand(0, 4));
 			continue;
 		}
 
@@ -162,7 +162,7 @@ 
 		 */
 
     $tries++;
-		sleep(1+rand(0,4));
+		sleep(1 + rand(0, 4));
   }
 
 	/*
@@ -211,7 +211,7 @@ 
  */
 
 // http://doc.spip.org/@spip_nfsunlock
-function spip_nfsunlock($fichier, $birth, $max_age=0, $test = false) {
+function spip_nfsunlock($fichier, $birth, $max_age = 0, $test = false) {
 	$id = creer_uniqid();
 	if (!$max_age) $max_age = _DEFAULT_LOCKTIME;
 
@@ -221,7 +221,7 @@ 
 	 */
 
 	$tpath = _DIR_TMP."stime.$id";
-  $tmpfd = @fopen($tpath,'w');
+  $tmpfd = @fopen($tpath, 'w');
   if ((!$tmpfd)
     OR (@fputs($tmpfd, "zz", 2) != 2)
 		OR !($our_tmp = fstat($tmpfd))) {
@@ -231,14 +231,14 @@ 
 		return false; //(NFSL_SYSF);
 	}
 
-	@fclose($tmpfd);		/* We don't need this once we have our_tmp.st_ctime. */
+	@fclose($tmpfd); /* We don't need this once we have our_tmp.st_ctime. */
 	spip_unlink($tpath);
 
 	/*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file
 	 */
 
-	$lock_file = _DIR_TMP . _NAME_LOCK . "-" . substr(md5($fichier),0,8);
+	$lock_file = _DIR_TMP._NAME_LOCK."-".substr(md5($fichier), 0, 8);
 
 	/*
 	 * 3. If the ctime hasn't been modified, unlink the file and return. If the
@@ -247,15 +247,15 @@ 
 	 *    tries to relock the file.
 	 */
 
-  if ( ($old_stat=@lstat($lock_file))	/* stat succeeds so file is there */
+  if (($old_stat = @lstat($lock_file))	/* stat succeeds so file is there */
 	  && ($old_stat['ctime'] == $birth)) {	/* hasn't been modified since birth */
 	  if (!$test)
-    	spip_unlink($lock_file);			/* so the lock is ours to remove */
-    if ($our_tmp['ctime'] >= $birth + $max_age){	/* the lock has expired */
+    	spip_unlink($lock_file); /* so the lock is ours to remove */
+    if ($our_tmp['ctime'] >= $birth + $max_age) {	/* the lock has expired */
     	if (!$test) return false; //(NFSL_LOST);
-      sleep(1+(random(0,4)));		/* so sleep a bit */
+      sleep(1 + (random(0, 4))); /* so sleep a bit */
     }
-		return true;//(NFSL_OK);			/* success */
+		return true; //(NFSL_OK);			/* success */
   }
 
 	/*
@@ -290,7 +290,7 @@ 
  */
 
 // http://doc.spip.org/@spip_nfslock_test
-function spip_nfslock_test($fichier, $birth, $max_age=0) {
+function spip_nfslock_test($fichier, $birth, $max_age = 0) {
 	return spip_nfsunlock($fichier, $birth, $max_age, true);
 }
 

Braces +26 added lines, -10 removed lines

@@ -84,7 +84,9 @@ 
  *
  */
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/acces');
 define('_DEFAULT_LOCKTIME',60);
@@ -94,7 +96,9 @@ 
 function spip_nfslock($fichier,$max_age=0) {
 	$tries = 0;
 		
-	if (!$max_age) $max_age = _DEFAULT_LOCKTIME;
+	if (!$max_age) {
+	    $max_age = _DEFAULT_LOCKTIME;
+	}
 	$lock_file = _DIR_TMP . _NAME_LOCK . "-" . substr(md5($fichier),0,8);
 
   
@@ -141,8 +145,10 @@ 
 		 */
 
 		$old_stat = lstat($lock_file);
-		if (@fputs($tmpfd, "zz", 2)!=2 || !$our_tmp=fstat($tmpfd))
-			break; /* something bogus is going on */
+		if (@fputs($tmpfd, "zz", 2)!=2 || !$our_tmp=fstat($tmpfd)) {
+					break;
+		}
+		/* something bogus is going on */
 
 
 		if ($old_stat!=false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
@@ -213,7 +219,9 @@ 
 // http://doc.spip.org/@spip_nfsunlock
 function spip_nfsunlock($fichier, $birth, $max_age=0, $test = false) {
 	$id = creer_uniqid();
-	if (!$max_age) $max_age = _DEFAULT_LOCKTIME;
+	if (!$max_age) {
+	    $max_age = _DEFAULT_LOCKTIME;
+	}
 
 	/*
 	 * 1. Build a temp file and stat that to get an idea of what the server
@@ -249,10 +257,15 @@ 
 
   if ( ($old_stat=@lstat($lock_file))	/* stat succeeds so file is there */
 	  && ($old_stat['ctime'] == $birth)) {	/* hasn't been modified since birth */
-	  if (!$test)
-    	spip_unlink($lock_file);			/* so the lock is ours to remove */
+	  if (!$test) {
+	      	spip_unlink($lock_file);
+	  }
+	  /* so the lock is ours to remove */
     if ($our_tmp['ctime'] >= $birth + $max_age){	/* the lock has expired */
-    	if (!$test) return false; //(NFSL_LOST);
+    	if (!$test) {
+    	    return false;
+    	}
+    	//(NFSL_LOST);
       sleep(1+(random(0,4)));		/* so sleep a bit */
     }
 		return true;//(NFSL_OK);			/* success */
@@ -265,8 +278,11 @@ 
 	 *    someone else has grabbed the file, so return NFSL_LOST.
 	 */
 
-	if ($our_tmp['ctime'] < $birth + $max_age)	/* lock was stolen */
-		return false; //(NFSL_STOLEN);
+	if ($our_tmp['ctime'] < $birth + $max_age) {
+	    /* lock was stolen */
+		return false;
+	}
+	//(NFSL_STOLEN);
 
 	return false; //(NFSL_LOST);	/* The lock must have expired first. */
 }

ecrire/inc/plonger.php

Indentation +74 added lines, -74 removed lines

@@ -16,95 +16,95 @@
 
 // http://doc.spip.org/@inc_plonger_dist
 function inc_plonger_dist($id_rubrique, $idom="", $list=array(), $col = 1, $exclu=0, $do='aff') {
-	global  $spip_lang_left;
+    global  $spip_lang_left;
 	
-	if ($list) $id_rubrique = $list[$col-1];
+    if ($list) $id_rubrique = $list[$col-1];
 	
-	$ret = '';
+    $ret = '';
 
-	# recherche les filles et petites-filles de la rubrique donnee
-	# en excluant une eventuelle rubrique interdite (par exemple, lorsqu'on
-	# deplace une rubrique, on peut la deplacer partout a partir de la
-	# racine... sauf vers elle-meme ou sa propre branche)
-	$ordre = array();
-	$rub = array();
+    # recherche les filles et petites-filles de la rubrique donnee
+    # en excluant une eventuelle rubrique interdite (par exemple, lorsqu'on
+    # deplace une rubrique, on peut la deplacer partout a partir de la
+    # racine... sauf vers elle-meme ou sa propre branche)
+    $ordre = array();
+    $rub = array();
 
-	$res = sql_select("rub1.id_rubrique, rub1.titre, rub1.id_parent, rub1.lang, rub1.langue_choisie, rub2.id_rubrique AS id_enfant",
-			  "spip_rubriques AS rub1 LEFT JOIN spip_rubriques AS rub2 ON (rub1.id_rubrique = rub2.id_parent)",
-			  "rub1.id_parent = ".sql_quote($id_rubrique)."
+    $res = sql_select("rub1.id_rubrique, rub1.titre, rub1.id_parent, rub1.lang, rub1.langue_choisie, rub2.id_rubrique AS id_enfant",
+                "spip_rubriques AS rub1 LEFT JOIN spip_rubriques AS rub2 ON (rub1.id_rubrique = rub2.id_parent)",
+                "rub1.id_parent = ".sql_quote($id_rubrique)."
 			AND rub1.id_rubrique!=".sql_quote($exclu)."
 			AND (rub2.id_rubrique IS NULL OR rub2.id_rubrique!=".sql_quote($exclu).")",  "", "0+rub1.titre,rub1.titre");
 
-	while ($row = sql_fetch($res)) {
-		if (autoriser('voir','rubrique',$row['id_rubrique'])){
-			$rub[$row['id_rubrique']]['enfants'] = $row['id_enfant'];
-			if ($row['id_parent'] == $id_rubrique) {
-				$t = trim(typo(supprimer_numero($row['titre'])));
-				if ($row['langue_choisie'] != 'oui')
-					$t .= ' <small title="'
-						.traduire_nom_langue($row['lang'])
-						.'">[' . $row['lang'] . ']</small>';
-					$ordre[$row['id_rubrique']] = $t;
-			}
-		}
-	}
-	$next = isset($list[$col]) ? $list[$col] : 0;
-	if ($ordre) {
-		$rec = generer_url_ecrire('plonger',"rac=$idom&exclus=$exclu&do=$do&col=".($col+1));
-		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
-		$args = "'$idom',this,$col,'$spip_lang_left','$info',event";
-		while (list($id, $titrebrut) = each($ordre)) {
+    while ($row = sql_fetch($res)) {
+        if (autoriser('voir','rubrique',$row['id_rubrique'])){
+            $rub[$row['id_rubrique']]['enfants'] = $row['id_enfant'];
+            if ($row['id_parent'] == $id_rubrique) {
+                $t = trim(typo(supprimer_numero($row['titre'])));
+                if ($row['langue_choisie'] != 'oui')
+                    $t .= ' <small title="'
+                        .traduire_nom_langue($row['lang'])
+                        .'">[' . $row['lang'] . ']</small>';
+                    $ordre[$row['id_rubrique']] = $t;
+            }
+        }
+    }
+    $next = isset($list[$col]) ? $list[$col] : 0;
+    if ($ordre) {
+        $rec = generer_url_ecrire('plonger',"rac=$idom&exclus=$exclu&do=$do&col=".($col+1));
+        $info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
+        $args = "'$idom',this,$col,'$spip_lang_left','$info',event";
+        while (list($id, $titrebrut) = each($ordre)) {
 
-			$titre = supprimer_numero($titrebrut);
+            $titre = supprimer_numero($titrebrut);
 
-			$classe1 = $id_rubrique ? 'petite-rubrique' : "petit-secteur";
-			if (isset($rub[$id]["enfants"])) {
-				$classe2 = " class='rub-ouverte'";
-				$url = "\nhref='$rec&amp;id=$id'" ;
-			} else {  $url = $classe2 = '' ; }
+            $classe1 = $id_rubrique ? 'petite-rubrique' : "petit-secteur";
+            if (isset($rub[$id]["enfants"])) {
+                $classe2 = " class='rub-ouverte'";
+                $url = "\nhref='$rec&amp;id=$id'" ;
+            } else {  $url = $classe2 = '' ; }
 
-			$js_func = $do . '_selection_titre';
-			$click = "\nonclick=\"changerhighlight(this.parentNode.parentNode.parentNode);\nreturn "
-			. (!is_array($list) ? ' false' 
-			   : "aff_selection_provisoire($id,$args)")
+            $js_func = $do . '_selection_titre';
+            $click = "\nonclick=\"changerhighlight(this.parentNode.parentNode.parentNode);\nreturn "
+            . (!is_array($list) ? ' false' 
+               : "aff_selection_provisoire($id,$args)")
 # ce lien provoque la selection (directe) de la rubrique cliquee
 # et l'affichage de son titre dans le bandeau
-			. "\"\nondblclick=\""
-			. "$js_func(this."
-			. "firstChild.nodeValue,"
-			. $id
-			. ",'selection_rubrique','id_parent');"
-			. "\nreturn aff_selection_provisoire($id,$args);"
-			. "\"";
+            . "\"\nondblclick=\""
+            . "$js_func(this."
+            . "firstChild.nodeValue,"
+            . $id
+            . ",'selection_rubrique','id_parent');"
+            . "\nreturn aff_selection_provisoire($id,$args);"
+            . "\"";
 
-			$ret .= "<div class='"
-			. (($id == $next) ? "item on" : "item")
-			. "'><div class='"
-			. $classe1
-			. "'><div$classe2><a"
-			. $url
-			. $click
-			. ">"
-			. $titre
-			. "</a></div></div></div>";
-		}
-	}
+            $ret .= "<div class='"
+            . (($id == $next) ? "item on" : "item")
+            . "'><div class='"
+            . $classe1
+            . "'><div$classe2><a"
+            . $url
+            . $click
+            . ">"
+            . $titre
+            . "</a></div></div></div>";
+        }
+    }
 
-	$idom2 = $idom . "_col_".($col+1);
-	$left = ($col*150);
+    $idom2 = $idom . "_col_".($col+1);
+    $left = ($col*150);
 
-	return http_img_pack("searching.gif", "*", "style='visibility: hidden; position: absolute; $spip_lang_left: "
-	. ($left-30)
-	. "px; top: 2px; z-index: 2;' id='img_$idom2'")
-	. "<div style='width: 150px; height: 100%; overflow: auto; position: absolute; top: 0px; $spip_lang_left: "
-	.($left-150)
-	."px;'>"
-	. $ret
-	. "\n</div>\n<div id='$idom2'>"
-	. ($next
-	   ? inc_plonger_dist($id_rubrique, $idom, $list, $col+1, $exclu)
-	   : "")
-	. "\n</div>";
+    return http_img_pack("searching.gif", "*", "style='visibility: hidden; position: absolute; $spip_lang_left: "
+    . ($left-30)
+    . "px; top: 2px; z-index: 2;' id='img_$idom2'")
+    . "<div style='width: 150px; height: 100%; overflow: auto; position: absolute; top: 0px; $spip_lang_left: "
+    .($left-150)
+    ."px;'>"
+    . $ret
+    . "\n</div>\n<div id='$idom2'>"
+    . ($next
+       ? inc_plonger_dist($id_rubrique, $idom, $list, $col+1, $exclu)
+       : "")
+    . "\n</div>";
 }
 
 ?>

Spacing +14 added lines, -14 removed lines

@@ -15,10 +15,10 @@ 
 include_spip('inc/texte');
 
 // http://doc.spip.org/@inc_plonger_dist
-function inc_plonger_dist($id_rubrique, $idom="", $list=array(), $col = 1, $exclu=0, $do='aff') {
+function inc_plonger_dist($id_rubrique, $idom = "", $list = array(), $col = 1, $exclu = 0, $do = 'aff') {
 	global  $spip_lang_left;
 	
-	if ($list) $id_rubrique = $list[$col-1];
+	if ($list) $id_rubrique = $list[$col - 1];
 	
 	$ret = '';
 
@@ -33,24 +33,24 @@ 
 			  "spip_rubriques AS rub1 LEFT JOIN spip_rubriques AS rub2 ON (rub1.id_rubrique = rub2.id_parent)",
 			  "rub1.id_parent = ".sql_quote($id_rubrique)."
 			AND rub1.id_rubrique!=".sql_quote($exclu)."
-			AND (rub2.id_rubrique IS NULL OR rub2.id_rubrique!=".sql_quote($exclu).")",  "", "0+rub1.titre,rub1.titre");
+			AND (rub2.id_rubrique IS NULL OR rub2.id_rubrique!=".sql_quote($exclu).")", "", "0+rub1.titre,rub1.titre");
 
 	while ($row = sql_fetch($res)) {
-		if (autoriser('voir','rubrique',$row['id_rubrique'])){
+		if (autoriser('voir', 'rubrique', $row['id_rubrique'])) {
 			$rub[$row['id_rubrique']]['enfants'] = $row['id_enfant'];
 			if ($row['id_parent'] == $id_rubrique) {
 				$t = trim(typo(supprimer_numero($row['titre'])));
 				if ($row['langue_choisie'] != 'oui')
 					$t .= ' <small title="'
 						.traduire_nom_langue($row['lang'])
-						.'">[' . $row['lang'] . ']</small>';
+						.'">['.$row['lang'].']</small>';
 					$ordre[$row['id_rubrique']] = $t;
 			}
 		}
 	}
 	$next = isset($list[$col]) ? $list[$col] : 0;
 	if ($ordre) {
-		$rec = generer_url_ecrire('plonger',"rac=$idom&exclus=$exclu&do=$do&col=".($col+1));
+		$rec = generer_url_ecrire('plonger', "rac=$idom&exclus=$exclu&do=$do&col=".($col + 1));
 		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
 		$args = "'$idom',this,$col,'$spip_lang_left','$info',event";
 		while (list($id, $titrebrut) = each($ordre)) {
@@ -60,10 +60,10 @@ 
 			$classe1 = $id_rubrique ? 'petite-rubrique' : "petit-secteur";
 			if (isset($rub[$id]["enfants"])) {
 				$classe2 = " class='rub-ouverte'";
-				$url = "\nhref='$rec&amp;id=$id'" ;
-			} else {  $url = $classe2 = '' ; }
+				$url = "\nhref='$rec&amp;id=$id'";
+			} else {  $url = $classe2 = ''; }
 
-			$js_func = $do . '_selection_titre';
+			$js_func = $do.'_selection_titre';
 			$click = "\nonclick=\"changerhighlight(this.parentNode.parentNode.parentNode);\nreturn "
 			. (!is_array($list) ? ' false' 
 			   : "aff_selection_provisoire($id,$args)")
@@ -90,19 +90,19 @@ 
 		}
 	}
 
-	$idom2 = $idom . "_col_".($col+1);
-	$left = ($col*150);
+	$idom2 = $idom."_col_".($col + 1);
+	$left = ($col * 150);
 
 	return http_img_pack("searching.gif", "*", "style='visibility: hidden; position: absolute; $spip_lang_left: "
-	. ($left-30)
+	. ($left - 30)
 	. "px; top: 2px; z-index: 2;' id='img_$idom2'")
 	. "<div style='width: 150px; height: 100%; overflow: auto; position: absolute; top: 0px; $spip_lang_left: "
-	.($left-150)
+	.($left - 150)
 	."px;'>"
 	. $ret
 	. "\n</div>\n<div id='$idom2'>"
 	. ($next
-	   ? inc_plonger_dist($id_rubrique, $idom, $list, $col+1, $exclu)
+	   ? inc_plonger_dist($id_rubrique, $idom, $list, $col + 1, $exclu)
 	   : "")
 	. "\n</div>";
 }

Braces +9 added lines, -4 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/texte');
 
@@ -18,7 +20,9 @@ 
 function inc_plonger_dist($id_rubrique, $idom="", $list=array(), $col = 1, $exclu=0, $do='aff') {
 	global  $spip_lang_left;
 	
-	if ($list) $id_rubrique = $list[$col-1];
+	if ($list) {
+	    $id_rubrique = $list[$col-1];
+	}
 	
 	$ret = '';
 
@@ -40,10 +44,11 @@ 
 			$rub[$row['id_rubrique']]['enfants'] = $row['id_enfant'];
 			if ($row['id_parent'] == $id_rubrique) {
 				$t = trim(typo(supprimer_numero($row['titre'])));
-				if ($row['langue_choisie'] != 'oui')
-					$t .= ' <small title="'
+				if ($row['langue_choisie'] != 'oui') {
+									$t .= ' <small title="'
 						.traduire_nom_langue($row['lang'])
 						.'">[' . $row['lang'] . ']</small>';
+				}
 					$ordre[$row['id_rubrique']] = $t;
 			}
 		}

ecrire/inc/iconifier.php

Indentation +3 added lines, -3 removed lines

@@ -16,10 +16,10 @@
 
 // http://doc.spip.org/@inc_iconifier_dist
 function inc_iconifier_dist($objet, $id,  $script, $visible=false, $flag_modif=true) {
-	// compat avec anciens appels
-	$objet = objet_type($objet);
+    // compat avec anciens appels
+    $objet = objet_type($objet);
 	
-	return recuperer_fond('prive/objets/editer/logo',array('objet'=>$objet,'id_objet'=>$id,'editable'=>$flag_modif));
+    return recuperer_fond('prive/objets/editer/logo',array('objet'=>$objet,'id_objet'=>$id,'editable'=>$flag_modif));
 }
 
 ?>

Spacing +2 added lines, -2 removed lines

@@ -15,11 +15,11 @@
 include_spip('inc/actions');
 
 // http://doc.spip.org/@inc_iconifier_dist
-function inc_iconifier_dist($objet, $id,  $script, $visible=false, $flag_modif=true) {
+function inc_iconifier_dist($objet, $id, $script, $visible = false, $flag_modif = true) {
 	// compat avec anciens appels
 	$objet = objet_type($objet);
 	
-	return recuperer_fond('prive/objets/editer/logo',array('objet'=>$objet,'id_objet'=>$id,'editable'=>$flag_modif));
+	return recuperer_fond('prive/objets/editer/logo', array('objet'=>$objet, 'id_objet'=>$id, 'editable'=>$flag_modif));
 }
 
 ?>

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/actions');
 

ecrire/inc/filtres_mime.php

Indentation +36 added lines, -36 removed lines

@@ -25,27 +25,27 @@ 
 
 // http://doc.spip.org/@filtre_text_txt_dist
 function filtre_text_dist($t) {
-	static $t1 = array('&', '<', '>');
-	static $t2 = array('&amp;', '&lt;', '&gt;');
-	return '<pre>' . str_replace($t1, $t2, $t) . '</pre>';
+    static $t1 = array('&', '<', '>');
+    static $t2 = array('&amp;', '&lt;', '&gt;');
+    return '<pre>' . str_replace($t1, $t2, $t) . '</pre>';
 }
 
 // http://doc.spip.org/@filtre_text_csv_dist
 function filtre_text_csv_dist($t) {
-	include_spip('inc/csv');
-	list($entete, $lignes) = analyse_csv($t);
-	foreach ($lignes as &$l)
-		$l = join('|', $l);
-	$corps = join("\n", $lignes) . "\n";
-	$corps = $caption .
-		"\n|{{" .
-		join('}}|{{',$entete) .
-		"}}|" .
-		"\n|" .
-		str_replace("\n", "|\n|",$corps);
-	$corps = str_replace('&#34#','&#34;',$corps);
-	include_spip('inc/texte');
-	return propre($corps);
+    include_spip('inc/csv');
+    list($entete, $lignes) = analyse_csv($t);
+    foreach ($lignes as &$l)
+        $l = join('|', $l);
+    $corps = join("\n", $lignes) . "\n";
+    $corps = $caption .
+        "\n|{{" .
+        join('}}|{{',$entete) .
+        "}}|" .
+        "\n|" .
+        str_replace("\n", "|\n|",$corps);
+    $corps = str_replace('&#34#','&#34;',$corps);
+    include_spip('inc/texte');
+    return propre($corps);
 }
 
 // Incrustation de HTML, si on est capable de le securiser
@@ -53,33 +53,33 @@ 
 // http://doc.spip.org/@filtre_text_html_dist
 function filtre_text_html_dist($t)
 {
-	if (!preg_match(',^(.*?)<body[^>]*>(.*)</body>,is', $t, $r))
-		return appliquer_filtre($t,'text/plain');
+    if (!preg_match(',^(.*?)<body[^>]*>(.*)</body>,is', $t, $r))
+        return appliquer_filtre($t,'text/plain');
 
-	list(,$h,$t) = $r;
+    list(,$h,$t) = $r;
 
-	$style = '';
-	// recuperer les styles internes
-	if (preg_match_all(',<style>(.*?)</style>,is', $h, $r, PREG_PATTERN_ORDER))
-		$style =  join("\n",$r[1]);
-	// ... et externes
+    $style = '';
+    // recuperer les styles internes
+    if (preg_match_all(',<style>(.*?)</style>,is', $h, $r, PREG_PATTERN_ORDER))
+        $style =  join("\n",$r[1]);
+    // ... et externes
 
-	include_spip('inc/distant');
-	if (preg_match_all(',<link[^>]+type=.text/css[^>]*>,is', $h, $r, PREG_PATTERN_ORDER))
-		foreach($r[0] as $l) {
-			preg_match("/href='([^']*)'/", str_replace('"',"'",$l), $m);
-			$style .= "\n/* $l */\n"
-			. str_replace('<','',recuperer_page($m[1]));
-		}
-	// Pourquoi SafeHtml transforme-t-il en texte les scripts dans Body ?
-	$t = safehtml(preg_replace(',<script'.'.*?</script>,is','',$t));
-	return (!$style ? '' : "\n<style>".$style."</style>") . $t;
+    include_spip('inc/distant');
+    if (preg_match_all(',<link[^>]+type=.text/css[^>]*>,is', $h, $r, PREG_PATTERN_ORDER))
+        foreach($r[0] as $l) {
+            preg_match("/href='([^']*)'/", str_replace('"',"'",$l), $m);
+            $style .= "\n/* $l */\n"
+            . str_replace('<','',recuperer_page($m[1]));
+        }
+    // Pourquoi SafeHtml transforme-t-il en texte les scripts dans Body ?
+    $t = safehtml(preg_replace(',<script'.'.*?</script>,is','',$t));
+    return (!$style ? '' : "\n<style>".$style."</style>") . $t;
 }
 
 // http://doc.spip.org/@filtre_audio_x_pn_realaudio
 function filtre_audio_x_pn_realaudio($id)
 {
-  return "
+    return "
 	<param name='controls' value='PositionSlider' />
 	<param name='controls' value='ImageWindow' />
 	<param name='controls' value='PlayButton' />

Spacing +23 added lines, -23 removed lines

@@ -16,18 +16,18 @@ 
 // Fichier des filtres d'incrustation d'un document selon son type MIME
 // Les 7 familles de base ne font rien sauf celle des textes
 
-function filtre_image_dist($t) {return '';}
-function filtre_audio_dist($t) {return '';}
-function filtre_video_dist($t) {return '';}
-function filtre_application_dist($t) {return '';}
-function filtre_message_dist($t) {return '';}
-function filtre_multipart_dist($t) {return '';}
+function filtre_image_dist($t) {return ''; }
+function filtre_audio_dist($t) {return ''; }
+function filtre_video_dist($t) {return ''; }
+function filtre_application_dist($t) {return ''; }
+function filtre_message_dist($t) {return ''; }
+function filtre_multipart_dist($t) {return ''; }
 
 // http://doc.spip.org/@filtre_text_txt_dist
 function filtre_text_dist($t) {
 	static $t1 = array('&', '<', '>');
 	static $t2 = array('&amp;', '&lt;', '&gt;');
-	return '<pre>' . str_replace($t1, $t2, $t) . '</pre>';
+	return '<pre>'.str_replace($t1, $t2, $t).'</pre>';
 }
 
 // http://doc.spip.org/@filtre_text_csv_dist
@@ -36,14 +36,14 @@ 
 	list($entete, $lignes) = analyse_csv($t);
 	foreach ($lignes as &$l)
 		$l = join('|', $l);
-	$corps = join("\n", $lignes) . "\n";
-	$corps = $caption .
-		"\n|{{" .
-		join('}}|{{',$entete) .
-		"}}|" .
-		"\n|" .
-		str_replace("\n", "|\n|",$corps);
-	$corps = str_replace('&#34#','&#34;',$corps);
+	$corps = join("\n", $lignes)."\n";
+	$corps = $caption.
+		"\n|{{".
+		join('}}|{{', $entete).
+		"}}|".
+		"\n|".
+		str_replace("\n", "|\n|", $corps);
+	$corps = str_replace('&#34#', '&#34;', $corps);
 	include_spip('inc/texte');
 	return propre($corps);
 }
@@ -54,26 +54,26 @@ 
 function filtre_text_html_dist($t)
 {
 	if (!preg_match(',^(.*?)<body[^>]*>(.*)</body>,is', $t, $r))
-		return appliquer_filtre($t,'text/plain');
+		return appliquer_filtre($t, 'text/plain');
 
-	list(,$h,$t) = $r;
+	list(,$h, $t) = $r;
 
 	$style = '';
 	// recuperer les styles internes
 	if (preg_match_all(',<style>(.*?)</style>,is', $h, $r, PREG_PATTERN_ORDER))
-		$style =  join("\n",$r[1]);
+		$style = join("\n", $r[1]);
 	// ... et externes
 
 	include_spip('inc/distant');
 	if (preg_match_all(',<link[^>]+type=.text/css[^>]*>,is', $h, $r, PREG_PATTERN_ORDER))
-		foreach($r[0] as $l) {
-			preg_match("/href='([^']*)'/", str_replace('"',"'",$l), $m);
+		foreach ($r[0] as $l) {
+			preg_match("/href='([^']*)'/", str_replace('"', "'", $l), $m);
 			$style .= "\n/* $l */\n"
-			. str_replace('<','',recuperer_page($m[1]));
+			. str_replace('<', '', recuperer_page($m[1]));
 		}
 	// Pourquoi SafeHtml transforme-t-il en texte les scripts dans Body ?
-	$t = safehtml(preg_replace(',<script'.'.*?</script>,is','',$t));
-	return (!$style ? '' : "\n<style>".$style."</style>") . $t;
+	$t = safehtml(preg_replace(',<script'.'.*?</script>,is', '', $t));
+	return (!$style ? '' : "\n<style>".$style."</style>").$t;
 }
 
 // http://doc.spip.org/@filtre_audio_x_pn_realaudio

Braces +15 added lines, -9 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 include_spip('inc/filtres');
 
 // Fichier des filtres d'incrustation d'un document selon son type MIME
@@ -34,8 +36,9 @@ 
 function filtre_text_csv_dist($t) {
 	include_spip('inc/csv');
 	list($entete, $lignes) = analyse_csv($t);
-	foreach ($lignes as &$l)
-		$l = join('|', $l);
+	foreach ($lignes as &$l) {
+			$l = join('|', $l);
+	}
 	$corps = join("\n", $lignes) . "\n";
 	$corps = $caption .
 		"\n|{{" .
@@ -53,21 +56,24 @@ 
 // http://doc.spip.org/@filtre_text_html_dist
 function filtre_text_html_dist($t)
 {
-	if (!preg_match(',^(.*?)<body[^>]*>(.*)</body>,is', $t, $r))
-		return appliquer_filtre($t,'text/plain');
+	if (!preg_match(',^(.*?)<body[^>]*>(.*)</body>,is', $t, $r)) {
+			return appliquer_filtre($t,'text/plain');
+	}
 
 	list(,$h,$t) = $r;
 
 	$style = '';
 	// recuperer les styles internes
-	if (preg_match_all(',<style>(.*?)</style>,is', $h, $r, PREG_PATTERN_ORDER))
-		$style =  join("\n",$r[1]);
+	if (preg_match_all(',<style>(.*?)</style>,is', $h, $r, PREG_PATTERN_ORDER)) {
+			$style =  join("\n",$r[1]);
+	}
 	// ... et externes
 
 	include_spip('inc/distant');
-	if (preg_match_all(',<link[^>]+type=.text/css[^>]*>,is', $h, $r, PREG_PATTERN_ORDER))
-		foreach($r[0] as $l) {
+	if (preg_match_all(',<link[^>]+type=.text/css[^>]*>,is', $h, $r, PREG_PATTERN_ORDER)) {
+			foreach($r[0] as $l) {
 			preg_match("/href='([^']*)'/", str_replace('"',"'",$l), $m);
+	}
 			$style .= "\n/* $l */\n"
 			. str_replace('<','',recuperer_page($m[1]));
 		}

ecrire/inc/json.php

Indentation +45 added lines, -45 removed lines

@@ -23,59 +23,59 @@
 
 // http://doc.spip.org/@var2js
 function var2js($var) {
-	$asso = false;
-	switch (true) {
-		case is_null($var) :
-			return 'null';
-		case is_string($var) :
-			return '"' .addcslashes($var, "\"\\\n\r/") . '"';
-		case is_bool($var) :
-			return $var ? 'true' : 'false';
-		case is_scalar($var) :
-			return (string)$var;
-		case is_object( $var) :
-			$var = get_object_vars($var);
-			$asso = true;
-		case is_array($var) :
-			$keys = array_keys($var);
-			$ikey = count($keys);
-			while (!$asso && $ikey--) {
-				$asso = $ikey !== $keys[$ikey];
-			}
-			$sep = '';
-			if ($asso) {
-				$ret = '{';
-				foreach ($var as $key => $elt) {
-					$ret .= $sep . '"' . $key . '":' . var2js($elt);
-					$sep = ',';
-				}
-				return $ret ."}";
-			} else {
-				$ret = '[';
-				foreach ($var as $elt) {
-					$ret .= $sep . var2js($elt);
-					$sep = ',';
-				}
-				return $ret ."]";
-			}
-	}
-	return false;
+    $asso = false;
+    switch (true) {
+        case is_null($var) :
+            return 'null';
+        case is_string($var) :
+            return '"' .addcslashes($var, "\"\\\n\r/") . '"';
+        case is_bool($var) :
+            return $var ? 'true' : 'false';
+        case is_scalar($var) :
+            return (string)$var;
+        case is_object( $var) :
+            $var = get_object_vars($var);
+            $asso = true;
+        case is_array($var) :
+            $keys = array_keys($var);
+            $ikey = count($keys);
+            while (!$asso && $ikey--) {
+                $asso = $ikey !== $keys[$ikey];
+            }
+            $sep = '';
+            if ($asso) {
+                $ret = '{';
+                foreach ($var as $key => $elt) {
+                    $ret .= $sep . '"' . $key . '":' . var2js($elt);
+                    $sep = ',';
+                }
+                return $ret ."}";
+            } else {
+                $ret = '[';
+                foreach ($var as $elt) {
+                    $ret .= $sep . var2js($elt);
+                    $sep = ',';
+                }
+                return $ret ."]";
+            }
+    }
+    return false;
 }
 
 if(!function_exists('json_encode')) {
-	function json_encode($v) { return var2js($v); }
+    function json_encode($v) { return var2js($v); }
 }
 
 // http://doc.spip.org/@json_export
 function json_export($var) {
-	$var = json_encode($var);
+    $var = json_encode($var);
 
-	// flag indiquant qu'on est en iframe et qu'il faut proteger nos
-	// donnees dans un <textarea> ; attention $_FILES a ete vide par array_pop
-	if (defined('FILE_UPLOAD'))
-		return "<textarea>".spip_htmlspecialchars($var)."</textarea>";
-	else
-		return $var;
+    // flag indiquant qu'on est en iframe et qu'il faut proteger nos
+    // donnees dans un <textarea> ; attention $_FILES a ete vide par array_pop
+    if (defined('FILE_UPLOAD'))
+        return "<textarea>".spip_htmlspecialchars($var)."</textarea>";
+    else
+        return $var;
 }
 
 

Spacing +8 added lines, -8 removed lines

@@ -28,12 +28,12 @@ 
 		case is_null($var) :
 			return 'null';
 		case is_string($var) :
-			return '"' .addcslashes($var, "\"\\\n\r/") . '"';
+			return '"'.addcslashes($var, "\"\\\n\r/").'"';
 		case is_bool($var) :
 			return $var ? 'true' : 'false';
 		case is_scalar($var) :
-			return (string)$var;
-		case is_object( $var) :
+			return (string) $var;
+		case is_object($var) :
 			$var = get_object_vars($var);
 			$asso = true;
 		case is_array($var) :
@@ -46,23 +46,23 @@ 
 			if ($asso) {
 				$ret = '{';
 				foreach ($var as $key => $elt) {
-					$ret .= $sep . '"' . $key . '":' . var2js($elt);
+					$ret .= $sep.'"'.$key.'":'.var2js($elt);
 					$sep = ',';
 				}
-				return $ret ."}";
+				return $ret."}";
 			} else {
 				$ret = '[';
 				foreach ($var as $elt) {
-					$ret .= $sep . var2js($elt);
+					$ret .= $sep.var2js($elt);
 					$sep = ',';
 				}
-				return $ret ."]";
+				return $ret."]";
 			}
 	}
 	return false;
 }
 
-if(!function_exists('json_encode')) {
+if (!function_exists('json_encode')) {
 	function json_encode($v) { return var2js($v); }
 }
 

Braces +9 added lines, -6 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // Les fonctions de toggg pour faire du JSON
 
@@ -72,11 +74,12 @@ 
 
 	// flag indiquant qu'on est en iframe et qu'il faut proteger nos
 	// donnees dans un <textarea> ; attention $_FILES a ete vide par array_pop
-	if (defined('FILE_UPLOAD'))
-		return "<textarea>".spip_htmlspecialchars($var)."</textarea>";
-	else
-		return $var;
-}
+	if (defined('FILE_UPLOAD')) {
+			return "<textarea>".spip_htmlspecialchars($var)."</textarea>";
+	} else {
+			return $var;
+	}
+	}
 
 
 

ecrire/inc/modifier.php

Indentation +181 added lines, -181 removed lines

@@ -37,28 +37,28 @@ 
  *     Tableau des champs et valeurs collectées 
  */
 function collecter_requests($white_list, $black_list, $set=null, $tous=false){
-	$c = $set;
-	if (!$c){
-		$c = array();
-		foreach($white_list as $champ) {
-			// on ne collecte que les champs reellement envoyes par defaut.
-			// le cas d'un envoi de valeur NULL peut du coup poser probleme.
-			$val = _request($champ);
-			if ($tous OR $val !== NULL) {
-				$c[$champ] = $val;
-			}
-		}
-		// on ajoute toujours la lang en saisie possible
-		// meme si pas prevu au depart pour l'objet concerne
-		if ($l = _request('changer_lang')){
-			$c['lang'] = $l;
-		}
-	}
-	foreach($black_list as $champ) {
-		unset($c[$champ]);
-	}
+    $c = $set;
+    if (!$c){
+        $c = array();
+        foreach($white_list as $champ) {
+            // on ne collecte que les champs reellement envoyes par defaut.
+            // le cas d'un envoi de valeur NULL peut du coup poser probleme.
+            $val = _request($champ);
+            if ($tous OR $val !== NULL) {
+                $c[$champ] = $val;
+            }
+        }
+        // on ajoute toujours la lang en saisie possible
+        // meme si pas prevu au depart pour l'objet concerne
+        if ($l = _request('changer_lang')){
+            $c['lang'] = $l;
+        }
+    }
+    foreach($black_list as $champ) {
+        unset($c[$champ]);
+    }
 	
-	return $c;
+    return $c;
 }
 
 /**
@@ -95,183 +95,183 @@ 
  *     - chaîne : Texte d'un message d'erreur
  */
 function objet_modifier_champs($objet, $id_objet, $options, $c=null, $serveur='') {
-	if (!$id_objet = intval($id_objet)) {
-		spip_log('Erreur $id_objet non defini', 'warn');
-		return _T('erreur_technique_enregistrement_impossible');
-	}
+    if (!$id_objet = intval($id_objet)) {
+        spip_log('Erreur $id_objet non defini', 'warn');
+        return _T('erreur_technique_enregistrement_impossible');
+    }
 
-	include_spip('inc/filtres');
+    include_spip('inc/filtres');
 
-	$table_objet = table_objet($objet,$serveur);
-	$spip_table_objet = table_objet_sql($objet,$serveur);
-	$id_table_objet = id_table_objet($objet,$serveur);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($spip_table_objet, $serveur);
+    $table_objet = table_objet($objet,$serveur);
+    $spip_table_objet = table_objet_sql($objet,$serveur);
+    $id_table_objet = id_table_objet($objet,$serveur);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($spip_table_objet, $serveur);
 
-	// Appels incomplets (sans $c)
-	if (!is_array($c)) {
-		spip_log('erreur appel objet_modifier_champs('.$objet.'), manque $c');
-		return _T('erreur_technique_enregistrement_impossible');
-	}
+    // Appels incomplets (sans $c)
+    if (!is_array($c)) {
+        spip_log('erreur appel objet_modifier_champs('.$objet.'), manque $c');
+        return _T('erreur_technique_enregistrement_impossible');
+    }
 
-	// Securite : certaines variables ne sont jamais acceptees ici
-	// car elles ne relevent pas de autoriser(xxx, modifier) ;
-	// il faut passer par instituer_XX()
-	// TODO: faut-il passer ces variables interdites
-	// dans un fichier de description separe ?
-	unset($c['statut']);
-	unset($c['id_parent']);
-	unset($c['id_rubrique']);
-	unset($c['id_secteur']);
+    // Securite : certaines variables ne sont jamais acceptees ici
+    // car elles ne relevent pas de autoriser(xxx, modifier) ;
+    // il faut passer par instituer_XX()
+    // TODO: faut-il passer ces variables interdites
+    // dans un fichier de description separe ?
+    unset($c['statut']);
+    unset($c['id_parent']);
+    unset($c['id_rubrique']);
+    unset($c['id_secteur']);
 
-	// Gerer les champs non vides
-	if (isset($options['nonvide']) AND is_array($options['nonvide'])) {
-		foreach ($options['nonvide'] as $champ => $sinon) {
-			if ($c[$champ] === '') {
-				$c[$champ] = $sinon;
-			}
-		}
-	}
+    // Gerer les champs non vides
+    if (isset($options['nonvide']) AND is_array($options['nonvide'])) {
+        foreach ($options['nonvide'] as $champ => $sinon) {
+            if ($c[$champ] === '') {
+                $c[$champ] = $sinon;
+            }
+        }
+    }
 
 
-	// N'accepter que les champs qui existent
-	// TODO: ici aussi on peut valider les contenus
-	// en fonction du type
-	$champs = array();
-	foreach($desc['field'] as $champ => $ignore)
-		if (isset($c[$champ]))
-			$champs[$champ] = $c[$champ];
+    // N'accepter que les champs qui existent
+    // TODO: ici aussi on peut valider les contenus
+    // en fonction du type
+    $champs = array();
+    foreach($desc['field'] as $champ => $ignore)
+        if (isset($c[$champ]))
+            $champs[$champ] = $c[$champ];
 
-	// Nettoyer les valeurs
-	$champs = array_map('corriger_caracteres', $champs);
+    // Nettoyer les valeurs
+    $champs = array_map('corriger_caracteres', $champs);
 
-	// Envoyer aux plugins
-	$champs = pipeline('pre_edition',
-		array(
-			'args' => array(
-				'table' => $spip_table_objet, // compatibilite
-				'table_objet' => $table_objet,
-				'spip_table_objet' => $spip_table_objet,
-				'type' =>$objet,
-				'id_objet' => $id_objet,
-				'champs' => isset($options['champs']) ? $options['champs'] : array(), // [doc] c'est quoi ?
-				'serveur' => $serveur,
-				'action' => isset($options['action']) ? $options['action'] : 'modifier'
-			),
-			'data' => $champs
-		)
-	);
+    // Envoyer aux plugins
+    $champs = pipeline('pre_edition',
+        array(
+            'args' => array(
+                'table' => $spip_table_objet, // compatibilite
+                'table_objet' => $table_objet,
+                'spip_table_objet' => $spip_table_objet,
+                'type' =>$objet,
+                'id_objet' => $id_objet,
+                'champs' => isset($options['champs']) ? $options['champs'] : array(), // [doc] c'est quoi ?
+                'serveur' => $serveur,
+                'action' => isset($options['action']) ? $options['action'] : 'modifier'
+            ),
+            'data' => $champs
+        )
+    );
 
-	if (!$champs) return false;
+    if (!$champs) return false;
 
 
-	// marquer le fait que l'objet est travaille par toto a telle date
-	if ($GLOBALS['meta']['articles_modif'] != 'non') {
-		include_spip('inc/drapeau_edition');
-		signale_edition ($id_objet, $GLOBALS['visiteur_session'], $objet);
-	}
+    // marquer le fait que l'objet est travaille par toto a telle date
+    if ($GLOBALS['meta']['articles_modif'] != 'non') {
+        include_spip('inc/drapeau_edition');
+        signale_edition ($id_objet, $GLOBALS['visiteur_session'], $objet);
+    }
 
-	// Verifier si les mises a jour sont pertinentes, datees, en conflit etc
-	include_spip('inc/editer');
-	if (!isset($options['data']) OR is_null($options['data'])){
-		$options['data'] = &$_POST;
-	}
-	$conflits = controler_md5($champs, $options['data'], $objet, $id_objet, $serveur);
-	// cas hypothetique : normalement inc/editer verifie en amont le conflit edition
-	// et gere l'interface
-	// ici on ne renvoie donc qu'un messsage d'erreur, au cas ou on y arrive quand meme
-	if ($conflits)
-		return _T('titre_conflit_edition');
+    // Verifier si les mises a jour sont pertinentes, datees, en conflit etc
+    include_spip('inc/editer');
+    if (!isset($options['data']) OR is_null($options['data'])){
+        $options['data'] = &$_POST;
+    }
+    $conflits = controler_md5($champs, $options['data'], $objet, $id_objet, $serveur);
+    // cas hypothetique : normalement inc/editer verifie en amont le conflit edition
+    // et gere l'interface
+    // ici on ne renvoie donc qu'un messsage d'erreur, au cas ou on y arrive quand meme
+    if ($conflits)
+        return _T('titre_conflit_edition');
 
-	if ($champs) {
-		// cas particulier de la langue : passer par instituer_langue_objet
-		if (isset($champs['lang'])){
-			if ($changer_lang=$champs['lang']){
-				$id_rubrique = 0;
-				if ($desc['field']['id_rubrique']){
-					$parent = ($objet=='rubrique')?'id_parent':'id_rubrique';
-					$id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=".intval($id_objet));
-				}
-				$instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
-				$champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang, $serveur);
-			}
-			// on laisse 'lang' dans $champs,
-			// ca permet de passer dans le pipeline post_edition et de journaliser
-			// et ca ne gene pas qu'on refasse un sql_updateq dessus apres l'avoir
-			// deja pris en compte
-		}
+    if ($champs) {
+        // cas particulier de la langue : passer par instituer_langue_objet
+        if (isset($champs['lang'])){
+            if ($changer_lang=$champs['lang']){
+                $id_rubrique = 0;
+                if ($desc['field']['id_rubrique']){
+                    $parent = ($objet=='rubrique')?'id_parent':'id_rubrique';
+                    $id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=".intval($id_objet));
+                }
+                $instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
+                $champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang, $serveur);
+            }
+            // on laisse 'lang' dans $champs,
+            // ca permet de passer dans le pipeline post_edition et de journaliser
+            // et ca ne gene pas qu'on refasse un sql_updateq dessus apres l'avoir
+            // deja pris en compte
+        }
 
-		// la modif peut avoir lieu
+        // la modif peut avoir lieu
 
-		// faut-il ajouter date_modif ?
-		if (isset($options['date_modif']) AND $options['date_modif']
-		AND !isset($champs[$options['date_modif']]))
-			$champs[$options['date_modif']] = date('Y-m-d H:i:s');
+        // faut-il ajouter date_modif ?
+        if (isset($options['date_modif']) AND $options['date_modif']
+        AND !isset($champs[$options['date_modif']]))
+            $champs[$options['date_modif']] = date('Y-m-d H:i:s');
 
-		// allez on commit la modif
-		sql_updateq($spip_table_objet, $champs, "$id_table_objet=".intval($id_objet), $serveur);
+        // allez on commit la modif
+        sql_updateq($spip_table_objet, $champs, "$id_table_objet=".intval($id_objet), $serveur);
 
-		// on verifie si elle est bien passee
-		$moof = sql_fetsel(array_keys($champs), $spip_table_objet, "$id_table_objet=".intval($id_objet), array(), array(), '', array(), $serveur);
-		// si difference entre les champs, reperer les champs mal enregistres
-		if ($moof != $champs) {
-			$liste = array();
-			foreach($moof as $k=>$v)
-				if ($v !== $champs[$k]
-					// ne pas alerter si le champ est numerique est que les valeurs sont equivalentes
-					AND (!is_numeric($v) OR intval($v)!=intval($champs[$k]))
-					) {
-					$liste[] = $k;
-					$conflits[$k]['post'] = $champs[$k];
-					$conflits[$k]['save'] = $v;
-				}
-			// si un champ n'a pas ete correctement enregistre, loger et retourner une erreur
-			// c'est un cas exceptionnel
-			if (count($liste)){
-				spip_log("Erreur enregistrement en base $objet/$id_objet champs :".var_export($conflits,true),'modifier.'._LOG_CRITIQUE);
-				return _T('erreur_technique_enregistrement_champs',array('champs'=>"<i>'".implode("'</i>,<i>'",$liste)."'</i>"));
-			}
-		}
+        // on verifie si elle est bien passee
+        $moof = sql_fetsel(array_keys($champs), $spip_table_objet, "$id_table_objet=".intval($id_objet), array(), array(), '', array(), $serveur);
+        // si difference entre les champs, reperer les champs mal enregistres
+        if ($moof != $champs) {
+            $liste = array();
+            foreach($moof as $k=>$v)
+                if ($v !== $champs[$k]
+                    // ne pas alerter si le champ est numerique est que les valeurs sont equivalentes
+                    AND (!is_numeric($v) OR intval($v)!=intval($champs[$k]))
+                    ) {
+                    $liste[] = $k;
+                    $conflits[$k]['post'] = $champs[$k];
+                    $conflits[$k]['save'] = $v;
+                }
+            // si un champ n'a pas ete correctement enregistre, loger et retourner une erreur
+            // c'est un cas exceptionnel
+            if (count($liste)){
+                spip_log("Erreur enregistrement en base $objet/$id_objet champs :".var_export($conflits,true),'modifier.'._LOG_CRITIQUE);
+                return _T('erreur_technique_enregistrement_champs',array('champs'=>"<i>'".implode("'</i>,<i>'",$liste)."'</i>"));
+            }
+        }
 
-		// Invalider les caches
-		if (isset($options['invalideur']) and $options['invalideur']) {
-			include_spip('inc/invalideur');
-			if (is_array($options['invalideur']))
-				array_map('suivre_invalideur',$options['invalideur']);
-			else
-				suivre_invalideur($options['invalideur']);
-		}
+        // Invalider les caches
+        if (isset($options['invalideur']) and $options['invalideur']) {
+            include_spip('inc/invalideur');
+            if (is_array($options['invalideur']))
+                array_map('suivre_invalideur',$options['invalideur']);
+            else
+                suivre_invalideur($options['invalideur']);
+        }
 
-		// Notifications, gestion des revisions...
-		// en standard, appelle |nouvelle_revision ci-dessous
-		pipeline('post_edition',
-			array(
-				'args' => array(
-					'table' => $spip_table_objet,
-					'table_objet' => $table_objet,
-					'spip_table_objet' => $spip_table_objet,
-					'type' =>$objet,
-					'id_objet' => $id_objet,
-					'champs' => isset($options['champs']) ? $options['champs'] : array(), // [doc] kesako ?
-					'serveur' => $serveur,
-					'action' => isset($options['action']) ? $options['action'] : 'modifier'
-				),
-				'data' => $champs
-			)
-		);
-	}
+        // Notifications, gestion des revisions...
+        // en standard, appelle |nouvelle_revision ci-dessous
+        pipeline('post_edition',
+            array(
+                'args' => array(
+                    'table' => $spip_table_objet,
+                    'table_objet' => $table_objet,
+                    'spip_table_objet' => $spip_table_objet,
+                    'type' =>$objet,
+                    'id_objet' => $id_objet,
+                    'champs' => isset($options['champs']) ? $options['champs'] : array(), // [doc] kesako ?
+                    'serveur' => $serveur,
+                    'action' => isset($options['action']) ? $options['action'] : 'modifier'
+                ),
+                'data' => $champs
+            )
+        );
+    }
 
-	// journaliser l'affaire
-	// message a affiner :-)
-	include_spip('inc/filtres_mini');
-	$qui = ((isset($GLOBALS['visiteur_session']['nom']) AND $GLOBALS['visiteur_session']['nom'])?$GLOBALS['visiteur_session']['nom']:$GLOBALS['ip']);
-	journal(_L($qui.' a &#233;dit&#233; l&#8217;'.$objet.' '.$id_objet.' ('.join('+',array_diff(array_keys($champs), array('date_modif'))).')'), array(
-		'faire' => 'modifier',
-		'quoi' => $objet,
-		'id' => $id_objet
-	));
+    // journaliser l'affaire
+    // message a affiner :-)
+    include_spip('inc/filtres_mini');
+    $qui = ((isset($GLOBALS['visiteur_session']['nom']) AND $GLOBALS['visiteur_session']['nom'])?$GLOBALS['visiteur_session']['nom']:$GLOBALS['ip']);
+    journal(_L($qui.' a &#233;dit&#233; l&#8217;'.$objet.' '.$id_objet.' ('.join('+',array_diff(array_keys($champs), array('date_modif'))).')'), array(
+        'faire' => 'modifier',
+        'quoi' => $objet,
+        'id' => $id_objet
+    ));
 
-	return '';
+    return '';
 }
 
 /**
@@ -296,8 +296,8 @@ 
  *     false sinon (erreur ou aucun champ modifié)
  */
 function modifier_contenu($type, $id, $options, $c=null, $serveur='') {
-	$res = objet_modifier_champs($type, $id, $options, $c, $serveur);
-	return ($res===''?true:false);
+    $res = objet_modifier_champs($type, $id, $options, $c, $serveur);
+    return ($res===''?true:false);
 }
 
 /**
@@ -318,9 +318,9 @@ 
  * @return mixed|string
  */
 function revision_objet($objet,$id_objet,$c=null){
-	$objet = objet_type($objet); // securite
-	include_spip('action/editer_objet');
-	return objet_modifier($objet,$id_objet,$c);
+    $objet = objet_type($objet); // securite
+    include_spip('action/editer_objet');
+    return objet_modifier($objet,$id_objet,$c);
 }
 
 

Braces +23 added lines, -14 removed lines

@@ -16,7 +16,9 @@ 
  * @package SPIP\Objets\Modifications
 **/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Collecte des champs postés
@@ -138,9 +140,10 @@ 
 	// TODO: ici aussi on peut valider les contenus
 	// en fonction du type
 	$champs = array();
-	foreach($desc['field'] as $champ => $ignore)
-		if (isset($c[$champ]))
+	foreach($desc['field'] as $champ => $ignore) {
+			if (isset($c[$champ]))
 			$champs[$champ] = $c[$champ];
+	}
 
 	// Nettoyer les valeurs
 	$champs = array_map('corriger_caracteres', $champs);
@@ -162,7 +165,9 @@ 
 		)
 	);
 
-	if (!$champs) return false;
+	if (!$champs) {
+	    return false;
+	}
 
 
 	// marquer le fait que l'objet est travaille par toto a telle date
@@ -180,8 +185,9 @@ 
 	// cas hypothetique : normalement inc/editer verifie en amont le conflit edition
 	// et gere l'interface
 	// ici on ne renvoie donc qu'un messsage d'erreur, au cas ou on y arrive quand meme
-	if ($conflits)
-		return _T('titre_conflit_edition');
+	if ($conflits) {
+			return _T('titre_conflit_edition');
+	}
 
 	if ($champs) {
 		// cas particulier de la langue : passer par instituer_langue_objet
@@ -205,8 +211,9 @@ 
 
 		// faut-il ajouter date_modif ?
 		if (isset($options['date_modif']) AND $options['date_modif']
-		AND !isset($champs[$options['date_modif']]))
-			$champs[$options['date_modif']] = date('Y-m-d H:i:s');
+		AND !isset($champs[$options['date_modif']])) {
+					$champs[$options['date_modif']] = date('Y-m-d H:i:s');
+		}
 
 		// allez on commit la modif
 		sql_updateq($spip_table_objet, $champs, "$id_table_objet=".intval($id_objet), $serveur);
@@ -216,12 +223,13 @@ 
 		// si difference entre les champs, reperer les champs mal enregistres
 		if ($moof != $champs) {
 			$liste = array();
-			foreach($moof as $k=>$v)
-				if ($v !== $champs[$k]
+			foreach($moof as $k=>$v) {
+							if ($v !== $champs[$k]
 					// ne pas alerter si le champ est numerique est que les valeurs sont equivalentes
 					AND (!is_numeric($v) OR intval($v)!=intval($champs[$k]))
 					) {
 					$liste[] = $k;
+			}
 					$conflits[$k]['post'] = $champs[$k];
 					$conflits[$k]['save'] = $v;
 				}
@@ -236,10 +244,11 @@ 
 		// Invalider les caches
 		if (isset($options['invalideur']) and $options['invalideur']) {
 			include_spip('inc/invalideur');
-			if (is_array($options['invalideur']))
-				array_map('suivre_invalideur',$options['invalideur']);
-			else
-				suivre_invalideur($options['invalideur']);
+			if (is_array($options['invalideur'])) {
+							array_map('suivre_invalideur',$options['invalideur']);
+			} else {
+							suivre_invalideur($options['invalideur']);
+			}
 		}
 
 		// Notifications, gestion des revisions...

Spacing +28 added lines, -28 removed lines

@@ -36,11 +36,11 @@ 
  * @return array
  *     Tableau des champs et valeurs collectées 
  */
-function collecter_requests($white_list, $black_list, $set=null, $tous=false){
+function collecter_requests($white_list, $black_list, $set = null, $tous = false) {
 	$c = $set;
-	if (!$c){
+	if (!$c) {
 		$c = array();
-		foreach($white_list as $champ) {
+		foreach ($white_list as $champ) {
 			// on ne collecte que les champs reellement envoyes par defaut.
 			// le cas d'un envoi de valeur NULL peut du coup poser probleme.
 			$val = _request($champ);
@@ -50,11 +50,11 @@ 
 		}
 		// on ajoute toujours la lang en saisie possible
 		// meme si pas prevu au depart pour l'objet concerne
-		if ($l = _request('changer_lang')){
+		if ($l = _request('changer_lang')) {
 			$c['lang'] = $l;
 		}
 	}
-	foreach($black_list as $champ) {
+	foreach ($black_list as $champ) {
 		unset($c[$champ]);
 	}
 	
@@ -94,7 +94,7 @@ 
  *     - chaîne vide : Vide si tout s'est bien passé
  *     - chaîne : Texte d'un message d'erreur
  */
-function objet_modifier_champs($objet, $id_objet, $options, $c=null, $serveur='') {
+function objet_modifier_champs($objet, $id_objet, $options, $c = null, $serveur = '') {
 	if (!$id_objet = intval($id_objet)) {
 		spip_log('Erreur $id_objet non defini', 'warn');
 		return _T('erreur_technique_enregistrement_impossible');
@@ -102,9 +102,9 @@ 
 
 	include_spip('inc/filtres');
 
-	$table_objet = table_objet($objet,$serveur);
-	$spip_table_objet = table_objet_sql($objet,$serveur);
-	$id_table_objet = id_table_objet($objet,$serveur);
+	$table_objet = table_objet($objet, $serveur);
+	$spip_table_objet = table_objet_sql($objet, $serveur);
+	$id_table_objet = id_table_objet($objet, $serveur);
 	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($spip_table_objet, $serveur);
 
@@ -138,7 +138,7 @@ 
 	// TODO: ici aussi on peut valider les contenus
 	// en fonction du type
 	$champs = array();
-	foreach($desc['field'] as $champ => $ignore)
+	foreach ($desc['field'] as $champ => $ignore)
 		if (isset($c[$champ]))
 			$champs[$champ] = $c[$champ];
 
@@ -168,12 +168,12 @@ 
 	// marquer le fait que l'objet est travaille par toto a telle date
 	if ($GLOBALS['meta']['articles_modif'] != 'non') {
 		include_spip('inc/drapeau_edition');
-		signale_edition ($id_objet, $GLOBALS['visiteur_session'], $objet);
+		signale_edition($id_objet, $GLOBALS['visiteur_session'], $objet);
 	}
 
 	// Verifier si les mises a jour sont pertinentes, datees, en conflit etc
 	include_spip('inc/editer');
-	if (!isset($options['data']) OR is_null($options['data'])){
+	if (!isset($options['data']) OR is_null($options['data'])) {
 		$options['data'] = &$_POST;
 	}
 	$conflits = controler_md5($champs, $options['data'], $objet, $id_objet, $serveur);
@@ -185,11 +185,11 @@ 
 
 	if ($champs) {
 		// cas particulier de la langue : passer par instituer_langue_objet
-		if (isset($champs['lang'])){
-			if ($changer_lang=$champs['lang']){
+		if (isset($champs['lang'])) {
+			if ($changer_lang = $champs['lang']) {
 				$id_rubrique = 0;
-				if ($desc['field']['id_rubrique']){
-					$parent = ($objet=='rubrique')?'id_parent':'id_rubrique';
+				if ($desc['field']['id_rubrique']) {
+					$parent = ($objet == 'rubrique') ? 'id_parent' : 'id_rubrique';
 					$id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=".intval($id_objet));
 				}
 				$instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
@@ -216,10 +216,10 @@ 
 		// si difference entre les champs, reperer les champs mal enregistres
 		if ($moof != $champs) {
 			$liste = array();
-			foreach($moof as $k=>$v)
+			foreach ($moof as $k=>$v)
 				if ($v !== $champs[$k]
 					// ne pas alerter si le champ est numerique est que les valeurs sont equivalentes
-					AND (!is_numeric($v) OR intval($v)!=intval($champs[$k]))
+					AND (!is_numeric($v) OR intval($v) != intval($champs[$k]))
 					) {
 					$liste[] = $k;
 					$conflits[$k]['post'] = $champs[$k];
@@ -227,9 +227,9 @@ 
 				}
 			// si un champ n'a pas ete correctement enregistre, loger et retourner une erreur
 			// c'est un cas exceptionnel
-			if (count($liste)){
-				spip_log("Erreur enregistrement en base $objet/$id_objet champs :".var_export($conflits,true),'modifier.'._LOG_CRITIQUE);
-				return _T('erreur_technique_enregistrement_champs',array('champs'=>"<i>'".implode("'</i>,<i>'",$liste)."'</i>"));
+			if (count($liste)) {
+				spip_log("Erreur enregistrement en base $objet/$id_objet champs :".var_export($conflits, true), 'modifier.'._LOG_CRITIQUE);
+				return _T('erreur_technique_enregistrement_champs', array('champs'=>"<i>'".implode("'</i>,<i>'", $liste)."'</i>"));
 			}
 		}
 
@@ -237,7 +237,7 @@ 
 		if (isset($options['invalideur']) and $options['invalideur']) {
 			include_spip('inc/invalideur');
 			if (is_array($options['invalideur']))
-				array_map('suivre_invalideur',$options['invalideur']);
+				array_map('suivre_invalideur', $options['invalideur']);
 			else
 				suivre_invalideur($options['invalideur']);
 		}
@@ -264,8 +264,8 @@ 
 	// journaliser l'affaire
 	// message a affiner :-)
 	include_spip('inc/filtres_mini');
-	$qui = ((isset($GLOBALS['visiteur_session']['nom']) AND $GLOBALS['visiteur_session']['nom'])?$GLOBALS['visiteur_session']['nom']:$GLOBALS['ip']);
-	journal(_L($qui.' a &#233;dit&#233; l&#8217;'.$objet.' '.$id_objet.' ('.join('+',array_diff(array_keys($champs), array('date_modif'))).')'), array(
+	$qui = ((isset($GLOBALS['visiteur_session']['nom']) AND $GLOBALS['visiteur_session']['nom']) ? $GLOBALS['visiteur_session']['nom'] : $GLOBALS['ip']);
+	journal(_L($qui.' a &#233;dit&#233; l&#8217;'.$objet.' '.$id_objet.' ('.join('+', array_diff(array_keys($champs), array('date_modif'))).')'), array(
 		'faire' => 'modifier',
 		'quoi' => $objet,
 		'id' => $id_objet
@@ -295,9 +295,9 @@ 
  *     true si quelque chose est modifié correctement
  *     false sinon (erreur ou aucun champ modifié)
  */
-function modifier_contenu($type, $id, $options, $c=null, $serveur='') {
+function modifier_contenu($type, $id, $options, $c = null, $serveur = '') {
 	$res = objet_modifier_champs($type, $id, $options, $c, $serveur);
-	return ($res===''?true:false);
+	return ($res === '' ?true:false);
 }
 
 /**
@@ -317,10 +317,10 @@ 
  *     Couples des champs/valeurs modifiées
  * @return mixed|string
  */
-function revision_objet($objet,$id_objet,$c=null){
+function revision_objet($objet, $id_objet, $c = null) {
 	$objet = objet_type($objet); // securite
 	include_spip('action/editer_objet');
-	return objet_modifier($objet,$id_objet,$c);
+	return objet_modifier($objet, $id_objet, $c);
 }
 
 

ecrire/inc/securiser_action.php

Indentation +104 added lines, -104 removed lines

@@ -34,18 +34,18 @@ 
  */
 function inc_securiser_action_dist($action='', $arg='', $redirect="", $mode=false, $att='', $public=false)
 {
-	if ($action)
-		return securiser_action_auteur($action, $arg, $redirect, $mode, $att, $public);
-	else {
-		$arg = _request('arg');
-		$hash = _request('hash');
-		$action = _request('action')?_request('action'):_request('formulaire_action');
-		if ($a = verifier_action_auteur("$action-$arg", $hash))
-			return $arg;
-		include_spip('inc/minipres');
-		echo minipres();
-		exit;
-	}
+    if ($action)
+        return securiser_action_auteur($action, $arg, $redirect, $mode, $att, $public);
+    else {
+        $arg = _request('arg');
+        $hash = _request('hash');
+        $action = _request('action')?_request('action'):_request('formulaire_action');
+        if ($a = verifier_action_auteur("$action-$arg", $hash))
+            return $arg;
+        include_spip('inc/minipres');
+        echo minipres();
+        exit;
+    }
 }
 
 /**
@@ -68,27 +68,27 @@ 
  */
 function securiser_action_auteur($action, $arg, $redirect="", $mode=false, $att='', $public=false) {
 
-	// mode URL ou array
-	if (!is_string($mode)){
-		$hash = calculer_action_auteur("$action-$arg",is_numeric($att)?$att:null);
+    // mode URL ou array
+    if (!is_string($mode)){
+        $hash = calculer_action_auteur("$action-$arg",is_numeric($att)?$att:null);
 
-		$r = rawurlencode($redirect);
-		if ($mode===-1)
-			return array('action'=>$action,'arg'=>$arg,'hash'=>$hash);
-		else
-			return generer_url_action($action, "arg=$arg&hash=$hash" . (!$r ? '' : "&redirect=$r"), $mode, $public);
-	}
+        $r = rawurlencode($redirect);
+        if ($mode===-1)
+            return array('action'=>$action,'arg'=>$arg,'hash'=>$hash);
+        else
+            return generer_url_action($action, "arg=$arg&hash=$hash" . (!$r ? '' : "&redirect=$r"), $mode, $public);
+    }
 
-	// mode formulaire
-	$hash = calculer_action_auteur("$action-$arg");
-	$att .= " style='margin: 0px; border: 0px'";
-	if ($redirect)
-		$redirect = "\n\t\t<input name='redirect' type='hidden' value='". str_replace("'", '&#39;', $redirect) ."' />";
-	$mode .= $redirect . "
+    // mode formulaire
+    $hash = calculer_action_auteur("$action-$arg");
+    $att .= " style='margin: 0px; border: 0px'";
+    if ($redirect)
+        $redirect = "\n\t\t<input name='redirect' type='hidden' value='". str_replace("'", '&#39;', $redirect) ."' />";
+    $mode .= $redirect . "
 <input name='hash' type='hidden' value='$hash' />
 <input name='arg' type='hidden' value='$arg' />";
 
-	return generer_form_action($action, $mode, $att, $public);
+    return generer_form_action($action, $mode, $att, $public);
 }
 
 /**
@@ -100,41 +100,41 @@ 
  * @return array
  */
 function caracteriser_auteur($id_auteur=null) {
-	static $caracterisation = array();
+    static $caracterisation = array();
 
-	if (is_null($id_auteur) AND !isset($GLOBALS['visiteur_session']['id_auteur'])) {
-	// si l'auteur courant n'est pas connu alors qu'il peut demander une action
-	// c'est une connexion par php_auth ou 1 instal, on se rabat sur le cookie.
-	// S'il n'avait pas le droit de realiser cette action, le hash sera faux.
-		if (isset($_COOKIE['spip_session'])
-		AND (preg_match('/^(\d+)/',$_COOKIE['spip_session'],$r))) {
-			  return array($r[1], '');
-			  // Necessaire aux forums anonymes.
-			  // Pour le reste, ca echouera.
-		} else return array('0','');
-	}
-	// Eviter l'acces SQL si le pass est connu de PHP
-	if (is_null($id_auteur)){
-		$id_auteur = isset($GLOBALS['visiteur_session']['id_auteur'])?$GLOBALS['visiteur_session']['id_auteur']:0;
-		if (isset($GLOBALS['visiteur_session']['pass']) AND $GLOBALS['visiteur_session']['pass'])
-			return $caracterisation[$id_auteur] = array($id_auteur, $GLOBALS['visiteur_session']['pass']);
-	}
+    if (is_null($id_auteur) AND !isset($GLOBALS['visiteur_session']['id_auteur'])) {
+    // si l'auteur courant n'est pas connu alors qu'il peut demander une action
+    // c'est une connexion par php_auth ou 1 instal, on se rabat sur le cookie.
+    // S'il n'avait pas le droit de realiser cette action, le hash sera faux.
+        if (isset($_COOKIE['spip_session'])
+        AND (preg_match('/^(\d+)/',$_COOKIE['spip_session'],$r))) {
+                return array($r[1], '');
+                // Necessaire aux forums anonymes.
+                // Pour le reste, ca echouera.
+        } else return array('0','');
+    }
+    // Eviter l'acces SQL si le pass est connu de PHP
+    if (is_null($id_auteur)){
+        $id_auteur = isset($GLOBALS['visiteur_session']['id_auteur'])?$GLOBALS['visiteur_session']['id_auteur']:0;
+        if (isset($GLOBALS['visiteur_session']['pass']) AND $GLOBALS['visiteur_session']['pass'])
+            return $caracterisation[$id_auteur] = array($id_auteur, $GLOBALS['visiteur_session']['pass']);
+    }
 
-	if (isset($caracterisation[$id_auteur])) return $caracterisation[$id_auteur];
+    if (isset($caracterisation[$id_auteur])) return $caracterisation[$id_auteur];
 
-	if ($id_auteur) {
-		include_spip('base/abstract_sql');
-		$t = sql_fetsel("id_auteur, pass", "spip_auteurs", "id_auteur=$id_auteur");
-		if ($t)
-			return $caracterisation[$id_auteur] = array($t['id_auteur'], $t['pass']);
-		include_spip('inc/minipres');
-		echo minipres();
-		exit;
-	}
-	// Visiteur anonyme, pour ls forums par exemple
-	else {
-		return array('0','');
-	}
+    if ($id_auteur) {
+        include_spip('base/abstract_sql');
+        $t = sql_fetsel("id_auteur, pass", "spip_auteurs", "id_auteur=$id_auteur");
+        if ($t)
+            return $caracterisation[$id_auteur] = array($t['id_auteur'], $t['pass']);
+        include_spip('inc/minipres');
+        echo minipres();
+        exit;
+    }
+    // Visiteur anonyme, pour ls forums par exemple
+    else {
+        return array('0','');
+    }
 }
 
 /**
@@ -151,25 +151,25 @@ 
  * @return string
  */
 function _action_auteur($action, $id_auteur, $pass, $alea) {
-	static $sha = array();
-	if (!isset($sha[$id_auteur.$pass.$alea])){
-		if (!isset($GLOBALS['meta'][$alea]) AND _request('exec')!=='install') {
-			include_spip('base/abstract_sql');
-			$GLOBALS['meta'][$alea] = sql_getfetsel('valeur', 'spip_meta', "nom=" . sql_quote($alea));
-			if (!($GLOBALS['meta'][$alea])) {
-				include_spip('inc/minipres');
-				echo minipres();
-				spip_log("$alea indisponible");
-				exit;
-			}
-		}
-		include_spip('auth/sha256.inc');
-		$sha[$id_auteur.$pass.$alea] = _nano_sha256($id_auteur.$pass.@$GLOBALS['meta'][$alea]);
-	}
-	if (function_exists('sha1'))
-		return sha1($action.$sha[$id_auteur.$pass.$alea]);
-	else
-		return md5($action.$sha[$id_auteur.$pass.$alea]);
+    static $sha = array();
+    if (!isset($sha[$id_auteur.$pass.$alea])){
+        if (!isset($GLOBALS['meta'][$alea]) AND _request('exec')!=='install') {
+            include_spip('base/abstract_sql');
+            $GLOBALS['meta'][$alea] = sql_getfetsel('valeur', 'spip_meta', "nom=" . sql_quote($alea));
+            if (!($GLOBALS['meta'][$alea])) {
+                include_spip('inc/minipres');
+                echo minipres();
+                spip_log("$alea indisponible");
+                exit;
+            }
+        }
+        include_spip('auth/sha256.inc');
+        $sha[$id_auteur.$pass.$alea] = _nano_sha256($id_auteur.$pass.@$GLOBALS['meta'][$alea]);
+    }
+    if (function_exists('sha1'))
+        return sha1($action.$sha[$id_auteur.$pass.$alea]);
+    else
+        return md5($action.$sha[$id_auteur.$pass.$alea]);
 }
 
 /**
@@ -181,8 +181,8 @@ 
  * @return string
  */
 function calculer_action_auteur($action, $id_auteur=null) {
-	list($id_auteur, $pass) = caracteriser_auteur($id_auteur);
-	return _action_auteur($action, $id_auteur, $pass, 'alea_ephemere');
+    list($id_auteur, $pass) = caracteriser_auteur($id_auteur);
+    return _action_auteur($action, $id_auteur, $pass, 'alea_ephemere');
 }
 
 
@@ -196,12 +196,12 @@ 
  * @return bool
  */
 function verifier_action_auteur($action, $hash) {
-	list($id_auteur, $pass) = caracteriser_auteur();
-	if ($hash == _action_auteur($action, $id_auteur, $pass, 'alea_ephemere'))
-		return true;
-	if ($hash == _action_auteur($action, $id_auteur, $pass, 'alea_ephemere_ancien'))
-		return true;
-	return false;
+    list($id_auteur, $pass) = caracteriser_auteur();
+    if ($hash == _action_auteur($action, $id_auteur, $pass, 'alea_ephemere'))
+        return true;
+    if ($hash == _action_auteur($action, $id_auteur, $pass, 'alea_ephemere_ancien'))
+        return true;
+    return false;
 }
 
 //
@@ -219,18 +219,18 @@ 
  * @return string
  */
 function secret_du_site() {
-	if (!isset($GLOBALS['meta']['secret_du_site'])){
-		include_spip('base/abstract_sql');
-		$GLOBALS['meta']['secret_du_site'] = sql_getfetsel('valeur', 'spip_meta', "nom='secret_du_site'");
-	}
-	if (!isset($GLOBALS['meta']['secret_du_site'])
-	  OR (strlen($GLOBALS['meta']['secret_du_site'])<64)) {
-		include_spip('inc/acces');
-		include_spip('auth/sha256.inc');
-		ecrire_meta('secret_du_site', _nano_sha256($_SERVER["DOCUMENT_ROOT"] . $_SERVER["SERVER_SIGNATURE"] . creer_uniqid()), 'non');
-		lire_metas(); // au cas ou ecrire_meta() ne fonctionne pas
-	}
-	return $GLOBALS['meta']['secret_du_site'];
+    if (!isset($GLOBALS['meta']['secret_du_site'])){
+        include_spip('base/abstract_sql');
+        $GLOBALS['meta']['secret_du_site'] = sql_getfetsel('valeur', 'spip_meta', "nom='secret_du_site'");
+    }
+    if (!isset($GLOBALS['meta']['secret_du_site'])
+      OR (strlen($GLOBALS['meta']['secret_du_site'])<64)) {
+        include_spip('inc/acces');
+        include_spip('auth/sha256.inc');
+        ecrire_meta('secret_du_site', _nano_sha256($_SERVER["DOCUMENT_ROOT"] . $_SERVER["SERVER_SIGNATURE"] . creer_uniqid()), 'non');
+        lire_metas(); // au cas ou ecrire_meta() ne fonctionne pas
+    }
+    return $GLOBALS['meta']['secret_du_site'];
 }
 
 /**
@@ -241,10 +241,10 @@ 
  * @return string
  */
 function calculer_cle_action($action) {
-	if (function_exists('sha1'))
-		return sha1($action . secret_du_site());
-	else
-		return md5($action . secret_du_site());
+    if (function_exists('sha1'))
+        return sha1($action . secret_du_site());
+    else
+        return md5($action . secret_du_site());
 }
 
 /**
@@ -256,7 +256,7 @@ 
  * @return bool
  */
 function verifier_cle_action($action, $cle) {
-	return ($cle == calculer_cle_action($action));
+    return ($cle == calculer_cle_action($action));
 }
 
 ?>

Braces +47 added lines, -32 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * interface d'appel:
@@ -34,14 +36,15 @@ 
  */
 function inc_securiser_action_dist($action='', $arg='', $redirect="", $mode=false, $att='', $public=false)
 {
-	if ($action)
-		return securiser_action_auteur($action, $arg, $redirect, $mode, $att, $public);
-	else {
+	if ($action) {
+			return securiser_action_auteur($action, $arg, $redirect, $mode, $att, $public);
+	} else {
 		$arg = _request('arg');
 		$hash = _request('hash');
 		$action = _request('action')?_request('action'):_request('formulaire_action');
-		if ($a = verifier_action_auteur("$action-$arg", $hash))
-			return $arg;
+		if ($a = verifier_action_auteur("$action-$arg", $hash)) {
+					return $arg;
+		}
 		include_spip('inc/minipres');
 		echo minipres();
 		exit;
@@ -73,17 +76,19 @@ 
 		$hash = calculer_action_auteur("$action-$arg",is_numeric($att)?$att:null);
 
 		$r = rawurlencode($redirect);
-		if ($mode===-1)
-			return array('action'=>$action,'arg'=>$arg,'hash'=>$hash);
-		else
-			return generer_url_action($action, "arg=$arg&hash=$hash" . (!$r ? '' : "&redirect=$r"), $mode, $public);
+		if ($mode===-1) {
+					return array('action'=>$action,'arg'=>$arg,'hash'=>$hash);
+		} else {
+					return generer_url_action($action, "arg=$arg&hash=$hash" . (!$r ? '' : "&redirect=$r"), $mode, $public);
+		}
 	}
 
 	// mode formulaire
 	$hash = calculer_action_auteur("$action-$arg");
 	$att .= " style='margin: 0px; border: 0px'";
-	if ($redirect)
-		$redirect = "\n\t\t<input name='redirect' type='hidden' value='". str_replace("'", '&#39;', $redirect) ."' />";
+	if ($redirect) {
+			$redirect = "\n\t\t<input name='redirect' type='hidden' value='". str_replace("'", '&#39;', $redirect) ."' />";
+	}
 	$mode .= $redirect . "
 <input name='hash' type='hidden' value='$hash' />
 <input name='arg' type='hidden' value='$arg' />";
@@ -111,22 +116,28 @@ 
 			  return array($r[1], '');
 			  // Necessaire aux forums anonymes.
 			  // Pour le reste, ca echouera.
-		} else return array('0','');
+		} else {
+		    return array('0','');
+		}
 	}
 	// Eviter l'acces SQL si le pass est connu de PHP
 	if (is_null($id_auteur)){
 		$id_auteur = isset($GLOBALS['visiteur_session']['id_auteur'])?$GLOBALS['visiteur_session']['id_auteur']:0;
-		if (isset($GLOBALS['visiteur_session']['pass']) AND $GLOBALS['visiteur_session']['pass'])
-			return $caracterisation[$id_auteur] = array($id_auteur, $GLOBALS['visiteur_session']['pass']);
+		if (isset($GLOBALS['visiteur_session']['pass']) AND $GLOBALS['visiteur_session']['pass']) {
+					return $caracterisation[$id_auteur] = array($id_auteur, $GLOBALS['visiteur_session']['pass']);
+		}
 	}
 
-	if (isset($caracterisation[$id_auteur])) return $caracterisation[$id_auteur];
+	if (isset($caracterisation[$id_auteur])) {
+	    return $caracterisation[$id_auteur];
+	}
 
 	if ($id_auteur) {
 		include_spip('base/abstract_sql');
 		$t = sql_fetsel("id_auteur, pass", "spip_auteurs", "id_auteur=$id_auteur");
-		if ($t)
-			return $caracterisation[$id_auteur] = array($t['id_auteur'], $t['pass']);
+		if ($t) {
+					return $caracterisation[$id_auteur] = array($t['id_auteur'], $t['pass']);
+		}
 		include_spip('inc/minipres');
 		echo minipres();
 		exit;
@@ -166,11 +177,12 @@ 
 		include_spip('auth/sha256.inc');
 		$sha[$id_auteur.$pass.$alea] = _nano_sha256($id_auteur.$pass.@$GLOBALS['meta'][$alea]);
 	}
-	if (function_exists('sha1'))
-		return sha1($action.$sha[$id_auteur.$pass.$alea]);
-	else
-		return md5($action.$sha[$id_auteur.$pass.$alea]);
-}
+	if (function_exists('sha1')) {
+			return sha1($action.$sha[$id_auteur.$pass.$alea]);
+	} else {
+			return md5($action.$sha[$id_auteur.$pass.$alea]);
+	}
+	}
 
 /**
  * Calculer le hash qui signe une action pour un auteur
@@ -197,10 +209,12 @@ 
  */
 function verifier_action_auteur($action, $hash) {
 	list($id_auteur, $pass) = caracteriser_auteur();
-	if ($hash == _action_auteur($action, $id_auteur, $pass, 'alea_ephemere'))
-		return true;
-	if ($hash == _action_auteur($action, $id_auteur, $pass, 'alea_ephemere_ancien'))
-		return true;
+	if ($hash == _action_auteur($action, $id_auteur, $pass, 'alea_ephemere')) {
+			return true;
+	}
+	if ($hash == _action_auteur($action, $id_auteur, $pass, 'alea_ephemere_ancien')) {
+			return true;
+	}
 	return false;
 }
 
@@ -241,11 +255,12 @@ 
  * @return string
  */
 function calculer_cle_action($action) {
-	if (function_exists('sha1'))
-		return sha1($action . secret_du_site());
-	else
-		return md5($action . secret_du_site());
-}
+	if (function_exists('sha1')) {
+			return sha1($action . secret_du_site());
+	} else {
+			return md5($action . secret_du_site());
+	}
+	}
 
 /**
  * Verifier la cle de signature d'une action valable pour le site

Spacing +25 added lines, -25 removed lines

@@ -32,14 +32,14 @@ 
  * @param bool $public
  * @return array|string
  */
-function inc_securiser_action_dist($action='', $arg='', $redirect="", $mode=false, $att='', $public=false)
+function inc_securiser_action_dist($action = '', $arg = '', $redirect = "", $mode = false, $att = '', $public = false)
 {
 	if ($action)
 		return securiser_action_auteur($action, $arg, $redirect, $mode, $att, $public);
 	else {
 		$arg = _request('arg');
 		$hash = _request('hash');
-		$action = _request('action')?_request('action'):_request('formulaire_action');
+		$action = _request('action') ?_request('action') : _request('formulaire_action');
 		if ($a = verifier_action_auteur("$action-$arg", $hash))
 			return $arg;
 		include_spip('inc/minipres');
@@ -66,25 +66,25 @@ 
  * @param bool $public
  * @return array|string
  */
-function securiser_action_auteur($action, $arg, $redirect="", $mode=false, $att='', $public=false) {
+function securiser_action_auteur($action, $arg, $redirect = "", $mode = false, $att = '', $public = false) {
 
 	// mode URL ou array
-	if (!is_string($mode)){
-		$hash = calculer_action_auteur("$action-$arg",is_numeric($att)?$att:null);
+	if (!is_string($mode)) {
+		$hash = calculer_action_auteur("$action-$arg", is_numeric($att) ? $att : null);
 
 		$r = rawurlencode($redirect);
-		if ($mode===-1)
-			return array('action'=>$action,'arg'=>$arg,'hash'=>$hash);
+		if ($mode === -1)
+			return array('action'=>$action, 'arg'=>$arg, 'hash'=>$hash);
 		else
-			return generer_url_action($action, "arg=$arg&hash=$hash" . (!$r ? '' : "&redirect=$r"), $mode, $public);
+			return generer_url_action($action, "arg=$arg&hash=$hash".(!$r ? '' : "&redirect=$r"), $mode, $public);
 	}
 
 	// mode formulaire
 	$hash = calculer_action_auteur("$action-$arg");
 	$att .= " style='margin: 0px; border: 0px'";
 	if ($redirect)
-		$redirect = "\n\t\t<input name='redirect' type='hidden' value='". str_replace("'", '&#39;', $redirect) ."' />";
-	$mode .= $redirect . "
+		$redirect = "\n\t\t<input name='redirect' type='hidden' value='".str_replace("'", '&#39;', $redirect)."' />";
+	$mode .= $redirect."
 <input name='hash' type='hidden' value='$hash' />
 <input name='arg' type='hidden' value='$arg' />";
 
@@ -99,7 +99,7 @@ 
  * @param int|null $id_auteur
  * @return array
  */
-function caracteriser_auteur($id_auteur=null) {
+function caracteriser_auteur($id_auteur = null) {
 	static $caracterisation = array();
 
 	if (is_null($id_auteur) AND !isset($GLOBALS['visiteur_session']['id_auteur'])) {
@@ -107,15 +107,15 @@ 
 	// c'est une connexion par php_auth ou 1 instal, on se rabat sur le cookie.
 	// S'il n'avait pas le droit de realiser cette action, le hash sera faux.
 		if (isset($_COOKIE['spip_session'])
-		AND (preg_match('/^(\d+)/',$_COOKIE['spip_session'],$r))) {
+		AND (preg_match('/^(\d+)/', $_COOKIE['spip_session'], $r))) {
 			  return array($r[1], '');
 			  // Necessaire aux forums anonymes.
 			  // Pour le reste, ca echouera.
-		} else return array('0','');
+		} else return array('0', '');
 	}
 	// Eviter l'acces SQL si le pass est connu de PHP
-	if (is_null($id_auteur)){
-		$id_auteur = isset($GLOBALS['visiteur_session']['id_auteur'])?$GLOBALS['visiteur_session']['id_auteur']:0;
+	if (is_null($id_auteur)) {
+		$id_auteur = isset($GLOBALS['visiteur_session']['id_auteur']) ? $GLOBALS['visiteur_session']['id_auteur'] : 0;
 		if (isset($GLOBALS['visiteur_session']['pass']) AND $GLOBALS['visiteur_session']['pass'])
 			return $caracterisation[$id_auteur] = array($id_auteur, $GLOBALS['visiteur_session']['pass']);
 	}
@@ -133,7 +133,7 @@ 
 	}
 	// Visiteur anonyme, pour ls forums par exemple
 	else {
-		return array('0','');
+		return array('0', '');
 	}
 }
 
@@ -152,10 +152,10 @@ 
  */
 function _action_auteur($action, $id_auteur, $pass, $alea) {
 	static $sha = array();
-	if (!isset($sha[$id_auteur.$pass.$alea])){
-		if (!isset($GLOBALS['meta'][$alea]) AND _request('exec')!=='install') {
+	if (!isset($sha[$id_auteur.$pass.$alea])) {
+		if (!isset($GLOBALS['meta'][$alea]) AND _request('exec') !== 'install') {
 			include_spip('base/abstract_sql');
-			$GLOBALS['meta'][$alea] = sql_getfetsel('valeur', 'spip_meta', "nom=" . sql_quote($alea));
+			$GLOBALS['meta'][$alea] = sql_getfetsel('valeur', 'spip_meta', "nom=".sql_quote($alea));
 			if (!($GLOBALS['meta'][$alea])) {
 				include_spip('inc/minipres');
 				echo minipres();
@@ -180,7 +180,7 @@ 
  * @param int|null $id_auteur
  * @return string
  */
-function calculer_action_auteur($action, $id_auteur=null) {
+function calculer_action_auteur($action, $id_auteur = null) {
 	list($id_auteur, $pass) = caracteriser_auteur($id_auteur);
 	return _action_auteur($action, $id_auteur, $pass, 'alea_ephemere');
 }
@@ -219,15 +219,15 @@ 
  * @return string
  */
 function secret_du_site() {
-	if (!isset($GLOBALS['meta']['secret_du_site'])){
+	if (!isset($GLOBALS['meta']['secret_du_site'])) {
 		include_spip('base/abstract_sql');
 		$GLOBALS['meta']['secret_du_site'] = sql_getfetsel('valeur', 'spip_meta', "nom='secret_du_site'");
 	}
 	if (!isset($GLOBALS['meta']['secret_du_site'])
-	  OR (strlen($GLOBALS['meta']['secret_du_site'])<64)) {
+	  OR (strlen($GLOBALS['meta']['secret_du_site']) < 64)) {
 		include_spip('inc/acces');
 		include_spip('auth/sha256.inc');
-		ecrire_meta('secret_du_site', _nano_sha256($_SERVER["DOCUMENT_ROOT"] . $_SERVER["SERVER_SIGNATURE"] . creer_uniqid()), 'non');
+		ecrire_meta('secret_du_site', _nano_sha256($_SERVER["DOCUMENT_ROOT"].$_SERVER["SERVER_SIGNATURE"].creer_uniqid()), 'non');
 		lire_metas(); // au cas ou ecrire_meta() ne fonctionne pas
 	}
 	return $GLOBALS['meta']['secret_du_site'];
@@ -242,9 +242,9 @@ 
  */
 function calculer_cle_action($action) {
 	if (function_exists('sha1'))
-		return sha1($action . secret_du_site());
+		return sha1($action.secret_du_site());
 	else
-		return md5($action . secret_du_site());
+		return md5($action.secret_du_site());
 }
 
 /**

ecrire/inc/layer.php

Indentation +87 added lines, -87 removed lines

@@ -14,69 +14,69 @@ 
 
 // http://doc.spip.org/@cadre_depliable
 function cadre_depliable($icone,$titre,$deplie,$contenu,$ids='',$style_cadre='r'){
-	$bouton = bouton_block_depliable($titre,$deplie,$ids);
-	return 
-		debut_cadre($style_cadre,$icone,'',$bouton, '', '', false)
-		. debut_block_depliable($deplie,$ids)
-		. "<div class='cadre_padding'>\n"
-		. $contenu
-		. "</div>\n"
-		. fin_block()
-		. fin_cadre();
+    $bouton = bouton_block_depliable($titre,$deplie,$ids);
+    return 
+        debut_cadre($style_cadre,$icone,'',$bouton, '', '', false)
+        . debut_block_depliable($deplie,$ids)
+        . "<div class='cadre_padding'>\n"
+        . $contenu
+        . "</div>\n"
+        . fin_block()
+        . fin_cadre();
 }
 
 // http://doc.spip.org/@block_parfois_visible
 function block_parfois_visible($nom, $invite, $masque, $style='', $visible=false){
-	return "\n"
-	. bouton_block_depliable($invite,$visible,$nom)
-	. debut_block_depliable($visible,$nom)
-	. $masque
-	. fin_block();
+    return "\n"
+    . bouton_block_depliable($invite,$visible,$nom)
+    . debut_block_depliable($visible,$nom)
+    . $masque
+    . fin_block();
 }
 
 // http://doc.spip.org/@debut_block_depliable
 function debut_block_depliable($deplie,$id=""){
-	$class=' blocdeplie';
-	// si on n'accepte pas js, ne pas fermer
-	if (!$deplie)
-		$class=" blocreplie";
-	return "<div ".($id?"id='$id' ":"")."class='bloc_depliable$class'>";	
+    $class=' blocdeplie';
+    // si on n'accepte pas js, ne pas fermer
+    if (!$deplie)
+        $class=" blocreplie";
+    return "<div ".($id?"id='$id' ":"")."class='bloc_depliable$class'>";	
 }
 // http://doc.spip.org/@fin_block
 function fin_block() {
-	return "<div class='nettoyeur'></div>\n</div>";
+    return "<div class='nettoyeur'></div>\n</div>";
 }
 // $texte : texte du bouton
 // $deplie : true (deplie) ou false (plie) ou -1 (inactif) ou 'incertain' pour que le bouton s'auto init au chargement de la page 
 // $ids : id des div lies au bouton (facultatif, par defaut c'est le div.bloc_depliable qui suit)
 // http://doc.spip.org/@bouton_block_depliable
 function bouton_block_depliable($texte,$deplie,$ids=""){
-	$bouton_id = 'b'.substr(md5($texte.microtime()),0,8);
+    $bouton_id = 'b'.substr(md5($texte.microtime()),0,8);
 
-	$class = ($deplie===true)?" deplie":(($deplie==-1)?" impliable":" replie");
-	if (strlen($ids)){
-		$cible = explode(',',$ids);
-		$cible = '#'.implode(",#",$cible);
-	}
-	else{
-		$cible = "#$bouton_id + div.bloc_depliable";
-	}
+    $class = ($deplie===true)?" deplie":(($deplie==-1)?" impliable":" replie");
+    if (strlen($ids)){
+        $cible = explode(',',$ids);
+        $cible = '#'.implode(",#",$cible);
+    }
+    else{
+        $cible = "#$bouton_id + div.bloc_depliable";
+    }
 
-	$b = (strpos($texte,"<h")===false?'h3':'div');
-	return "<$b "
-	  .($bouton_id?"id='$bouton_id' ":"")
-	  ."class='titrem$class'"
-	  . (($deplie===-1)
-	  	?""
-	  	:" onmouseover=\"jQuery(this).depliant('$cible');\""
-	  )
-	  .">"
-	  // une ancre pour rendre accessible au clavier le depliage du sous bloc
-	  . "<a href='#' onclick=\"return jQuery(this).depliant_clicancre('$cible');\" class='titremancre'></a>"
-	  . "$texte</$b>"
-	  . http_script( ($deplie==='incertain')
-			? "jQuery(document).ready(function(){if (jQuery('$cible').is(':visible')) $('#$bouton_id').addClass('deplie').removeClass('replie');});"
-			: '');
+    $b = (strpos($texte,"<h")===false?'h3':'div');
+    return "<$b "
+        .($bouton_id?"id='$bouton_id' ":"")
+        ."class='titrem$class'"
+        . (($deplie===-1)
+          ?""
+          :" onmouseover=\"jQuery(this).depliant('$cible');\""
+        )
+        .">"
+        // une ancre pour rendre accessible au clavier le depliage du sous bloc
+        . "<a href='#' onclick=\"return jQuery(this).depliant_clicancre('$cible');\" class='titremancre'></a>"
+        . "$texte</$b>"
+        . http_script( ($deplie==='incertain')
+            ? "jQuery(document).ready(function(){if (jQuery('$cible').is(':visible')) $('#$bouton_id').addClass('deplie').removeClass('replie');});"
+            : '');
 }
 
 //
@@ -85,51 +85,51 @@ 
 // http://doc.spip.org/@verif_butineur
 function verif_butineur() {
 
-	global $browser_name, $browser_version;
-	global $browser_description, $browser_rev, $browser_barre;
-	preg_match(",^([A-Za-z]+)/([0-9]+\.[0-9]+) (.*)$,", $_SERVER['HTTP_USER_AGENT'], $match);
-	$browser_name = $match[1];
-	$browser_version = $match[2];
-	$browser_description = $match[3];
-	$GLOBALS['browser_layer'] = ' '; // compat avec vieux scripts qui testent la valeur
-	$browser_barre = '';
+    global $browser_name, $browser_version;
+    global $browser_description, $browser_rev, $browser_barre;
+    preg_match(",^([A-Za-z]+)/([0-9]+\.[0-9]+) (.*)$,", $_SERVER['HTTP_USER_AGENT'], $match);
+    $browser_name = $match[1];
+    $browser_version = $match[2];
+    $browser_description = $match[3];
+    $GLOBALS['browser_layer'] = ' '; // compat avec vieux scripts qui testent la valeur
+    $browser_barre = '';
 
-	if (!preg_match(",opera,i", $browser_description)&&preg_match(",opera,i", $browser_name)) {
-		$browser_name = "Opera";
-		$browser_version = $match[2];
-		$browser_barre = ($browser_version >= 8.5);
-	}
-	else if (preg_match(",opera,i", $browser_description)) {
-		preg_match(",Opera ([^\ ]*),i", $browser_description, $match);
-		$browser_name = "Opera";
-		$browser_version = $match[1];
-		$browser_barre = ($browser_version >= 8.5);
-	}
-	else if (preg_match(",msie,i", $browser_description)) {
-		preg_match(",MSIE ([^;]*),i", $browser_description, $match);
-		$browser_name = "MSIE";
-		$browser_version = $match[1];
-		$browser_barre = ($browser_version >= 5.5);
-	}
-	else if (preg_match(",KHTML,i", $browser_description) &&
-		preg_match(",Safari/([^;]*),", $browser_description, $match)) {
-		$browser_name = "Safari";
-		$browser_version = $match[1];
-		$browser_barre = ($browser_version >= 5.0);
-	}
-	else if (preg_match(",mozilla,i", $browser_name) AND $browser_version >= 5) {
-		// Numero de version pour Mozilla "authentique"
-		if (preg_match(",rv:([0-9]+\.[0-9]+),", $browser_description, $match))
-			$browser_rev = doubleval($match[1]);
-		// Autres Gecko => equivalents 1.4 par defaut (Galeon, etc.)
-		else if (strpos($browser_description, "Gecko") and !strpos($browser_description, "KHTML"))
-			$browser_rev = 1.4;
-		// Machins quelconques => equivalents 1.0 par defaut (Konqueror, etc.)
-		else $browser_rev = 1.0;
-		$browser_barre = $browser_rev >= 1.3;
-	}
+    if (!preg_match(",opera,i", $browser_description)&&preg_match(",opera,i", $browser_name)) {
+        $browser_name = "Opera";
+        $browser_version = $match[2];
+        $browser_barre = ($browser_version >= 8.5);
+    }
+    else if (preg_match(",opera,i", $browser_description)) {
+        preg_match(",Opera ([^\ ]*),i", $browser_description, $match);
+        $browser_name = "Opera";
+        $browser_version = $match[1];
+        $browser_barre = ($browser_version >= 8.5);
+    }
+    else if (preg_match(",msie,i", $browser_description)) {
+        preg_match(",MSIE ([^;]*),i", $browser_description, $match);
+        $browser_name = "MSIE";
+        $browser_version = $match[1];
+        $browser_barre = ($browser_version >= 5.5);
+    }
+    else if (preg_match(",KHTML,i", $browser_description) &&
+        preg_match(",Safari/([^;]*),", $browser_description, $match)) {
+        $browser_name = "Safari";
+        $browser_version = $match[1];
+        $browser_barre = ($browser_version >= 5.0);
+    }
+    else if (preg_match(",mozilla,i", $browser_name) AND $browser_version >= 5) {
+        // Numero de version pour Mozilla "authentique"
+        if (preg_match(",rv:([0-9]+\.[0-9]+),", $browser_description, $match))
+            $browser_rev = doubleval($match[1]);
+        // Autres Gecko => equivalents 1.4 par defaut (Galeon, etc.)
+        else if (strpos($browser_description, "Gecko") and !strpos($browser_description, "KHTML"))
+            $browser_rev = 1.4;
+        // Machins quelconques => equivalents 1.0 par defaut (Konqueror, etc.)
+        else $browser_rev = 1.0;
+        $browser_barre = $browser_rev >= 1.3;
+    }
 
-	if (!$browser_name) $browser_name = "Mozilla";
+    if (!$browser_name) $browser_name = "Mozilla";
 }
 
 verif_butineur();

Spacing +24 added lines, -24 removed lines

@@ -13,11 +13,11 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 // http://doc.spip.org/@cadre_depliable
-function cadre_depliable($icone,$titre,$deplie,$contenu,$ids='',$style_cadre='r'){
-	$bouton = bouton_block_depliable($titre,$deplie,$ids);
+function cadre_depliable($icone, $titre, $deplie, $contenu, $ids = '', $style_cadre = 'r') {
+	$bouton = bouton_block_depliable($titre, $deplie, $ids);
 	return 
-		debut_cadre($style_cadre,$icone,'',$bouton, '', '', false)
-		. debut_block_depliable($deplie,$ids)
+		debut_cadre($style_cadre, $icone, '', $bouton, '', '', false)
+		. debut_block_depliable($deplie, $ids)
 		. "<div class='cadre_padding'>\n"
 		. $contenu
 		. "</div>\n"
@@ -26,21 +26,21 @@ 
 }
 
 // http://doc.spip.org/@block_parfois_visible
-function block_parfois_visible($nom, $invite, $masque, $style='', $visible=false){
+function block_parfois_visible($nom, $invite, $masque, $style = '', $visible = false) {
 	return "\n"
-	. bouton_block_depliable($invite,$visible,$nom)
-	. debut_block_depliable($visible,$nom)
+	. bouton_block_depliable($invite, $visible, $nom)
+	. debut_block_depliable($visible, $nom)
 	. $masque
 	. fin_block();
 }
 
 // http://doc.spip.org/@debut_block_depliable
-function debut_block_depliable($deplie,$id=""){
-	$class=' blocdeplie';
+function debut_block_depliable($deplie, $id = "") {
+	$class = ' blocdeplie';
 	// si on n'accepte pas js, ne pas fermer
 	if (!$deplie)
-		$class=" blocreplie";
-	return "<div ".($id?"id='$id' ":"")."class='bloc_depliable$class'>";	
+		$class = " blocreplie";
+	return "<div ".($id ? "id='$id' " : "")."class='bloc_depliable$class'>";	
 }
 // http://doc.spip.org/@fin_block
 function fin_block() {
@@ -50,31 +50,31 @@ 
 // $deplie : true (deplie) ou false (plie) ou -1 (inactif) ou 'incertain' pour que le bouton s'auto init au chargement de la page 
 // $ids : id des div lies au bouton (facultatif, par defaut c'est le div.bloc_depliable qui suit)
 // http://doc.spip.org/@bouton_block_depliable
-function bouton_block_depliable($texte,$deplie,$ids=""){
-	$bouton_id = 'b'.substr(md5($texte.microtime()),0,8);
+function bouton_block_depliable($texte, $deplie, $ids = "") {
+	$bouton_id = 'b'.substr(md5($texte.microtime()), 0, 8);
 
-	$class = ($deplie===true)?" deplie":(($deplie==-1)?" impliable":" replie");
-	if (strlen($ids)){
-		$cible = explode(',',$ids);
-		$cible = '#'.implode(",#",$cible);
+	$class = ($deplie === true) ? " deplie" : (($deplie == -1) ? " impliable" : " replie");
+	if (strlen($ids)) {
+		$cible = explode(',', $ids);
+		$cible = '#'.implode(",#", $cible);
 	}
-	else{
+	else {
 		$cible = "#$bouton_id + div.bloc_depliable";
 	}
 
-	$b = (strpos($texte,"<h")===false?'h3':'div');
+	$b = (strpos($texte, "<h") === false ? 'h3' : 'div');
 	return "<$b "
-	  .($bouton_id?"id='$bouton_id' ":"")
+	  .($bouton_id ? "id='$bouton_id' " : "")
 	  ."class='titrem$class'"
-	  . (($deplie===-1)
-	  	?""
+	  . (($deplie === -1)
+	  	? ""
 	  	:" onmouseover=\"jQuery(this).depliant('$cible');\""
 	  )
 	  .">"
 	  // une ancre pour rendre accessible au clavier le depliage du sous bloc
 	  . "<a href='#' onclick=\"return jQuery(this).depliant_clicancre('$cible');\" class='titremancre'></a>"
 	  . "$texte</$b>"
-	  . http_script( ($deplie==='incertain')
+	  . http_script(($deplie === 'incertain')
 			? "jQuery(document).ready(function(){if (jQuery('$cible').is(':visible')) $('#$bouton_id').addClass('deplie').removeClass('replie');});"
 			: '');
 }
@@ -94,7 +94,7 @@ 
 	$GLOBALS['browser_layer'] = ' '; // compat avec vieux scripts qui testent la valeur
 	$browser_barre = '';
 
-	if (!preg_match(",opera,i", $browser_description)&&preg_match(",opera,i", $browser_name)) {
+	if (!preg_match(",opera,i", $browser_description) && preg_match(",opera,i", $browser_name)) {
 		$browser_name = "Opera";
 		$browser_version = $match[2];
 		$browser_barre = ($browser_version >= 8.5);

Braces +24 added lines, -20 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // http://doc.spip.org/@cadre_depliable
 function cadre_depliable($icone,$titre,$deplie,$contenu,$ids='',$style_cadre='r'){
@@ -38,8 +40,9 @@ 
 function debut_block_depliable($deplie,$id=""){
 	$class=' blocdeplie';
 	// si on n'accepte pas js, ne pas fermer
-	if (!$deplie)
-		$class=" blocreplie";
+	if (!$deplie) {
+			$class=" blocreplie";
+	}
 	return "<div ".($id?"id='$id' ":"")."class='bloc_depliable$class'>";	
 }
 // http://doc.spip.org/@fin_block
@@ -57,8 +60,7 @@ 
 	if (strlen($ids)){
 		$cible = explode(',',$ids);
 		$cible = '#'.implode(",#",$cible);
-	}
-	else{
+	} else{
 		$cible = "#$bouton_id + div.bloc_depliable";
 	}
 
@@ -98,39 +100,41 @@ 
 		$browser_name = "Opera";
 		$browser_version = $match[2];
 		$browser_barre = ($browser_version >= 8.5);
-	}
-	else if (preg_match(",opera,i", $browser_description)) {
+	} else if (preg_match(",opera,i", $browser_description)) {
 		preg_match(",Opera ([^\ ]*),i", $browser_description, $match);
 		$browser_name = "Opera";
 		$browser_version = $match[1];
 		$browser_barre = ($browser_version >= 8.5);
-	}
-	else if (preg_match(",msie,i", $browser_description)) {
+	} else if (preg_match(",msie,i", $browser_description)) {
 		preg_match(",MSIE ([^;]*),i", $browser_description, $match);
 		$browser_name = "MSIE";
 		$browser_version = $match[1];
 		$browser_barre = ($browser_version >= 5.5);
-	}
-	else if (preg_match(",KHTML,i", $browser_description) &&
+	} else if (preg_match(",KHTML,i", $browser_description) &&
 		preg_match(",Safari/([^;]*),", $browser_description, $match)) {
 		$browser_name = "Safari";
 		$browser_version = $match[1];
 		$browser_barre = ($browser_version >= 5.0);
-	}
-	else if (preg_match(",mozilla,i", $browser_name) AND $browser_version >= 5) {
+	} else if (preg_match(",mozilla,i", $browser_name) AND $browser_version >= 5) {
 		// Numero de version pour Mozilla "authentique"
-		if (preg_match(",rv:([0-9]+\.[0-9]+),", $browser_description, $match))
-			$browser_rev = doubleval($match[1]);
+		if (preg_match(",rv:([0-9]+\.[0-9]+),", $browser_description, $match)) {
+					$browser_rev = doubleval($match[1]);
+		}
 		// Autres Gecko => equivalents 1.4 par defaut (Galeon, etc.)
-		else if (strpos($browser_description, "Gecko") and !strpos($browser_description, "KHTML"))
-			$browser_rev = 1.4;
+		else if (strpos($browser_description, "Gecko") and !strpos($browser_description, "KHTML")) {
+					$browser_rev = 1.4;
+		}
 		// Machins quelconques => equivalents 1.0 par defaut (Konqueror, etc.)
-		else $browser_rev = 1.0;
+		else {
+		    $browser_rev = 1.0;
+		}
 		$browser_barre = $browser_rev >= 1.3;
 	}
 
-	if (!$browser_name) $browser_name = "Mozilla";
-}
+	if (!$browser_name) {
+	    $browser_name = "Mozilla";
+	}
+	}
 
 verif_butineur();
 ?>

ecrire/inc/charger_php_extension.php

Indentation +43 added lines, -43 removed lines

@@ -16,53 +16,53 @@
 //
 // http://doc.spip.org/@inc_charger_php_extension_dist
 function inc_charger_php_extension_dist($module){
-	if (extension_loaded($module)) {
-		return true;
-	}
+    if (extension_loaded($module)) {
+        return true;
+    }
 
-	// A-t-on le droit de faire un dl() ; si on peut, on memorise la reponse,
-	// lourde a calculer, dans les meta
-	if (!isset($GLOBALS['meta']['dl_allowed'])) {
-		if (!@ini_get('safe_mode') 
-		  && @ini_get('enable_dl')
-		  && @function_exists('dl')) {
-			ob_start();
-			phpinfo(INFO_GENERAL); /* Only general info */
-			$a = strip_tags(ob_get_contents());
-			ob_end_clean();
-			if (preg_match('@Thread Safety[[:space:]]*enabled@', $a)) {
-				if (preg_match('@Server API[[:space:]]*\(CGI\|CLI\)@', $a)) {
-					$GLOBALS['meta']['dl_allowed'] = true;
-				} else {
-					$GLOBALS['meta']['dl_allowed'] = false;
-				}
-			} else {
-				$GLOBALS['meta']['dl_allowed'] = true;
-			}
-		} else {
-			$GLOBALS['meta']['dl_allowed'] = false;
-		}
+    // A-t-on le droit de faire un dl() ; si on peut, on memorise la reponse,
+    // lourde a calculer, dans les meta
+    if (!isset($GLOBALS['meta']['dl_allowed'])) {
+        if (!@ini_get('safe_mode') 
+          && @ini_get('enable_dl')
+          && @function_exists('dl')) {
+            ob_start();
+            phpinfo(INFO_GENERAL); /* Only general info */
+            $a = strip_tags(ob_get_contents());
+            ob_end_clean();
+            if (preg_match('@Thread Safety[[:space:]]*enabled@', $a)) {
+                if (preg_match('@Server API[[:space:]]*\(CGI\|CLI\)@', $a)) {
+                    $GLOBALS['meta']['dl_allowed'] = true;
+                } else {
+                    $GLOBALS['meta']['dl_allowed'] = false;
+                }
+            } else {
+                $GLOBALS['meta']['dl_allowed'] = true;
+            }
+        } else {
+            $GLOBALS['meta']['dl_allowed'] = false;
+        }
 
-		// Attention, le ecrire_meta() echouera si on le tente ici ;
-		// donc on ne fait rien, et on attend qu'un prochain ecrire_meta()
-		// se produisant apres cette sequence enregistre sa valeur.
-		#include_spip('inc/meta');
-		#ecrire_meta('dl_allowed', $GLOBALS['meta']['dl_allowed'], 'non');
-	}
+        // Attention, le ecrire_meta() echouera si on le tente ici ;
+        // donc on ne fait rien, et on attend qu'un prochain ecrire_meta()
+        // se produisant apres cette sequence enregistre sa valeur.
+        #include_spip('inc/meta');
+        #ecrire_meta('dl_allowed', $GLOBALS['meta']['dl_allowed'], 'non');
+    }
 
-	if (!$GLOBALS['meta']['dl_allowed']) {
-		return false;
-	}
+    if (!$GLOBALS['meta']['dl_allowed']) {
+        return false;
+    }
 
-	/* Once we require PHP >= 4.3, we might use PHP_SHLIB_SUFFIX here */
-	if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
-		$module_file = 'php_' . $module . '.dll';
-	} elseif (PHP_OS=='HP-UX') {
-		$module_file = $module . '.sl';
-	} else {
-		$module_file = $module . '.so';
-	}
+    /* Once we require PHP >= 4.3, we might use PHP_SHLIB_SUFFIX here */
+    if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
+        $module_file = 'php_' . $module . '.dll';
+    } elseif (PHP_OS=='HP-UX') {
+        $module_file = $module . '.sl';
+    } else {
+        $module_file = $module . '.so';
+    }
 
-	return @dl($module_file);
+    return @dl($module_file);
 }
 ?>

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
 // - 3) si l'on peut, on charge le module par la fonction dl()
 //
 // http://doc.spip.org/@inc_charger_php_extension_dist
-function inc_charger_php_extension_dist($module){
+function inc_charger_php_extension_dist($module) {
 	if (extension_loaded($module)) {
 		return true;
 	}
@@ -56,11 +56,11 @@ 
 
 	/* Once we require PHP >= 4.3, we might use PHP_SHLIB_SUFFIX here */
 	if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
-		$module_file = 'php_' . $module . '.dll';
-	} elseif (PHP_OS=='HP-UX') {
-		$module_file = $module . '.sl';
+		$module_file = 'php_'.$module.'.dll';
+	} elseif (PHP_OS == 'HP-UX') {
+		$module_file = $module.'.sl';
 	} else {
-		$module_file = $module . '.so';
+		$module_file = $module.'.so';
 	}
 
 	return @dl($module_file);

Braces +3 added lines, -1 removed lines

@@ -1,6 +1,8 @@
 <?php
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // cette fonction (adaptee de phpMyAdmin)
 // permet de charger un module php