[arvados] updated: 2.7.0-6057-ge99e454b41

git repository hosting git at public.arvados.org
Thu Mar 21 14:41:32 UTC 2024


Summary of changes:
 .../components/multiselect-toolbar/MultiselectToolbar.tsx    | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

       via  e99e454b4134724fecc12cb53549a99b19849f02 (commit)
      from  fb969936f88f70d6aeba224a44c44a6787b8b600 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit e99e454b4134724fecc12cb53549a99b19849f02
Author: Lisa Knox <lisaknox83 at gmail.com>
Date:   Thu Mar 21 10:41:23 2024 -0400

    21224: removed vestigial inputSelectedUuid param in toolbar Arvados-DCO-1.1-Signed-off-by: Lisa Knox <lisa.knox at curii.com>

diff --git a/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx b/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
index 81eaaa8638..6d33d6bf76 100644
--- a/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
+++ b/services/workbench2/src/components/multiselect-toolbar/MultiselectToolbar.tsx
@@ -83,7 +83,7 @@ export type MultiselectToolbarProps = {
     auth: AuthState;
     location: string;
     isSubPanel?: boolean;
-    executeMulti: (action: ContextMenuAction | MultiSelectMenuAction, inputSelectedUuid: string | null, checkedList: TCheckedList, resources: ResourcesState) => void;
+    executeMulti: (action: ContextMenuAction | MultiSelectMenuAction, checkedList: TCheckedList, resources: ResourcesState) => void;
 };
 
 type IconProps = {
@@ -149,7 +149,7 @@ export const MultiselectToolbar = connect(
                                         <IconButton
                                             data-cy='multiselect-button'
                                             disabled={disabledButtons.has(name)}
-                                            onClick={() => props.executeMulti(action, selectedResourceUuid, checkedList, iconProps.resources)}
+                                            onClick={() => props.executeMulti(action, checkedList, iconProps.resources)}
                                             className={classes.icon}
                                         >
                                             {currentPathIsTrash || (useAlts && useAlts(selectedResourceUuid, iconProps)) ? altIcon && altIcon({}) : icon({})}
@@ -168,7 +168,7 @@ export const MultiselectToolbar = connect(
                                         <IconButton
                                             data-cy='multiselect-button'
                                             onClick={() => {
-                                                props.executeMulti(action, selectedResourceUuid, checkedList, iconProps.resources)}}
+                                                props.executeMulti(action, checkedList, iconProps.resources)}}
                                             className={classes.icon}
                                         >
                                             {action.icon({})}
@@ -346,9 +346,9 @@ function mapStateToProps({auth, multiselect, resources, favorites, publicFavorit
 
 function mapDispatchToProps(dispatch: Dispatch) {
     return {
-        executeMulti: (selectedAction: ContextMenuAction, inputSelectedUuid: string | null, checkedList: TCheckedList, resources: ResourcesState): void => {
-            const kindGroups = inputSelectedUuid ? groupByKind({[inputSelectedUuid]: true}, resources) : groupByKind(checkedList, resources);
-            const currentList = inputSelectedUuid ? [inputSelectedUuid] : selectedToArray(checkedList)
+        executeMulti: (selectedAction: ContextMenuAction, checkedList: TCheckedList, resources: ResourcesState): void => {
+            const kindGroups = groupByKind(checkedList, resources);
+            const currentList = selectedToArray(checkedList)
             switch (selectedAction.name) {
                 case MultiSelectMenuActionNames.MOVE_TO:
                 case MultiSelectMenuActionNames.REMOVE:

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list