spip / SPIP

ecrire/action/converser.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 		action_converser_post($lang);
 	} elseif ($lang = _request('var_lang_ecrire')) {
 		if ($update_session) {
-			sql_updateq('spip_auteurs', ['lang' => $lang], 'id_auteur = ' . $GLOBALS['visiteur_session']['id_auteur']);
+			sql_updateq('spip_auteurs', ['lang' => $lang], 'id_auteur = '.$GLOBALS['visiteur_session']['id_auteur']);
 			$GLOBALS['visiteur_session']['lang'] = $lang;
 			$session = charger_fonction('session', 'inc');
 			if ($spip_session = $session($GLOBALS['visiteur_session'])) {

Indentation +43 added lines, -43 removed lines

@@ -16,7 +16,7 @@ 
  * @package SPIP\Core\Langue
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/cookie');
@@ -30,21 +30,21 @@ 
  * @return void
  */
 function action_converser_dist() {
-	$update_session = false;
-	if (_request('arg') and spip_connect()) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$securiser_action();
-		$update_session = true;
-	}
+    $update_session = false;
+    if (_request('arg') and spip_connect()) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $securiser_action();
+        $update_session = true;
+    }
 
-	$lang = action_converser_changer_langue($update_session);
-	$redirect = rawurldecode(_request('redirect'));
+    $lang = action_converser_changer_langue($update_session);
+    $redirect = rawurldecode(_request('redirect'));
 
-	if (!$redirect) {
-		$redirect = _DIR_RESTREINT_ABS;
-	}
-	$redirect = parametre_url($redirect, 'lang', $lang, '&');
-	redirige_par_entete($redirect, true);
+    if (!$redirect) {
+        $redirect = _DIR_RESTREINT_ABS;
+    }
+    $redirect = parametre_url($redirect, 'lang', $lang, '&');
+    redirige_par_entete($redirect, true);
 }
 
 /**
@@ -56,23 +56,23 @@ 
  * @return string
  */
 function action_converser_changer_langue($update_session) {
-	if ($lang = _request('var_lang')) {
-		action_converser_post($lang);
-	} elseif ($lang = _request('var_lang_ecrire')) {
-		if ($update_session) {
-			sql_updateq('spip_auteurs', ['lang' => $lang], 'id_auteur = ' . $GLOBALS['visiteur_session']['id_auteur']);
-			$GLOBALS['visiteur_session']['lang'] = $lang;
-			$session = charger_fonction('session', 'inc');
-			if ($spip_session = $session($GLOBALS['visiteur_session'])) {
-				spip_setcookie('spip_session', $spip_session, [
-					'expires' => time() + 3600 * 24 * 14
-				]);
-			}
-		}
-		action_converser_post($lang, true);
-	}
+    if ($lang = _request('var_lang')) {
+        action_converser_post($lang);
+    } elseif ($lang = _request('var_lang_ecrire')) {
+        if ($update_session) {
+            sql_updateq('spip_auteurs', ['lang' => $lang], 'id_auteur = ' . $GLOBALS['visiteur_session']['id_auteur']);
+            $GLOBALS['visiteur_session']['lang'] = $lang;
+            $session = charger_fonction('session', 'inc');
+            if ($spip_session = $session($GLOBALS['visiteur_session'])) {
+                spip_setcookie('spip_session', $spip_session, [
+                    'expires' => time() + 3600 * 24 * 14
+                ]);
+            }
+        }
+        action_converser_post($lang, true);
+    }
 
-	return $lang;
+    return $lang;
 }
 
 /**
@@ -83,17 +83,17 @@ 
  * @return void
  */
 function action_converser_post($lang, $ecrire = false) {
-	if ($lang) {
-		include_spip('inc/lang');
-		if (changer_langue($lang)) {
-			spip_setcookie('spip_lang', $_COOKIE['spip_lang'] = $lang, [
-				'expires' => time() + 365 * 24 * 3600
-			]);
-			if ($ecrire) {
-				spip_setcookie('spip_lang_ecrire', $_COOKIE['spip_lang_ecrire'] = $lang, [
-					'expires' => time() + 365 * 24 * 3600
-				]);
-			}
-		}
-	}
+    if ($lang) {
+        include_spip('inc/lang');
+        if (changer_langue($lang)) {
+            spip_setcookie('spip_lang', $_COOKIE['spip_lang'] = $lang, [
+                'expires' => time() + 365 * 24 * 3600
+            ]);
+            if ($ecrire) {
+                spip_setcookie('spip_lang_ecrire', $_COOKIE['spip_lang_ecrire'] = $lang, [
+                    'expires' => time() + 365 * 24 * 3600
+                ]);
+            }
+        }
+    }
 }

ecrire/action/annuler_job.php

Indentation +9 added lines, -9 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -26,13 +26,13 @@ 
  * @return void
  */
 function action_annuler_job_dist() {
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$id_job = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $id_job = $securiser_action();
 
-	if (
-		$id_job = intval($id_job)
-		and autoriser('annuler', 'job', $id_job)
-	) {
-		job_queue_remove($id_job);
-	}
+    if (
+        $id_job = intval($id_job)
+        and autoriser('annuler', 'job', $id_job)
+    ) {
+        job_queue_remove($id_job);
+    }
 }

ecrire/action/tester.php

Spacing +18 added lines, -18 removed lines

@@ -43,7 +43,7 @@ 
 			} else {
 				# Attention GD sait lire le gif mais pas forcement l'ecrire
 				if (function_exists('ImageCreateFromGIF')) {
-					$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK . 'test.gif');
+					$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK.'test.gif');
 					if ($srcImage) {
 						$gd_formats_read_gif = ',gif';
 						ImageDestroy($srcImage);
@@ -67,28 +67,28 @@ 
 			# les formats disponibles en ecriture... (cf. inc_logos)
 
 			if (function_exists('ImageCreateFromJPEG')) {
-				$srcImage = @ImageCreateFromJPEG(_ROOT_IMG_PACK . 'test.jpg');
+				$srcImage = @ImageCreateFromJPEG(_ROOT_IMG_PACK.'test.jpg');
 				if ($srcImage) {
 					$gd_formats[] = 'jpg';
 					ImageDestroy($srcImage);
 				}
 			}
 			if (function_exists('ImageCreateFromGIF')) {
-				$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK . 'test.gif');
+				$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK.'test.gif');
 				if ($srcImage) {
 					$gd_formats[] = 'gif';
 					ImageDestroy($srcImage);
 				}
 			}
 			if (function_exists('ImageCreateFromPNG')) {
-				$srcImage = @ImageCreateFromPNG(_ROOT_IMG_PACK . 'test.png');
+				$srcImage = @ImageCreateFromPNG(_ROOT_IMG_PACK.'test.png');
 				if ($srcImage) {
 					$gd_formats[] = 'png';
 					ImageDestroy($srcImage);
 				}
 			}
 			if (function_exists('ImageCreateFromWEBP')) {
-				$srcImage = @ImageCreateFromWEBP(_ROOT_IMG_PACK . 'test.webp');
+				$srcImage = @ImageCreateFromWEBP(_ROOT_IMG_PACK.'test.webp');
 				if ($srcImage) {
 					$gd_formats[] = 'webp';
 					ImageDestroy($srcImage);
@@ -96,10 +96,10 @@ 
 			}
 		}
 
