refactor(DownloadFilesButton): Remove useless code

cozy · Nov 27, 2024 · 668bf97 · 668bf97
1 parent ebca3c1
commit 668bf97
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 20 deletions.
diff --git a/src/modules/public/DownloadButton.jsx → src/modules/public/DownloadFilesButton.jsx b/src/modules/public/DownloadButton.jsx → src/modules/public/DownloadFilesButton.jsx
@@ -10,33 +10,32 @@ import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 
 import { downloadFiles } from 'modules/actions/utils'
 
-const DownloadButton = ({ onDownload, ...props }) => (
-  <Button
-    onClick={() => onDownload()}
-    startIcon={<Icon icon={DownloadIcon} />}
-    {...props}
-  />
-)
-
-export default DownloadButton
-
-export const DownloadFilesButton = ({ files, className }) => {
+export const DownloadFilesButton = ({
+  files,
+  variant = 'secondary',
+  ...props
+}) => {
   const { t } = useI18n()
   const client = useClient()
   const { showAlert } = useAlert()
 
+  const handleClick = () => {
+    downloadFiles(client, files, { showAlert, t })
+  }
+
   return (
-    <DownloadButton
+    <Button
       label={t('toolbar.menu_download')}
       data-testid="fil-public-download"
-      onDownload={() => {
-        downloadFiles(client, files, { showAlert, t })
-      }}
-      variant="secondary"
-      className={className}
+      startIcon={<Icon icon={DownloadIcon} />}
+      onClick={handleClick}
+      variant={variant}
+      {...props}
     />
   )
 }
+
 DownloadFilesButton.propTypes = {
-  files: PropTypes.array.isRequired
+  files: PropTypes.array.isRequired,
+  variant: PropTypes.string
 }
diff --git a/src/modules/public/PublicToolbarByLink.jsx b/src/modules/public/PublicToolbarByLink.jsx
@@ -14,7 +14,7 @@ import AddMenuProvider from 'modules/drive/AddMenu/AddMenuProvider'
 import AddButton from 'modules/drive/Toolbar/components/AddButton'
 import AddMenuItem from 'modules/drive/Toolbar/components/AddMenuItem'
 import SelectableItem from 'modules/drive/Toolbar/selectable/SelectableItem'
-import { DownloadFilesButton } from 'modules/public/DownloadButton'
+import { DownloadFilesButton } from 'modules/public/DownloadFilesButton'
 import PublicToolbarMoreMenu from 'modules/public/PublicToolbarMoreMenu'
 import { useSelectionContext } from 'modules/selection/SelectionProvider'
 

diff --git a/src/modules/public/PublicToolbarCozyToCozy.jsx b/src/modules/public/PublicToolbarCozyToCozy.jsx
@@ -9,7 +9,7 @@ import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 import { BarRightOnMobile } from 'components/Bar'
 import { download, openExternalLink } from 'modules/actions'
 import SelectableItem from 'modules/drive/Toolbar/selectable/SelectableItem'
-import { DownloadFilesButton } from 'modules/public/DownloadButton'
+import { DownloadFilesButton } from 'modules/public/DownloadFilesButton'
 import PublicToolbarMoreMenu from 'modules/public/PublicToolbarMoreMenu'
 import { useSelectionContext } from 'modules/selection/SelectionProvider'