Bug #19183
Bug sur lien "Contient les permissions des rôles :"
100%
Description
- aucun résultat sur l'instance de test de Dreux (exemple : https://connexion-dreux.test.entrouvert.org/manage/roles/20/)
- un résultat surprenant (demande de code de suivi !, cf. copie écran jointe) sur l'instance de test de Blois (exemple : https://connexion-blois.test.entrouvert.org/manage/roles/48/)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Pierre Cros il y a plus de 6 ans
À noter qu'on parle de la recette (je reproduis pour le Nord). Pas de
problème en prod.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Lié à Development #18901: amélioration du back-office pour le CUT ajouté
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-fix-add-child-parent-role-popup-19183.patch ajouté
- Statut changé de Nouveau à Résolu (à déployer)
- Patch proposed changé de Non à Oui
Un patch, je ne sais pas trop quelles sont les repercutions sur le CUT pour l'instant.
La modif vient de ce commit :
commit 17ddb72acdac81c44790d927cad931b8e9c2d476 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Jul 18 11:36:44 2017 +0200 manager: do not use a popup for adding users (#18901) diff --git a/src/authentic2/manager/templates/authentic2/manager/users.html b/src/authentic2/manager/templates/authentic2/manager/users.html index 378f2738..f962a54a 100644 --- a/src/authentic2/manager/templates/authentic2/manager/users.html +++ b/src/authentic2/manager/templates/authentic2/manager/users.html @@ -8,7 +8,7 @@ {% block appbar %} {{ block.super }} {% if view.can_add and search_form.cleaned_data.ou %} - <a rel="popup" + <a href="{% url "a2-manager-user-add" ou_pk=search_form.cleaned_data.ou.pk %}" id="add-user-btn"> {% trans "Add user" %}
Benj doit avoir plus d'info sur la modif.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
Ah non me suis trompé désolé. plutot ce commit
commit 50f8b97d22ea4cd48a31139b76cbbfb9d11c2f66 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Jul 11 11:49:11 2017 +0200 manager: rework use of TitleMixin and get_title() (#18901)
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier
0001-fix-add-child-parent-role-popup-19183.patchsupprimé
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-fix-add-child-parent-role-popup-19183.patch 0001-fix-add-child-parent-role-popup-19183.patch ajouté
Un patch
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Josué si tu as compris le problème peux-tu l'expliquer ici s'il te plaît ?
Mis à jour par Josué Kouka il y a plus de 6 ans
Benjamin Dauvergne a écrit :
Josué si tu as compris le problème peux-tu l'expliquer ici s'il te plaît ?
Pour le premier cas exposé par Brice (... aucun résultat sur l'instance de test de Dreux ..) le problème vient du fait lors de l'ajout d'un fils/parent on a une erreur 500 parce que les classes RoleAddChildView
et RoleAddParentView
n'ont pas de méthode get_tile
.
Le cas un résultat surprenant (demande de code de suivi !, cf. copie écran jointe) sur l'instance de test de Blois
semble etre propre à Blois et je n'ai toujours pas compris.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|c7d06b4e7c65ef28bd210dd1c00812b99a438ac9.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Correction poussée, si on peut vérifier que ça marche mieux sur la dév, je ferme.
Mis à jour par Pierre Cros il y a plus de 6 ans
Ça ne marche pas sur
https://authentic-demo.dev.entrouvert.org/manage/roles/14/
Dialog box loading failed: error INTERNAL SERVER ERROR
Mis à jour par Josué Kouka il y a plus de 6 ans
Pierre Cros a écrit :
Ça ne marche pas sur
https://authentic-demo.dev.entrouvert.org/manage/roles/14/Dialog box loading failed: error INTERNAL SERVER ERROR
ça doit etre ok maintenant, la dev n'était pas à jour.
Mis à jour par Brice Mallet il y a plus de 6 ans
- Statut changé de Fermé à Information nécessaire
- Assigné à changé de Benjamin Dauvergne à Josué Kouka
Ce n'est probablement pas déployé en recette SaaS (eh oui, d'ailleurs je le vois en marge droite;-) puisque ne marche toujours pas sur Dreux (https://connexion-dreux.test.entrouvert.org/manage/roles/18/), or j'y fais une formation demain à compter de 9h00 !! Envisageable que soit corrigé courant de matinée ? Merci.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Ce n'est probablement pas déployé en recette SaaS [...]
Ça ne l'était pas, je viens de pousser sur notre SaaS de recette. Cela étant, il y a un autre bug sur cette instance qui empêche l'affaire de fonctionner, avec un trace similaire à #19168. Et là je ne peux rien faire comme ça.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Hack posé sur la recette pour que ça passe demain matin :
for block in blocks: cls.update_default(block) + if isinstance(block.get('basedn'), unicode): + block['basedn'] = block['basedn'].encode('utf-8')
manager: fix AttributeError on get_title() in RoleAddChildView and RoleAddParentView (fixes #19183)
Bug introduced by fix to #18901.