-		if (! empty($gd_formats)) {
+		if (!empty($gd_formats)) {
 			$gd_formats = join(',', $gd_formats);
 		}
-		ecrire_meta('gd_formats_read', $gd_formats . $gd_formats_read_gif);
+		ecrire_meta('gd_formats_read', $gd_formats.$gd_formats_read_gif);
 		ecrire_meta('gd_formats', $gd_formats);
 	} // verifier les formats netpbm
 	else {
@@ -123,9 +123,9 @@ 
 				_PNMSCALE_COMMAND
 			);
 
-			$vignette = _ROOT_IMG_PACK . 'test.jpg';
-			$dest = _DIR_VAR . 'test-jpg.jpg';
-			$commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+			$vignette = _ROOT_IMG_PACK.'test.jpg';
+			$dest = _DIR_VAR.'test-jpg.jpg';
+			$commande = "$jpegtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 			spip_log($commande);
 			exec($commande);
 			if ($taille = @getimagesize($dest)) {
@@ -135,9 +135,9 @@ 
 			}
 			$giftopnm_command = str_replace('pnmscale', 'giftopnm', _PNMSCALE_COMMAND);
 			$pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', _PNMSCALE_COMMAND);
-			$vignette = _ROOT_IMG_PACK . 'test.gif';
-			$dest = _DIR_VAR . 'test-gif.jpg';
-			$commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+			$vignette = _ROOT_IMG_PACK.'test.gif';
+			$dest = _DIR_VAR.'test-gif.jpg';
+			$commande = "$giftopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 			spip_log($commande);
 			exec($commande);
 			if ($taille = @getimagesize($dest)) {
@@ -147,9 +147,9 @@ 
 			}
 
 			$pngtopnm_command = str_replace('pnmscale', 'pngtopnm', _PNMSCALE_COMMAND);
