multi if statement for document type alias

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

Martin 278 posts 661 karma points

Sep 18, 2015 @ 12:36

Multi If Statement for Document Type Alias

Hi,

Im trying apply a navigation structure depending on the document type alias used.

Im checking if the pages have any children, and then the doc type alias.

Im running in to problems with duplicate nodes.

Any help would be grateful.

Martin

 @foreach (var childPage in home.Children.Where("Visible")) {
if (childPage.Children.Any()) {
    if(childPage.DocumentTypeAlias == "DocType-1" ) { <li class=""><a href="">Doc Type 1</a></li>
    }

    if(childPage.DocumentTypeAlias == "DocType-2" ) { <li class=""><a href="">Doc Type 2</a></li>
    }

    else { <li class=""><a href="">Any Doc Type</a></li>
    }
}
else { <li class=""><a href="">Any Doc Type</a></li>
} }

Copy Link

Dennis Aaen 4457 posts 17970 karma points admin hq c-trib

Sep 18, 2015 @ 13:06

Hi Martin,

Could you try this code and see if it works like you wanted it to work.

@{ 
    var home = CurrentPage.AncestorOrSelf(1);

    foreach (var childPage in home.Children.Where("Visible")) {
        if (childPage.Children.Any()) {
            if(childPage.DocumentTypeAlias == "DocType1Alias" ) { 
                <li class=""><a href="">Doc Type 1</a></li>

                }

            if(childPage.DocumentTypeAlias == "DocType2Alias" ) { 
                <li class=""><a href="">Doc Type 2</a></li>
            }

            else { 
                <li class=""><a href="">Any Doc Type</a></li>
            }
        }
        else { 
            <li class=""><a href="">Any Doc Type</a></li>
        } 

    }   
}

Hope this helps,

/Dennis

Copy Link

Ajay Karwal 31 posts 149 karma points

Sep 18, 2015 @ 13:08

Reformating your code for readability, you're probably getting the duplicates due to the 2 if statements

@foreach (var childPage in home.Children.Where("Visible")) {

  if (childPage.Children.Any()) {
      if(childPage.DocumentTypeAlias == "DocType-1" ) { 
        <li class=""><a href="">Doc Type 1</a></li>
      }

      if(childPage.DocumentTypeAlias == "DocType-2" ) {
        <li class=""><a href="">Doc Type 2</a></li>
      } else {
        <li class=""><a href="">Any Doc Type</a></li>
      }
  } else { 
    <li class=""><a href="">Any Doc Type</a></li>
  }

}

Now consider your doctype is DocType-1. That would satisfy TURE for the first IF

if(childPage.DocumentTypeAlias == "DocType-1" )  {
  <li class=""><a href="">Doc Type 1</a></li>
}

and also be rendered in the ELSE

if(childPage.DocumentTypeAlias == "DocType-2" )  {
  <li class=""><a href="">Doc Type 2</a></li>
} else {
  <li class=""><a href="">Any Doc Type (can also be Doc Type 1 again)</a></li>
}

Hence the double result.

Changing you're whole code block to this should fix things.

@foreach (var childPage in home.Children.Where("Visible")) {

  if (childPage.Children.Any()) {
      if(childPage.DocumentTypeAlias == "DocType-1" ) { 
        <li class=""><a href="">Doc Type 1</a></li>
      } else if(childPage.DocumentTypeAlias == "DocType-2" )  {
        <li class=""><a href="">Doc Type 2</a></li>
      } else {
        <li class=""><a href="">Any Doc Type</a></li>
      }
  } else { 
    <li class=""><a href="">Any Doc Type</a></li>
  } 

}

Also, the last else {} probably isn't needed. What would that render?

Copy Link

Martin 278 posts 661 karma points

Sep 18, 2015 @ 14:42

0

Thanks Ajay.

Copy Link
is working on a reply...

This forum is in read-only mode while we transition to the new forum.

You can continue this topic on the new forum by tapping the "Continue discussion" link below.

Please Sign in or register to post replies