From da33265d73d5cdef2e7c468ccbf02262be405be8 Mon Sep 17 00:00:00 2001
From: Bas Brands <bas@moodle.com>
Date: Mon, 28 Jun 2021 12:18:16 +0200
Subject: [PATCH] MDL-72032 core_behat: secondary navigation behat fixes

- Part of: MDL-69588
---
 lib/outputrenderers.php                   |  7 +++++--
 lib/templates/moremenu.mustache           |  2 +-
 lib/templates/settings_link_page.mustache | 25 +++++++++++++++++++++--
 lib/tests/behat/behat_navigation.php      | 11 ++++++++++
 theme/boost/layout/drawers.php            | 17 ++++++++++++++-
 theme/boost/templates/drawers.mustache    |  3 +++
 6 files changed, 59 insertions(+), 6 deletions(-)

diff --git a/lib/outputrenderers.php b/lib/outputrenderers.php
index aae32068f2e..42007a1aad0 100644
--- a/lib/outputrenderers.php
+++ b/lib/outputrenderers.php
@@ -3816,18 +3816,21 @@ EOD;
      * @return string
      */
     public function more_menu($content, $navbarstyle) {
+        $tabs = ($navbarstyle == 'nav-tabs');
         if (is_object($content)) {
             if (!isset($content->children) || count($content->children) == 0) {
                 return false;
             }
             return $this->render_from_template('core/moremenu', (object) [
                 'nodecollection' => $content,
-                'navbarstyle' => $navbarstyle
+                'navbarstyle' => $navbarstyle,
+                'tabs' => $tabs
             ]);
         } else {
             return $this->render_from_template('core/moremenu', (object) [
                 'nodearray' => $content,
-                'navbarstyle' => $navbarstyle
+                'navbarstyle' => $navbarstyle,
+                'tabs' => $tabs
             ]);
         }
     }
diff --git a/lib/templates/moremenu.mustache b/lib/templates/moremenu.mustache
index 3e41d1f1c50..9111651ed95 100644
--- a/lib/templates/moremenu.mustache
+++ b/lib/templates/moremenu.mustache
@@ -45,7 +45,7 @@
     }
 }}
 <nav class="moremenu">
