Skip to content
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

Fix compatibility with Apache WebConsole 4.7.0 #1618

Merged

Conversation

sfeilmeier
Copy link
Contributor

Latest version of Apache WebConsole does not explicitely store unchanged default configuration properties. This has certain implications on the way OpenEMS uses properties, e.g. it means that a Component that is created via Apache WebConsole with default 'id' does not have the 'id' stored as property.

See this issue for details:
https://issues.apache.org/jira/browse/FELIX-6436?focusedCommentId=17412472&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17412472

@sfeilmeier sfeilmeier merged commit ce020e9 into develop Sep 9, 2021
@sfeilmeier sfeilmeier deleted the feature/compatibility-with-apache-webconsole-4_7_0 branch September 9, 2021 15:03
clehne pushed a commit to clehne/openems that referenced this pull request Sep 12, 2021
Latest version of Apache WebConsole does not explicitely store unchanged default configuration properties. This has certain implications on the way OpenEMS uses properties, e.g. it means that a Component that is created via Apache WebConsole with default 'id' does not have the 'id' stored as property.

See this issue for details:
https://issues.apache.org/jira/browse/FELIX-6436?focusedCommentId=17412472&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17412472

* Read default property values from Factory
* Implement custom listConfigurations(componentId) method: this method also checks for the Component default value of the 'id' property
* Fix handling of static Component Property ID for singletons

Offtopic
* AbstractWebsocketServer: test for isShutdown
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant