switch inside of a foreach loop

Brad 14 posts 105 karma points

Nov 30, 2017 @ 23:20

Umbraco 7

Why does this break?

    <table align="center" class="container" style="font-size: 14px; padding: 15px;">
        @foreach (var field in Model.Fields)
        {
            <tr class="row">
                <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">@field.Name</td>
                @{
                switch(field.FieldType)
                {
                    case "FieldType.FileUpload.cshtml":
                    {
                        <td class="large-12"style=" padding:10px; font-weight:300;  border-bottom: 1px solid #ccc;"><a href="@(siteDomain/@field.GetValue())" target="_blank" style="color: #00AEA2;">@field.GetValue()</a></td>
                     }
                        break;

                    case "FieldType.DatePicker.cshtml":
                    {
                        <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">@(Convert.ToDateTime(field.GetValue()).ToString("MMMM d, yyyy"))</td>
                    }
                    break;

                    case "FieldType.CheckboxList.cshtml":
                    {
                        <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">
                            @foreach (var color in field.GetValues())
                            {
                                @color<br/>
                            }
                        </td>
                     }
                     break;
                    default:
                    {
                        <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">@field.GetValue()</td>
                    }
                     break;
                }
               } 
            </tr>
        }

    </table>

Copy Link

Alex Skrypnyk 6133 posts 23952 karma points MVP 7x admin c-trib

Nov 30, 2017 @ 23:22

Hi Brad

It's hard to say why exactly it's braking, can you show all razor code?

What line is braking?

Thanks,

Alex

Copy Link

Brad 14 posts 105 karma points

Nov 30, 2017 @ 23:25

I'm trying to modify an email template for a form, but it doesn't seem to agree with the switch portion. If I take it out, the email goes through when I test the form. But when I put it back in, the email doesn't come through. Maybe it's something else??

<body style="background-color: #f4f4f4; margin: 0 !important; padding: 0 !important;">
    <table align="center" class="container" style="font-size: 14px; padding: 15px;">
        @foreach (var field in Model.Fields)
        {
            <tr class="row">
                <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">@field.Name</td>

                switch(field.FieldType)
                {
                    case "FieldType.FileUpload.cshtml":
                    {
                        <td class="large-12"style=" padding:10px; font-weight:300;  border-bottom: 1px solid #ccc;"><a href="@(siteDomain/@field.GetValue())" target="_blank" style="color: #00AEA2;">@field.GetValue()</a></td>
                     }
                        break;

                    case "FieldType.DatePicker.cshtml":
                    {
                        <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">@(Convert.ToDateTime(field.GetValue()).ToString("MMMM d, yyyy"))</td>
                    }
                    break;

                    case "FieldType.CheckboxList.cshtml":
                    {
                        <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">
                            foreach (var color in field.GetValues())
                            {
                                @color<br/>
                            }
                        </td>
                     }
                     break;
                    default:
                    {
                        <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">@field.GetValue()</td>
                    }
                     break;
                }

            </tr>
        }

    </table>

Copy Link

Nigel Wilson 944 posts 2076 karma points

Nov 30, 2017 @ 23:38

Hi Brad

Are you not able to debug in Visual Studio ?

One thought is that a date is not set (null) and so breaks when in the case for datepicker - have no idea if this is or is not possible, just a thought.

Cheers Nigel

Copy Link

Ben Palmer 176 posts 842 karma points c-trib

Dec 01, 2017 @ 08:33

Hi Brad,

Check your error log found in the ~/App_Data/Logs folder and post any errors you get related to this here. It should tell us/you exactly what's going wrong.

Copy Link

Matthew Wise 271 posts 1373 karma points MVP 4x c-trib

Dec 01, 2017 @ 08:57

Probably not a full fix but the foreach is missing an @ in this section

case "FieldType.CheckboxList.cshtml":
                {
                    <td class="large-12" style=" padding:10px; font-weight:300; border-bottom: 1px solid #ccc;">
                        foreach (var color in field.GetValues())
                        {
                            @color<br/>
                        }
                    </td>
                 }

Copy Link

Brad 14 posts 105 karma points

Jan 05, 2018 @ 21:24

Thanks for the responses guys! I was away for a while so I wasn't able to repsond.

It appears the datepicker issue that Nigel mentioned was the issue.

Copy Link

is working on a reply...

Flag this post as spam?

Switch inside of a foreach loop