Association between features and realizing aspects not handled properly
Issue #298
resolved
- When associating an existing model (that is not part of the models reference, i.e., it does not show up in the aspect list), it is not added to that reference (or the list is not updated right away)
- Deletion of a model from the aspect container should be done using controllers/handlers.
- When deleting the currently selected aspect, menu buttons do not show up again though nothing is selected anymore in the aspects container.
- When deleting a model from the aspect container, the file is deleted, then removed from realizing the features, but when the concern is saved, an exception is thrown. It seems that the model wasn't removed from the models reference, but it doesn't always occur. It once also happened when a new aspect is created for a feature, saved (the existing one overwritten) and then the concern saved.
java.lang.NullPointerException
at ca.mcgill.sel.core.ui.handler.impl.DisplayConcernSceneHandler.save(DisplayConcernSceneHandler.java:87)
at ca.mcgill.sel.core.ui.scenes.DisplayConcernScene.actionPerformed(DisplayConcernScene.java:247)
- Related to the issue above, when deleting an aspect, all references are not removed (ie references for extensions of this aspect are not removed). This results in NullPointerExceptions when loading or weaving the aspect that extends the deleted aspect. We should either remove the references or prevent deletion if an aspect extends the one we wish to delete.
Comments (6)
-
reporter -
reporter References
#298: creation of aspect from feature now properly command-based + always display the new aspect.→ <<cset d902827c0ca1>>
-
Issue
#272was marked as a duplicate of this issue. -
reporter References
#298: remove previous realizations references of an external aspect when it is associated to a new concern→ <<cset 728ff1964fd8>>
-
reporter References
#298: deleting aspect now unloads its resource→ <<cset b33a639f341a>>
-
reporter - changed status to resolved
- Log in to comment
References
#298: we can now undo properly the addition of a new child feature→ <<cset ab3f3c401b76>>