Problem/Motivation

EntityFile currently is local only. This is not necessary. It also calls deprecated methods. This is not necessary either.

Proposed resolution

Inject file_system and stream_wrapper_manager. Mourn we do not require PHP 5.6. Code around the stupid old PHP restrictions. Use the services.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
we_want_php_56.patch5.36 KBchx

Comments

Status: Needs review » Needs work

The last submitted patch, we_want_php_56.patch, failed testing.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Needs work » Closed (duplicate)
Related issues: +#2417975: EntityFile destination incorrectly assumes temporary files are absolute paths

The fixes in this patch overlap with the work being done over in #2417975: EntityFile destination incorrectly assumes temporary files are absolute paths. I'll integrate these changes into the patch being developed there.