Merge pull request #9282 from Devportobello/menuItemAttrRel

[SEO] rel attribute for external url menu item
joomla · Mar 3, 2016 · f07003c · f07003c
2 parents 83cda46 + 3d35139
commit f07003c
Show file tree

Hide file tree

Showing 4 changed files with 38 additions and 2 deletions.
diff --git a/administrator/components/com_menus/models/forms/item_url.xml b/administrator/components/com_menus/models/forms/item_url.xml
@@ -12,6 +12,26 @@
 				type="text" label="COM_MENUS_ITEM_FIELD_ANCHOR_CSS_LABEL"
 				description="COM_MENUS_ITEM_FIELD_ANCHOR_CSS_DESC" />
 
+			<field name="menu-anchor_rel" type="list"
+				label="COM_MENUS_ITEM_FIELD_ANCHOR_REL_LABEL"
+				description="COM_MENUS_ITEM_FIELD_ANCHOR_REL_DESC"
+				default=""
+			>
+				<option value="">JNONE</option>
+				<option value="alternate">COM_MENUS_FIELD_VALUE_ALTERNATE</option>
+				<option value="author">COM_MENUS_FIELD_VALUE_AUTHOR</option>
+				<option value="bookmark">COM_MENUS_FIELD_VALUE_BOOKMARK</option>
+				<option value="help">COM_MENUS_FIELD_VALUE_HELP</option>
+				<option value="license">COM_MENUS_FIELD_VALUE_LICENSE</option>
+				<option value="next">COM_MENUS_FIELD_VALUE_NEXT</option>
+				<option value="nofollow">COM_MENUS_FIELD_VALUE_NOFOLLOW</option>
+				<option value="noreferrer">COM_MENUS_FIELD_VALUE_NOREFERRER</option>
+				<option value="prefetch">COM_MENUS_FIELD_VALUE_PREFETCH</option>
+				<option value="prev">COM_MENUS_FIELD_VALUE_PREV</option>
+				<option value="search">COM_MENUS_FIELD_VALUE_SEARCH</option>
+				<option value="tag">COM_MENUS_FIELD_VALUE_TAG</option>
+			</field>
+
 			<field name="menu_image" type="media"
 				label="COM_MENUS_ITEM_FIELD_MENU_IMAGE_LABEL"
 				description="COM_MENUS_ITEM_FIELD_MENU_IMAGE_DESC" />

diff --git a/administrator/language/en-GB/en-GB.com_menus.ini b/administrator/language/en-GB/en-GB.com_menus.ini
@@ -66,6 +66,20 @@ COM_MENUS_ITEM_FIELD_ANCHOR_CSS_DESC="An optional, custom style to apply to the
 COM_MENUS_ITEM_FIELD_ANCHOR_CSS_LABEL="Link CSS Style"
 COM_MENUS_ITEM_FIELD_ANCHOR_TITLE_DESC="An optional, custom description for the title attribute of the menu hyperlink."
 COM_MENUS_ITEM_FIELD_ANCHOR_TITLE_LABEL="Link Title Attribute"
+COM_MENUS_ITEM_FIELD_ANCHOR_REL_DESC="An optional, custom rel attribute to get more information about the menu hyperlink (search engines purpose)."
+COM_MENUS_ITEM_FIELD_ANCHOR_REL_LABEL="Link Rel Attribute"
+COM_MENUS_FIELD_VALUE_ALTERNATE="Alternate"
+COM_MENUS_FIELD_VALUE_AUTHOR="Author"
+COM_MENUS_FIELD_VALUE_BOOKMARK="Bookmark"
+COM_MENUS_FIELD_VALUE_HELP="Help"
+COM_MENUS_FIELD_VALUE_LICENSE="License"
+COM_MENUS_FIELD_VALUE_NEXT="Next"
+COM_MENUS_FIELD_VALUE_NOFOLLOW="Nofollow"
+COM_MENUS_FIELD_VALUE_NOREFERRER="Noreferrer"
+COM_MENUS_FIELD_VALUE_PREFETCH="Prefetch"
+COM_MENUS_FIELD_VALUE_PREV="Prev"
+COM_MENUS_FIELD_VALUE_SEARCH="Search"
+COM_MENUS_FIELD_VALUE_TAG="Tag"
 COM_MENUS_ITEM_FIELD_ASSIGNED_DESC="Shows which menu the link will appear in."
 COM_MENUS_ITEM_FIELD_ASSIGNED_LABEL=" Menu"
 COM_MENUS_ITEM_FIELD_ASSOCIATION_NO_VALUE="- No association -"

diff --git a/modules/mod_menu/helper.php b/modules/mod_menu/helper.php
@@ -127,6 +127,7 @@ public static function getList(&$params)
 					$item->title        = htmlspecialchars($item->title, ENT_COMPAT, 'UTF-8', false);
 					$item->anchor_css   = htmlspecialchars($item->params->get('menu-anchor_css', ''), ENT_COMPAT, 'UTF-8', false);
 					$item->anchor_title = htmlspecialchars($item->params->get('menu-anchor_title', ''), ENT_COMPAT, 'UTF-8', false);
+					$item->anchor_rel = htmlspecialchars($item->params->get('menu-anchor_rel', ''), ENT_COMPAT, 'UTF-8', false);
 					$item->menu_image   = $item->params->get('menu_image', '') ?
 						htmlspecialchars($item->params->get('menu_image', ''), ENT_COMPAT, 'UTF-8', false) : '';
 				}

diff --git a/modules/mod_menu/tmpl/default_url.php b/modules/mod_menu/tmpl/default_url.php
@@ -12,6 +12,7 @@
 // Note. It is important to remove spaces between elements.
 $class = $item->anchor_css ? 'class="' . $item->anchor_css . '" ' : '';
 $title = $item->anchor_title ? 'title="' . $item->anchor_title . '" ' : '';
+$rel   = $item->anchor_rel ? 'rel="' . $item->anchor_rel . '" ' : '';
 
 if ($item->menu_image)
 {
@@ -30,11 +31,11 @@
 switch ($item->browserNav) :
 	default:
 	case 0:
-?><a <?php echo $class; ?>href="<?php echo $flink; ?>" <?php echo $title; ?>><?php echo $linktype; ?></a><?php
+?><a <?php echo $class; ?>href="<?php echo $flink; ?>" <?php echo $title . $rel; ?>><?php echo $linktype; ?></a><?php
 		break;
 	case 1:
 		// _blank
-?><a <?php echo $class; ?>href="<?php echo $flink; ?>" target="_blank" <?php echo $title; ?>><?php echo $linktype; ?></a><?php
+?><a <?php echo $class; ?>href="<?php echo $flink; ?>" target="_blank" <?php echo $title . $rel; ?>><?php echo $linktype; ?></a><?php
 		break;
 	case 2:
 		// Use JavaScript "window.open"