From 005a41456153f37638e01ad04607960efd739657 Mon Sep 17 00:00:00 2001
From: SaadYousaf <saadyousaf@A006-00314.local>
Date: Wed, 13 Jan 2021 14:11:49 +0500
Subject: [PATCH] [TNL-7361] - Restrict Instructor Tab from masquerading user
 with student role.

---
 .../courseware/course_navigation.html         | 30 +++++++++++--------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/lms/templates/courseware/course_navigation.html b/lms/templates/courseware/course_navigation.html
index 16257629c7..b0fa4e4307 100644
--- a/lms/templates/courseware/course_navigation.html
+++ b/lms/templates/courseware/course_navigation.html
@@ -44,19 +44,23 @@ if course is not None:
                     <%
                     tab_is_active = tab.tab_id in (active_page, default_tab)
                     %>
-                    <li class="nav-item ${'active' if tab_is_active else ''}">
-                        <a href="${tab.link_func(course, reverse)}" class="nav-link">
-                            ${_(tab.name)}
-                            % if tab_is_active:
-                                <span class="sr-only">, ${_('current location')}</span>
-                            %endif
-                            % if tab_image:
-                                ## Translators: 'needs attention' is an alternative string for the
-                                ## notification image that indicates the tab "needs attention".
-                                <img src="${tab_image}" alt="${_('needs attention')}" />
-                            %endif
-                        </a>
-                    </li>
+                        % if masquerade is not None and masquerade.role == 'student' and tab.name == 'Instructor':
+                            <% continue %>
+                        % else:
+                            <li class="nav-item ${'active' if tab_is_active else ''}">
+                                <a href="${tab.link_func(course, reverse)}" class="nav-link">
+                                    ${_(tab.name)}
+                                    % if tab_is_active:
+                                        <span class="sr-only">, ${_('current location')}</span>
+                                    %endif
+                                    % if tab_image:
+                                        ## Translators: 'needs attention' is an alternative string for the
+                                        ## notification image that indicates the tab "needs attention".
+                                        <img src="${tab_image}" alt="${_('needs attention')}" />
+                                    %endif
+                                </a>
+                            </li>
+                        %endif
                 % endfor
             </ul>
         </nav>