fix: don't pass metadataCallback to the options if metadata present

README.md

@@ -141,6 +141,9 @@ The option can be used to set metadata object associated with the uploaded file.
 
 Note that metadata supports `string` values only, any non-string value will be converted to `string`, including `boolean`, `number`, `null` and `undefined`.
 
+**WARNING**: If this option is specified, option `metadataCallback` will be overridden (without merging).
+This is opposite from the uploadcare-widget's behavior.
+
 See [Metadata docs][uc-docs-metadata] for details.
 
 <br>
@@ -149,8 +152,6 @@ See [Metadata docs][uc-docs-metadata] for details.
 
 Defines the function that specifies the actual metadata object a file uploader should use to associate with the uploaded file. It's helpful in the case of dynamic metadata object.
 
-If this option is specified, option `metadata` will be overridden (without merging).
-
 Note that metadata supports `string` values only, any non-string value will be converted to `string`, including `boolean`, `number`, `null` and `undefined`.
 
 See [Metadata docs][uc-docs-metadata] for details.

src/dialog.js

@@ -95,7 +95,9 @@ const useDialog = (props, uploadcare) => {
         multipleMax: restProps.multiple ? undefined : 1,
         ...restProps,
         multiple: true,
-        metadataCallback: metadataCommitedCallback,
+        metadataCallback: restProps.metadata
+          ? undefined
+          : metadataCommitedCallback,
         previewUrlCallback: previewUrlCommitedCallback
       }
     )
@@ -137,8 +139,13 @@ const useDialog = (props, uploadcare) => {
       dialogApi.fileColl.onReplace.remove(onChangeWrapper)
       dialogApi.fileColl.onSort.remove(onChangeWrapper)
     }
-  // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [...Object.values(restProps), onTabChangeCallback, onChangeCallback, onProgressCallback])
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [
+    ...Object.values(restProps),
+    onTabChangeCallback,
+    onChangeCallback,
+    onProgressCallback
+  ])
 
   useImperativeHandle(
     apiRef,

src/uploader.js

@@ -83,7 +83,9 @@ const useWidget = (
     const inputEl = input.current
     widget.current = uploadcare.Widget(inputEl, {
       ...widgetOptions,
-      metadataCallback: metadataCommitedCallback,
+      metadataCallback: widgetOptions.metadata
+        ? undefined
+        : metadataCommitedCallback,
       previewUrlCallback: previewUrlCommitedCallback
     })