multi if statement for document type alias

Go to solution

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 662 karma points

Sep 18, 2015 @ 12:36

Razor

Umbraco 7

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 4499 posts 18254 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 662 karma points

Sep 18, 2015 @ 14:42

0

Thanks Ajay.

Copy Link
Carlos Mosqueda 240 posts 431 karma points

Oct 26, 2021 @ 18:40
0
Resurrecting this one for Umbraco 7.15 at least (yep I know U8 and U9 are out), and maybe U8 will work too. The "IsDocumentType()" method worked nicely for me.
```
if(childPage.IsDocumentType("doctype-1") ) {
//Do something here
}else if(childPage.IsDocumentType("doctype-1") ) {
//Do something here
}else{//blah}
```
Copy Link
is working on a reply...

Please Sign in or register to post replies