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

Revive BadRequest() #888

Closed
javiercn opened this issue Jul 28, 2014 · 4 comments
Closed

Revive BadRequest() #888

javiercn opened this issue Jul 28, 2014 · 4 comments
Labels
3 - Done enhancement
Milestone
1.0.0-beta3

Comments

@javiercn
Copy link
Member

Bring it back as HttpBadRequest() for consistency with MVC
@javiercn javiercn mentioned this issue Jul 28, 2014
Closed
@javiercn
Copy link
Member Author

1 day

@javiercn javiercn self-assigned this Jul 28, 2014
@javiercn javiercn added this to the 6.0.0-alpha3 milestone Jul 28, 2014
@danroth27 danroth27 modified the milestones: 6.0.0-alpha3, 6.0.0-alpha4 Aug 12, 2014
javiercn added a commit that referenced this issue Aug 26, 2014
@javiercn
[Fixes #888] Revive BadRequest()
0578da3 
1. Added an HttpBadRequest() method on Controller.
2. Added an HttpBadRequestResult class that extends HttpStatusCodeResult
   and sets the status code to 400 following the similar pattern we do in
   HttpNotFoundResult.
3. Added a sample in MvcSample.Web and unit tests.
@javiercn
Copy link
Member Author

On hold until we decide if it should be included or not.

@danroth27 danroth27 modified the milestones: 6.0.0-rc1, 6.0.0-alpha4 Sep 24, 2014
@danroth27
Copy link
Member

We will do this as part of the Web API compat shim work.

@yishaigalatzer
Copy link
Contributor

https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/Controller.cs#L636

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
3 - Done enhancement
Projects
None yet
Milestone
1.0.0-beta3
Development

No branches or pull requests
3 participants
@yishaigalatzer @danroth27 @javiercn