dolibarr icon indicating copy to clipboard operation
dolibarr copied to clipboard

Allow moveUploadedFile hook to receive more parameters

Open vdegrandpre opened this issue 1 year ago • 1 comments

Feature Request

Add parameters to hook function so that $disablevirusscan and $uploaderrorcode are passed along.

Use case

When needed to implement a hook function for moveUploadedFile, we need to be able to reproduce the standard behaviour.

In our case, we need to append a suffix to the file name when the uploaded file already exists (i.e. same filename is given when the uploaded picture is took with an iPad).

We need to keep the actual behaviour of AV scanning and error core handling in the hook function.

Suggested implementation

Add those parameters in the array that is sent to the hook function.

Suggested steps

No response

vdegrandpre avatar Apr 26 '23 14:04 vdegrandpre

This issue is stale because it has been open 1 year with no activity. If this is a bug, please comment to confirm it is still present on latest stable version. if this is a feature request, please comment to notify the request is still relevant and not yet covered by latest stable version. This issue may be closed automatically by stale bot in 10 days (you should still be able to re-open it if required).

github-actions[bot] avatar Apr 25 '24 21:04 github-actions[bot]