-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow creating classifier when setting subcomponent or feature classifier #1944
Allow creating classifier when setting subcomponent or feature classifier #1944
Conversation
the ClassifierHandler
…ier_when_setting_classifier
…ier_when_setting_classifier
…ier_when_setting_classifier
…ier_when_setting_classifier
…ier_when_setting_classifier
1198_create_classifier_when_setting_classifier Conflicts: ge/org.osate.ge/src/org/osate/ge/internal/ui/properties/SetFeatureClassifierPropertySection.java ge/org.osate.ge/src/org/osate/ge/internal/ui/properties/SetSubcomponentClassifierPropertySection.java
The code uses Optional.isEmpty(). This doesn't exist in Java 8: org/osate/ge/internal/util/EditingUtil.java:[240] |
Fixed and I've updated my dev environment to ensure it's using JDK 8. |
@RyanMcilnay fixed issue that you raised and merged master into the branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
@@ -61,6 +63,18 @@ public AgeBusinessObjectSelection(final Collection<? extends BusinessObjectConte | |||
modify(boc -> c.cast(boc.getBusinessObject()), (bo, boc) -> modifier.accept(bo)); | |||
} | |||
|
|||
@Override | |||
public <T extends EObject, O> void modifyWithOperation(final OperationBuilder<O> opBuilder, Class<T> c, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add final to parameters
1198 create classifier when setting classifier
Closes #1198