Developing (v7+)

5.7.5.1 Autonav SEO

Permalink 1 user found helpful August 28, 2015 at 11:05 AM

Hey all,

As some of you know, our firm is extremely big on perfecting SEO throughout the concrete5 platform. It's important to our team and clientele that each link on the website have the proper SEO elements associated with it.

In 5.6, we were able to add a custom page attribute for adding custom title tags to the autonav's links (echoed in the autonav block's view.php). In 5.7.5.1, we are unable to accomplish this simple addition.

Is there a way to add a custom page attribute into the CMS with the handle "page_title_meta", for example, and echo it into the autonav's view.php? This way, the links in the main navigation for website's would be customized on a per page basis. For example:

An autonav's page link would appear like this in the code:

<a href="/page-name" title="Custom Page Attribute">Page Name</a>

I know this is possible, but I can't figure out why we're unable to echo it accordingly. Can anyone help?

We really appreciate any advice you could provide. Thank you so much!

JohntheFish replied on Aug 28, 2015 at 11:23 am Permalink Reply

I haven't done any custom autonav work for 5.7, but I had a look at the 5.7 code while recently working on a series of 5.6 autonav customisations and there is very little difference. Its still the same old backward compatible series of kludges inside.

Whatever custom attribute tests you developed for 5.6 autonavs should be easily transferable to 5.7.

creativesolutions replied on Aug 28, 2015 at 11:33 am Permalink Reply

Pardon me for this, but I believe I mentioned the incorrect block above. I meant to say the amiant CSS3 menu instead. Here is the entire view.php for that block that worked perfectly in 5.6. Please note the title="'.$_c->getCollectionAttributeValue('title_link_meta').'" area:

<?php   
   defined('C5_EXECUTE') or die("Access Denied.");
?>
   <div class="amiant-css3-menu-navigation-bar">
