Skip to content

Sql Server Spatial support#100

Open
ligos wants to merge 1 commit into
SamSaffron:masterfrom
ligos:master
Open

Sql Server Spatial support#100
ligos wants to merge 1 commit into
SamSaffron:masterfrom
ligos:master

Conversation

@ligos

@ligos ligos commented Dec 19, 2012

Copy link
Copy Markdown

A very simple addition to support spatial data types in SQL Server. Addresses this error:

http://community.miniprofiler.com/permalinks/36/the-provider-did-not-return-a-dbspatialservices-instance

I've tested basic reading and writing of entities using EF 5, but nothing else. And, given this was my very first usage of spatial data types, well, buyer beware!

The big hitch is it requires targeting .net 4.5. I'm not entirely sure what the best way will be to handle that, but once 4.5 is available the code changes are pretty minor.

Murray

@SamSaffron

Copy link
Copy Markdown
Owner

@dixon whats the status on this one?

@dixon

dixon commented Jan 1, 2013

Copy link
Copy Markdown
Collaborator

I am holding off merging until support for multiple framework versions is in place.

@stephen776

Copy link
Copy Markdown

Any idea when this will get fixed? I am currently running to this issue when using spatial data types with EF5 and SQL Server 2008

@wbraun

wbraun commented Sep 26, 2013

Copy link
Copy Markdown

I used the override fix provided by Trevor from this post (http://community.miniprofiler.com/permalinks/36/the-provider-did-not-return-a-dbspatialservices-instance) and it seems to be working well for me. I hope this finds its way into the next release. I don't want to have to constantly make my own miniprofiler builds every time there is an update.

@dixon

dixon commented Oct 31, 2013

Copy link
Copy Markdown
Collaborator

We have moved this project to https://github.com/MiniProfiler/dotnet - would you please make your changes there? Thank you!

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.

7 participants