Software / code / prosody
Changeset
9090:56c52cb4d44e
util.dataforms: Exclude descriptive text fields from forms of type 'submit'
The receiving end presumably already have the original form, so these
potentially long text fields are of little value.
| author | Kim Alvefur <zash@zash.se> |
|---|---|
| date | Fri, 03 Aug 2018 21:45:55 +0200 |
| parents | 9089:3a7a0b9f42f3 |
| children | 9091:519dea077d20 |
| files | util/dataforms.lua |
| diffstat | 1 files changed, 13 insertions(+), 9 deletions(-) [+] |
line wrap: on
line diff
--- a/util/dataforms.lua Fri Aug 03 21:06:59 2018 +0200 +++ b/util/dataforms.lua Fri Aug 03 21:45:55 2018 +0200 @@ -31,19 +31,23 @@ if formtype == "cancel" then return form; end - if layout.title then - form:tag("title"):text(layout.title):up(); - end - if layout.instructions then - form:tag("instructions"):text(layout.instructions):up(); + if formtype ~= "submit" then + if layout.title then + form:tag("title"):text(layout.title):up(); + end + if layout.instructions then + form:tag("instructions"):text(layout.instructions):up(); + end end for _, field in ipairs(layout) do local field_type = field.type or "text-single"; -- Add field tag - form:tag("field", { type = field_type, var = field.name, label = field.label }); + form:tag("field", { type = field_type, var = field.name, label = formtype ~= "submit" and field.label or nil }); - if field.desc then - form:text_tag("desc", field.desc); + if formtype ~= "submit" then + if field.desc then + form:text_tag("desc", field.desc); + end end local value; @@ -120,7 +124,7 @@ form:up(); end - if field.required then + if formtype == "form" and field.required then form:tag("required"):up(); end
