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

Bug 320609

Summary: NPE in ProjectDescription class
Product: [WebTools] WTP Source Editing Reporter: Rakesh <rakes123>
Component: jst.jspAssignee: Rakesh <rakes123>
Status: RESOLVED FIXED QA Contact: Nitin Dahyabhai <thatnitind>
Severity: normal    
Priority: P3 CC: davidmichaelkarr, doug.satchwell, nsand.dev, thebravoman
Version: 3.2.1Flags: nsand.dev: review+
Target Milestone: 3.2.2   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
patch
none
patch nsand.dev: iplog+

Description Rakesh CLA 2010-07-22 06:32:55 EDT
Build Identifier: I20100608-0911

If resource is outofsych with local file system , zip returned is null and we get an NPE.

Reproducible: Always
Comment 1 Rakesh CLA 2010-07-22 06:34:22 EDT
Created attachment 174954 [details]
patch 

Null Pointer check added.
Comment 2 Nick Sandonato CLA 2010-07-27 14:37:29 EDT
While I know it's probably negligible, instead of checking if zip is null on each iteration of the loop, we only really have to do it once before we enter the loop. Can you please provide an update?
Comment 3 Rakesh CLA 2010-07-28 02:51:12 EDT
Created attachment 175369 [details]
patch

NP checked outside loop.
Comment 4 Nick Sandonato CLA 2010-07-28 15:29:22 EDT
Thanks, Rakesh.
Comment 5 Nitin Dahyabhai CLA 2010-07-30 23:18:52 EDT
*** Bug 321412 has been marked as a duplicate of this bug. ***
Comment 6 Nick Sandonato CLA 2010-08-09 15:32:07 EDT
Released.
Comment 7 Nitin Dahyabhai CLA 2010-08-26 13:24:35 EDT
*** Bug 319095 has been marked as a duplicate of this bug. ***
Comment 8 Nitin Dahyabhai CLA 2010-09-17 15:33:03 EDT
*** Bug 325643 has been marked as a duplicate of this bug. ***