Hi @guenole,
I have updated the code you pushed to fix the current issue. The array_slice method had a problem, but it is now working properly.
prefer #MR !2 for the fix
ajay-mallah → made their first commit to this issue’s fork.
Hello,
I've followed the steps mentioned by @belazoth and I was able to reproduce the issue.
After that, I applied the patch #3, and the issue was fixed.
Updated the branch, Please review.
@avpaderno,
Based on your suggestions, I reviewed the module code, refactored it, and removed unnecessary sections to enhance the module's clarity and performance.
@avpaderno, I have made the changes as per the suggestions in #13. I was occupied with a project, so I assigned the co-maintainer @vishal-prasad to verify the changes and commit them to the discussion thread. In the future, I will handle the suggested changes myself.
ajay-mallah → created an issue.
Hii @vishalkadam
I have made the suggested changes can you please review them, if I need to make more changes
@geoanders
I have tested the new feature developed by @vishal prasad and it is working properly.
These are the test cases that I tested
- Opted download and save option, and files are getting downloaded as well as stored at selected file stream location.
- If the path is selected to custom file, files are getting stored at the given path relative to the default file stream path
- Also tested drush commands.
Attaching the updated form's UI.
ajay-mallah → created an issue.