- Issue created by @dan_metille
- ๐ฐ๐ฌKyrgyzstan dan_metille
Make file save to the fieldโs configured file_directory instead of hard-coded public://
- ๐บ๐ธUnited States CoffeyMachine
Hi @dan_metille, this is a really good catch, thank you! It's a use case that I totally missed.
Looking at this I noticed that the
public://
upload destination is hard-coded. It would be good to also include support for the Private files upload destination with this enhancement.I'm also having trouble applying your patch- I'm getting
error: corrupt patch at line 46
. Would you be able to create an issue fork instead? See: https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... โ - Merge request !4Issue#3522556: Fix images ignore Media Image field File Directory setting. โ (Open) created by divyansh.gupta
- ๐ฎ๐ณIndia divyansh.gupta Jaipur
@coffeymachine, Converted the patch provided by @dan_metille into MR, Please review and merge!!
- ๐บ๐ธUnited States CoffeyMachine
@divyansh.gupta Thank you for creating the MR.
This enhancement still needs to support the Private files upload destination which is not included in the current version of the MR. So, I am moving this back to "Needs work" status.