Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 353735 - Album.loadAlbum(boolean) throws NPE as it does not set systemId
Summary: Album.loadAlbum(boolean) throws NPE as it does not set systemId
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-edc (show other bugs)
Version: 8.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Ken Ryall CLA
QA Contact: Ken Ryall CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-03 07:12 EDT by Daniel Thomas CLA
Modified: 2012-05-22 20:37 EDT (History)
2 users (show)

See Also:

Attachments
Provide a systemId so that NPE does not occur. (1.37 KB, patch)
2011-08-03 07:12 EDT, Daniel Thomas CLA 		cdtdoug: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.