Revert "Fix issue 7301 - broken vhost link (backport #7304) (backport #7314)"

deps/rabbitmq_management/priv/www/js/tmpl/permissions.ejs

@@ -1,4 +1,4 @@
-<div class="section" id="permissions">
+<div class="section">
   <h2>Permissions</h2>
   <div class="hider">
     <h3>Current permissions</h3>
@@ -55,8 +55,8 @@ for (var i = 0; i < permissions.length; i++) {
           <td>
             <input type="hidden" name="vhost" value="<%= fmt_string(parent.name) %>"/>
             <select name="username">
-              <% for (var i = 0; i < users.length; i++) { %>
-              <option value="<%= fmt_string(users[i].name) %>"><%= fmt_string(users[i].name) %></option>
+              <% for (var i = 0; i < users.items.length; i++) { %>
+              <option value="<%= fmt_string(users.items[i].name) %>"><%= fmt_string(users.items[i].name) %></option>
               <% } %>
             </select>
           </td>

deps/rabbitmq_management/priv/www/js/tmpl/topic-permissions.ejs

@@ -1,4 +1,4 @@
-<div class="section" id="topic-permissions">
+<div class="section">
   <h2>Topic permissions</h2>
   <div class="hider">
     <h3>Current topic permissions</h3>
@@ -56,8 +56,8 @@ for (var i = 0; i < topic_permissions.length; i++) {
           <td>
             <input type="hidden" name="vhost" value="<%= fmt_string(parent.name) %>"/>
             <select name="username">
-              <% for (var i = 0; i < users.length; i++) { %>
-              <option value="<%= fmt_string(users[i].name) %>"><%= fmt_string(users[i].name) %></option>
+              <% for (var i = 0; i < users.items.length; i++) { %>
+              <option value="<%= fmt_string(users.items[i].name) %>"><%= fmt_string(users.items[i].name) %></option>
               <% } %>
             </select>
           </td>

deps/rabbitmq_management/priv/www/js/tmpl/vhost.ejs

@@ -8,11 +8,11 @@
 <% } %>
 
 <% if (!disable_stats) { %>
-<div class="section" id="overview">
+<div class="section">
   <h2>Overview</h2>
   <div class="hider updatable">
     <%= queue_lengths('lengths-vhost', vhost) %>
-<% if (rates_mode != 'none' && vhost.message_stats) { %>
+<% if (rates_mode != 'none') { %>
     <%= message_rates('msg-rates-vhost', vhost.message_stats) %>
 <% } %>
     <%= data_rates('data-rates-vhost', vhost, 'Data rates') %>
@@ -39,7 +39,6 @@
             </td>
             </tr>
         <% } %>
-
         </table>
         </td>
       </tr>
@@ -52,7 +51,7 @@
 
 <%= format('topic-permissions', {'mode': 'vhost', 'topic_permissions': topic_permissions, 'users':users, 'parent': vhost, 'exchanges': exchanges}) %>
 
-<div class="section-hidden" id="delete-vhost">
+<div class="section-hidden">
 <h2>Delete this vhost</h2>
 <div class="hider">
 <form action="#/vhosts" method="delete" class="confirm">

deps/rabbitmq_management/selenium/test/pageobjects/AdminTab.js

@@ -8,18 +8,10 @@ const FILTER_USER = By.css('input#users-name')
 const CHECKBOX_REGEX = By.css('input#filter-regex-mode')
 const FILTERED_USER = By.css('span.filter-highlight')
 
-// RHM : RIGHT HAND MENU
-const USERS_ON_RHM = By.css('div#rhs ul li a[href="#/users"]')
-const LIMITS_ON_RHM = By.css('div#rhs ul li a[href="#/limits"]')
-const VHOSTS_ON_RHM = By.css('div#rhs ul li a[href="#/vhosts"]')
-
 module.exports = class AdminTab extends OverviewPage {
   async isLoaded () {
     await this.waitForDisplayed(ADMIN_TAB)
   }
-  async clickOnVhosts() {
-    await this.click(VHOSTS_ON_RHM)
-  }
 
   async searchForUser(user, regex = false) {