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

Sql Server Spatial support #100

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Sql Server Spatial support #100

wants to merge 1 commit into from

Conversation

ligos
Copy link

@ligos ligos commented Dec 19, 2012

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
Owner

@dixon whats the status on this one?

@dixon
Copy link
Collaborator

dixon commented Jan 1, 2013

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

@stephen776
Copy link

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
Copy link

wbraun commented Sep 26, 2013

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
Copy link
Collaborator

dixon commented Oct 31, 2013

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