From ef5a145c7669c7d815c99befec444bf46a32a364 Mon Sep 17 00:00:00 2001
From: Julia Radzhabova <julia.radzhabova@onlyoffice.com>
Date: Mon, 28 Nov 2022 22:19:29 +0300
Subject: [PATCH] Refactoring

---
 .../main/app/view/RolesManagerDlg.js          | 20 +++----------------
 1 file changed, 3 insertions(+), 17 deletions(-)

diff --git a/apps/documenteditor/main/app/view/RolesManagerDlg.js b/apps/documenteditor/main/app/view/RolesManagerDlg.js
index f585efc27..e36647623 100644
--- a/apps/documenteditor/main/app/view/RolesManagerDlg.js
+++ b/apps/documenteditor/main/app/view/RolesManagerDlg.js
@@ -224,9 +224,8 @@ define([  'text!documenteditor/main/app/template/RolesManagerDlg.template',
                 handler : function(result, settings) {
                     if (result == 'ok' && settings) {
                         var color = settings.color,
-                            name = settings.name,
-                            store = me.rolesList.store;
-                        this.lastSelectedRole = name;
+                            name = settings.name;
+                        me.lastSelectedRole = name;
                         if (isEdit) {
                             // me.api.asc_editRole(settings);
                             rec.set('name', name);
@@ -236,19 +235,6 @@ define([  'text!documenteditor/main/app/template/RolesManagerDlg.template',
                             role.asc_putName(name);
                             role.asc_putColor(color);
                             me.oformManager.asc_addRole(role);
-                            // rec = store.push({
-                            //     name: name,
-                            //     color: color,
-                            //     fields: 0,
-                            //     index: store.length,
-                            //     scope: me
-                            // });
-                            // if (rec) {
-                            //     me.rolesList.selectRecord(rec);
-                            //     setTimeout(function() {
-                            //         me.rolesList.scrollToRecord(rec);
-                            //     }, 50);
-                            // }
                         }
                         me.updateButtons();
                     }
@@ -279,7 +265,7 @@ define([  'text!documenteditor/main/app/template/RolesManagerDlg.template',
 
             var callback = function(toRole) {
                 var index = store.indexOf(rec);
-                this.lastSelectedRole = index;
+                me.lastSelectedRole = index;
                 me.api.asc_delRole(rec.get('name'), toRole); // remove role and move it's fields
 
                 // if (toRole) {