OPUS4 / opus4-common

Core package for OPUS 4.
Other
0 stars 5 forks source link

Methode zur Abfrage von Konfigurationsoptionen von Application nach Common verschieben #93

Closed extracts closed 1 year ago

extracts commented 1 year ago

Die Application-Methode Application_Configuration->getValueFromConfig() ermöglicht es, in der Config.ini gesetzte Werte auszulesen. Um diese Funktionalität auch in anderen Modulen (z.B. opus4-pdf) verfügbar zu machen, sollte diese Methode (als auch die entsprechende Application_Configuration->setValueInConfig() Methode) in die Basis-Klasse (Opus\Common\Config) verschoben werden.

j3nsch commented 1 year ago

Da gibt es vermutlich Tests. Bitte mit verschieben und falls nötig anpassen.

extracts commented 1 year ago

Da gibt es vermutlich Tests. Bitte mit verschieben und falls nötig anpassen.

In Application_ConfigurationTest gibt es eine Reihe an Tests, die Application_Configuration->getValue() testen, nicht aber Application_Configuration->getValueFromConfig() direkt. Die getValue() Methode würde aber in Application_Configuration verbleiben, oder?

j3nsch commented 1 year ago

Ja, nur direkte Tests der beiden statischen Funktionen verschieben.

j3nsch commented 1 year ago

Das sollte ja erledigt sein.