Opened 8 years ago

Last modified 8 years ago

#16388 closed defect

RelationEditor#registerRelationEditor not used? — at Initial Version

Reported by: michael2402 Owned by: team
Priority: normal Milestone: 18.07
Component: Core Version:
Keywords: Cc: stoecker, Biswesh

Description

When trying to register an extended relation editor for the PT assistant plugin, we noticed that the methods to register new relation editors seem not to be functional.

There are several issues with it:

  • The registerRelationEditor is not static - it probably should be
  • The type of the parameter should be Class<? extends RelationEditor>. Otherwise, you can only register exact relation editor instances.
  • We are not checking the full constructor signature of that class, we cannot do this since the second argument is contains a generic type info.

Is it used? If not, we should drop it and replace it by a more robust registration mechanism that does not rely on unchecked reflective accesses and uses a factory interface instead.

@stoecker: You seem to have added the code, what was it used for?

Change History (0)

Note: See TracTickets for help on using tickets.