Skip to content

S
sdkjs
Commit 531e97aa authored by Igor.Zotov's avatar Igor.Zotov
Browse files
Options

изменена обработка вставки(удаления) строк/столбцов, содержащих форматированные таблицы или а/ф 

git-svn-id: svn://192.168.3.15/activex/AVS/Sources/TeamlabOffice/trunk/OfficeWeb@51383 954022d7-b5bf-4e40-9824-e11837661b57
parent 1a673061
Hide whitespace changes
Inline Side-by-side
Showing with 136 additions and 24 deletions
Excel/model/autofilters.js
View file @ 531e97aa
...@@ -1646,7 +1646,7 @@ ...@@ -1646,7 +1646,7 @@
ws._isLockedCells (sortRange1, /*subType*/null, onSortAutoFilterCallback); ws._isLockedCells (sortRange1, /*subType*/null, onSortAutoFilterCallback);
}, },
isEmptyAutoFilters: function(ar, turnOnHistory, insCells) isEmptyAutoFilters: function(ar, turnOnHistory, insCells, deleteFilterAfterDeleteColRow)
{ {
if(turnOnHistory) if(turnOnHistory)
{ {
...@@ -1694,7 +1694,7 @@ ...@@ -1694,7 +1694,7 @@
//открываем скрытые строки //открываем скрытые строки
aWs.setRowHidden(false, bbox.r1, bbox.r2); aWs.setRowHidden(false, bbox.r1, bbox.r2);
//заносим в историю //заносим в историю
this._addHistoryObj(oCurFilter, historyitem_AutoFilter_Empty, {activeCells: activeCells}); this._addHistoryObj(oCurFilter, historyitem_AutoFilter_Empty, {activeCells: activeCells},deleteFilterAfterDeleteColRow);
} }
else else
{ {
...@@ -1853,7 +1853,7 @@ ...@@ -1853,7 +1853,7 @@
aWs.TableParts = []; aWs.TableParts = [];
aWs.TableParts[aWs.TableParts.length] = cloneData; aWs.TableParts[aWs.TableParts.length] = cloneData;
var splitRange = cloneData.Ref.split(':'); var splitRange = cloneData.Ref.split(':');
if(!gUndoInsDelCellsFlag) /*if(!gUndoInsDelCellsFlag)
{ {
gUndoInsDelCellsFlag = gUndoInsDelCellsFlag =
{ {
...@@ -1862,7 +1862,7 @@ ...@@ -1862,7 +1862,7 @@
data: cloneData data: cloneData
} }
} }
else else*/
this._setColorStyleTable(splitRange[0], splitRange[1], cloneData, null, true); this._setColorStyleTable(splitRange[0], splitRange[1], cloneData, null, true);
this._addButtonAF({result: cloneData.result,isVis: true}); this._addButtonAF({result: cloneData.result,isVis: true});
} }
...@@ -2036,18 +2036,83 @@ ...@@ -2036,18 +2036,83 @@
return objOptions; return objOptions;
}, },
//если селект затрагивает часть хотя бы одной форматированной таблицы(для случая insert(delete) cells) //если селект затрагивает часть хотя бы одной форматированной таблицы(для случая insert(delete) cells)
isActiveCellsCrossHalfFTable: function(activeCells, val, prop) isActiveCellsCrossHalfFTable: function(activeCells, val, prop, bUndoRedo)
{ {
var InsertCellsAndShiftDown = (val == c_oAscInsertOptions.InsertCellsAndShiftDown && prop == 'insCell') ? true : false; var InsertCellsAndShiftDown = (val == c_oAscInsertOptions.InsertCellsAndShiftDown && prop == 'insCell') ? true : false;
var InsertCellsAndShiftRight = (val == c_oAscInsertOptions.InsertCellsAndShiftRight && prop == 'insCell') ? true : false; var InsertCellsAndShiftRight = (val == c_oAscInsertOptions.InsertCellsAndShiftRight && prop == 'insCell') ? true : false;
var DeleteCellsAndShiftLeft = (val == c_oAscDeleteOptions.DeleteCellsAndShiftLeft && prop == 'delCell') ? true : false;; var DeleteCellsAndShiftLeft = (val == c_oAscDeleteOptions.DeleteCellsAndShiftLeft && prop == 'delCell') ? true : false;
var DeleteCellsAndShiftTop = (val == c_oAscDeleteOptions.DeleteCellsAndShiftTop && prop == 'delCell') ? true : false;; var DeleteCellsAndShiftTop = (val == c_oAscDeleteOptions.DeleteCellsAndShiftTop && prop == 'delCell') ? true : false;
var DeleteColumns = (val == c_oAscDeleteOptions.DeleteColumns && prop == 'delCell') ? true : false;
var DeleteRows = (val == c_oAscDeleteOptions.DeleteRows && prop == 'delCell') ? true : false;
var ws = this.worksheet; var ws = this.worksheet;
var aWs = this._getCurrentWS(); var aWs = this._getCurrentWS();
var tableParts = aWs.TableParts; var tableParts = aWs.TableParts;
var autoFilter = aWs.AutoFilter; var autoFilter = aWs.AutoFilter;
var result = true; var result = true;
if(DeleteColumns || DeleteRows)
{
//меняем активную область
var newActiveRange;
if(DeleteRows)
{
newActiveRange =
{
c1: ws.visibleRange.c1,
c2: ws.visibleRange.c2,
r1: activeCells.r1,
r2: activeCells.r2
}
}
else
{
newActiveRange =
{
c1: activeCells.c1,
c2: activeCells.c2,
r1: ws.visibleRange.r1,
r2: ws.visibleRange.r2
}
}
//если активной областью захвачена полнотью форматированная таблица(или её часть) + часть форматированной таблицы - выдаём ошибку
if(tableParts)
{
var tableRange;
var isExp = false;
var isPart = false;
for(var i = 0; i < tableParts.length; i++ )
{
tableRange = this._refToRange(tableParts[i].Ref);
//если хотя бы одна ячейка активной области попадает внутрь форматированной таблицы
if(this._rangeHitInAnRange(newActiveRange, tableRange))
{
if(isExp && isPart)//часть + целая
{
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical);
return false;
}
if(newActiveRange.c1 <= tableRange.c1 && newActiveRange.c2 >= tableRange.c2 && newActiveRange.r1 <= tableRange.r1 && newActiveRange.r2 >= tableRange.r2)
isExp = true;
else if(isExp)
{
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical);
return false;
}
else if(isPart)//уже часть захвачена + ещё одна часть
{
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical);
return false;
}
else
isPart = true;
}
}
}
return result;
}
//проверка на то, что захвачен кусок форматированной таблицы //проверка на то, что захвачен кусок форматированной таблицы
if(tableParts)//при удалении в MS Excel ошибка может возникать только в случае форматированных таблиц if(tableParts)//при удалении в MS Excel ошибка может возникать только в случае форматированных таблиц
{ {
...@@ -2079,7 +2144,8 @@ ...@@ -2079,7 +2144,8 @@
} }
if(!isExp) if(!isExp)
{ {
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical); if(!bUndoRedo)
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical);
return false; return false;
} }
} }
...@@ -2092,7 +2158,8 @@ ...@@ -2092,7 +2158,8 @@
//если данный фильтр находится справа //если данный фильтр находится справа
if(tableRange.c1 > activeCells.c1 && (tableRange.r1 < activeCells.r1 || tableRange.r2 > activeCells.r2)) if(tableRange.c1 > activeCells.c1 && (tableRange.r1 < activeCells.r1 || tableRange.r2 > activeCells.r2))
{ {
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical); if(!bUndoRedo)
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical);
return false; return false;
} }
} }
...@@ -2101,7 +2168,8 @@ ...@@ -2101,7 +2168,8 @@
//если данный фильтр находится внизу //если данный фильтр находится внизу
if(tableRange.r1 > activeCells.r1 && (tableRange.c1 < activeCells.c1 || tableRange.c2 > activeCells.c2)) if(tableRange.r1 > activeCells.r1 && (tableRange.c1 < activeCells.c1 || tableRange.c2 > activeCells.c2))
{ {
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical); if(!bUndoRedo)
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical);
return false; return false;
} }
...@@ -2111,7 +2179,8 @@ ...@@ -2111,7 +2179,8 @@
//если данный фильтр находится справа //если данный фильтр находится справа
if(tableRange.c1 > activeCells.c1 && (tableRange.r1 < activeCells.r1 || tableRange.r2 > activeCells.r2)) if(tableRange.c1 > activeCells.c1 && (tableRange.r1 < activeCells.r1 || tableRange.r2 > activeCells.r2))
{ {
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical); if(!bUndoRedo)
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical);
return false; return false;
} }
} }
...@@ -2120,7 +2189,8 @@ ...@@ -2120,7 +2189,8 @@
//если данный фильтр находится внизу //если данный фильтр находится внизу
if(tableRange.r1 > activeCells.r1 && (tableRange.c1 < activeCells.c1 || tableRange.c2 > activeCells.c2)) if(tableRange.r1 > activeCells.r1 && (tableRange.c1 < activeCells.c1 || tableRange.c2 > activeCells.c2))
{ {
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical); if(!bUndoRedo)
ws.model.workbook.handlers.trigger("asc_onError", c_oAscError.ID.AutoFilterChangeFormatTableError, c_oAscError.Level.NoCritical);
return false; return false;
} }
} }
...@@ -4431,24 +4501,30 @@ ...@@ -4431,24 +4501,30 @@
}, },
//change filters after insert column //change filters after insert column
_changeFiltersAfterColumn: function(col,val,type) _changeFiltersAfterColumn: function(col, val, type)
{ {
History.TurnOff(); History.TurnOff();
var aWs = this._getCurrentWS(); var aWs = this._getCurrentWS();
var bUndoChanges = this.worksheet.model.workbook.bUndoChanges;
if(aWs.AutoFilter) if(aWs.AutoFilter)
{ {
var ref = aWs.AutoFilter.Ref.split(':'); var ref = aWs.AutoFilter.Ref.split(':');
var doNotChangesHeadString = false;
var newCol = col;
if(val > 0 && bUndoChanges && ((col == this._idToRange(ref[0]).c1 && type == "insCol") || (col == this._idToRange(ref[0]).r1 && type == "insRow")))
newCol++;
var options = { var options = {
ref:ref, ref:ref,
val:val, val:val,
type:type, type:type,
col:col col:newCol
}; };
//внутри данного фильтра располагается колонка(колонки) //внутри данного фильтра располагается колонка(колонки)
this._changeFilterAfterInsertColumn(options,type); this._changeFilterAfterInsertColumn(options,type);
} }
if(aWs.TableParts && aWs.TableParts.length > 0) if(aWs.TableParts && aWs.TableParts.length > 0)
{ {
var length;
for(var lT = 0; lT < aWs.TableParts.length; lT++) for(var lT = 0; lT < aWs.TableParts.length; lT++)
{ {
var ref = aWs.TableParts[lT].Ref.split(':'); var ref = aWs.TableParts[lT].Ref.split(':');
...@@ -4459,8 +4535,11 @@ ...@@ -4459,8 +4535,11 @@
col:col, col:col,
index: lT index: lT
}; };
length = aWs.TableParts.length;
//внутри данного фильтра располагается колонка //внутри данного фильтра располагается колонка
this._changeFilterAfterInsertColumn(options,type); this._changeFilterAfterInsertColumn(options,type);
if(length > aWs.TableParts.length)
lT--;
} }
} }
// ToDo - от _reDrawFilters в будущем стоит избавиться, ведь она проставляет стили ячейкам, а это не нужно делать (сменить отрисовку) // ToDo - от _reDrawFilters в будущем стоит избавиться, ведь она проставляет стили ячейкам, а это не нужно делать (сменить отрисовку)
...@@ -4485,6 +4564,7 @@ ...@@ -4485,6 +4564,7 @@
var colStart = col; var colStart = col;
var colEnd = col + Math.abs(val) - 1; var colEnd = col + Math.abs(val) - 1;
//диапазон фильтра
var startRangeCell = startRange.c1; var startRangeCell = startRange.c1;
var endRangeCell = endRange.c2; var endRangeCell = endRange.c2;
if(type == 'insRow') if(type == 'insRow')
...@@ -4520,8 +4600,9 @@ ...@@ -4520,8 +4600,9 @@
{ {
var valNew = startRangeCell - colEnd - 1; var valNew = startRangeCell - colEnd - 1;
var val2 = colStart - startRangeCell; var val2 = colStart - startRangeCell;
this._editFilterAfterInsertColumn(range,valNew,startRangeCell,type); var retVal = this._editFilterAfterInsertColumn(range,valNew,startRangeCell,type);
this._editFilterAfterInsertColumn(range,val2,undefined,type); if(!retVal)
this._editFilterAfterInsertColumn(range,val2,undefined,type);
} }
else else
{ {
...@@ -4598,8 +4679,8 @@ ...@@ -4598,8 +4679,8 @@
r2: endCell.r1, r2: endCell.r1,
c2: endCell.c1 c2: endCell.c1
} }
this.isEmptyAutoFilters(activeRange, true, true); this.isEmptyAutoFilters(activeRange, true, true, true);
return; return true;
} }
} }
...@@ -4631,6 +4712,7 @@ ...@@ -4631,6 +4712,7 @@
//change result into filter and change info in button //change result into filter and change info in button
if(filter.result && filter.result.length > 0) if(filter.result && filter.result.length > 0)
{ {
var insertIndexes = [];
for(var filR = 0; filR < filter.result.length; filR++) for(var filR = 0; filR < filter.result.length; filR++)
{ {
var curFilter = filter.result[filR]; var curFilter = filter.result[filR];
...@@ -4664,11 +4746,12 @@ ...@@ -4664,11 +4746,12 @@
{ {
for(var b = 0; b < buttons.length; b++) for(var b = 0; b < buttons.length; b++)
{ {
if(buttons[b].id == curFilter.id) if(buttons[b].id == curFilter.id && !insertIndexes[b])
{ {
buttons[b].inFilter = filter.Ref; buttons[b].inFilter = filter.Ref;
buttons[b].id = id; buttons[b].id = id;
buttons[b].idNext = nextId; buttons[b].idNext = nextId;
insertIndexes[b] = true;
break; break;
} }
} }
...@@ -4699,6 +4782,7 @@ ...@@ -4699,6 +4782,7 @@
var newResult = []; var newResult = [];
var n = 0; var n = 0;
var isChangeColumn = false; var isChangeColumn = false;
var insertIndexes = [];
for(var filR = 0; filR < filter.result.length; filR++) for(var filR = 0; filR < filter.result.length; filR++)
{ {
var endCount = 0; var endCount = 0;
...@@ -4828,6 +4912,21 @@ ...@@ -4828,6 +4912,21 @@
var id = this._rangeToId(newFirstCol); var id = this._rangeToId(newFirstCol);
var nextId = this._rangeToId(newNextCol); var nextId = this._rangeToId(newNextCol);
if(buttons && buttons.length)
{
for(var b = 0; b < buttons.length; b++)
{
if(buttons[b].id == curFilter.id && !insertIndexes[b])
{
buttons[b].inFilter = inFilter;
buttons[b].id = id;
buttons[b].idNext = nextId;
insertIndexes[b] = true;
break;
}
}
}
curFilter.inFilter = inFilter; curFilter.inFilter = inFilter;
curFilter.id = id; curFilter.id = id;
curFilter.idNext = nextId; curFilter.idNext = nextId;
...@@ -5594,7 +5693,7 @@ ...@@ -5594,7 +5693,7 @@
} }
}, },
_addHistoryObj: function (oldObj, type, redoObject) { _addHistoryObj: function (oldObj, type, redoObject, deleteFilterAfterDeleteColRow) {
var ws = this.worksheet; var ws = this.worksheet;
var oHistoryObject = new UndoRedoData_AutoFilter(); var oHistoryObject = new UndoRedoData_AutoFilter();
oHistoryObject.undo = oldObj; oHistoryObject.undo = oldObj;
...@@ -5609,6 +5708,11 @@ ...@@ -5609,6 +5708,11 @@
oHistoryObject.moveTo = redoObject.arnTo; oHistoryObject.moveTo = redoObject.arnTo;
History.Add(g_oUndoRedoAutoFilters, type, ws.model.getId(), null, oHistoryObject); History.Add(g_oUndoRedoAutoFilters, type, ws.model.getId(), null, oHistoryObject);
if(deleteFilterAfterDeleteColRow && History.CurPoint && History.CurPoint.Items && History.CurPoint.Items.length)
{
var popEl = History.CurPoint.Items.pop();
History.CurPoint.Items.unshift(popEl);
}
}, },
_isAddNameColumn: function(range) _isAddNameColumn: function(range)
...@@ -5716,6 +5820,8 @@ ...@@ -5716,6 +5820,8 @@
var isSequence = false; var isSequence = false;
if(indexInsertColumn != undefined) if(indexInsertColumn != undefined)
{ {
if(indexInsertColumn < 0)
indexInsertColumn = 0;
var nameStart; var nameStart;
var nameEnd; var nameEnd;
if(tableColumns[indexInsertColumn] && tableColumns[indexInsertColumn].Name) if(tableColumns[indexInsertColumn] && tableColumns[indexInsertColumn].Name)
......
Excel/view/WorksheetView.js
View file @ 531e97aa
...@@ -8149,7 +8149,7 @@ ...@@ -8149,7 +8149,7 @@
switch (val) { switch (val) {
case c_oAscInsertOptions.InsertCellsAndShiftRight: case c_oAscInsertOptions.InsertCellsAndShiftRight:
functionModelAction = function () { functionModelAction = function () {
var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeIns, c_oAscInsertOptions.InsertCellsAndShiftRight, prop); var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeIns, c_oAscInsertOptions.InsertCellsAndShiftRight, prop, bUndoRedo);
if(!isCheckChangeAutoFilter) if(!isCheckChangeAutoFilter)
return; return;
if (range.addCellsShiftRight()) { if (range.addCellsShiftRight()) {
...@@ -8175,7 +8175,7 @@ ...@@ -8175,7 +8175,7 @@
return; return;
case c_oAscInsertOptions.InsertCellsAndShiftDown: case c_oAscInsertOptions.InsertCellsAndShiftDown:
functionModelAction = function () { functionModelAction = function () {
var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeIns, c_oAscInsertOptions.InsertCellsAndShiftDown, prop); var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeIns, c_oAscInsertOptions.InsertCellsAndShiftDown, prop, bUndoRedo);
if(!isCheckChangeAutoFilter) if(!isCheckChangeAutoFilter)
return; return;
if (range.addCellsShiftBottom()) { if (range.addCellsShiftBottom()) {
...@@ -8252,7 +8252,7 @@ ...@@ -8252,7 +8252,7 @@
range = t.model.getRange3(_updateRangeDel.r1, _updateRangeDel.c1, _updateRangeDel.r2, _updateRangeDel.c2); range = t.model.getRange3(_updateRangeDel.r1, _updateRangeDel.c1, _updateRangeDel.r2, _updateRangeDel.c2);
switch (val) { switch (val) {
case c_oAscDeleteOptions.DeleteCellsAndShiftLeft: case c_oAscDeleteOptions.DeleteCellsAndShiftLeft:
var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeDel, c_oAscDeleteOptions.DeleteCellsAndShiftLeft, prop); var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeDel, c_oAscDeleteOptions.DeleteCellsAndShiftLeft, prop, bUndoRedo);
if(!isCheckChangeAutoFilter) if(!isCheckChangeAutoFilter)
return; return;
...@@ -8278,7 +8278,7 @@ ...@@ -8278,7 +8278,7 @@
gc_nMaxCol0, _updateRangeDel.r2), null, onChangeWorksheetCallback); gc_nMaxCol0, _updateRangeDel.r2), null, onChangeWorksheetCallback);
return; return;
case c_oAscDeleteOptions.DeleteCellsAndShiftTop: case c_oAscDeleteOptions.DeleteCellsAndShiftTop:
var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeDel, c_oAscDeleteOptions.DeleteCellsAndShiftTop, prop); var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeDel, c_oAscDeleteOptions.DeleteCellsAndShiftTop, prop, bUndoRedo);
if(!isCheckChangeAutoFilter) if(!isCheckChangeAutoFilter)
return; return;
...@@ -8304,6 +8304,9 @@ ...@@ -8304,6 +8304,9 @@
_updateRangeDel.c2, gc_nMaxRow0), null, onChangeWorksheetCallback); _updateRangeDel.c2, gc_nMaxRow0), null, onChangeWorksheetCallback);
return; return;
case c_oAscDeleteOptions.DeleteColumns: case c_oAscDeleteOptions.DeleteColumns:
var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeDel, c_oAscDeleteOptions.DeleteColumns, prop, bUndoRedo);
if(!isCheckChangeAutoFilter)
return;
functionModelAction = function () { functionModelAction = function () {
fullRecalc = true; fullRecalc = true;
History.Create_NewPoint(); History.Create_NewPoint();
...@@ -8324,6 +8327,9 @@ ...@@ -8324,6 +8327,9 @@
onChangeWorksheetCallback); onChangeWorksheetCallback);
return; return;
case c_oAscDeleteOptions.DeleteRows: case c_oAscDeleteOptions.DeleteRows:
var isCheckChangeAutoFilter = t.autoFilters.isActiveCellsCrossHalfFTable(_updateRangeDel, c_oAscDeleteOptions.DeleteRows, prop, bUndoRedo);
if(!isCheckChangeAutoFilter)
return;
functionModelAction = function () { functionModelAction = function () {
fullRecalc = true; fullRecalc = true;
History.Create_NewPoint(); History.Create_NewPoint();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7