search

scijava / scijava-common

A plugin framework and application container with built-in extensibility mechanism :electric_plug:
BSD 2-Clause "Simplified" License
85 stars 53 forks source link

Fix priority of `LoggerPreprocessor` #447

Closed imagejan closed 1 year ago

imagejan commented 1 year ago

Logger parameters should be processed before InitPreprocessor, so that plugins can use a Logger in their initializer.

Closes #446.

ctrueden commented 1 year ago

@imagejan Thanks! So the change to VERY_HIGH ends up moving the LoggerPreprocessor to before the InitPreprocessor (which is HIGH) but still after the ServicePreprocessor (which is 2*VERY_HIGH). That's intended, right?

imagejan commented 1 year ago

Thanks for merging, @ctrueden! Exactly, moving LoggerPreprocessor priority in between ServicePreprocessor and InitPreprocessor was intended.