From ca138aa9feb68bf76876ace2d161fcd1add9fe8a Mon Sep 17 00:00:00 2001 From: David McKnight Date: Tue, 12 May 2009 17:59:25 +0000 Subject: [PATCH] [190001] [refresh] Avoid unnecessary duplicate queries during drag&drop to filter --- .../UI/org/eclipse/rse/internal/ui/view/SystemView.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemView.java b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemView.java index de6276a09b9..9f9d5f683c3 100644 --- a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemView.java +++ b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemView.java @@ -68,6 +68,7 @@ * David McKnight (IBM) - [251625] Widget disposed exception when renaming/pasting a folder * David McKnight (IBM) - [257721] Doubleclick doing special handling and expanding * David McKnight (IBM) - [190805] [performance][dstore] Right-click > Disconnect on a dstore connection is slow and spawns many Jobs + * David McKnight (IBM) - [190001] [refresh] Avoid unnecessary duplicate queries during drag&drop to filter ********************************************************************************/ package org.eclipse.rse.internal.ui.view; @@ -2649,7 +2650,11 @@ public class SystemView extends SafeTreeViewer if (selectedFilters == null) selectedFilters = new ArrayList(); selectedFilters.add(filterItem); } - smartRefresh(new TreeItem[] { filterItem }, null, true); + + Object filterRef = filterItem.getData(); + if (filterRef != remoteResourceParent){ // don't refresh it here if the filter reference is the object to refresh anyway + smartRefresh(new TreeItem[] { filterItem }, null, true); + } } } // now, refresh all occurrences of the remote parent object.