orbeon / orbeon-forms

Orbeon Forms is an open source web forms solution. It includes an XForms engine, the Form Builder web-based form editor, and the Form Runner runtime.
http://www.orbeon.com/
GNU Lesser General Public License v2.1
514 stars 221 forks source link

Configuration banner shows for custom persistence provider #6300

Closed ebruchez closed 4 months ago

ebruchez commented 5 months ago

The reason is that we check all active providers, then make sure that there is a datasource configured for that provider. However, an external provider, only identified by URI with oxf.fr.persistence.my-provider.uri, should not have its datasource checked.

So somehow we must exclude such external providers.

+1 from customer

ebruchez commented 5 months ago

We could consider that providers with an absolute URL are external and should not be datasource-checked.

Or we could consider that providers that match known internal URLs are internal and should be datasource-checked.

avernet commented 4 months ago

We could consider that providers with an absolute URL are external and should not be datasource-checked.

@ebruchez Wouldn't this technique fail to exclude the filesystem provider, which doesn't have a data source?

ebruchez commented 4 months ago

We don't really have a filesystem provider: we have it only for attachments, and the configuration property for that looks different:

oxf.fr.persistence.provider.*.*.*.attachments
avernet commented 4 months ago

Try adding the property. That provider will end up in activeInternalProviders, and then in problematicDataSources since it doesn't have a datasource.

<property as="xs:string" name="oxf.fr.persistence.provider.*.*.*.attachments" value="filesystem"/>
ebruchez commented 4 months ago

+1 from customer for a related issue where the filesystem provider causes the banner to show.

ebruchez commented 4 months ago

New fix, with more unit tests!