Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Revisit IJsonOutputFormatter #2715

Closed
danroth27 opened this issue Jun 18, 2015 · 4 comments
Closed

Revisit IJsonOutputFormatter #2715

danroth27 opened this issue Jun 18, 2015 · 4 comments

Comments

@danroth27
Copy link
Member

IJsonOutputFormatter is currently just used as a marker interface to indicate that a formatter outputs JSON. We should revisit this approach and see if we can come up with something cleaner and more general.

@danroth27
Copy link
Member Author

We decided to remove the dependency on formatting in JsonResult and JsonViewComponentResult and instead just use JSON.NET directly. Remove IJsonOutputFormatter as part of this work.

@ajaybhargavb
Copy link
Contributor

Part 1 - f206cf6

@JamesNK
Copy link
Member

JamesNK commented Jun 28, 2015

Are the serializer settings from MvcJsonOptions still used by default?

I don't have an environment to test on right now but judging by https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Extensions/Controller.cs#L400 I'm guessing they aren't.

@rynowak
Copy link
Member

rynowak commented Jun 29, 2015

I look a look here and @JamesNK is right. This should be respecting MvcJsonOptions

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants