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

Bug 394805

Summary: "The parameter 'path' must not be null" when running RAP with Workbench
Product: [RT] RAP Reporter: Stefan Milchram <stefan.milchram>
Component: WorkbenchAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P1    
Version: 2.0   
Target Milestone: 2.0 M4   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Stefan Milchram CLA 2012-11-21 11:06:00 EST
MILESTONE: M3
Since M3 you have to specify a path for your IApplication directly in org.eclipse.core.runtime.applications, otherwise an exception is thrown.
But the "The parameter 'path' must not be null" exception is present despite specifiying the path for your own IApplication because the plugin "org.eclipse.equinox.app" declares a "ErrorApplication" which of course doesnt have a "path" set and the exception is thrown. The same if you use birt, there is also a ReportExecutor registered as an IApplication.
Comment 1 Ralf Sternberg CLA 2012-11-21 12:25:21 EST
Apparently, we must ignore applications that doesn't have a path parameter.
Comment 2 Ralf Sternberg CLA 2012-12-06 11:23:10 EST
Another side-effect of this change is that an existing IApplication cannot be made available in RAP without modification. This was possible before.
I'd suggest to change the way IApplications are mapped to paths and brandings as follows:

In an entrypoint extension, we allow to specify the id of an IApplication extension alternatively to an EntryPoint class. To make an IApplication available in RAP, you have to provide an entrypoint extension that refers to its id. All related properties (path, brandingId) are configured in the entrypoint extension. Consequences:

* We need to add an additional attribute to the entrypoint extension.
* We don't evaluate IApplication properties, there's also no risk of collisions.
* For every servlet path, there is exactly one entrypoint extension.
* There's only one place to specify path and branding.
Comment 3 Ralf Sternberg CLA 2012-12-08 12:11:41 EST
Changed as suggested in comment 2 with commit 30c54bb0ba10530748f782258c0d6fa20224f665.