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

Bug 323906

Summary: [Tooling] Branding defaultEntrypointId browser should honor IApplication extensions
Product: [RT] RAP Reporter: Benjamin Wolff <eclipse>
Component: ToolsAssignee: Project Inbox <rap-inbox>
Status: RESOLVED WONTFIX QA Contact:
Severity: enhancement    
Priority: P3 CC: artur.kronenberg, elias
Version: 1.4   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 221540    
Bug Blocks:    

Description Benjamin Wolff CLA 2010-08-29 09:22:26 EDT
When setting a defaultEntrypointId, the Browse-Wizard does only check for IEntrypoint extension IDs. It should also check for IApplication extension IDs and show them to the user to choose from.
Comment 1 Artur Kronenberg CLA 2010-09-06 08:20:35 EDT
This bug seems to be blocked by bug Nr. 221540 for now. Currently it seems to be only possible to reference one identifier and we'd need to reference two.
Comment 2 Elias Volanakis CLA 2011-01-13 06:53:51 EST
As a workaround I propose to use the type String instead of Id (in the extension point).

The problem is that when using an application id, one gets a warning "referenced identifier 'app.id' could not be found", because the tooling looks for an entry point id.

This would be solved by using a String. Drawback: no 'Browse' button.
Comment 3 Ralf Sternberg CLA 2011-01-14 09:53:51 EST
I agree with the suggestion in comment 2. Since both entrypoint ids and an application extension ids are admissible as values for this attribute, an identifier that restricts the values to entrypoint ids only is wrong.

BTW, the application attribute to the o.e.core.runtime.products extension point also accepts Strings instead of identifiers, so it's not completely uncommon. Applied change to CVS HEAD.
Comment 4 Ivan Furnadjiev CLA 2013-01-14 04:51:19 EST
In RAP 2.0 there is no more defaultEntrypointId in branding. Close as WONTFIX.