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

ezt: handle NoneFieldVar values (#78017) #353

Merged
fpeters merged 1 commits from wip/78017-handle-NoneFieldVar-in-ezt-templates into main 2023-05-30 18:22:00 +02:00
Conversation 0 Commits 1 Files Changed 2 +9
tnoel commented 2023-05-30 16:35:06 +02:00
Owner
Copy Link

J'étais parti pour faire un unlazy général avec « if hasattr(ob, 'get_value'): ob = ob.get_value() » ... mais ça me semble un peu bourrin, et je préfère limiter le patch à la gestion du NoneFieldVar, même si c'est un peu moche.

J'étais parti pour faire un unlazy général avec « if hasattr(ob, 'get_value'): ob = ob.get_value() » ... mais ça me semble un peu bourrin, et je préfère limiter le patch à la gestion du NoneFieldVar, même si c'est un peu moche.
lguerin reviewed 2023-05-30 16:37:39 +02:00
tests/test_formdata.py Outdated
@ -699,6 +699,7 @@ def variable_test_data(pub):
fields.StringField(id='12', label='float1', varname='value'),
fields.PasswordField(id='13', label='pwd', varname='pwd'),
fields.EmailField(id='14', label='email', varname='email'),
fields.EmailField(id='15', label='item', items=['aa', 'ab', 'ac'], varname='itemfield'),
lguerin commented 2023-05-30 16:37:30 +02:00
Owner
Copy Link

ItemField, non ? mais a priori ça marche aussi avec EmailField :)

ItemField, non ? mais a priori ça marche aussi avec EmailField :)
tnoel commented 2023-05-30 16:41:27 +02:00
Author
Owner
Copy Link

La fatigue. En fait oui ça marche pour n'importe quel field qui est vide (None). Mais je vais corriger ça...

La fatigue. En fait oui ça marche pour n'importe quel field qui est vide (None). Mais je vais corriger ça...
tnoel marked this conversation as resolved
tnoel force-pushed wip/78017-handle-NoneFieldVar-in-ezt-templates from beb9a54ce8 to 4d983e59ec 2023-05-30 16:51:03 +02:00 Compare
lguerin approved these changes 2023-05-30 16:52:34 +02:00
tnoel force-pushed wip/78017-handle-NoneFieldVar-in-ezt-templates from 4d983e59ec to 56f20bcf29 2023-05-30 17:37:39 +02:00 Compare
fpeters merged commit 56f20bcf29 into main 2023-05-30 18:22:00 +02:00
fpeters deleted branch wip/78017-handle-NoneFieldVar-in-ezt-templates 2023-05-30 18:22:01 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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#353
No description provided.
Delete Branch "wip/78017-handle-NoneFieldVar-in-ezt-templates"

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?