version 4.0.0, compatible SPIP 4.1 et 4.2
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;
}