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

Impossible to remove file from BinaryFile and Media fields

    Details

      Description

      Steps to reproduce
      • Create one video object with one video file (Video1.mp4)
      • Publish the video
      • Edit the video object and remove the video file
      • Publish the video

      Here we can see that the video file is still on the video

      I couldn't see any error either on logs or in browser console.

        Issue Links

          Activity

          Hide
          Damien Pobel (Inactive) added a comment - - edited

          Note: the default Video content type uses a BinaryFile field but the same issue appears with Media fields.

          When removing the file from a BinaryFile/Media field, the fieldValue of the field is set to null in the VersionUpdate struct but this seems to be ignored by the REST API (also tried false, an empty object, an empty string, ...). As as I can remember this used to work as expected.

          Show
          Damien Pobel (Inactive) added a comment - - edited Note: the default Video content type uses a BinaryFile field but the same issue appears with Media fields. When removing the file from a BinaryFile/Media field, the fieldValue of the field is set to null in the VersionUpdate struct but this seems to be ignored by the REST API (also tried false , an empty object, an empty string, ...). As as I can remember this used to work as expected.
          Hide
          Bertrand Dunogier added a comment -

          We might very well have a regression on that part, yes. Do you approximately remember "when" it worked as expected, [~damien.pobel@ez.no] ?

          Show
          Bertrand Dunogier added a comment - We might very well have a regression on that part, yes. Do you approximately remember "when" it worked as expected, [~damien.pobel@ez.no] ?
          Show
          Bertrand Dunogier added a comment - Pull-request https://github.com/ezsystems/ezpublish-kernel/pull/1483 .
          Show
          Bertrand Dunogier added a comment - https://github.com/ezsystems/ezpublish-kernel/pull/1483 merged to master@84d98c1 .
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

          Show
          Paulo Nunes (Inactive) added a comment - QA Approved

            People

            • Assignee:
              Unassigned
              Reporter:
              Paulo Nunes (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              3 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 - 1 hour, 10 minutes
                1h 10m