autoriser() doit retourner un bool

cf https://git.spip.net/spip/medias/src/branch/master/medias_autoriser.php#L287 on retourne htaccess en lien avec le plugin acces_restreint.

A trouver où et quoi et comment gérer ça proprement

@cerdic a ajouté le label amélioration

@cerdic a fait référence à ce ticket depuis un commentaire de spip/spip#5341 Fatale avec un lien vers un document inexistant et Accès restreint protégeant les documents

@marcimat a fait référence à ce ticket depuis un commentaire de spip/spip#5649 pb retour fonction autoriser()

@touti a fait référence à ce ticket depuis un commentaire de spip-contrib-extensions/acces_restreint#17 Deprecated: Function autoriser_document_voir should returns a boolean instead of string

Mon analyse à deux sous :

• Les seuls appels de la fonction autoriser('voir','document') se font dans les plugins medias et acces_restreint
• Dans ces 2 plugins la seule fois où l'on s'interesse à la valeur htaccess sont
  • https://git.spip.net/spip-contrib-extensions/acces_restreint/src/branch/master/urls/generer_url_document.php#L49
  • https://git.spip.net/spip/medias/src/branch/master/inc/documents.php#L84

Les 2 fois c'est suite à un appel à autoriser('voir','document') sans id donc, et dans ce cas, on tombe donc sur https://git.spip.net/spip/medias/src/branch/master/medias_autoriser.php#L281 qui renvoie donc toujours false.

Donc les seules fois où l'on attends la valeur htaccess on ne peut pas l'avoir car on a un false.

Les autres fois ou la fonction autoriser renvoie htaccess c'est homogène à un true.

@cerdic a fait référence à ce ticket depuis !4943 fix: normaliser la fonction autoriser_document_voir() pour lui faire retourner uniquement true ou false

@cerdic a référencé ce ticket depuis une révision fix: prendre en compte l'option htaccess dans l'appel a autoriser('voir','document'...), et utiliser l'option à bon escient

@cerdic a fait référence à ce ticket depuis spip-contrib-extensions/acces_restreint!20 fix: prendre en compte l'option htaccess dans l'appel a autoriser('voir','document'...), et utiliser l'option à bon escient

cf #4943 pour le plugin medias et https://git.spip.net/spip-contrib-extensions/acces_restreint/pulls/20 pour acces_restreint, si je ne me trompe pas

@marcimat a référencé ce ticket depuis une révision docs(Changelog): pour #4931

@marcimat a référencé ce ticket depuis une révision docs(Changelog): pour #4931

@marcimat a référencé ce ticket depuis une révision fix: prendre en compte l'option htaccess dans l'appel a autoriser('voir','document'...), et utiliser l'option à bon escient

Intégré, on ferme.

@b_b a fermé ce ticket

@b_b a ajouté ce ticket au jalon spip-4.2

@touti a fait référence à ce ticket depuis un commentaire de spip/spip#5874 PHP Deprecated: Function autoriser_document_voir should returns a boolean

Pff, raa, les logs apache que je lis date de septembre huhu, je ferme !

mentioned in commit 917d2047

mentioned in commit b241b5b6