Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323906 - [Tooling] Branding defaultEntrypointId browser should honor IApplication extensions
Summary: [Tooling] Branding defaultEntrypointId browser should honor IApplication exte...
Status: RESOLVED WONTFIX
Alias: None
Product: RAP
Classification: RT
Component: Tools (show other bugs)
Version: 1.4   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 221540
Blocks:
  Show dependency tree
 
Reported: 2010-08-29 09:22 EDT by Benjamin Wolff CLA
Modified: 2013-01-14 04:51 EST (History)
2 users (show)

See Also:

Attachments
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 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.