Error in file_unmanaged_copy "The specified file %file could not be copied, because the destination directory is not properly configured. ..." cause by file_entity_uuid_presave that didn't check file size and destination directory. Some streamwrapper classed, like MediaReadOnlyStreamWrapper didn't provide destination directory.

Initial patch attached.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pingers’s picture

Issue summary: View changes
FileSize
1.1 KB

Re-roll against alpha6.

RoSk0’s picture

Assigned: RoSk0 » Unassigned
joseph.olstad’s picture

Marking this as RTBC.

We've been using uuid-empty_file-2121031-1.patch for quite some time (more than 8 months) on previous revisions of uuid and this patch is now being used and tested in dev/qa/production evironments with uuid revision a7bf2db (the latest).

Please commit uuid-empty_file-2121031-1.patch

joseph.olstad’s picture

Status: Needs review » Reviewed & tested by the community

I'm marking uuid-empty_file-2121031-1.patch as RTBC

as for the other one, it's no longer needed. So just commit uuid-empty_file-2121031-1.patch

joseph.olstad’s picture

maijs’s picture

I'm updating the patch to match the latest code changes.

skwashd’s picture

Thanks for the patch. I finally committed it.

  • skwashd committed bc2a94d on 7.x-1.x authored by pingers
    Issue #2121031 by RoSk0, pingers, maijs: Error when trying to create...
skwashd’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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