search

leekelleher / umbraco-contentment

Contentment for Umbraco - a state of happiness and satisfaction
https://marketplace.umbraco.com/package/umbraco.community.contentment
Mozilla Public License 2.0
157 stars 72 forks source link

[v3.0] Data List - SQL CE data source (on Umbraco v9.0) #172

Closed leekelleher closed 2 years ago

leekelleher commented 3 years ago

Which Contentment version are you using?

v3.0.0

Which Umbraco version are you using? For example: 8.14.1 - don't just write v8

9.0.0

Bug summary

With Umbraco v9, the SQL CE support is available in separate library - (meaning that it does not come as default with Umbraco). To this, I've been unsure how to make the Data List's SQL data-source support querying SQL CE.

My original code notes are here: /DataEditors/DataList/DataSources/SqlDataListSource.cs#L170-L174

// NOTE: SQLCE uses a different connection/command. I'm trying to keep this as generic as possible, without resorting to using NPoco. [LK]
// I've tried digging around Umbraco's `IUmbracoDatabase` layer, but I couldn't get my head around it.
// At the end of the day, if the user has SQLCE configured, it'd be nice for them to query it.
// But I don't want to add an assembly dependency (for SQLCE) to Contentment itself. I'd like to leverage Umbraco's code.

Steps to reproduce

Try querying a SQL CE database, will result in an error/exception.

Expected result / actual result

This is an open issue, welcoming any help/guidance/contributions, in order to support querying a SQL CE database, without having a dependency on Umbraco's SQL CE library.

Do you have Umbraco ModelsBuilder enabled?

What browsers are you seeing the problem on?

No response

leekelleher commented 2 years ago

Closing this issue off. Support for SQL CE in Umbraco has been dropped for version 10, being replaced with SQLite.

Given that I haven't had much (any) demand for supporting SQL CE in the Data List's SQL data source for Umbraco v9, I am not inclined to spend much (free) time on the issue.

If there is a sudden surge of interest, then this issue can be re-opened.