6

I'm writing an MVC2 app using DataAnnotations. I have a following Model:

public class FooModel 
{
    [ScaffoldColumn("false")]
    public long FooId { get; set; }

    [UIHint("BarTemplate")]
    public DateTime? Bar { get; set;}
}

I want to create a custom display template for Bar. I have created following template:

<%@ Control Language="C#" Inherits="System.Web.Mvc.ViewUserControl<DateTime?>" %>

<div class="display-label">
    <span><%: Html.LabelForModel() %></span>
</div>
<div class="display-field">
    <span><%: Html.DisplayForModel()%></span>
    <%: Html.ActionLink("Some link", "Action", new { id = ??FooId?? }) %>
</div>

Now, my problem is that inside template for Bar I want to access another property from my model. I don't want to create a separate template for FooModel because than I will have to hardcode all other FooModel properties.

After a brief investigation with a debugger I can see that:

  1. this.ViewData.ModelMetadata.ContainerType is FooModel (as expected)
  2. this.ViewData.TemplateInfo has a non-public property VisitedObjects (of type System.Collections.Generic.HashSet<object>) which contains two elements: FooModel and DateTime?.

How can I get access to my FooModel? I don't want to hack my way around using Reflection.

Update:

I've accepted mootinator's answer as it looks to me as the best solution that allows type-safety. I've also upvoted Tx3's answer, as mootinator's answer builds upon it. Nevertheless, I think that there should be a better support form MVC in those kind of scenarios, which I believe are quite common in real world but missing from sample apps.

smartcaveman
  • 41,281
  • 29
  • 127
  • 212
Jakub Konecki
  • 45,581
  • 7
  • 87
  • 126
  • @Jakub: The model of Bar.cshtml is type of `DateTime?`, there is no `m.Bar` I think. – Second Person Shooter Feb 15 '11 at 10:54
  • @Recycle Bin - Cheers, edited the question. – Jakub Konecki Feb 15 '11 at 11:20
  • @Jakub: I don't understand why you need to access `FooModel` from within `DateTime?`. It does not make sense. :-) – Second Person Shooter Feb 15 '11 at 11:25
  • @Recycle Bin - imagine I have a UserDetailsModel that has a DateTime? property called LastLoginDate. I want to create a template for this datetime property that will be used by EditorForModel() to render a date time picker and a link to login history page, for which I need UserId. – Jakub Konecki Feb 15 '11 at 12:34
  • @Jakub: The Creative Commons license that Wikipedia uses requires citations. Please review your tag wiki edits, and add the needed citation to each. See here for an example: http://stackoverflow.com/tags/smtp/info. Note the Wikipedia link I've added to the bottom. – Robert Harvey Feb 28 '11 at 15:59
  • @Robert - Thank you, will do that – Jakub Konecki Feb 28 '11 at 16:03

5 Answers5

4

Maybe you could create new class, let's say UserDateTime and it would contain nullable DateTime and rest of the information you need. Then you would use custom display template for UserDateTime and get access to information you require.

I realize that you might be looking for other kind of solution.

Tx3
  • 6,796
  • 4
  • 37
  • 52
  • This would mean changing my UserDetailsModel, which I would rather not do. Should I have 5 DateTime properties I would have to keep creating different classes and copy data around. My point is: I already have data I need in my Model. Thanks anyway! – Jakub Konecki Feb 15 '11 at 13:22
  • Good points. I am also interested to hear what is the solution. – Tx3 Feb 15 '11 at 13:44
2

I think you may be better off extracting this functionality to an HtmlHelper call from the Parent View.

Something like RenderSpecialDateTime<TModel>(this HtmlHelper html, Expression<Func<TModel,DateTime?>> getPropertyExpression) would probably do the job.

Otherwise, you will have to do something like what Tx3 suggested. I upvoted his answer, but posted this as an alternative.

smartcaveman
  • 41,281
  • 29
  • 127
  • 212
  • Sorry, I don't get it. 1) Extract *what* functionality? 2) How am I supposed to call Html.RenderSpecialDateTime() in the Parent View when all I'm doing in the Parent View is Html.DisplayForModel()? – Jakub Konecki Feb 18 '11 at 06:17
1

It would appear that somewhere between MVC 5.0 and 5.2.2 a "Container" property was added on to the ModelMetadata class.

However, because all of the methods in a provider responsible for metadata creation (GetMetadataForProperty, Create etc) do not have container in their signature, the Container property is assigned only in certain cases (GetMetadataForProperties and GetMetadataFromProvider according to reflected code) and in my case was usually null.

So what I ended up doing is overriding the GetMetadataForProperty in a new metadata provider and setting it there:

public override ModelMetadata GetMetadataForProperty(Func<object> modelAccessor, Type containerType, string propertyName)
{
  var propMetaData = base.GetMetadataForProperty(modelAccessor, containerType, propertyName);
  Object container = modelAccessor.Target.GetType().GetField("container").GetValue(modelAccessor.Target);
  propMetaData.Container = container;
  return propMetaData;
}

I know this is reflection but it's fairly succinct. It would appear that MS is correcting this oversite so maybe it will be possible to replace the reflection code in the future.

b_levitt
  • 7,059
  • 2
  • 41
  • 56
1

Couldn't you use the ViewData dictionary object in the controller and then grab that in the ViewUserControl? It wouldn't be strongly typed but...you could write a helper to do nothing if it's empty, and link to say the example login history page if it had a value.

Webjedi
  • 4,677
  • 7
  • 42
  • 59
0

Sorry if this suggestion seems daft, I haven't tried it, but couldn't you do what Tx3 suggested without having to create a bunch of new classes by defining a generic class to reference whatever type of parent you want?

    public class FooModel 
    {
        [ScaffoldColumn("false")]
        public long FooId { get; set; }

        [UIHint("BarTemplate")]
        public ParentedDateTime<FooModel> Bar { get; set;}

        public FooModel()
        {
            Bar = new ParentedDateTime<FooModel>(this);
        }
    }


    public class ParentedDateTime<T>
    {
        public T Parent {get; set;}
        public DateTime? Babar {get; set; }

        public ParentedDateTime(T parent)
        {
            Parent = parent;
        }

}

You could expand that to encapsulate any old type with a <Parent, Child> typed generic, even.

That would also give you the benefit that your strongly typed template would be for

Inherits="System.Web.Mvc.ViewUserControl<ParentedDateTime<FooType>> thus you would not have to explicity name which template to use anywhere. This is more how things are intended to work.

Kevin Stricker
  • 17,178
  • 5
  • 45
  • 71
  • You could go a step further and create ParentedField. You would need 1 class but still your model would be 'ugly' as FooModel properties will be of type ParentedField etc. It still smells for me. My point is that ModelMetadata already has my parent object - I don't want to 'polute' my model (image how it would look like when serialized to JSON, which I also do). – Jakub Konecki Feb 21 '11 at 22:07
  • @Jakub I would probably just keep a separate ViewModel for JSON. I can see why you'd want to avoid the duplication, if possible. – Kevin Stricker Feb 21 '11 at 22:35
  • The syntax which specifically exists to accomplish what you want still involves using ViewData: http://www.dalsoft.co.uk/blog/index.php/2010/07/29/asp-net-mvc-2-template-helpers-allow-you-to-specify-extra-view-data/ – Kevin Stricker Feb 21 '11 at 22:54