There is a one ( too many in the if-statement.

Dave

budy plz guide for the correct format for above code

Hi Shine,

It would be like this: 

 <% if ((umbraco.library.GetItem("pageheaderLink") == "")) { %>
  <umbraco:Item field="pageheader" runat="server" />
<% } else { %>
 <a href='<umbraco:Item field="pageheaderLink" runat="server" />'><umbraco:Item field="pageheader" runat="server" /></a>
<% } %>  

 However, I would recommend that you use a inline Razor macro instead, this would be like this:

<umbraco:Macro runat="server" language="cshtml">
@{
    if (!Model.HasValue("pageheaderLink")){            
      <span>@Model.pageheader</span>
    } else {
      <a href="@Model.pageheaderLink">@Model.pageheader</a>
    }
}
</umbraco:Macro>

Thanks,

Jeavon

is working on a reply...