Refactoring
This commit is contained in:
parent
0848f083f7
commit
ef5a145c76
|
@ -224,9 +224,8 @@ define([ 'text!documenteditor/main/app/template/RolesManagerDlg.template',
|
||||||
handler : function(result, settings) {
|
handler : function(result, settings) {
|
||||||
if (result == 'ok' && settings) {
|
if (result == 'ok' && settings) {
|
||||||
var color = settings.color,
|
var color = settings.color,
|
||||||
name = settings.name,
|
name = settings.name;
|
||||||
store = me.rolesList.store;
|
me.lastSelectedRole = name;
|
||||||
this.lastSelectedRole = name;
|
|
||||||
if (isEdit) {
|
if (isEdit) {
|
||||||
// me.api.asc_editRole(settings);
|
// me.api.asc_editRole(settings);
|
||||||
rec.set('name', name);
|
rec.set('name', name);
|
||||||
|
@ -236,19 +235,6 @@ define([ 'text!documenteditor/main/app/template/RolesManagerDlg.template',
|
||||||
role.asc_putName(name);
|
role.asc_putName(name);
|
||||||
role.asc_putColor(color);
|
role.asc_putColor(color);
|
||||||
me.oformManager.asc_addRole(role);
|
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();
|
me.updateButtons();
|
||||||
}
|
}
|
||||||
|
@ -279,7 +265,7 @@ define([ 'text!documenteditor/main/app/template/RolesManagerDlg.template',
|
||||||
|
|
||||||
var callback = function(toRole) {
|
var callback = function(toRole) {
|
||||||
var index = store.indexOf(rec);
|
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
|
me.api.asc_delRole(rec.get('name'), toRole); // remove role and move it's fields
|
||||||
|
|
||||||
// if (toRole) {
|
// if (toRole) {
|
||||||
|
|
Loading…
Reference in a new issue