Bugzilla – Full Text Bug Listing
| Summary: | NPE when use Operation "Open Dialog" in Tree Representation | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Modeling] Sirius | Reporter: | M V <mv> | ||||
| Component: | Tree | Assignee: | Florian Barbin <florian.barbin> | ||||
| Status: | CLOSED FIXED | QA Contact: | Julien Dupont <julien.dupont> | ||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | florian.barbin, julien.dupont, laurent.fasani, pierre-charles.david | ||||
| Version: | 5.0.0 | Keywords: | triaged | ||||
| Target Milestone: | 5.1.0 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| See Also: |
https://git.eclipse.org/r/104969 https://git.eclipse.org/r/105195 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=3d40d8a67cd9fda2c99f8f90d21fec2d92c6cddb https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=0d59a7a926dc90fc3c49634b7079dd33b03a4e7c |
||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
M V
Hi, from the look of the stack, one possible explanation would be that the org.eclipse.sirius.ui.properties is not present in the runtime context (where you actually run your modeler). The "Open Dialog" and "Open Wizard" operations are not actually contributed by the core of Sirius, but by the the (optional) EEF-based properties view support feature. It's possible that you have this feature installed in your development environment (it must be because you were able to create an "Open Dialog" operation), but that it is not available in your runtime. If this explanation is correct, you can probably fix the issue by having an explicit dependency from your modeler plug-in to org.eclipse.sirius.ui.properties. Can you confirm? We'll need to document this (https://www.eclipse.org/sirius/doc/specifier/general/Model_Operations.html#dialogandwizard does not mention it), and make the code more robust to avoid the NPE anyway. This issue is effectively linked to that the plug-in that contains the Dialog declaration is not present in the runtime. Scenario to reproduce: * launch a runtime which have org.eclipse.sirius.ui.properties and oes.ui.properties.ext.widgets.reference * import both vsp and Sirius modeling project(attached with this bug) into the runtime * open testOpenDialogOp project and "my_diag1" diagram * use create_class tool and click on diagram -> a dialog box is opened * close dialog -> a class has been created * Now, relaunch the runtime but without any of oes.ui.properties.xxx plug-in (change launch config and uncheck plug-in) * open testOpenDialogOp project and "my_diag1" diagram * use create_class tool and click on diagram -> NPE Created attachment 269995 [details]
vsp and testOpenDialogOp modeling project
(In reply to Pierre-Charles David from comment #1) > Hi, > > from the look of the stack, one possible explanation would be that the > org.eclipse.sirius.ui.properties is not present in the runtime context > (where you actually run your modeler). The "Open Dialog" and "Open Wizard" > operations are not actually contributed by the core of Sirius, but by the > the (optional) EEF-based properties view support feature. It's possible that > you have this feature installed in your development environment (it must be > because you were able to create an "Open Dialog" operation), but that it is > not available in your runtime. If this explanation is correct, you can > probably fix the issue by having an explicit dependency from your modeler > plug-in to org.eclipse.sirius.ui.properties. > > Can you confirm? > > We'll need to document this > (https://www.eclipse.org/sirius/doc/specifier/general/Model_Operations. > html#dialogandwizard does not mention it), and make the code more robust to > avoid the NPE anyway. Yes, I can confirm that this was fixing the problem. (In reply to M V from comment #4) > (In reply to Pierre-Charles David from comment #1) > > Hi, > > > > from the look of the stack, one possible explanation would be that the > > org.eclipse.sirius.ui.properties is not present in the runtime context > > (where you actually run your modeler). The "Open Dialog" and "Open Wizard" > > operations are not actually contributed by the core of Sirius, but by the > > the (optional) EEF-based properties view support feature. It's possible that > > you have this feature installed in your development environment (it must be > > because you were able to create an "Open Dialog" operation), but that it is > > not available in your runtime. If this explanation is correct, you can > > probably fix the issue by having an explicit dependency from your modeler > > plug-in to org.eclipse.sirius.ui.properties. > > > > Can you confirm? > > > > We'll need to document this > > (https://www.eclipse.org/sirius/doc/specifier/general/Model_Operations. > > html#dialogandwizard does not mention it), and make the code more robust to > > avoid the NPE anyway. > > Yes, I can confirm that this was fixing the problem. Thanks for the confirmation. We'll improve the documentation and update the code to provide a more useful error message if it occurs at runtime. New Gerrit change created: https://git.eclipse.org/r/104969 New Gerrit change created: https://git.eclipse.org/r/105195 Gerrit change https://git.eclipse.org/r/104969 was merged to [master]. Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=3d40d8a67cd9fda2c99f8f90d21fec2d92c6cddb Will be marked as fixed once the Junit test (that fails on hudson for now) will be merged. Gerrit change https://git.eclipse.org/r/105195 was merged to [master]. Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=0d59a7a926dc90fc3c49634b7079dd33b03a4e7c Available in Sirius 5.1.0, see https://wiki.eclipse.org/Sirius/5.1.0. |