édition d'une fiche dans une popup (#73689) #83
fpeters
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/73689-card-edit-popup"
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?
WIP: édition d'une fiche dans une popupto WIP: édition d'une fiche dans une popup (#73689)2217df620cto67267053c567267053c5to729d3f9fb0729d3f9fb0to54410cd36454410cd364tof8bf6abaf8WIP: édition d'une fiche dans une popup (#73689)to édition d'une fiche dans une popup (#73689)@ -1217,0 +1217,4 @@query = get_request().form.get('q', '')limit = get_request().form.get('page_limit')edit_related = info.get('edit_related')with_edit_related_url = query and limit and edit_relatedpour l'autocomplete, on a toujours un param q et une pagination; par précaution, si l'un est manquant, pas de vérification des droits du user pour éviter de jouer ça sur toutes les fiches
@ -59,0 +73,4 @@include_category=True,language=get_publisher().current_language,)+ 'wfedit-%s' % _item.id(j'ai repris la même url que dans la méthode submit_form de l'action editable)
@ -1216,1 +1216,4 @@custom_view.filters = info['dynamic_custom_view_filters']query = get_request().form.get('q', '')limit = get_request().form.get('page_limit')edit_related = info.get('edit_related')(edit_related est passé dans le context du token)
@ -2269,0 +2264,4 @@if carddef.can_user_add_cards(get_request().user):kwargs['add_related_url'] = carddef.get_backoffice_submission_url()kwargs['edit_related'] = Trueurl_kwargs['edit_related'] = Truepour ajout dans le contexte du token
@ -2318,2 +2321,3 @@url_kwargs['edit_related'] = True# store display value in session to be used by select2url = data_source.get_jsonp_url()url = data_source.get_jsonp_url(**url_kwargs)pas sûre que cette modif soit nécessaire ?
@ -1054,3 +1054,3 @@if self.edit_mode:existing_formdata = self.edited_data.dataif not get_request().form:request_data = {k: v for k, v in get_request().form.items() if k != '_popup'}sinon pas de données initiales dans le form d'édition en mode popup ...
(j'ai ajouté un test pour ça)
@ -334,0 +336,4 @@if (data.edit_related_url) {$(data.element).attr('data-edit-related-url', data.edit_related_url);}return data.text;je me suis un peu perdue dans le js, ça marche comme ça mais ça éparpille pas mal les choses ...
là c'est pour ajouter un data attribute sur les options du select à l'autocompletion
@ -407,1 +409,3 @@elem.options[elem.options.length] = new Option(newRepr, newId, true, true);var $option = $('<option />').val(newId).html(newRepr).prop('selected', true);if (edit_related_url) {$option.attr('data-edit-related-url', edit_related_url);ajout d'un data attribute sur l'option sélectionnée à la fermeture de la popup
@ -373,0 +380,4 @@// update edit-related button href$(elem).siblings('.edit-related').attr('href', '').hide();if ($selected.attr('data-edit-related-url')) {$(elem).siblings('.edit-related').attr('href', $selected.attr('data-edit-related-url') + '?_popup=1').show();affichage du lien d'édition si on a un data attribute qui va bien sur l'option sélectionnée
@ -379,2 +391,4 @@option.appendTo($(elem));option.text($input_display_value.val());if ($(elem).data('initial-edit-related-url')) {option.attr('data-edit-related-url', $(elem).data('initial-edit-related-url'));pour gérer la valeur initiale du champ, et avoir le bouton d'édition
Je serais tenté par remplacer le crayon 🖉 par une réelle icône (mais elle n'existe pas pour le survol, j'ai créé entrouvert/gadjo#2)
J'ai une branche rebasée qui ajoute ça mais il y avait un conflit au rebase, il me semble que c'est juste get_display_value qui est déplacée mais je préfère être frileux et juste attacher ici mon patch supplémentaire plutôt qu'écraser ta branche.
nouvelle version avec l'icône proposée