Skip to content

version 4.0.0, compatible SPIP 4.1 et 4.2

b_b a demandé de fusionner gh-c4bb7565/12/unknown/refs/pull/12/head vers master

J'allais pousser le niveau de phpstan à 3 mais je tombe sur ces erreurs :

  Line   plugins/compositions/compositions_fonctions.php                                 
 ------ -------------------------------------------------------------------------------- 
  493    Function compositions_verrouiller() should return string but returns true.      
  506    Function compositions_verrouiller() should return string but returns true.      
  511    Function compositions_verrouiller() should return string but returns false.     
  514    Function compositions_verrouiller() should return string but returns false.     
  527    Function compositions_verrou_branche() should return string but returns false.  
  531    Function compositions_verrou_branche() should return string but returns true. 

J'ai comme l'impression que le phpdoc est foireux et que le @return devrait être un bool pour ces deux fonctions, vos avis sur le patch qui va en ce sens ?

diff --git a/compositions_fonctions.php b/compositions_fonctions.php
index c6ecf9c..84f3ff2 100644
--- a/compositions_fonctions.php
+++ b/compositions_fonctions.php
@@ -485,7 +485,7 @@ function balise_COMPOSITION_dist($p) {
  * @param string $type
  * @param integer $id
  * @param string $serveur
- * @return string
+ * @return bool
  */
 function compositions_verrouiller($type, $id, $serveur = '') {
 	$config = (isset($GLOBALS['meta']['compositions']) ? unserialize($GLOBALS['meta']['compositions']) : []);
@@ -519,7 +519,7 @@ function compositions_verrouiller($type, $id, $serveur = '') {
  * Indique si les objets d'une branche sont verrouilles
  * @param integer $id_rubrique
  * @param string $serveur
- * @return string
+ * @return bool
  */
 function compositions_verrou_branche($id_rubrique, $serveur = '') {
 
@@ -533,5 +533,5 @@ function compositions_verrou_branche($id_rubrique, $serveur = '') {
 			return compositions_verrou_branche($infos_rubrique['id_parent'], $serveur);
 		}
 	}
-	return '';
+	return false;
 }

Rapports de requête de fusion

Chargement en cours