Problem/Motivation

Sometimes files aren't uploaded because they couldn't be moved from the temp directory to the private files directory on submit of the MediaBulkUploadForm. Users have no way of knowing this until they check the file overview afterwards and see there's no file present. An error message should immediately make it clear to the user that a file wasn't uploaded.

Steps to reproduce

  1. Open the media bulk upload form.
  2. Drop files into the DropzoneJS field.
  3. Remove files from the tmp directory.
  4. Submit the media bulk upload form.
  5. Drupal wouldn't show an error message that the files weren't added.

Proposed resolution

Drupal should display an error message to the user which clearly indicates which files weren't uploaded correctly.

Remaining tasks

  • Decide whether the file entity status should be temporary instead of permanent before the file_move succeeds.

User interface changes

-

API changes

-

Data model changes

-

CommentFileSizeAuthor
#4 media_bulk_upload-display_file_move_error_message-3207649-1.patch793 byteskevin.-

Issue fork media_bulk_upload-3207649

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Kevin.- created an issue. See original summary.

Kevin.- opened merge request !1

Kevin.- closed merge request !1

kevin.-’s picture

kevin.- commented
Status: Active » Needs review
StatusFileSize
new media_bulk_upload-display_file_move_error_message-3207649-1.patch793 bytes

  • Nitebreed committed c8e34b1 on 8.x-1.x authored by Kevin.- 
    Issue #3207649 by Kevin.-: No error message shown if moving file from...
nitebreed’s picture

nitebreed
Primary language Dutch
commented
Status: Needs review » Fixed

Committed, thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.