-			$vignette = _ROOT_IMG_PACK . 'test.png';
-			$dest = _DIR_VAR . 'test-gif.jpg';
-			$commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+			$vignette = _ROOT_IMG_PACK.'test.png';
+			$dest = _DIR_VAR.'test-gif.jpg';
+			$commande = "$pngtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 			spip_log($commande);
 			exec($commande);
 			if ($taille = @getimagesize($dest)) {
@@ -173,9 +173,9 @@ 
 		include_spip('inc/filtres');
 		include_spip('inc/filtres_images_mini');
 		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
+		$image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
 
-		$image['fichier_dest'] = _DIR_VAR . "test_$arg";
+		$image['fichier_dest'] = _DIR_VAR."test_$arg";
 
 		if (
 			$preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true)

Indentation +152 added lines, -152 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -30,155 +30,155 @@ 
  * redirige sur l'image ainsi créée (sinon sur une image d'echec).
  **/
 function action_tester_dist() {
-	$arg = _request('arg');
-
-	$gd_formats = [];
-	$gd_formats_read_gif = '';
-	// verifier les formats acceptes par GD
-	if ($arg == 'gd1') {
-		// Si GD est installe et php >= 4.0.2
-		if (function_exists('imagetypes')) {
-			if (imagetypes() & IMG_GIF) {
-				$gd_formats[] = 'gif';
-			} else {
-				# Attention GD sait lire le gif mais pas forcement l'ecrire
-				if (function_exists('ImageCreateFromGIF')) {
-					$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK . 'test.gif');
-					if ($srcImage) {
-						$gd_formats_read_gif = ',gif';
-						ImageDestroy($srcImage);
-					}
-				}
-			}
-
-			if (imagetypes() & IMG_JPG) {
-				$gd_formats[] = 'jpg';
-			}
-			if (imagetypes() & IMG_PNG) {
-				$gd_formats[] = 'png';
-			}
-			if (imagetypes() & IMG_WEBP) {
-				$gd_formats[] = 'webp';
-			}
-		} else {
-			# ancienne methode de detection des formats, qui en plus
-			# est bugguee car elle teste les formats en lecture
-			# alors que la valeur deduite sert a identifier
-			# les formats disponibles en ecriture... (cf. inc_logos)
-
-			if (function_exists('ImageCreateFromJPEG')) {
-				$srcImage = @ImageCreateFromJPEG(_ROOT_IMG_PACK . 'test.jpg');
-				if ($srcImage) {
-					$gd_formats[] = 'jpg';
-					ImageDestroy($srcImage);
-				}
-			}
-			if (function_exists('ImageCreateFromGIF')) {
-				$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK . 'test.gif');
-				if ($srcImage) {
-					$gd_formats[] = 'gif';
-					ImageDestroy($srcImage);
-				}
-			}
-			if (function_exists('ImageCreateFromPNG')) {
-				$srcImage = @ImageCreateFromPNG(_ROOT_IMG_PACK . 'test.png');
-				if ($srcImage) {
-					$gd_formats[] = 'png';
-					ImageDestroy($srcImage);
-				}
-			}
-			if (function_exists('ImageCreateFromWEBP')) {
-				$srcImage = @ImageCreateFromWEBP(_ROOT_IMG_PACK . 'test.webp');
-				if ($srcImage) {
-					$gd_formats[] = 'webp';
-					ImageDestroy($srcImage);
-				}
-			}
-		}
-
-		if (! empty($gd_formats)) {
-			$gd_formats = join(',', $gd_formats);
-		}
-		ecrire_meta('gd_formats_read', $gd_formats . $gd_formats_read_gif);
-		ecrire_meta('gd_formats', $gd_formats);
-	} // verifier les formats netpbm
-	else {
-		if ($arg == 'netpbm') {
-			if (!defined('_PNMSCALE_COMMAND')) {
-				define('_PNMSCALE_COMMAND', 'pnmscale');
-			} // Securite : mes_options.php peut preciser le chemin absolu
-			if (_PNMSCALE_COMMAND == '') {
-				return;
-			}
-			$netpbm_formats = [];
-
-			$jpegtopnm_command = str_replace(
-				'pnmscale',
-				'jpegtopnm',
-				_PNMSCALE_COMMAND
-			);
-			$pnmtojpeg_command = str_replace(
-				'pnmscale',
-				'pnmtojpeg',
-				_PNMSCALE_COMMAND
-			);
-
-			$vignette = _ROOT_IMG_PACK . 'test.jpg';
-			$dest = _DIR_VAR . 'test-jpg.jpg';
-			$commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-			spip_log($commande);
-			exec($commande);
-			if ($taille = @getimagesize($dest)) {
-				if ($taille[1] == 10) {
-					$netpbm_formats[] = 'jpg';
-				}
-			}
-			$giftopnm_command = str_replace('pnmscale', 'giftopnm', _PNMSCALE_COMMAND);
-			$pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', _PNMSCALE_COMMAND);
-			$vignette = _ROOT_IMG_PACK . 'test.gif';
-			$dest = _DIR_VAR . 'test-gif.jpg';
-			$commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-			spip_log($commande);
-			exec($commande);
-			if ($taille = @getimagesize($dest)) {
-				if ($taille[1] == 10) {
-					$netpbm_formats[] = 'gif';
-				}
-			}
-
-			$pngtopnm_command = str_replace('pnmscale', 'pngtopnm', _PNMSCALE_COMMAND);
-			$vignette = _ROOT_IMG_PACK . 'test.png';
-			$dest = _DIR_VAR . 'test-gif.jpg';
-			$commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-			spip_log($commande);
-			exec($commande);
-			if ($taille = @getimagesize($dest)) {
-				if ($taille[1] == 10) {
-					$netpbm_formats[] = 'png';
-				}
-			}
-
-			ecrire_meta('netpbm_formats', join(',', $netpbm_formats ?: []));
-		}
-	}
-
-	// et maintenant envoyer la vignette de tests
-	if (in_array($arg, ['gd1', 'gd2', 'imagick', 'convert', 'netpbm'])) {
-		include_spip('inc/filtres');
-		include_spip('inc/filtres_images_mini');
-		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
-
-		$image['fichier_dest'] = _DIR_VAR . "test_$arg";
-
-		if (
-			$preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true)
-			and ($preview['width'] * $preview['height'] > 0)
-		) {
-			redirige_par_entete($preview['fichier']);
-		}
-	}
-
-	# image echec
-	redirige_par_entete(chemin_image('echec-reducteur-xx.svg'));
+    $arg = _request('arg');
+
+    $gd_formats = [];
+    $gd_formats_read_gif = '';
+    // verifier les formats acceptes par GD
+    if ($arg == 'gd1') {
+        // Si GD est installe et php >= 4.0.2
+        if (function_exists('imagetypes')) {
+            if (imagetypes() & IMG_GIF) {
+                $gd_formats[] = 'gif';
+            } else {
+                # Attention GD sait lire le gif mais pas forcement l'ecrire
+                if (function_exists('ImageCreateFromGIF')) {
+                    $srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK . 'test.gif');
+                    if ($srcImage) {
+                        $gd_formats_read_gif = ',gif';
+                        ImageDestroy($srcImage);
+                    }
+                }
+            }
+
+            if (imagetypes() & IMG_JPG) {
+                $gd_formats[] = 'jpg';
+            }
+            if (imagetypes() & IMG_PNG) {
+                $gd_formats[] = 'png';
+            }
+            if (imagetypes() & IMG_WEBP) {
+                $gd_formats[] = 'webp';
+            }
+        } else {
+            # ancienne methode de detection des formats, qui en plus
+            # est bugguee car elle teste les formats en lecture
+            # alors que la valeur deduite sert a identifier
+            # les formats disponibles en ecriture... (cf. inc_logos)
+
+            if (function_exists('ImageCreateFromJPEG')) {
+                $srcImage = @ImageCreateFromJPEG(_ROOT_IMG_PACK . 'test.jpg');
+                if ($srcImage) {
+                    $gd_formats[] = 'jpg';
+                    ImageDestroy($srcImage);
+                }
+            }
+            if (function_exists('ImageCreateFromGIF')) {
+                $srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK . 'test.gif');
+                if ($srcImage) {
+                    $gd_formats[] = 'gif';
+                    ImageDestroy($srcImage);
+                }
+            }
+            if (function_exists('ImageCreateFromPNG')) {
+                $srcImage = @ImageCreateFromPNG(_ROOT_IMG_PACK . 'test.png');
+                if ($srcImage) {
+                    $gd_formats[] = 'png';
+                    ImageDestroy($srcImage);
+                }
+            }
+            if (function_exists('ImageCreateFromWEBP')) {
+                $srcImage = @ImageCreateFromWEBP(_ROOT_IMG_PACK . 'test.webp');
+                if ($srcImage) {
+                    $gd_formats[] = 'webp';
+                    ImageDestroy($srcImage);
+                }
+            }
+        }
+
+        if (! empty($gd_formats)) {
+            $gd_formats = join(',', $gd_formats);
+        }
+        ecrire_meta('gd_formats_read', $gd_formats . $gd_formats_read_gif);
+        ecrire_meta('gd_formats', $gd_formats);
+    } // verifier les formats netpbm
+    else {
+        if ($arg == 'netpbm') {
+            if (!defined('_PNMSCALE_COMMAND')) {
+                define('_PNMSCALE_COMMAND', 'pnmscale');
+            } // Securite : mes_options.php peut preciser le chemin absolu
+            if (_PNMSCALE_COMMAND == '') {
+                return;
+            }
+            $netpbm_formats = [];
+
+            $jpegtopnm_command = str_replace(
+                'pnmscale',
+                'jpegtopnm',
+                _PNMSCALE_COMMAND
+            );
+            $pnmtojpeg_command = str_replace(
+                'pnmscale',
+                'pnmtojpeg',
+                _PNMSCALE_COMMAND
+            );
+
+            $vignette = _ROOT_IMG_PACK . 'test.jpg';
+            $dest = _DIR_VAR . 'test-jpg.jpg';
+            $commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+            spip_log($commande);
+            exec($commande);
+            if ($taille = @getimagesize($dest)) {
+                if ($taille[1] == 10) {
+                    $netpbm_formats[] = 'jpg';
+                }
+            }
+            $giftopnm_command = str_replace('pnmscale', 'giftopnm', _PNMSCALE_COMMAND);
+            $pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', _PNMSCALE_COMMAND);
+            $vignette = _ROOT_IMG_PACK . 'test.gif';
+            $dest = _DIR_VAR . 'test-gif.jpg';
+            $commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+            spip_log($commande);
+            exec($commande);
+            if ($taille = @getimagesize($dest)) {
+                if ($taille[1] == 10) {
+                    $netpbm_formats[] = 'gif';
+                }
+            }
+
+            $pngtopnm_command = str_replace('pnmscale', 'pngtopnm', _PNMSCALE_COMMAND);
+            $vignette = _ROOT_IMG_PACK . 'test.png';
+            $dest = _DIR_VAR . 'test-gif.jpg';
+            $commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+            spip_log($commande);
+            exec($commande);
+            if ($taille = @getimagesize($dest)) {
+                if ($taille[1] == 10) {
+                    $netpbm_formats[] = 'png';
+                }
+            }
+
+            ecrire_meta('netpbm_formats', join(',', $netpbm_formats ?: []));
+        }
+    }
+
+    // et maintenant envoyer la vignette de tests
+    if (in_array($arg, ['gd1', 'gd2', 'imagick', 'convert', 'netpbm'])) {
+        include_spip('inc/filtres');
+        include_spip('inc/filtres_images_mini');
+        $taille_preview = 150;
+        $image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
+
+        $image['fichier_dest'] = _DIR_VAR . "test_$arg";
+
+        if (
+            $preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true)
+            and ($preview['width'] * $preview['height'] > 0)
+        ) {
+            redirige_par_entete($preview['fichier']);
+        }
+    }
+
+    # image echec
+    redirige_par_entete(chemin_image('echec-reducteur-xx.svg'));
 }

ecrire/action/logout.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
 				_T('login_deconnexion_ok'),
 				_T('login_verifiez_navigateur'),
 				_T('login_retour_public'),
-				'redirect=' . _DIR_RESTREINT_ABS,
+				'redirect='._DIR_RESTREINT_ABS,
 				_T('login_test_navigateur'),
 				true
 			);

Indentation +80 added lines, -80 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/cookie');
@@ -34,73 +34,73 @@ 
  *
  */
 function action_logout_dist() {
-	$logout = _request('logout');
-	$url = securiser_redirect_action(_request('url'));
-	// cas particulier, logout dans l'espace public
-	if ($logout == 'public' and !$url) {
-		$url = url_de_base();
-	}
+    $logout = _request('logout');
+    $url = securiser_redirect_action(_request('url'));
+    // cas particulier, logout dans l'espace public
+    if ($logout == 'public' and !$url) {
+        $url = url_de_base();
+    }
 
-	// seul le loge peut se deloger (mais id_auteur peut valoir 0 apres une restauration avortee)
-	if (
-		isset($GLOBALS['visiteur_session']['id_auteur'])
-		and is_numeric($GLOBALS['visiteur_session']['id_auteur'])
-		// des sessions anonymes avec id_auteur=0 existent, mais elle n'ont pas de statut : double check
-		and isset($GLOBALS['visiteur_session']['statut'])
-	) {
-		// il faut un jeton pour fermer la session (eviter les CSRF)
-		if (
-			!$jeton = _request('jeton')
-			or !verifier_jeton_logout($jeton, $GLOBALS['visiteur_session'])
-		) {
-			$jeton = generer_jeton_logout($GLOBALS['visiteur_session']);
-			$action = generer_url_action('logout', "jeton=$jeton");
-			$action = parametre_url($action, 'logout', _request('logout'));
-			$action = parametre_url($action, 'url', _request('url'));
-			include_spip('inc/minipres');
-			include_spip('inc/filtres');
-			$texte = bouton_action(_T('spip:icone_deconnecter'), $action);
-			$texte = "<div class='boutons'>$texte</div>";
-			$texte .= '<script type="text/javascript">document.write("<style>body{visibility:hidden;}</style>");window.document.forms[0].submit();</script>';
-			$res = minipres(_T('spip:icone_deconnecter'), $texte, ['all_inline' => true]);
-			echo $res;
+    // seul le loge peut se deloger (mais id_auteur peut valoir 0 apres une restauration avortee)
+    if (
+        isset($GLOBALS['visiteur_session']['id_auteur'])
+        and is_numeric($GLOBALS['visiteur_session']['id_auteur'])
+        // des sessions anonymes avec id_auteur=0 existent, mais elle n'ont pas de statut : double check
+        and isset($GLOBALS['visiteur_session']['statut'])
+    ) {
+        // il faut un jeton pour fermer la session (eviter les CSRF)
+        if (
+            !$jeton = _request('jeton')
+            or !verifier_jeton_logout($jeton, $GLOBALS['visiteur_session'])
+        ) {
+            $jeton = generer_jeton_logout($GLOBALS['visiteur_session']);
+            $action = generer_url_action('logout', "jeton=$jeton");
+            $action = parametre_url($action, 'logout', _request('logout'));
+            $action = parametre_url($action, 'url', _request('url'));
+            include_spip('inc/minipres');
+            include_spip('inc/filtres');
+            $texte = bouton_action(_T('spip:icone_deconnecter'), $action);
+            $texte = "<div class='boutons'>$texte</div>";
+            $texte .= '<script type="text/javascript">document.write("<style>body{visibility:hidden;}</style>");window.document.forms[0].submit();</script>';
+            $res = minipres(_T('spip:icone_deconnecter'), $texte, ['all_inline' => true]);
+            echo $res;
 
-			return;
-		}
+            return;
+        }
 
-		include_spip('inc/auth');
-		auth_trace($GLOBALS['visiteur_session'], '0000-00-00 00:00:00');
-		// le logout explicite vaut destruction de toutes les sessions
-		if (isset($_COOKIE['spip_session'])) {
-			$session = charger_fonction('session', 'inc');
-			$session($GLOBALS['visiteur_session']['id_auteur']);
-			spip_setcookie('spip_session', $_COOKIE['spip_session'], [
-				'expires' => time() - 3600
-			]);
-		}
-		// si authentification http, et que la personne est loge,
-		// pour se deconnecter, il faut proposer un nouveau formulaire de connexion http
-		if (
-			isset($_SERVER['PHP_AUTH_USER'])
-			and !$GLOBALS['ignore_auth_http']
-			and $GLOBALS['auth_can_disconnect']
-		) {
-			ask_php_auth(
-				_T('login_deconnexion_ok'),
-				_T('login_verifiez_navigateur'),
-				_T('login_retour_public'),
-				'redirect=' . _DIR_RESTREINT_ABS,
-				_T('login_test_navigateur'),
-				true
-			);
-		}
-	}
+        include_spip('inc/auth');
+        auth_trace($GLOBALS['visiteur_session'], '0000-00-00 00:00:00');
+        // le logout explicite vaut destruction de toutes les sessions
+        if (isset($_COOKIE['spip_session'])) {
+            $session = charger_fonction('session', 'inc');
+            $session($GLOBALS['visiteur_session']['id_auteur']);
+            spip_setcookie('spip_session', $_COOKIE['spip_session'], [
+                'expires' => time() - 3600
+            ]);
+        }
+        // si authentification http, et que la personne est loge,
+        // pour se deconnecter, il faut proposer un nouveau formulaire de connexion http
+        if (
+            isset($_SERVER['PHP_AUTH_USER'])
+            and !$GLOBALS['ignore_auth_http']
+            and $GLOBALS['auth_can_disconnect']
+        ) {
+            ask_php_auth(
+                _T('login_deconnexion_ok'),
+                _T('login_verifiez_navigateur'),
+                _T('login_retour_public'),
+                'redirect=' . _DIR_RESTREINT_ABS,
+                _T('login_test_navigateur'),
+                true
+            );
+        }
+    }
 
-	// Rediriger en contrant le cache navigateur (Safari3)
-	include_spip('inc/headers');
-	redirige_par_entete($url
-		? parametre_url($url, 'var_hasard', uniqid(random_int(0, mt_getrandmax())), '&')
-		: generer_url_public('login'));
+    // Rediriger en contrant le cache navigateur (Safari3)
+    include_spip('inc/headers');
+    redirige_par_entete($url
+        ? parametre_url($url, 'var_hasard', uniqid(random_int(0, mt_getrandmax())), '&')
+        : generer_url_public('login'));
 }
 
 /**
@@ -111,17 +111,17 @@ 
  * @return string
  */
 function generer_jeton_logout($session, $alea = null) {
-	if (is_null($alea)) {
-		include_spip('inc/acces');
-		$alea = charger_aleas();
-	}
+    if (is_null($alea)) {
+        include_spip('inc/acces');
+        $alea = charger_aleas();
+    }
 
-	$jeton = md5($session['date_session']
-		. $session['id_auteur']
-		. $session['statut']
-		. $alea);
+    $jeton = md5($session['date_session']
+        . $session['id_auteur']
+        . $session['statut']
+        . $alea);
 
-	return $jeton;
+    return $jeton;
 }
 
 /**
@@ -135,13 +135,13 @@ 
  * @return bool
  */
 function verifier_jeton_logout($jeton, $session) {
-	if (generer_jeton_logout($session) === $jeton) {
-		return true;
-	}
+    if (generer_jeton_logout($session) === $jeton) {
+        return true;
+    }
 
-	if (generer_jeton_logout($session, $GLOBALS['meta']['alea_ephemere_ancien']) === $jeton) {
-		return true;
-	}
+    if (generer_jeton_logout($session, $GLOBALS['meta']['alea_ephemere_ancien']) === $jeton) {
+        return true;
+    }
 
-	return false;
+    return false;
 }

