Attachment 118515 Details for Bug 256139 – backported patch

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] backported patch

patch.txt (text/plain), 2.78 KB, created by David McKnight

on 2008-11-21 14:51:32 EST

(hide)

Description:

Filename:

MIME Type:

Creator: David McKnight

Created: 2008-11-21 14:51:32 EST

Size: 2.78 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.rse.ui
>Index: UI/org/eclipse/rse/internal/ui/view/SystemViewPart.java
>===================================================================
>RCS file: /cvsroot/dsdp/org.eclipse.tm.rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemViewPart.java,v
>retrieving revision 1.39
>diff -u -r1.39 SystemViewPart.java
>--- UI/org/eclipse/rse/internal/ui/view/SystemViewPart.java	22 Aug 2008 13:00:36 -0000	1.39
>+++ UI/org/eclipse/rse/internal/ui/view/SystemViewPart.java	21 Nov 2008 19:39:15 -0000
>@@ -33,6 +33,7 @@
>  * David Dykstal (IBM) - [216858] Need the ability to Import/Export RSE connections for sharing
>  * Kevin Doyle 	 (IBM) - [186769] Enable Contributions to Drop Down menu of Remote Systems view -> Preferences
>  * David McKnight (IBM)  - [244807] System view does not handle restore from cache
>+ * Li Ding        (IBM)          - [256135] Subsystem not restored in system view tree if subsystem configuration does not support filter
>  *******************************************************************************/
> 
> package org.eclipse.rse.internal.ui.view;
>@@ -1316,6 +1317,12 @@
> 					break;
> 					// filter pool or filter (depends on showFilterPools)
> 				case 3 :
>+					
>+					if (!(subsystem.getSubSystemConfiguration().supportsFilters())) {
>+						remoteObject = new RemoteObject(token, subsystem, null, null);
>+						break;
>+					}
>+					
> 					if (showFilterPools)
> 					{
> 						if (subsystem != null)
>@@ -1360,6 +1367,12 @@
> 					break;
> 					// filter or filter string (depends on showFilterPools) or remote object (depends on showFilterStrings)
> 				case 4 :
>+					
>+					if (!(subsystem.getSubSystemConfiguration().supportsFilters())) {
>+						remoteObject = new RemoteObject(token, subsystem, null, null);
>+						break;
>+					}
>+					
> 					if (showFilterPools) // definitely a filter
> 					{
> 						index = token.indexOf('=');
>@@ -1396,6 +1409,12 @@
> 					break;
> 					// filter string (depends on showFilterStrings) or remote object
> 				case 5 :
>+					
>+					if (!(subsystem.getSubSystemConfiguration().supportsFilters())) {
>+						remoteObject = new RemoteObject(token, subsystem, null, null);
>+						break;
>+					}
>+					
> 					if (showFilterPools && showFilterStrings) // definitely a filter string
> 					{
> 						// at this point we know the parent filter reference as that was parsed in case 4
>@@ -1416,6 +1435,12 @@
> 
> 					break;
> 				default : // definitely a remote object
>+					
>+					if (!(subsystem.getSubSystemConfiguration().supportsFilters())) {
>+						remoteObject = new RemoteObject(token, subsystem, null, null);
>+						break;
>+					}
>+				
> 					if ((subsystem != null) && (fRef != null))
> 						remoteObject = new RemoteObject(token, subsystem, fRef, fsRef);
> 			}

Actions: View | Diff

Attachments on bug 256139: 118515