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

RichText: input in the internal format is validated twice

    Details

      Description

      Input value is validated in the acceptValue(), using validator dispatcher. As both input the result of conversion is validated, input in internal format gets validated twice.

      Internal format validator should be passed to the constructor separately from validator dispatcher, and validation if internal format should be moved to the validate() method.

        Issue Links

          Activity

          Show
          Petar Spanja (Inactive) added a comment - Pull request: https://github.com/ezsystems/ezpublish-kernel/pull/1274
          Show
          Petar Spanja (Inactive) added a comment - Merged in https://github.com/ezsystems/ezpublish-kernel/commit/941666113ace2400241de9b6aed06e24f0824980
          Hide
          Rui Silva (Inactive) added a comment -

          Cannot save / publish richtext field contents.
          Currently being blocked EZP-24907.

          Show
          Rui Silva (Inactive) added a comment - Cannot save / publish richtext field contents. Currently being blocked EZP-24907 .
          Hide
          Rui Silva (Inactive) added a comment -

          Currently being blocked by EZP-24732, which doesn't allow to publish / save richtext fieldtype contents, thus blocking the validation to proceed.

          Show
          Rui Silva (Inactive) added a comment - Currently being blocked by EZP-24732 , which doesn't allow to publish / save richtext fieldtype contents, thus blocking the validation to proceed.
          Hide
          Rui Silva (Inactive) added a comment -

          Sanity tests were executed on RichText, by using its various subfields and chaining them with several formattings and styles.
          Remaining validation seems to work fine (after the removal of one other validation process).
          (Had to manually remove the last "empty" paragraph from richtext field that doesn't allow publishing of the richtext content, which is an issue)
          Approved by QA, for master.

          Show
          Rui Silva (Inactive) added a comment - Sanity tests were executed on RichText, by using its various subfields and chaining them with several formattings and styles. Remaining validation seems to work fine (after the removal of one other validation process). (Had to manually remove the last "empty" paragraph from richtext field that doesn't allow publishing of the richtext content, which is an issue) Approved by QA, for master.

            People

            • Assignee:
              Unassigned
              Reporter:
              Petar Spanja (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 2 hours, 15 minutes
                2h 15m