YDA-5956: remove uneeded API call to retrieve group manager view pref…

…erence

group_manager/group_manager.py

@@ -16,7 +16,6 @@
 @group_manager_bp.route('/')
 def index() -> Response:
     response = api.call('group_data', data={})
-
     group_hierarchy = response['data']['group_hierarchy']
     user_type = response['data']['user_type']
     user_zone = response['data']['user_zone']
@@ -25,15 +24,8 @@ def index() -> Response:
     schema_ids = response['data']['schemas']
     schema_id_default = response['data']['schema_default']
 
-    # Group list view to be presented initially.
-    response = api.call('settings_load', data={})
-    show_list = "TREE"
-    if 'group_manager_view' in response['data']:
-        show_list = response['data']['group_manager_view']
-
     return render_template('group_manager/index.html',
                            group_hierarchy=group_hierarchy,
-                           show_list=show_list,
                            schema_ids=schema_ids,
                            schema_id_default=schema_id_default,
                            user_type=user_type,

group_manager/templates/group_manager/index.html

@@ -33,10 +33,10 @@ <h1 id="group-manager-text">Group manager</h1>
                 Groups
                 <ul class="nav nav-pills float-end" id="pills-tab" role="tablist">
                     <li class="nav-item" role="presentation">
-                        <button class="btn btn-sm me-1 {% if show_list == "TREE" %}active{% endif %}" id="pills-tree-tab" data-bs-toggle="pill" data-bs-target="#pills-tree" type="button" role="tab" aria-controls="pills-tree" aria-selected="{% if show_list == "TREE" %} true {% else %} false {% endif %}">Tree view</button>
+                        <button class="btn btn-sm me-1 {% if g.settings.group_manager_view == "TREE" %}active{% endif %}" id="pills-tree-tab" data-bs-toggle="pill" data-bs-target="#pills-tree" type="button" role="tab" aria-controls="pills-tree" aria-selected="{% if g.settings.group_manager_view == "TREE" %} true {% else %} false {% endif %}">Tree view</button>
                     </li>
                     <li class="nav-item" role="presentation">
-                        <button class="btn btn-sm {% if show_list == "LIST" %}active{% endif %}" id="pills-list-tab" data-bs-toggle="pill" data-bs-target="#pills-list" type="button" role="tab" aria-controls="pills-list" aria-selected="{% if show_list == "LIST" %} true {% else %} false {% endif %}">List view</button>
+                        <button class="btn btn-sm {% if g.settings.group_manager_view == "LIST" %}active{% endif %}" id="pills-list-tab" data-bs-toggle="pill" data-bs-target="#pills-list" type="button" role="tab" aria-controls="pills-list" aria-selected="{% if g.settings.group_manager_view == "LIST" %} true {% else %} false {% endif %}">List view</button>
                     </li>
                 </ul>
             </div>
@@ -46,7 +46,7 @@ <h1 id="group-manager-text">Group manager</h1>
                 </select>
             </div>
             <div class="tab-content" id="pills-tabContent">
-                <div class="tab-pane {% if show_list == "TREE" %} show active {% endif %}" id="pills-tree" role="tabpanel" aria-labelledby="pills-tree-tab">
+                <div class="tab-pane {% if g.settings.group_manager_view == "TREE" %} show active {% endif %}" id="pills-tree" role="tabpanel" aria-labelledby="pills-tree-tab">
                     <div class="card groups">
                         <div class="list-group" id="group-list">
 
@@ -78,7 +78,7 @@ <h1 id="group-manager-text">Group manager</h1>
                         </div>
                     </div>
                 </div>
-                <div class="tab-pane {% if show_list == "LIST" %} show active {% endif %}" id="pills-list" role="tabpanel" aria-labelledby="pills-list-tab">
+                <div class="tab-pane {% if g.settings.group_manager_view == "LIST" %} show active {% endif %}" id="pills-list" role="tabpanel" aria-labelledby="pills-list-tab">
                     <div class="card groups">
 
                         <div id="result-user-search-groups"></div>