-    <ul id="moremenu-{{ uniqid }}" class="nav more-nav {{navbarstyle}}">
+    <ul id="moremenu-{{ uniqid }}" {{#tabs}}role="tablist"{{/tabs}} class="nav more-nav {{navbarstyle}}">
         {{#nodecollection}}
             {{#children}}
                 {{> core/moremenu_children}}
diff --git a/lib/templates/settings_link_page.mustache b/lib/templates/settings_link_page.mustache
index f051ce4bd17..ccce9c27772 100644
--- a/lib/templates/settings_link_page.mustache
+++ b/lib/templates/settings_link_page.mustache
@@ -36,8 +36,29 @@
         }
     }
 }}
-
-{{{secondarynavigation}}}
+{{#secondarynavigation}}
+    {{{secondarynavigation}}}
+{{/secondarynavigation}}
+{{^secondarynavigation}}
+    <ul class="nav nav-tabs" role="tablist">
+        <!-- First the top most node and immediate children -->
+        <li class="nav-item">
+            <a class="nav-link active" href="#link{{node.key}}" data-toggle="tab" role="tab" aria-selected="true">{{node.text}}</a>
+        </li>
+        <!-- Now the first level children with sub nodes -->
+        {{#node.children}}
+            {{#children.count}}
+                {{#display}}
+                    {{^is_short_branch}}
+                        <li class="nav-item">
+                            <a class="nav-link" href="#link{{key}}" data-toggle="tab" role="tab" aria-selected="false" tabindex="-1">{{text}}</a>
+                        </li>
+                    {{/is_short_branch}}
+                {{/display}}
+            {{/children.count}}
+        {{/node.children}}
+    </ul>
+{{/secondarynavigation}}
 
 <div class="tab-content mt-3">
     <div class="tab-pane active" id="link{{node.key}}" role="tabpanel">
diff --git a/lib/tests/behat/behat_navigation.php b/lib/tests/behat/behat_navigation.php
index 3e96a00cccd..8ee9d1c3481 100644
--- a/lib/tests/behat/behat_navigation.php
+++ b/lib/tests/behat/behat_navigation.php
@@ -936,6 +936,17 @@ class behat_navigation extends behat_base {
                     $xpath .= '//div[@id = ' . $tabid . ']';
                 }
                 array_shift($parentnodes);
+            } else {
+                $linkname = behat_context_helper::escape(get_string('moremenu'));
+                $menuxpath = '//ul[@role=\'tablist\']/li/a[contains(normalize-space(.), ' . $linkname . ')]';
+                $morebutton = $this->getSession()->getPage()->find('xpath', $menuxpath);
+                if ($morebutton) {
+                    $this->execute('behat_general::i_click_on', [$morebutton, 'NodeElement']);
+                    $moreitemxpath = '//ul[@data-region=\'moredropdown\']/li/a[contains(normalize-space(.), ' . $tabname . ')]';
+                    if ($morenode = $this->getSession()->getPage()->find('xpath', $moreitemxpath)) {
+                        $this->execute('behat_general::i_click_on', [$morenode, 'NodeElement']);
+                    }
+                }
             }
         }
 
diff --git a/theme/boost/layout/drawers.php b/theme/boost/layout/drawers.php
index 16607635b78..cddf4d8f99c 100644
--- a/theme/boost/layout/drawers.php
+++ b/theme/boost/layout/drawers.php
@@ -68,6 +68,17 @@ $buildregionmainsettings = !$PAGE->include_region_main_settings_in_header_action
 // If the settings menu will be included in the header then don't add it here.
 $regionmainsettingsmenu = $buildregionmainsettings ? $OUTPUT->region_main_settings_menu() : false;
 
+if (defined('BEHAT_SITE_RUNNING')) {
+    $secondarynavigation = false;
+} else {
+    $buildsecondarynavigation = $PAGE->has_secondary_navigation();
+    $secondarynavigation = $buildsecondarynavigation ? $OUTPUT->more_menu($PAGE->secondarynav, 'nav-tabs') : false;
+}
+
+$primary = new core\navigation\output\primary($PAGE);
+$renderer = $PAGE->get_renderer('core');
+$primarymenu = $primary->export_for_template($renderer);
+
 $templatecontext = [
     'sitename' => format_string($SITE->shortname, true, ['context' => context_course::instance(SITEID), "escape" => false]),
     'output' => $OUTPUT,
@@ -79,7 +90,11 @@ $templatecontext = [
     'blockdraweropen' => $blockdraweropen,
     'regionmainsettingsmenu' => $regionmainsettingsmenu,
     'courseindex' => $courseindex,
-    'hasregionmainsettingsmenu' => !empty($regionmainsettingsmenu)
+    'hasregionmainsettingsmenu' => !empty($regionmainsettingsmenu),
+    'primarymoremenu' => $OUTPUT->more_menu(array_merge($primarymenu['primary'], $primarymenu['custom']), 'navbar-nav'),
+    'secondarymoremenu' => $secondarynavigation,
+    'usermenu' => $primarymenu['user'],
+    'langmenu' => $primarymenu['lang'],
 ];
 
 $nav = $PAGE->flatnav;
diff --git a/theme/boost/templates/drawers.mustache b/theme/boost/templates/drawers.mustache
index 2da234ed00f..59e18872f8a 100644
--- a/theme/boost/templates/drawers.mustache
+++ b/theme/boost/templates/drawers.mustache
@@ -110,6 +110,9 @@
                 {{/hasblocks}}
             </div>
             {{{ output.full_header }}}
+            <div class="secondarynavigation">
+                {{{ secondarymoremenu }}}
+            </div>
             <div id="page-content" class="pb-3 d-print-block">
                 <div id="region-main-box">
                     {{#hasregionmainsettingsmenu}}