entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

workflows: be explicit about global actions not supporting live forms (#89412) #1393

Open
fpeters wants to merge 1 commits from wip/89412-global-interactive-no-live into main
pull from: wip/89412-global-interactive-no-live
merge into: entrouvert:main
Conversation 0 Commits 1 Files Changed 9 +73 -7
fpeters commented 2024-04-11 09:44:20 +02:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2024-04-11 09:44:21 +02:00
fpeters force-pushed wip/89412-global-interactive-no-live from 965d0abb0a to 75d716742a 2024-04-11 19:49:18 +02:00 Compare
fpeters force-pushed wip/89412-global-interactive-no-live from 75d716742a to 635851cc6c 2024-04-11 20:01:20 +02:00 Compare
fpeters reviewed 2024-04-12 08:15:41 +02:00
wcs/admin/forms.py Outdated
@ -1271,2 +1271,3 @@
form = Form(action='#', use_tokens=False)
form.attrs['data-backoffice-preview'] = 'on'
form.attrs['data-backoffice-preview'] = 'true'
form.attrs['data-js-features'] = 'true'
fpeters commented 2024-04-12 08:12:02 +02:00
Author
Owner
Copy Link

Dans le js on a

  add_js_behaviours($('form[data-live-url], form[data-backoffice-preview], form[data-enable-select2]'));

un data-live-url non fonctionnel avait été ajouté sur les actions globales interactives (pour que les select2 fonctionnent, cf #80999).

Pour nettoyer ça je chnage les choses pour que les trucs JS soient ajoutés sur base de data-js-features, et j'ajoute donc l'attribut ici et là.

Dans le js on a ``` add_js_behaviours($('form[data-live-url], form[data-backoffice-preview], form[data-enable-select2]')); ``` un data-live-url non fonctionnel avait été ajouté sur les actions globales interactives (pour que les select2 fonctionnent, cf #80999). Pour nettoyer ça je chnage les choses pour que les trucs JS soient ajoutés sur base de data-js-features, et j'ajoute donc l'attribut ici et là.
wcs/admin/tests.py Outdated
@ -1049,3 +1049,3 @@
attrs={'data-dynamic-display-parent': 'true'},
)
form.attrs['data-enable-select2'] = 'on'
form.attrs['data-js-features'] = 'true'
fpeters commented 2024-04-12 08:12:44 +02:00
Author
Owner
Copy Link

Pareil pour avoir le js pour le select2 cet attribut avait été ajouté, avec aucun autre usage.

Pareil pour avoir le js pour le select2 cet attribut avait été ajouté, avec aucun autre usage.
wcs/formdef.py Outdated
@ -930,1 +930,3 @@
if not field.has_live_conditions(self, hidden_varnames=hidden_varnames):
if not getattr(form, 'has_live_form_support', True) or not field.has_live_conditions(
self, hidden_varnames=hidden_varnames
):
fpeters commented 2024-04-12 08:14:09 +02:00
Author
Owner
Copy Link

C'est la correction essentielle de cette PR, si le formulaire est déclaré explicitement avec has_live_form_support à False, alors les champs ne sont pas ajoutés du tout (vs autrement où ils sont ajoutés mais cachés en CSS/JS).

C'est la correction essentielle de cette PR, si le formulaire est déclaré explicitement avec `has_live_form_support` à False, alors les champs ne sont pas ajoutés du tout (vs autrement où ils sont ajoutés mais cachés en CSS/JS).
wcs/forms/actions.py Outdated
@ -184,4 +184,1 @@
messages = self.action.get_messages()
form.attrs['data-live-url'] = (
self.formdata.get_url(backoffice=get_request().is_in_backoffice()) + 'live'
)
fpeters commented 2024-04-12 08:14:51 +02:00
Author
Owner
Copy Link

Comme noté plus haut ce data-live-url était non fonctionnel, juste là pour permettre les select2.

Comme noté plus haut ce data-live-url était non fonctionnel, juste là pour permettre les select2.
wcs/forms/actions.py Outdated
@ -186,2 +185,2 @@
self.formdata.get_url(backoffice=get_request().is_in_backoffice()) + 'live'
)
get_response().add_javascript(['jquery.js', 'qommon.forms.js'])
form.attrs['data-js-features'] = 'true'
fpeters commented 2024-04-12 08:15:14 +02:00
Author
Owner
Copy Link

J'assure aussi que qommon.forms.js est ajouté, ça n'était pas vraiment garanti.

J'assure aussi que qommon.forms.js est ajouté, ça n'était pas vraiment garanti.
wcs/workflows.py Outdated
@ -2513,6 +2515,7 @@ class WorkflowGlobalAction(SerieOfActionsMixin):
name = None
triggers = None
backoffice_info_text = None
has_live_form_support = False
fpeters commented 2024-04-12 08:15:37 +02:00
Author
Owner
Copy Link

Pour les actions globales, pas de live.

Pour les actions globales, pas de live.
fpeters changed title from WIP: workflows: be explicit about global actions not supporting live forms (#89412) to workflows: be explicit about global actions not supporting live forms (#89412) 2024-04-12 08:15:49 +02:00
fpeters force-pushed wip/89412-global-interactive-no-live from 635851cc6c to b25d2b9d75 2024-04-19 14:37:12 +02:00 Compare
All checks were successful
gitea/wcs/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1393
No description provided.
Delete Branch "wip/89412-global-interactive-no-live"

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?