testdef, permettre d'indiquer qui clique sur un bouton d'action (#87543) #1232
fpeters
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/87543-testdef-permettre-d-indiquer-qui"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
e8d898df82to55a279f7b255a279f7b2todc6ebbdbd3dc6ebbdbd3to3c7cf4faba3c7cf4fabatoe27e711839WIP: testdef, permettre d'indiquer qui clique sur un bouton d'action (#87543)to testdef, permettre d'indiquer qui clique sur un bouton d'action (#87543)@ -236,0 +284,4 @@value=self.who,attrs={'data-dynamic-display-parent': 'true'},)user_options = [(x.id, str(x), x.id) for x in get_publisher().user_class.select(order_by='name')]Je crains la catastrophe sur une grosse instance avec plein d'utilisateurs; en regardant rapidement ce qu'on a pour la sélection de l'utilisateur lors de la saisie backoffice c'est beaucoup de js mais in fine, c'est un select2 qui utilise /api/users/; peut-être que ça marcherait tout seul ou presque en utilisant JsonpSingleSelectWidget et url='/api/users/'. (il pourrait y avoir un truc moche à faire pour remplir jsonp_display_values pour afficher la valeur sélectionnée).
Pas de soucis de ce côté là par contre ça a nécessité un peu de ruses côté JS, j'ai ajouté des commits je te laisse regarder
e27e711839tobd964389a9bd964389a9to015bce5336015bce5336to6da2e7c712@ -123,6 +123,212 @@ Responsive_table_widget.prototype.init = function () {});};function add_js_behaviours($base) {Je me demandais s'il n'y avait pas plus clean qu'ajouter la fonction au namespace global et certainement que oui mais c'est déjà ce qu'on fait pour le prepare_widget_list_elements() appelé un peu plus haut, donc ok ainsi en attendant un moment de grande passe sur l'organisation du js.
6da2e7c712to783f3a8bb4