ecrire/action/forcer_job.php

Indentation +11 added lines, -11 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -26,15 +26,15 @@ 
  * @return void
  */
 function action_forcer_job_dist() {
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$id_job = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $id_job = $securiser_action();
 
-	if (
-		$id_job = intval($id_job)
-		and autoriser('forcer', 'job', $id_job)
-	) {
-		include_spip('inc/queue');
-		include_spip('inc/genie');
-		queue_schedule([$id_job]);
-	}
+    if (
+        $id_job = intval($id_job)
+        and autoriser('forcer', 'job', $id_job)
+    ) {
+        include_spip('inc/queue');
+        include_spip('inc/genie');
+        queue_schedule([$id_job]);
+    }
 }

ecrire/action/purger.php

Indentation +47 added lines, -47 removed lines

@@ -16,7 +16,7 @@ 
  * @package SPIP\Core\Cache
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -39,57 +39,57 @@ 
  *     de l'action sécurisée.
  */
 function action_purger_dist($arg = null) {
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
 
-	include_spip('inc/invalideur');
+    include_spip('inc/invalideur');
 
-	spip_log("purger $arg");
+    spip_log("purger $arg");
 
-	switch ($arg) {
-		case 'inhibe_cache':
-			// inhiber le cache pendant 24h
-			ecrire_meta('cache_inhib', $_SERVER['REQUEST_TIME'] + 24 * 3600);
-			break;
-		case 'reactive_cache':
-			effacer_meta('cache_inhib');
-			break;
+    switch ($arg) {
+        case 'inhibe_cache':
+            // inhiber le cache pendant 24h
+            ecrire_meta('cache_inhib', $_SERVER['REQUEST_TIME'] + 24 * 3600);
+            break;
+        case 'reactive_cache':
+            effacer_meta('cache_inhib');
+            break;
 
-		case 'cache':
-			supprime_invalideurs();
-			@spip_unlink(_CACHE_RUBRIQUES);
-			@spip_unlink(_CACHE_CHEMIN);
-			@spip_unlink(_DIR_TMP . 'plugin_xml_cache.gz');
-			// on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
-			// _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
-			// pour eviter des problemes de concurence
-			// cf https://core.spip.net/issues/2989
-			//@spip_unlink(_CACHE_PIPELINES);
-			//@spip_unlink(_CACHE_PLUGINS_PATH);
-			//@spip_unlink(_CACHE_PLUGINS_FCT);
-			@spip_unlink(_CACHE_PLUGINS_OPT);
-			purger_repertoire(_DIR_CACHE, ['subdir' => true]);
-			purger_repertoire(_DIR_AIDE);
-			purger_repertoire(_DIR_VAR . 'cache-css');
-			purger_repertoire(_DIR_VAR . 'cache-js');
-			break;
+        case 'cache':
+            supprime_invalideurs();
+            @spip_unlink(_CACHE_RUBRIQUES);
+            @spip_unlink(_CACHE_CHEMIN);
+            @spip_unlink(_DIR_TMP . 'plugin_xml_cache.gz');
+            // on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
+            // _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
+            // pour eviter des problemes de concurence
+            // cf https://core.spip.net/issues/2989
+            //@spip_unlink(_CACHE_PIPELINES);
+            //@spip_unlink(_CACHE_PLUGINS_PATH);
+            //@spip_unlink(_CACHE_PLUGINS_FCT);
+            @spip_unlink(_CACHE_PLUGINS_OPT);
+            purger_repertoire(_DIR_CACHE, ['subdir' => true]);
+            purger_repertoire(_DIR_AIDE);
+            purger_repertoire(_DIR_VAR . 'cache-css');
+            purger_repertoire(_DIR_VAR . 'cache-js');
+            break;
 
-		case 'squelettes':
-			purger_repertoire(_DIR_SKELS);
-			break;
+        case 'squelettes':
+            purger_repertoire(_DIR_SKELS);
+            break;
 
-		case 'vignettes':
-			purger_repertoire(_DIR_VAR . 'cache-gd2', ['subdir' => true]);
-			purger_repertoire(_DIR_VAR . 'cache-texte', ['subdir' => true]);
-			purger_repertoire(_DIR_VAR . 'cache-vignettes', ['subdir' => true]);
-			purger_repertoire(_DIR_VAR . 'cache-TeX', ['subdir' => true]);
-			supprime_invalideurs();
-			purger_repertoire(_DIR_CACHE, ['subdir' => true]);
-			break;
-	}
+        case 'vignettes':
+            purger_repertoire(_DIR_VAR . 'cache-gd2', ['subdir' => true]);
+            purger_repertoire(_DIR_VAR . 'cache-texte', ['subdir' => true]);
+            purger_repertoire(_DIR_VAR . 'cache-vignettes', ['subdir' => true]);
+            purger_repertoire(_DIR_VAR . 'cache-TeX', ['subdir' => true]);
+            supprime_invalideurs();
+            purger_repertoire(_DIR_CACHE, ['subdir' => true]);
+            break;
+    }
 
-	// le faire savoir aux plugins
-	pipeline('trig_purger', $arg);
+    // le faire savoir aux plugins
+    pipeline('trig_purger', $arg);
 }

