chore: phpdoc & opérateurs &&

(cherry picked from commit d28ad01c)

formulaires/joindre_document.php

@@ -232,8 +232,8 @@ function formulaires_joindre_document_verifier_dist(
 				// regarder si il faut lister le contenu du zip et le presenter
 				if (
 					!count($erreurs)
-					and !_request('joindre_zip')
-					and $contenu_zip = joindre_verifier_zip($files)
+					&& !_request('joindre_zip')
+					&& ($contenu_zip = joindre_verifier_zip($files))
 				) {
 					[$fichiers, $erreurs, $tmp_zip] = $contenu_zip;
 					if ($fichiers) {

inc/joindre_document.php

@@ -248,17 +248,20 @@ function joindre_upload_error($error) {
  * Verifier si le fichier poste est un zip
  * Si on sait le deballer, proposer les options necessaires
  *
+ * Retourne [liste des fichiers, liste des erreurs, chemin du zip]
+ *
  * @param array $files
- * @return string
+ * @return false|array<array<string,array>, array<int,string>, string>
  */
 function joindre_verifier_zip($files) {
 	if (
 		function_exists('gzopen')
-		and (count($files) == 1)
-		and !isset($files[0]['distant'])
-		and
-		(preg_match('/\.zip$/i', $files[0]['name'])
-			or (isset($files[0]['type']) and $files[0]['type'] == 'application/zip'))
+		&& (count($files) == 1)
+		&& !isset($files[0]['distant'])
+		&& (
+			preg_match('/\.zip$/i', $files[0]['name'])
+			|| (isset($files[0]['type']) && $files[0]['type'] == 'application/zip')
+		)
 	) {
 		// on pose le fichier dans le repertoire zip
 		// (nota : copier_document n'ecrase pas un fichier avec lui-meme
@@ -275,11 +278,11 @@ function joindre_verifier_zip($files) {
 		include_spip('inc/archives');
 		if (
 			$zip
-			and $archive = new Spip\Archives\SpipArchives($zip)
-			and $infos = $archive->informer()
-			and $contenu = joindre_decrire_contenu_zip($infos)
-			and $tmp = sous_repertoire(_DIR_TMP, 'zip')
-			and rename($zip, $tmp = $tmp . basename($zip))
+			&& ($archive = new Spip\Archives\SpipArchives($zip))
+			&& ($infos = $archive->informer())
+			&& ($contenu = joindre_decrire_contenu_zip($infos))
+			&& ($tmp = sous_repertoire(_DIR_TMP, 'zip'))
+			&& rename($zip, $tmp = $tmp . basename($zip))
 		) {
 			$zip_to_clean = (isset($GLOBALS['visiteur_session']['zip_to_clean']) ?
 				unserialize($GLOBALS['visiteur_session']['zip_to_clean']) : []);
@@ -301,7 +304,7 @@ function joindre_verifier_zip($files) {
  * - une liste des erreurs (fichiers refuses)
  *
  * @param object $zip
- * @return array
+ * @return false|array<array<string,array>, array<int,string>>
  */
 function joindre_decrire_contenu_zip($infos) {
 	include_spip('action/ajouter_documents');