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

Bug 353735

Summary: Album.loadAlbum(boolean) throws NPE as it does not set systemId
Product: [Tools] CDT Reporter: Daniel Thomas <daniel.thomas>
Component: cdt-debug-edcAssignee: Ken Ryall <ken.ryall>
Status: RESOLVED FIXED QA Contact: Ken Ryall <ken.ryall>
Severity: normal    
Priority: P3 CC: cdtdoug, kirk.beitz
Version: 8.0   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Provide a systemId so that NPE does not occur. cdtdoug: iplog+

Description Daniel Thomas CLA 2011-08-03 07:12:19 EDT 
Build Identifier: 

Album#loadAlbum(boolean) does not call setSystemId on the InputSource it creates and so when this inputSource is parsed a NullPointerException is thrown inside the XML parser.
This causes a large number of messages about there not being recognised symbolics to be in the error log.

Reproducible: Always

Steps to Reproduce:
1. Debug AllEDCTests breaking on NullPointerExceptions
Comment 1 Daniel Thomas CLA 2011-08-03 07:12:45 EDT 
Created attachment 200796 [details]
Provide a systemId so that NPE does not occur.
Comment 2 Kirk Beitz CLA 2011-08-06 06:28:53 EDT 
the patch has been merged and nominally tested in our local source base.  waiting for final review and merge to eclipse edc git repository by committer Ken Ryall.