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

Bug 516771

Summary: The toString of class BundleModel always returns 'Unknown Bundle Model'
Product: [Eclipse Project] PDE Reporter: Olivier Prouvost <olivier.prouvost>
Component: UIAssignee: Andrey Loskutov <loskutov>
Status: VERIFIED FIXED QA Contact:
Severity: trivial    
Priority: P3 CC: Lars.Vogel, loskutov, Vikas.Chandra
Version: 4.5Flags: Lars.Vogel: review+
Vikas.Chandra: review+
Target Milestone: 4.7 RC1   
Hardware: PC   
OS: All   
See Also: https://git.eclipse.org/r/97303
https://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=439ecbb31ae6631cbc298552c2ab5f19b2a1466e
Whiteboard:

Description Olivier Prouvost CLA 2017-05-16 18:58:56 EDT
In the class : org.eclipse.pde.internal.core.bundle.BundleModel toString method is : 

	public String toString() {
		if (fBundle != null) {
			StringBuffer buf = new StringBuffer();
			buf.append(fBundle.getHeader(Constants.BUNDLE_SYMBOLICNAME));
			buf.append(" ("); //$NON-NLS-1$
			buf.append(fBundle.getHeader(Constants.BUNDLE_VERSION));
			buf.append(')');
		}
		return "Unknown bundle model"; //$NON-NLS-1$
	}

It always returns Unknown bundle model and this method is not overridden in the unique child (org.eclipse.pde.internal.core.bundle.WorkspaceBundleModel).
Comment 1 Eclipse Genie CLA 2017-05-17 03:24:53 EDT
New Gerrit change created: https://git.eclipse.org/r/97303
Comment 2 Andrey Loskutov CLA 2017-05-17 03:26:01 EDT
Good catch!
Comment 3 Vikas Chandra CLA 2017-05-17 03:34:33 EDT
I will triage this to 4.8M1 when that target milestone is available.
Comment 4 Lars Vogel CLA 2017-05-17 04:09:34 EDT
IMHO this qualifies as polish.
Comment 5 Vikas Chandra CLA 2017-05-17 05:37:18 EDT
Thanks Lars, I deliberated and was 50-50.

Sooner is always better :)
Comment 6 Lars Vogel CLA 2017-05-17 05:45:45 EDT
(In reply to Vikas Chandra from comment #5)
> Thanks Lars, I deliberated and was 50-50.
> 
> Sooner is always better :)

Thanks, I read this as "ok to merge for RC1". I merge soon if I do hear otherwise.
Comment 8 Vikas Chandra CLA 2017-05-17 06:05:09 EDT
Thanks Lars !
Comment 9 Andrey Loskutov CLA 2017-05-30 04:19:30 EDT
Verified in I20170528-2000