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?


