Skip to content

Commit

Permalink
[se] Fix bug 73213
Browse files Browse the repository at this point in the history
  • Loading branch information
GoshaZotov committed Feb 13, 2025
1 parent 68c9e4f commit 38e24d0
Showing 1 changed file with 13 additions and 14 deletions.
27 changes: 13 additions & 14 deletions cell/model/UndoRedo.js
Original file line number Diff line number Diff line change
Expand Up @@ -3448,7 +3448,6 @@ function (window, undefined) {
return;
}
var collaborativeEditing = wb.oApi.collaborativeEditing;
var workSheetView;
var changeFreezePane;
if (AscCH.historyitem_Worksheet_RemoveCell === Type) {
nRow = Data.nRow;
Expand Down Expand Up @@ -3530,9 +3529,9 @@ function (window, undefined) {

//нужно для того, чтобы грамотно выставлялись цвета в ф/т при ручном скрытии строк, затрагивающих ф/т(undo/redo)
//TODO для случая скрытия строк фильтром(undo), может два раза вызываться функция setColorStyleTable - пересмотреть
workSheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
workSheetView.model.autoFilters.reDrawFilter(null, index);
worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (worksheetView) {
worksheetView.model.autoFilters.reDrawFilter(null, index);
}
} else if (AscCH.historyitem_Worksheet_RowHide === Type) {
from = Data.from;
Expand All @@ -3556,9 +3555,9 @@ function (window, undefined) {

ws.setRowHidden(nRow, from, to);

workSheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
workSheetView.model.autoFilters.reDrawFilter(new Asc.Range(0, from, ws.nColsCount - 1, to));
worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (worksheetView) {
worksheetView.model.autoFilters.reDrawFilter(new Asc.Range(0, from, ws.nColsCount - 1, to));
}
} else if (AscCH.historyitem_Worksheet_AddRows === Type || AscCH.historyitem_Worksheet_RemoveRows === Type) {
from = Data.from;
Expand Down Expand Up @@ -3594,7 +3593,7 @@ function (window, undefined) {

// ToDo Так делать неправильно, нужно поправить (перенести логику в model, а отрисовку отделить)
worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
if (worksheetView) {
worksheetView.cellCommentator.updateCommentsDependencies(bInsert, operType, range);
worksheetView.shiftCellWatches(bInsert, operType, range);

Expand Down Expand Up @@ -3641,7 +3640,7 @@ function (window, undefined) {

// ToDo Так делать неправильно, нужно поправить (перенести логику в model, а отрисовку отделить)
worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
if (worksheetView) {
worksheetView.cellCommentator.updateCommentsDependencies(bInsert, operType, range);
worksheetView.shiftCellWatches(bInsert, operType, range);

Expand Down Expand Up @@ -3688,7 +3687,7 @@ function (window, undefined) {

// ToDo Так делать неправильно, нужно поправить (перенести логику в model, а отрисовку отделить)
worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
if (worksheetView) {
worksheetView.cellCommentator.updateCommentsDependencies(bInsert, operType, range.bbox);
worksheetView.shiftCellWatches(bInsert, operType, range.bbox);
}
Expand Down Expand Up @@ -3727,7 +3726,7 @@ function (window, undefined) {

// ToDo Так делать неправильно, нужно поправить (перенести логику в model, а отрисовку отделить)
worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
if (worksheetView) {
worksheetView.cellCommentator.updateCommentsDependencies(bInsert, operType, range.bbox);
worksheetView.shiftCellWatches(bInsert, operType, range.bbox);
}
Expand Down Expand Up @@ -3755,7 +3754,7 @@ function (window, undefined) {
range._sortByArray(bbox, places, null, sortByRow);

worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
if (worksheetView) {
worksheetView.model.autoFilters.resetTableStyles(bbox);
}
} else if (AscCH.historyitem_Worksheet_MoveRange == Type) {
Expand Down Expand Up @@ -3795,7 +3794,7 @@ function (window, undefined) {
}
}
worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
if (worksheetView) {
if (bUndo)//если на Undo перемещается диапазон из форматированной таблицы - стиль форматированной таблицы не должен цепляться
{
worksheetView.model.autoFilters._cleanStyleTable(to);
Expand Down Expand Up @@ -3933,7 +3932,7 @@ function (window, undefined) {
_c1 = updateData.c1;
}
worksheetView = wb.oApi.wb.getWorksheetById(nSheetId);
if (workSheetView) {
if (worksheetView) {
worksheetView._updateFreezePane(_c1, _r1, /*lockDraw*/true);
}
} else if (AscCH.historyitem_Worksheet_SetTabColor === Type) {
Expand Down

0 comments on commit 38e24d0

Please sign in to comment.