ASP.NET MVC return ViewResult

Question

I trying to return a ViewResult in OnActionExecuted method override from ActionFilterAttribute class

Like below ...

public override void OnActionExecuted(ActionExecutedContext filterContext)
{
    if (CreateCookie && filterContext.Exception == null)
    {
        LoginCookies lcookie = new LoginCookies(usuDs, usuSenha);
        lcookie.WriteCookie("SCE", 10);
    }
    else
    {
        filterContext.Result = new ViewResult() { ViewName = "Login" };
        filterContext.Result.ExecuteResult
                (filterContext.Controller.ControllerContext);
    }

It works fine to return to a view called "Login",but i need to pass the model object to this view (in this case model object is type of Users) and I don't know how to pass it using ViewResult class directly.

Any ideas?

UPDATED: I've solved my problem setting filterContext.ExceptionHandled to TRUE,but the main problem was not solved, I can't set Model property of View, it is always null.

The code is above.When i return ViewResult Login in code above,i need to pass the model to View as well,else view throws an exception cause Model is null.Problem is i cant set the model to this view. — ozsenegal
– ozsenegal, Commented Aug 26, 2010 at 17:37

tokyovariable · Accepted Answer · 2012-08-16 16:30:58Z

10

I ran into the same issue where my Model being passed to the View was always NULL. I was able to pass the Model to my view with the following:

Create your viewModel and set the properties. Create a new ViewResult giving it the name of your View and then pass your viewModel into the ViewData.

public override void OnActionExecuted(ActionExecutedContext filterContext)
{

    TestViewModel viewModel = new TestViewModel; 

    //Here set all the properties of your viewModel such as your exception message

    filterContext.Controller.ViewData.Model = viewModel;
    filterContext.Result = new ViewResult { ViewName = "Login", ViewData = new ViewDataDictionary(viewModel)};
    filterContext.ExceptionHandled = true;

}

answered Aug 16, 2012 at 16:30

tokyovariable

1,66615 silver badges23 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

joshlrogers · Accepted Answer · 2010-08-26 00:46:14Z

2

I may be mistaken, but I believe that the view data is part of the controller base and not actually part of the view itself. So you should be able to set the view data by doing such:

filterContext.Controller.ViewData.Model = <your view model>

I just tested and this worked for me. I don't see any reason why it shouldn't work for you:

public override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        TestClass1 viewModel = new TestClass1();

        viewModel.FirstName = "TestFilter";

        filterContext.Controller.ViewData.Model = viewModel;
    }

Referencing documentation

edited Aug 26, 2010 at 0:46

answered Aug 25, 2010 at 23:26

joshlrogers

1,1341 gold badge11 silver badges20 bronze badges

4 Comments

ozsenegal Over a year ago

wow,i thought it works,but nope,that Model is always null in the view

joshlrogers Over a year ago

@ozsenegal In your example you are in the OnActionExecuted event, but are you sure you are in it in your actual code? Your viewdata should be accessible at this point.

ozsenegal Over a year ago

yes,im sure.View data is acessible the way you show,but in the view the model property keeps null

joshlrogers Over a year ago

@ozsenegal check my update. My code worked for me, it changed the view model to what I added in the filter.

Michiel · Accepted Answer · 2010-10-15 13:18:46Z

2

Maybe this will work for you:

filterContext.Result = new ViewResult { ViewName = "Exception", ViewData = new ViewDataDictionary(new CmsExceptionViewData(filterContext.Exception, action, controllerName, errorMessage)) };

So the ViewData is created with a ViewDataDictionary which accepts either a dictionary or a model.

answered Oct 15, 2010 at 13:18

Michiel

6625 silver badges13 bronze badges

Collectives™ on Stack Overflow

ASP.NET MVC return ViewResult

3 Answers 3

Comments

4 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related