-
Notifications
You must be signed in to change notification settings - Fork 27
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
SO-1974 extension terminology selector #103
SO-1974 extension terminology selector #103
Conversation
to TerminologyRegistryService and to its implementation
the comparison is based on the code system's base version eg.: CodeSystems with these branchPaths MAIN/2016-07-31/SNOMEDCT-DK MAIN MAIN/2016-01-31/SNOMEDCT-DK will be ordered to this: MAIN/2016-01-31/SNOMEDCT-DK (smallest) MAIN/2016-07-31/SNOMEDCT-DK MAIN (biggest)
improoving default terminology extension computation: select the main by default.
@@ -0,0 +1,15 @@ | |||
<com.b2international.snowowl.core.api.preferences.ConfigNode> |
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.
If this config is only applicable on the client side, then please remove it from the server assembly.
return new ActiveCodeSystemPredicate(branchPathMap, repositoryUuid); | ||
} | ||
|
||
public static Predicate<ICodeSystem> mainCodeSystemPredicate() { |
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.
Use only a single instance for Predicate
s and Function
s. See the other comment about moving them to the ICodeSystem
interface.
persisting repo UUID to short name rather then POJO
* | ||
* Preference configuration for terminology extension selection. | ||
*/ | ||
public class TerminologyExtensionConfiguration extends PreferenceBase { |
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.
This class is still available on the server side, could you please move it to the ext repo? Thanks!
this pull request introduces a new preference page that allows the users to select the extension on which they will do their authoring work