Spacing +7 added lines, -7 removed lines

@@ -61,7 +61,7 @@ 
 			supprime_invalideurs();
 			@spip_unlink(_CACHE_RUBRIQUES);
 			@spip_unlink(_CACHE_CHEMIN);
-			@spip_unlink(_DIR_TMP . 'plugin_xml_cache.gz');
+			@spip_unlink(_DIR_TMP.'plugin_xml_cache.gz');
 			// on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
 			// _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
 			// pour eviter des problemes de concurence
@@ -72,8 +72,8 @@ 
 			@spip_unlink(_CACHE_PLUGINS_OPT);
 			purger_repertoire(_DIR_CACHE, ['subdir' => true]);
 			purger_repertoire(_DIR_AIDE);
-			purger_repertoire(_DIR_VAR . 'cache-css');
-			purger_repertoire(_DIR_VAR . 'cache-js');
+			purger_repertoire(_DIR_VAR.'cache-css');
+			purger_repertoire(_DIR_VAR.'cache-js');
 			break;
 
 		case 'squelettes':
@@ -81,10 +81,10 @@ 
 			break;
 
 		case 'vignettes':
-			purger_repertoire(_DIR_VAR . 'cache-gd2', ['subdir' => true]);
-			purger_repertoire(_DIR_VAR . 'cache-texte', ['subdir' => true]);
-			purger_repertoire(_DIR_VAR . 'cache-vignettes', ['subdir' => true]);
-			purger_repertoire(_DIR_VAR . 'cache-TeX', ['subdir' => true]);
+			purger_repertoire(_DIR_VAR.'cache-gd2', ['subdir' => true]);
+			purger_repertoire(_DIR_VAR.'cache-texte', ['subdir' => true]);
+			purger_repertoire(_DIR_VAR.'cache-vignettes', ['subdir' => true]);
+			purger_repertoire(_DIR_VAR.'cache-TeX', ['subdir' => true]);
 			supprime_invalideurs();
 			purger_repertoire(_DIR_CACHE, ['subdir' => true]);
 			break;

ecrire/action/confirmer_inscription.php