<?php  
   $aBlocks = $controller->generateNav();
   $c = Page::getCurrentPage();
   $containsPages = false;
   $nh = Loader::helper('navigation');
   //this will create an array of parent cIDs 
   $inspectC=$c;
   $selectedPathCIDs=array( $inspectC->getCollectionID() );
   $parentCIDnotZero=true;   
   while($parentCIDnotZero){
      $cParentID=$inspectC->cParentID;

Viewing 15 lines of 84 lines. View entire code block.

 
<?php   
   defined('C5_EXECUTE') or die("Access Denied.");
?>
   <div class="amiant-css3-menu-navigation-bar">
 
<?php  
   $aBlocks = $controller->generateNav();
   $c = Page::getCurrentPage();
   $containsPages = false;
 
   $nh = Loader::helper('navigation');
 
   //this will create an array of parent cIDs 
   $inspectC=$c;
   $selectedPathCIDs=array( $inspectC->getCollectionID() );
   $parentCIDnotZero=true;   
   while($parentCIDnotZero){
      $cParentID=$inspectC->cParentID;
      if(!intval($cParentID)){
         $parentCIDnotZero=false;
      }else{
         $selectedPathCIDs[]=$cParentID;
         $inspectC=Page::getById($cParentID);
      }
   }    
 
   foreach($aBlocks as $ni) {
      $_c = $ni->getCollectionObject();
      $_c_parent = Page::getByID($_c->getCollectionParentID(), $version = 'RECENT');
 
      if (!$_c->getCollectionAttributeValue('exclude_nav') && !$_c_parent->getCollectionAttributeValue('do_not_display_subpages_in_nav') ) {
 
 
         $target = $ni->getTarget();
         if ($target != '') {
            $target = 'target="' . $target . '"';
         }
         if (!$containsPages) {
            // this is the first time we've entered the loop so we print out the UL tag
            echo("<ul class=\"navigation-menu nav\">");
         }
 
         $containsPages = true;
 
         $thisLevel = $ni->getLevel();
         if ($thisLevel > $lastLevel) {
            echo("<ul>");
         } else if ($thisLevel < $lastLevel) {
            for ($j = $thisLevel; $j < $lastLevel; $j++) {
               if ($lastLevel - $j > 1) {
                  echo("</li></ul>");
               } else {
                  echo("</li></ul></li>");
               }
            }
         } else if ($i > 0) {
            echo("</li>");
         }
 
         $pageLink = false;
 
         if ($_c->getCollectionAttributeValue('replace_link_with_first_in_nav')) {
            $subPage = $_c->getFirstChild();
            if ($subPage instanceof Page) {
               $pageLink = $nh->getLinkToCollection($subPage);
            }
         }
 
         if (!$pageLink) {
            $pageLink = $ni->getURL();
         }
 
         $arrow = '';
         $nav_with_sub = '';
         if (($_c->getNumChildren() > 0) && ($_c->getCollectionID() != 1) && (!$_c->getCollectionAttributeValue('do_not_display_subpages_in_nav') )) {
            $arrow = '<span class="menu-arrow"></span>';
            $nav_with_sub = 'nav-with-sub';
         }
 
 
         if ($c->getCollectionID() == $_c->getCollectionID()) { 
            echo('<li class="nav-selected nav-path-selected '.$nav_with_sub.'"><a title="'.$_c->getCollectionAttributeValue('title_link_meta').'" class="nav-selected nav-path-selected '.$nav_with_sub.'" ' . $target . ' href="' . $pageLink . '">' . $ni->getName() . $arrow . '</a>');
         } elseif ( in_array($_c->getCollectionID(),$selectedPathCIDs) && ($_c->getCollectionID() != HOME_CID) ) {
            echo('<li class="nav-path-selected '.$nav_with_sub.'"><a title="'.$_c->getCollectionAttributeValue('title_link_meta').'" class="nav-path-selected '.$nav_with_sub.'" href="' . $pageLink . '" ' . $target . '>' . $ni->getName() . $arrow . '</a>');
         } else {
            echo('<li class="'.$nav_with_sub.'"><a title="'.$_c->getCollectionAttributeValue('title_link_meta').'" class="'.$nav_with_sub.'" href="' . $pageLink . '" ' . $target . ' >' . $ni->getName() . $arrow . '</a>');
         }   
         $lastLevel = $thisLevel;
         $i++;
 
 
      }
   }
 
   $thisLevel = 0;
   if ($containsPages) {
      for ($i = $thisLevel; $i <= $lastLevel; $i++) {
         echo("</li></ul>");
      }
   }
 
?>
 
   </div>

What I meant to say was that we are trying to make this part of the new autonav itself. Again, my apologies.

JohntheFish replied on Aug 28, 2015 at 12:57 pm Permalink Reply

That code is similar to what all autonav view templates looked like before 5.6. But the 5.6 nav controller included all the obsolete code necessary to be compatible with old pre-5.6 templates. You may just be lucky if you just cut and paste it into a 5.7 autonav view template and see what happens. The requisite excess baggage to work with that code may still be in there.

creativesolutions replied on Aug 28, 2015 at 1:14 pm Permalink Reply

We tried that exact solution when we first started, but to no avail. Is there anything we can do to add this functionality for the title tags on the autonav?

JohntheFish replied on Aug 28, 2015 at 1:41 pm Permalink Reply

Nothing short of developing a new 5.7 autonav template for amniant.

creativesolutions replied on Aug 28, 2015 at 1:56 pm Permalink Reply

Cool. In that case, how should I echo out the custom page attribute for the autonav's view.php? It should be something like this in 5.7+:

<?php echo $c->getCollectionAttributeValue('attribute_handle') ?>

This is basically all we are trying to accomplish so that the title tag can be added on the per link basis pulled from the page's attribute. If we had that, we'd be ready to rock!

Any advice?

JohntheFish replied on Aug 28, 2015 at 2:18 pm Permalink Best Answer Reply

There are a bunch of examples in the default view.php.

creativesolutions replied on Aug 28, 2015 at 2:48 pm Permalink Reply

I didn't even see those! Thanks for helping me with this. I was able to create an attribute that worked perfectly. Thanks JohntheFish!!!

Forums

Developing (v7+)

5.7.5.1 Autonav SEO

Code

Post Reply

Delete Post

Mark Post as Spam

Destroy Spammer

Sign In?