Autorisation et nom d'objet composé #3735

Closed
opened 7 years ago by miros · 2 comments
miros commented 7 years ago

Hello,

Il y a un sale bug dans le système d'autorisation:

Je voulais ajouter le support des documents sur les réponses de formidable (objet formulaires_reponse). J'ai donc activer cela dans l'admin. Mais ça ne fonctionne pas.
En réalité, autorisé "détruit" tout les "_" et il est impossible d'activer les documents sur un objet SPIP dont le nom serait composé:

if ($faire == 'joindredocument') {
	var_dump($type);
}

$type = str_replace('_', '', strncmp($type, "_", 1) == 0 ? $type : objet_type($type, false));

if ($faire == 'joindredocument') {
	var_dump($type);
	}

Renvoie :

string 'formulaires_reponse' (length=19)
string 'formulairesreponse' (length=18)

Je pense qu'il faut corriger la suppression des "" et ajouter la recherche d'une fonction pour le $type qui n'a pas "" pour ne pas casser les fonctions d'autorisation qui ont pris en compte ce bug.

Hello, Il y a un sale bug dans le système d'autorisation: Je voulais ajouter le support des documents sur les réponses de formidable (objet formulaires_reponse). J'ai donc activer cela dans l'admin. Mais ça ne fonctionne pas. En réalité, autorisé "détruit" tout les "_" et il est impossible d'activer les documents sur un objet SPIP dont le nom serait composé: <pre> if ($faire == 'joindredocument') { var_dump($type); } $type = str_replace('_', '', strncmp($type, "_", 1) == 0 ? $type : objet_type($type, false)); if ($faire == 'joindredocument') { var_dump($type); } </pre> Renvoie : <pre> string 'formulaires_reponse' (length=19) string 'formulairesreponse' (length=18) </pre> Je pense qu'il faut corriger la suppression des "_" et ajouter la recherche d'une fonction pour le $type qui n'a pas "_" pour ne pas casser les fonctions d'autorisation qui ont pris en compte ce bug.
Poster
Corrigé par http://zone.spip.org/trac/spip-zone/changeset/96025
b_b commented 7 years ago
Owner

< b_b‎ > mais type_surnoms m'a sauveR !

On ferme :)
Statut changé à Fermé

> < b_b‎ > mais type_surnoms m'a sauveR ! On ferme :) **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.