Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ServerSide flagged fields are inaccessible in custom script adapter #396

Open
sauzher opened this issue Jan 12, 2023 · 0 comments · May be fixed by #397
Open

ServerSide flagged fields are inaccessible in custom script adapter #396

sauzher opened this issue Jan 12, 2023 · 0 comments · May be fixed by #397

Comments

@sauzher
Copy link

sauzher commented Jan 12, 2023

collective.easyform/src/collective/easyform/actions.py

Line 641 in c33922c

resultData = self.sanifyFields(request.form)

Here onSuccess -> fields method paremeter comes with all fields submited and all server side flagged ones also, but calling sanifyFields will clear them all: the custom script will run on another fields parameter populated only with the fields in the request so without server side ones.
sauzher added a commit that referenced this issue Jan 12, 2023
@sauzher
Fix #396. Custom action scripts gets server side flagged fields in th…
ad9c911 
…e `fields` dict parameter under the `name_of_the_field` key.
sauzher added a commit to sauzher/collective.easyform that referenced this issue Jan 12, 2023
@sauzher
Fix collective#396. Custom action scripts gets server side flagged fi…
13f1c09 
…elds in the `fields` dict parameter under the `name_of_the_field` key.
sauzher added a commit to sauzher/collective.easyform that referenced this issue Jan 18, 2023
@sauzher
Fix collective#396. Custom action scripts gets server side flagged fi…
ed7f207 
…elds in the `fields` dict parameter under the `name_of_the_field` key.
@sauzher sauzher linked a pull request Jan 18, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Missing server side variable in custom script adapters sauzher/collective.easyform
1 participant
@sauzher