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

Cannot use Multi-file upload on Microsoft Edge

    Details

      Description

      Cannot use Multi-file upload on Microsoft Edge.

      Steps:

      1. Log in to admin panel using Edge as browser.
      2. Go to Content / Content structure.
      3. Enter a container item e.g. folder.
      4. Try using Multi-file upload by clicking on the Upload sub-items button and selecting files via Upload sub-items from the pop-up.
      5. Try using Multi-file upload by clicking on the Upload sub-items button and using drag & drop onto the pop-up.
      6. Try using Multi-file upload by using drag & drop onto the Sub-items list.

      Actual: Nothing happens in UI, in console error occurs: "SCRIPT5002: Function expected".
      Expected: Files are uploaded without errors.

        Activity

        Hide
        Jakub Brzegowski added a comment - - edited

        I debuged the problem and the error is caused by spread operator in line:
        https://github.com/ezsystems/ezplatform-admin-ui-modules/blob/fb8794adf0ca9a0cf98eb9d2474bff532ffd305e/src/modules/multi-file-upload/multi.file.upload.module.js#L153
        because `list.files` is not an array but rather `FileList`. Problem can be solved by using `Array.from(list.files)` instead of `[...list.files]`. The question is, is this the solution that we want.

        Show
        Jakub Brzegowski added a comment - - edited I debuged the problem and the error is caused by spread operator in line: https://github.com/ezsystems/ezplatform-admin-ui-modules/blob/fb8794adf0ca9a0cf98eb9d2474bff532ffd305e/src/modules/multi-file-upload/multi.file.upload.module.js#L153 because `list.files` is not an array but rather `FileList`. Problem can be solved by using `Array.from(list.files)` instead of ` [...list.files] `. The question is, is this the solution that we want.
        Show
        Michał Szołtysek added a comment - PR: https://github.com/ezsystems/ezplatform-admin-ui-modules/pull/115
        Hide
        Michał Szołtysek added a comment -

        QA Approved.

        Show
        Michał Szołtysek added a comment - QA Approved.
        Show
        Michał Szołtysek added a comment - Merged to 1.3: https://github.com/ezsystems/ezplatform-admin-ui-modules/commit/5e1c317afda3b5b1db61a20eced1a603bdf5653d

          People

          • Assignee:
            Unassigned
            Reporter:
            Michał Szołtysek
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: