Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-26240

Update the editing UI with server side validation

    Details

      Description

      After EZP-24853, the REST API now reports the field validation errors. The UI should be updated with that.

        Issue Links

          Activity

          Hide
          Damien Pobel (Inactive) added a comment -

          Rough technical way of implementing that:

          • the saveAction and publishAction event facades are already "enhanced" in Content Edit View to receive the fields values and the UI validation state of the form. There, it could also receive a callback that is supposed to be called when the server detects some validation issues.
          • this callback would given back the errors in a structure way to the Content Edit Form View which should then give those to the concerned field edit views.

          One thing to take into account at some point (not necessarily in this story): we are also saving the draft before a preview

          Show
          Damien Pobel (Inactive) added a comment - Rough technical way of implementing that: the saveAction and publishAction event facades are already "enhanced" in Content Edit View to receive the fields values and the UI validation state of the form. There, it could also receive a callback that is supposed to be called when the server detects some validation issues. this callback would given back the errors in a structure way to the Content Edit Form View which should then give those to the concerned field edit views. One thing to take into account at some point (not necessarily in this story): we are also saving the draft before a preview
          Show
          Stéphane Diot added a comment - PR : https://github.com/ezsystems/PlatformUIBundle/pull/672
          Hide
          Miguel das Neves Jacinto (Inactive) added a comment - - edited

          PR approved by QA

          Show
          Miguel das Neves Jacinto (Inactive) added a comment - - edited PR approved by QA
          Hide
          Stéphane Diot added a comment -

          merged in master : https://github.com/ezsystems/PlatformUIBundle/commit/e0500029079201d7902ea171f0a7be96d0a6b41e

          Instructions for doc team : Maybe this should be in user guide. This patch gives the edit views error messages coming from the serverside validation. So if the user removes the app validation he will have the serverside errors ( with more details than app ones )

          Show
          Stéphane Diot added a comment - merged in master : https://github.com/ezsystems/PlatformUIBundle/commit/e0500029079201d7902ea171f0a7be96d0a6b41e Instructions for doc team : Maybe this should be in user guide. This patch gives the edit views error messages coming from the serverside validation. So if the user removes the app validation he will have the serverside errors ( with more details than app ones )

            People

            • Assignee:
              Unassigned
              Reporter:
              Damien Pobel (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: