ASP.NET MVC compilation error with Razor syntax

Question

The following code snippet is causing a compilation error that I am unable to pinpoint. I tried removing the 2 from the last brace. I tried the AntiForgery without braces - no luck.

The error is CS1501: No overload for method 'Write' takes 0 arguments

Any suggestions?

<div id="header-wrapper">
    <div id="header">
        <div id="menu-wrapper">
            @if (Page.User.Identity.IsAuthenticated){
                @Html.Telerik().Menu().Name("mainnav").BindTo("main").Effects(x => x.Toggle())
            }     
        </div>
        @if (Page.User.Identity.IsAuthenticated){
            <form action="@Url.Action("Logoff", "Account")" id="formLogout" method="POST">
                @{ Html.AntiForgeryToken(); }
                <a href="" id="logout" onclick="$('#formLogout').submit();return false;"></a>
            </form>
        @}
        <div id="subnav"></div>
    </div>
</div>

enter image description here

Chris Fulstow · Accepted Answer · 2011-12-07 03:18:58Z

1

Try replacing:

@{ Html.AntiForgeryToken(); }

with:

@Html.AntiForgeryToken()

(and remove the @ from your if closing brace)

Code should be:

@if (Page.User.Identity.IsAuthenticated) {
  <form action="@Url.Action("Logoff", "Account")" id="formLogout" method="POST">
    @Html.AntiForgeryToken()
    <a href="" id="logout" onclick="$('#formLogout').submit();return false;"></a>
  </form>
}

edited Dec 7, 2011 at 3:18

answered Dec 7, 2011 at 3:13

Chris Fulstow

42k10 gold badges91 silver badges114 bronze badges

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

1 Comment

Leons Over a year ago

That worked! I was sure that I already tried it, but I obviously did not.

Erik Funkenbusch · Accepted Answer · 2011-12-07 04:48:31Z

1

Since Chris answered your problem, I'd like to point out several improvements to your code. Your code could be written this way:

@if (Page.User.Identity.IsAuthenticated){
 using (Html.BeginForm("Logoff", "Account", FormMethod.Post, new { id="formLogout" }) {
     Html.AntiForgeryToken();
     <a href="" id="logout">Logout</a>
 }
}

<script type="text/javascript">
    $(function() {
        $('#formLogout').click(function() { 
            $('#formLogout').submit(); return false; 
        });
    }};
</script>

This does a few things. First, it simplifies your markup. This is known as "unobtrusive" javascript, because it removes all those ugly "onclick" and other kinds of javascript related events.

Secondly, it seperates your javascript logic from your markup, making it easier to maintain.

Notice also that i'm using the BeginForm html helper, which simplifies your markup as well. Notice how there's only a single @ sign in the entire block?

answered Dec 7, 2011 at 4:48

Erik Funkenbusch

93.7k29 gold badges201 silver badges292 bronze badges

1 Comment

Leons Over a year ago

This is so much cleaner and definitely easier to maintain. +1 for making the improvement suggestions.

Collectives™ on Stack Overflow

ASP.NET MVC compilation error with Razor syntax

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related