Sterc / FormIt

A dynamic form processing Snippet for MODX Revolution
https://docs.modx.com/current/en/extras/formit
33 stars 58 forks source link

Missing trim on validate rules #237

Open Jako opened 4 years ago

Jako commented 4 years ago

Bug report

Summary

There is a missing trim on the validate rules. The last rule needs to be empty (by adding a trailing comma after the last rule) or all rules have to be finished directly with the backtick.

Step to reproduce

This does not work:

                      &validate=`
                        name:required,
                        email:email:required,
                        message:required,
                        dsgvo:required
                      `

This works fine:

                      &validate=`
                        name:required,
                        email:email:required,
                        message:required,
                        dsgvo:required,
                      `

This works fine, too:

                      &validate=`
                        name:required,
                        email:email:required,
                        message:required,
                        dsgvo:required`

Observed behavior

The last rule is not executed.

Expected behavior

The last rule should be executed.

Environment

FormIt 4.2.5, MODX 2.7.3