Indentation +44 added lines, -44 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,51 +28,51 @@ 
  * @return void
  */
 function action_confirmer_inscription_dist() {
-	$jeton = _request('jeton');
-	$email = _request('email');
+    $jeton = _request('jeton');
+    $email = _request('email');
 
-	include_spip('action/inscrire_auteur');
-	if (
-		$auteur = auteur_verifier_jeton($jeton)
-		and $auteur['email'] == $email
-		and $auteur['statut'] == 'nouveau'
-	) {
-		// d'abord on confirme son statut
-		$auteur = confirmer_statut_inscription($auteur);
+    include_spip('action/inscrire_auteur');
+    if (
+        $auteur = auteur_verifier_jeton($jeton)
+        and $auteur['email'] == $email
+        and $auteur['statut'] == 'nouveau'
+    ) {
+        // d'abord on confirme son statut
+        $auteur = confirmer_statut_inscription($auteur);
 
-		// OK c'est un nouvel inscrit qui confirme :
-		// on le loge => ca va confirmer son statut et c'est plus sympa
-		include_spip('inc/auth');
-		auth_loger($auteur);
+        // OK c'est un nouvel inscrit qui confirme :
+        // on le loge => ca va confirmer son statut et c'est plus sympa
+        include_spip('inc/auth');
+        auth_loger($auteur);
 
-		// et on efface son jeton
-		auteur_effacer_jeton($auteur['id_auteur']);
+        // et on efface son jeton
+        auteur_effacer_jeton($auteur['id_auteur']);
 
-		// si pas de redirection demandee, rediriger vers public ou prive selon le statut de l'auteur
-		// TODO: ne semble pas marcher si inscrit non visiteur, a debug
-		if (!_request('redirect')) {
-			// on passe id_auteur explicite pour forcer une lecture en base de toutes les infos
-			if (autoriser('ecrire', '', '', $auteur['id_auteur'])) {
-				// poser un cookie admin aussi
-				$cookie = charger_fonction('cookie', 'action');
-				$cookie('@' . $GLOBALS['visiteur_session']['login']);
-				$GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
-			} else {
-				$GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];
-			}
-		}
-	} else {
-		// lien perime :
-		if ($GLOBALS['visiteur_session']['id_auteur']) {
-			// on passe id_auteur explicite pour forcer une lecture en base de toutes les infos
-			if (autoriser('ecrire', '', '', $GLOBALS['visiteur_session']['id_auteur'])) {
-				$GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
-			} else {
-				$GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];
-			}
-		} else // rediriger vers la page de login si pas encore loge
-		{
-			$GLOBALS['redirect'] = parametre_url(generer_url_public('login', '', false), 'url', _request('redirect'));
-		}
-	}
+        // si pas de redirection demandee, rediriger vers public ou prive selon le statut de l'auteur
+        // TODO: ne semble pas marcher si inscrit non visiteur, a debug
+        if (!_request('redirect')) {
+            // on passe id_auteur explicite pour forcer une lecture en base de toutes les infos
+            if (autoriser('ecrire', '', '', $auteur['id_auteur'])) {
+                // poser un cookie admin aussi
+                $cookie = charger_fonction('cookie', 'action');
+                $cookie('@' . $GLOBALS['visiteur_session']['login']);
+                $GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
+            } else {
+                $GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];
+            }
+        }
+    } else {
+        // lien perime :
+        if ($GLOBALS['visiteur_session']['id_auteur']) {
+            // on passe id_auteur explicite pour forcer une lecture en base de toutes les infos
+            if (autoriser('ecrire', '', '', $GLOBALS['visiteur_session']['id_auteur'])) {
+                $GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
+            } else {
+                $GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];
+            }
+        } else // rediriger vers la page de login si pas encore loge
+        {
+            $GLOBALS['redirect'] = parametre_url(generer_url_public('login', '', false), 'url', _request('redirect'));
+        }
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 			if (autoriser('ecrire', '', '', $auteur['id_auteur'])) {
 				// poser un cookie admin aussi
 				$cookie = charger_fonction('cookie', 'action');
-				$cookie('@' . $GLOBALS['visiteur_session']['login']);
+				$cookie('@'.$GLOBALS['visiteur_session']['login']);
 				$GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
 			} else {
 				$GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];

ecrire/action/etre_webmestre.php

Indentation +21 added lines, -21 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -29,23 +29,23 @@ 
  * @return void
  */
 function action_etre_webmestre_dist() {
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$time = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $time = $securiser_action();
 
-	if (
-		time() - $time < 15 * 60
-		and $GLOBALS['visiteur_session']['statut'] == '0minirezo'
-		and $GLOBALS['visiteur_session']['webmestre'] !== 'oui'
-	) {
-		$action = _T('info_admin_etre_webmestre');
-		$admin = charger_fonction('admin', 'inc');
-		// lance la verif par ftp et l'appel
-		// a base_etre_webmestre_dist quand c'est OK
-		if ($r = $admin('etre_webmestre', $action)) {
-			echo $r;
-			exit;
-		}
-	}
+    if (
+        time() - $time < 15 * 60
+        and $GLOBALS['visiteur_session']['statut'] == '0minirezo'
+        and $GLOBALS['visiteur_session']['webmestre'] !== 'oui'
+    ) {
+        $action = _T('info_admin_etre_webmestre');
+        $admin = charger_fonction('admin', 'inc');
+        // lance la verif par ftp et l'appel
+        // a base_etre_webmestre_dist quand c'est OK
+        if ($r = $admin('etre_webmestre', $action)) {
+            echo $r;
+            exit;
+        }
+    }
 }
 
 /**
@@ -54,8 +54,8 @@ 
  * @return void
  */
 function base_etre_webmestre_dist() {
-	if ($GLOBALS['visiteur_session']['statut'] == '0minirezo' and $GLOBALS['visiteur_session']['webmestre'] !== 'oui') {
-		include_spip('action/editer_auteur');
-		auteur_instituer($GLOBALS['visiteur_session']['id_auteur'], ['webmestre' => 'oui'], true);
-	}
+    if ($GLOBALS['visiteur_session']['statut'] == '0minirezo' and $GLOBALS['visiteur_session']['webmestre'] !== 'oui') {
+        include_spip('action/editer_auteur');
+        auteur_instituer($GLOBALS['visiteur_session']['id_auteur'], ['webmestre' => 'oui'], true);
+    }
 }

ecrire/action/referencer_traduction.php

