Support #4590

Deprecate all of org.simantics.modeling.ui.modelBrowser.model

Added by Tuukka Lehtonen about 4 years ago. Updated about 4 years ago.

Status:ClosedStart date:
Priority:4Due date:
Assignee:Tuukka Lehtonen% Done:

100%

Category:Model browserSpent time:-
Target version:1.13.1
Release notes:
Story pointsS
Velocity based estimate-
ReleaseSimantics 1.13.2Release relationshipAuto

Description

The whole old INode/INode2 and all their implementations in org.simantics.modeling.ui.modelBrowser.model is very old and should no longer be used by anybody.

The contents of the package should all be deprecated now for 1.13 and removed in 1.14 release.

If there are important cases that are still using INode/INode2 (Comos?) we'll keep only the necessary interfaces/implementations around but everything will still be marked deprecated.

Associated revisions

Revision 28494
Added by Tuukka Lehtonen about 4 years ago

Deprecated the contents of org.simantics.modeling.ui.modelBrowser.model package.

refs #4590

History

#1 Updated by Tuukka Lehtonen about 4 years ago

  • Description updated (diff)
  • Category set to Model browser
  • Assignee set to Tuukka Lehtonen

#2 Updated by Tuukka Lehtonen about 4 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

#3 Updated by Marko Luukkainen about 4 years ago

Yes, Comos integration uses INode interface. We may have Comos development program the next year, which could be used for fix this.

#4 Updated by Tuukka Lehtonen about 4 years ago

Out of curiosity, do you have the possibility to check which classes/interfaces in that package in that package you are still using? I'm sure most of the package is rubbish that could be removed right away without problems.

#5 Updated by Marko Luukkainen about 4 years ago

I remembered the case incorrectly: Comos used INode interface, but now it uses org.simantics.interop.browsing.INode interface, so the should not be any dependencies between Comos intregation and org.simantics.modeling.ui.modelBrowser.model.

#6 Updated by Tuukka Lehtonen about 4 years ago

Ok, thanks for the info.

#7 Updated by Marko Luukkainen about 4 years ago

Note: interop.browsing uses org.simantics.browsing.ui.common.evaluatorBinding extension point.

#8 Updated by Tuukka Lehtonen about 4 years ago

Marko Luukkainen wrote:

Note: interop.browsing uses org.simantics.browsing.ui.common.evaluatorBinding extension point.

Yes, this is fine, those are not even under deprecation.

#9 Updated by Tuukka Lehtonen about 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF