Form system improvements #41

Closed
opened 2024-05-30 08:07:25 +00:00 by phryk · 0 comments
Owner

Expose fields through dict-style items (getitem, setitem, …).

Make sure setting assigns name/prefix (i.e. does what add_field does now.

Also, have a go at AutoForm; a common use case overrides its handle function with an entire copy of the logic,
with the only distinction being that redirects from downstream fieldsets are ignored in lieu for a simple reload (redirect('')).
-> look whether the downstream redirects are actually used anywhere, if not just change the default behavior, at least for AutoForm (debatable if for BaseForm)

Expose fields through dict-style items (__getitem__, __setitem__, …). Make sure setting assigns name/prefix (i.e. does what `add_field` does now. Also, have a go at `AutoForm`; a common use case overrides its `handle` function with an entire copy of the logic, with the only distinction being that redirects from downstream fieldsets are ignored in lieu for a simple reload (`redirect('')`). -> look whether the downstream redirects are actually used anywhere, if not just change the default behavior, at least for `AutoForm` (debatable if for `BaseForm`)
phryk added this to the post-deployment milestone 2024-05-30 08:07:25 +00:00
phryk added the
Maintainability
label 2024-05-30 08:07:25 +00:00
phryk closed this issue 2024-05-31 14:42:53 +00:00
Sign in to join this conversation.
No Milestone
No project
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: phryk/pdnew#41
No description provided.