Is @Html.Raw(...) working? ... or am I just a nOOb?

[guardrex]

Trying to use @Html.Raw(@ViewBag.xxx) in a view to inject some raw HTML into the view, but I'm receiving: RuntimeBinderException: 'Microsoft.AspNet.Mvc.Rendering.IHtmlHelper<object>' does not contain a definition for 'Raw'

I'm new to MVC, so I'm not surprised if I'm just doing something wrong.

[dougbu]

should work fine. suspect the issue is the second @. try @Html.Raw(ViewBag.xxx) instead.

[guardrex]

Thanks. I changed it, but I found out that ...

@Html.Raw("<meta name=\"xx\" content=\"xx\">")

works, but

@Html.Raw(ViewBag.Description)

throws the RuntimeBinderException even with that modification.

The string for that is simply in the top of the rendered body view:

@{
ViewBag.Description = "<meta name=\"description\" content=\"Description here\">";
}

Clearly, I'm learning MVC and not doing this right; however, IHtmlHelper<object> does contain a def for Raw, so getting that exception here seems incorrect regardless of how bad my coding is.

[guardrex]

Ok, so I get it now that

<meta name="description" content="@ViewBag.Description">

in the shared view with

@{
ViewBag.Description = "Description here";
}

works. It still seems strange that I can't just inject the entire HTML tag into the view. I'll leave this open while my concern about that RuntimeBinderException being thrown for what I was doing is the correct exception ... and even perhaps to find out why @Html.Raw(ViewBag.Description) fails.

[henkmollema]

Does casting ViewBag.Description to a string works when passing it to the @Html.Raw() method? Perhaps something goes wrong with the dynamic type in the Raw method, as the RuntimeBinderException suggests.

Something like:

@Html.Raw(ViewBag.Description as string))

[Bartmax]

This is likely a bug:

@{ 
    ViewBag.Description = "Hey!";
}

<p>@ViewBag.Description</p> // works
<p>Raw: @Html.Raw(ViewBag.Description)</p> //throws modelbinder exception
<p>Raw: @Html.Raw(ViewBag.Description as string)</p> // works

[henkmollema]

This does work as expected with MVC 5.

[dougbu]

interesting. there is absolutely nothing special about the @Html.Raw() methods and the ModelBinderException comes from much lower in the stack. could one of you with a test project please try it out with different DNX variants -- x86 / x64 and CLR / Core CLR?

also, which MVC and DNX versions are you testing?

[guardrex]

I was using CoreCLR: dnx-coreclr-x64-1.0.0-beta5-11556 (and MVC 6)

[guardrex]

@dougbu I'll be back in the office in three hours (1pm CST). If nobody else checks DNX variants by then, I'll do it and report back.

[Eilon]

Html.Raw used to be an instance method but is now an extension method. Extension methods unfortunately cannot be called with dynamic parameters, and everything returned by ViewBag is always dynamic. That's why casting works.

This is technically by design but we could consider a different approach here perhaps, such as getting rid of IHtmlHelper.

[dougbu]

@Eilon @Html.Raw() remains an instance method https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/Rendering/Html/HtmlHelper.cs#L524 casting probably works because the Raw(string) method is called instead of Raw(object) though why those are different remains unclear.

[guardrex]

The original exception in my app was RuntimeBinderException: 'Microsoft.AspNet.Mvc.Rendering.IHtmlHelper<object>' does not contain a definition for 'Raw', and that app doesn't have a model for the view I was working with.

When I went back and tested in my HelloMVC app, which in my version includes a @model User in the view, it threw: RuntimeBinderException: 'Microsoft.AspNet.Mvc.Rendering.IHtmlHelper<HelloMvc.Models.User>' ...

Sort of made me think that Viewbag (the object) is what @Html.Raw() choked on ... not ViewBag.xxx, which would be a, what?, a dynamic property of the ViewBag object?

... anyway, check this out ... When I try this @Html.Raw(ViewData["xxx"]) ... it works!!

Does this mean that @Html.Raw(Viewbag.xxx) isn't necessarily misinterpreting what it has been given? Is it just not resolving the dynamic object property value of Viewbag.xxx before performing its injection into the markup?

If an object is provided to @Html.Raw(), does it expect the passed object implement its own Raw definition? ... that would explain why it's saying the object doesn't have a Raw definition for ViewBag ... Viewbag the object ... it doesn't and shouldn't.

[Bartmax]

why those are different remains unclear.

@dougbu maybe this is because most cases you call raw with string and you avoid the null check ? I also don't think it's necessary.

I still don't know why the object one is throwing, this worked with dynamics in mvc5 with no problem.

[Eilon]

@dougbu oh yeah weird I guess it's the same as it always was... so not sure what's going on here.

[henkmollema]

@dougbu @Eilon
Looks like it goes wrong because the Html property of the view uses the generic interface IHtmlHelper<TModel>. This worked in MVC5 because there is no IHtmlHelper interface.

---

Repro:
An 'html helper' with a Raw method:

public interface IMyHtmlHelper
{
    HtmlString Raw(object x);
}

public interface IMyHtmlHelper<T> : IMyHtmlHelper { }

public class MyHtmlHelper : IMyHtmlHelper
{
    public HtmlString Raw(object value)
    {
        return new HtmlString(value == null ? null : value.ToString());
    }
}

public class MyHtmlHelper<T> : MyHtmlHelper, IMyHtmlHelper<T> { }

With this view:

@{
    IMyHtmlHelper<object> MyHtml = new MyHtmlHelper<object>();
    ViewBag.Test = "<strong>Hello World!</strong>";
}
<p>
    @MyHtml.Raw(ViewBag.Test)
</p>

Causes the same RuntimeBinderException:

at CallSite.Target(Closure , CallSite , IMyHtmlHelper`1 , Object )
at System.Dynamic.UpdateDelegates.UpdateAndExecute2[T0,T1,TRet](CallSite site, T0 arg0, T1 arg1)
at Asp.ASPV__Views_Home_Index_cshtml.<ExecuteAsync>d__13.MoveNext()

However, when I change this line:

IMyHtmlHelper<object> MyHtml = new MyHtmlHelper<object>();

to this:

MyHtmlHelper<object> MyHtml = new MyHtmlHelper<object>();

(notice that the type of MyHtml is the actual class implementation MyHtmlHelper<object>)

It does work.

Using the non-generic type works too:

IMyHtmlHelper MyHtml = new MyHtmlHelper();

Should this be a new issue?