Bug 182625 - Missing constructor in IncludedFeatureReference class
Summary: Missing constructor in IncludedFeatureReference class
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Update (deprecated - use Eclipse>Equinox>p2) (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M7   Edit
Assignee: Platform-Update-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-04-16 16:11 EDT by James D. Miles CLA
Modified: 2007-06-05 16:36 EDT (History)
1 user (show)

See Also:

Attachments
Adds constructor (1.54 KB, patch)
2007-04-16 16:15 EDT, James D. Miles CLA 		no flags 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 James D. Miles CLA 2007-04-16 16:11:19 EDT 
This includes a fix regressed by bug 140942, digest is not generated
correctly

In update.core.model.FeatureModel
	public void addIncludedFeatureReferenceModel(IncludedFeatureReferenceModel include) {
		assertIsWriteable();
		if (this.featureIncludes == null)
			this.featureIncludes = new ArrayList();
		//PERF: no ListContains()
		//if (!this.featureIncludes.contains(include))
		this.featureIncludes.add(new IncludedFeatureReference(include));
	}
There was no constructor for IncludedFeatureReferences(IncludedFeatureReference). This resulted in the filter information received from an update site being lost.

This patch adds the missing constructor.
Comment 1 James D. Miles CLA 2007-04-16 16:15:51 EDT 
Created attachment 63945 [details]
Adds constructor
Comment 2 James D. Miles CLA 2007-04-16 16:18:53 EDT 
Last note should have said:
There was no constructor for
IncludedFeatureReference(IncludedFeatureReferenceModel). This resulted in the
filter information received from an update site being lost.
Comment 3 Dejan Glozic CLA 2007-04-16 16:23:42 EDT 
Thanks.