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

Checkbox not stored on custom attribute if editor is disabled and there is no default value

    Details

      Description

      The fix from this ticket (https://jira.ez.no/browse/EZP-21124) makes it so that the checkbox is saved when the editor is enabled. However, if you disable the editor and the re-enable the editor, the checkbox is lost.

      To reproduce the issue, set up the same test case as https://jira.ez.no/browse/EZP-21124, save a checked checkbox when the editor is enabled, then disable the editor, then enable the editor. The checkbox will no longer be checked.

        Issue Links

          Activity

          Hide
          Peter Keung added a comment -

          Pull request: https://github.com/ezsystems/ezpublish-legacy/pull/897

          This fix just ensures that the value of the checkbox falls back to "1" instead of empty so that its "checked" property is preserved when disabling and then re-enabling the editor.

          Show
          Peter Keung added a comment - Pull request: https://github.com/ezsystems/ezpublish-legacy/pull/897 This fix just ensures that the value of the checkbox falls back to "1" instead of empty so that its "checked" property is preserved when disabling and then re-enabling the editor.
          Hide
          Damien Pobel (Inactive) added a comment - - edited
          Show
          Damien Pobel (Inactive) added a comment - - edited community PR waiting for reviews: https://github.com/ezsystems/ezpublish-legacy/pull/897
          Show
          Damien Pobel (Inactive) added a comment - PR merged in https://github.com/ezsystems/ezpublish-legacy/commit/6e7a4e93ffa831a338358ce9b3911c0bd2a30251
          Hide
          Damien Pobel (Inactive) added a comment - - edited

          @QA: mostly likely the patch for EZP-21124 needs to be applied as well

          Show
          Damien Pobel (Inactive) added a comment - - edited @QA: mostly likely the patch for EZP-21124 needs to be applied as well
          Hide
          Marcos Loureiro (Inactive) added a comment -

          QA Approved

          Show
          Marcos Loureiro (Inactive) added a comment - QA Approved

            People

            • Assignee:
              Unassigned
              Reporter:
              Peter Keung
            • Votes:
              0 Vote for this issue
              Watchers:
              5 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 - 3 hours, 45 minutes
                3h 45m