Indentation +45 added lines, -45 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -41,54 +41,54 @@ 
  */
 function action_referencer_traduction_dist($objet, $id_objet, $id_trad) {
 
-	// ne rien faire si id_trad est ambigu
-	if (!is_numeric($id_trad)) {
-		return false;
-	}
+    // ne rien faire si id_trad est ambigu
+    if (!is_numeric($id_trad)) {
+        return false;
+    }
 
-	$table_objet_sql = table_objet_sql($objet);
-	$id_table_objet = id_table_objet($objet);
+    $table_objet_sql = table_objet_sql($objet);
+    $id_table_objet = id_table_objet($objet);
 
-	// on a fourni un id_trad : affectation ou modification du groupe de trad
-	if ($id_trad) {
-		// selectionner l'objet cible, qui doit etre different de nous-meme,
-		// et quitter s'il n'existe pas
-		$id_lier = sql_getfetsel(
-			'id_trad',
-			$table_objet_sql,
-			"$id_table_objet=" . intval($id_trad) . " AND NOT($id_table_objet=" . intval($id_objet) . ')'
-		);
-		if ($id_lier === null) {
-			spip_log("echec lien de trad vers objet $objet/$id_objet incorrect ($id_trad)");
+    // on a fourni un id_trad : affectation ou modification du groupe de trad
+    if ($id_trad) {
+        // selectionner l'objet cible, qui doit etre different de nous-meme,
+        // et quitter s'il n'existe pas
+        $id_lier = sql_getfetsel(
+            'id_trad',
+            $table_objet_sql,
+            "$id_table_objet=" . intval($id_trad) . " AND NOT($id_table_objet=" . intval($id_objet) . ')'
+        );
+        if ($id_lier === null) {
+            spip_log("echec lien de trad vers objet $objet/$id_objet incorrect ($id_trad)");
 
-			return false;
-		}
+            return false;
+        }
 
-		// $id_lier est le numero du groupe de traduction
-		// Si l'objet vise n'est pas deja traduit, son identifiant devient
-		// le nouvel id_trad de ce nouveau groupe et on l'affecte aux deux
-		// objets
-		if ($id_lier == 0) {
-			sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "$id_table_objet IN ($id_trad, $id_objet)");
-		} // si id_lier = id_objet alors on veut changer la reference de tout le groupe de trad
-		elseif ($id_lier == $id_objet) {
-			sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "id_trad = $id_lier");
-		} // sinon ajouter notre objet dans le groupe
-		else {
-			sql_updateq($table_objet_sql, ['id_trad' => $id_lier], "$id_table_objet=" . intval($id_objet));
-		}
-	} // on a fourni un id_trad nul : sortir id_objet du groupe de trad
-	else {
-		$old_id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$id_table_objet=" . intval($id_objet));
-		// supprimer le lien de traduction
-		sql_updateq($table_objet_sql, ['id_trad' => 0], "$id_table_objet=" . intval($id_objet));
+        // $id_lier est le numero du groupe de traduction
+        // Si l'objet vise n'est pas deja traduit, son identifiant devient
+        // le nouvel id_trad de ce nouveau groupe et on l'affecte aux deux
+        // objets
+        if ($id_lier == 0) {
+            sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "$id_table_objet IN ($id_trad, $id_objet)");
+        } // si id_lier = id_objet alors on veut changer la reference de tout le groupe de trad
+        elseif ($id_lier == $id_objet) {
+            sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "id_trad = $id_lier");
+        } // sinon ajouter notre objet dans le groupe
+        else {
+            sql_updateq($table_objet_sql, ['id_trad' => $id_lier], "$id_table_objet=" . intval($id_objet));
+        }
+    } // on a fourni un id_trad nul : sortir id_objet du groupe de trad
+    else {
+        $old_id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$id_table_objet=" . intval($id_objet));
+        // supprimer le lien de traduction
+        sql_updateq($table_objet_sql, ['id_trad' => 0], "$id_table_objet=" . intval($id_objet));
 
-		// Verifier si l'ancien groupe ne comporte plus qu'un seul objet. Alors mettre a zero.
-		$cpt = sql_countsel($table_objet_sql, 'id_trad=' . intval($old_id_trad));
-		if ($cpt == 1) {
-			sql_updateq($table_objet_sql, ['id_trad' => 0], 'id_trad=' . intval($old_id_trad));
-		}
-	}
+        // Verifier si l'ancien groupe ne comporte plus qu'un seul objet. Alors mettre a zero.
+        $cpt = sql_countsel($table_objet_sql, 'id_trad=' . intval($old_id_trad));
+        if ($cpt == 1) {
+            sql_updateq($table_objet_sql, ['id_trad' => 0], 'id_trad=' . intval($old_id_trad));
+        }
+    }
 
-	return true;
+    return true;
 }

Spacing +6 added lines, -6 removed lines

@@ -56,7 +56,7 @@ 
 		$id_lier = sql_getfetsel(
 			'id_trad',
 			$table_objet_sql,
-			"$id_table_objet=" . intval($id_trad) . " AND NOT($id_table_objet=" . intval($id_objet) . ')'
+			"$id_table_objet=".intval($id_trad)." AND NOT($id_table_objet=".intval($id_objet).')'
 		);
 		if ($id_lier === null) {
 			spip_log("echec lien de trad vers objet $objet/$id_objet incorrect ($id_trad)");
@@ -75,18 +75,18 @@ 
 			sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "id_trad = $id_lier");
 		} // sinon ajouter notre objet dans le groupe
 		else {
-			sql_updateq($table_objet_sql, ['id_trad' => $id_lier], "$id_table_objet=" . intval($id_objet));
+			sql_updateq($table_objet_sql, ['id_trad' => $id_lier], "$id_table_objet=".intval($id_objet));
 		}
 	} // on a fourni un id_trad nul : sortir id_objet du groupe de trad
 	else {
-		$old_id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$id_table_objet=" . intval($id_objet));
+		$old_id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$id_table_objet=".intval($id_objet));
 		// supprimer le lien de traduction
-		sql_updateq($table_objet_sql, ['id_trad' => 0], "$id_table_objet=" . intval($id_objet));
+		sql_updateq($table_objet_sql, ['id_trad' => 0], "$id_table_objet=".intval($id_objet));
 
 		// Verifier si l'ancien groupe ne comporte plus qu'un seul objet. Alors mettre a zero.
-		$cpt = sql_countsel($table_objet_sql, 'id_trad=' . intval($old_id_trad));
+		$cpt = sql_countsel($table_objet_sql, 'id_trad='.intval($old_id_trad));
 		if ($cpt == 1) {
-			sql_updateq($table_objet_sql, ['id_trad' => 0], 'id_trad=' . intval($old_id_trad));
+			sql_updateq($table_objet_sql, ['id_trad' => 0], 'id_trad='.intval($old_id_trad));
 		}
 	}