From c46a90ccfa578f2086f129544a6cc10fc00c12b7 Mon Sep 17 00:00:00 2001 From: Xuan Chen Date: Fri, 4 Apr 2008 14:08:06 +0000 Subject: [PATCH] fix: [225685] NPE when running archive testcases --- .../rse/internal/ui/view/SystemTableViewPart.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemTableViewPart.java b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemTableViewPart.java index a44f761a325..588030344d6 100644 --- a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemTableViewPart.java +++ b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemTableViewPart.java @@ -29,6 +29,7 @@ * David McKnight (IBM) - [223103] [cleanup] fix broken externalized strings * David McKnight (IBM) - [224313] [api] Create RSE Events for MOVE and COPY holding both source and destination fields * David McKnight (IBM) - [225506] [api][breaking] RSE UI leaks non-API types + * Xuan Chen (IBM) - [225685] NPE when running archive testcases ********************************************************************************/ package org.eclipse.rse.internal.ui.view; @@ -1671,9 +1672,11 @@ public class SystemTableViewPart extends ViewPart boolean referToSameObject = false; if (registry instanceof SystemRegistry) - { - referToSameObject = ((SystemRegistry)registry).isSameObjectByAbsoluteName(input, null, child, event.getOldNames()[0]); // right now assuming only one resource - } + { + String[] oldNames = event.getOldNames(); + String oldName = (oldNames == null)? null : oldNames[0]; + referToSameObject =((SystemRegistry)registry).isSameObjectByAbsoluteName(input, null, child,oldName); // right now assuming only one resource + } if (input == child || child instanceof java.util.List || referToSameObject) {