search

dgarijo / Widoco

Wizard for documenting ontologies. WIDOCO is a step by step generator of HTML templates with the documentation of your ontology. It uses the LODE environment to create part of the template.
Apache License 2.0
284 stars 88 forks source link

Warnings printed when config.properties is not available #599

Open stefanogermano opened 1 year ago

stefanogermano commented 1 year ago

Describe the bug

Several warnings are printed if there is not a config.properties file:

[main] WARN widoco.Configuration - Error while reading configuration properties from [[...]/config/config.properties]: [...]/config/config.properties (No such file or directory)
[main] WARN widoco.Configuration - Error while loading the default property file: [...]/config/config.properties (No such file or directory)
[main] WARN widoco.Configuration - Error while reading configuration properties from []:  (No such file or directory)
[main] WARN widoco.Configuration - Error while reading configuration properties from []:  (No such file or directory)

To Reproduce

Steps to reproduce the behavior:

  1. Run Widoco without a config.properties file

Expected behavior

No warnings should appear.

ThomasThelen commented 1 week ago

AFAIK this is a feature, https://github.com/dgarijo/Widoco/blob/master/src/main/java/widoco/Configuration.java#L417

Since these are warnings and not error log level, maybe the wording can be changed to remove "Error" from it.

stefanogermano commented 1 week ago

They should probably just be "info" since, as noted in the comment you linked:

we can continue safely without a property file

And perhaps just have a concise message stating that a property file won't be used.

But I completely agree with your suggestion!