razor syntax error

Press Ctrl / CMD + C to copy this to your clipboard.

Copied to clipboard

Flag this post as spam?

This post will be reported to the moderators as potential spam to be looked at

Craig O'Mahony 364 posts 918 karma points

Aug 14, 2018 @ 13:05

Templates, (partial) views and macros

Umbraco 7

Hi all,

I've put together a fairly simple razor script where I'm building up a string and attempting to output the result to the page but I'm getting an error that I'm missing a } and for the life of me I can't see what's wrong. Could anyone assist please?

@inherits Umbraco.Web.Macros.PartialViewMacroPage

@{
        var folderNode = Umbraco.TypedContent(1068);
        var playList = "";
        var counter = 0;
        foreach(var node in folderNode.Children)
        {
            playList = "'" + @node.Name + "':[";
            foreach(var song in node.Children)
            {
                playList = playList + counter.ToString() + ",";
                counter++;
            }

            playList += "],";
        }
        @Html.Raw(playList);
}

Thanks, C

Copy Link

Nik 1593 posts 7151 karma points MVP 6x c-trib

Aug 14, 2018 @ 13:14
0
Hi Craig,

I think the error about a missing } is possibly a bit misleading.

The first thing that jumped out to me was this line here:
```
playList = "'" + @node.Name + "':[";
```
I'm pretty sure you don't need the @ before node.Name

Try removing that and see if it decides to behave for you.

Cheers

Nik
Copy Link

suman 30 posts 101 karma points

Aug 14, 2018 @ 13:21

Hi Craig,

Nik is correct.. below are your updated code.

@inherits Umbraco.Web.Macros.PartialViewMacroPage

var folderNode = Umbraco.TypedContent(1068);
var playList = "";
var counter = 0;
foreach (var node in folderNode.Children)
{
    playList = "'" + node.Name + "':[";
    foreach (var song in node.Children)
    {
        playList = playList + counter.ToString() + ",";
        counter++;
    }

    playList += "],";

}

@Html.Raw(playList);

}

Copy Link

Craig O'Mahony 364 posts 918 karma points

Aug 14, 2018 @ 13:24

0

Awesome! Couldn't see the wood for the trees there.

Thanks folks!

Copy Link
is working on a reply...

Please Sign in or register to post replies