Fix <FileInput> triggers onChange twice with latest version of react-dropzone

[slax57]

Problem

Fix #10401

Analysis

The issue only happens after upgrading react-dropzone to the latest version (14.3.5 at the time of writing). Version 14.2.3 does not show the issue.

At first I thought it seemed like a regression from the react-dropzone lib, but on second thought, I think we should not forward the onChange prop to the wrapping <Labeled> anyways. We should only call onChange ourselves, to make sure it's always fed with a File object or array.

Solution

Exclude the onChange prop from the ...rest parameters.

How To Test

Locally update the react-dropzone version: yarn up react-dropzone.
Then, run one of the following stories, and make sure they only trigger onChange once, with either a File object or array.

• http://localhost:9010/?path=/story/ra-ui-materialui-input-fileinput--on-change
• http://localhost:9010/?path=/story/ra-ui-materialui-input-fileinput--on-change-multiple

Additional Checks

• The PR targets master for a bugfix, or next for a feature
• The PR includes unit tests (if not possible, describe why) -> not possible, current unit tests about setting a file are disabled because jsdom doesn't support the drop event
• The PR includes one or several stories (if not possible, describe why)
• The documentation is up to date

Also, please make sure to read the contributing guidelines.