holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Etemplate: Don't try to delete row if it's not in the nextmatch

Browse Source
This commit is contained in:
nathangray 2020-08-26 11:24:15 -06:00
parent d525347275
commit 075c44a48b
2 changed files with 59 additions and 54 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

50
api/js/etemplate/et2_extension_nextmatch.js
View File

@ -501,31 +501,33 @@ var et2_nextmatch = /** @class */ (function (_super) {
// Record current & next index
var uid = _row_ids[0].toString().indexOf(this.controller.dataStorePrefix) == 0 ? _row_ids[0] : this.controller.dataStorePrefix + "::" + _row_ids[0];
var entry = this.controller._selectionMgr._getRegisteredRowsEntry(uid);
var next = (entry.ao ? entry.ao.getNext(_row_ids.length) : null);
if (next == null || !next.id || next.id == uid) {
// No next, select previous
next = (entry.ao ? entry.ao.getPrevious(1) : null);
if (entry && entry.idx !== null) {
var next = (entry.ao ? entry.ao.getNext(_row_ids.length) : null);
if (next == null || !next.id || next.id == uid) {
// No next, select previous
next = (entry.ao ? entry.ao.getPrevious(1) : null);
}
// Stop automatic updating
this.dataview.grid.doInvalidate = false;
for (var i = 0; i < _row_ids.length; i++) {
uid = _row_ids[i].toString().indexOf(this.controller.dataStorePrefix) == 0 ? _row_ids[i] : this.controller.dataStorePrefix + "::" + _row_ids[i];
// Delete from internal references
this.controller.deleteRow(uid);
}
// Select & focus next row
if (next && next.id) {
this.controller._selectionMgr.setSelected(next.id, true);
this.controller._selectionMgr.setFocused(next.id, true);
}
// Update the count
var total = this.dataview.grid._total - _row_ids.length;
// This will remove the last row!
// That's OK, because grid adds one in this.controller.deleteRow()
this.dataview.grid.setTotalCount(total);
// Re-enable automatic updating
this.dataview.grid.doInvalidate = true;
this.dataview.grid.invalidate();
}
// Stop automatic updating
this.dataview.grid.doInvalidate = false;
for (var i = 0; i < _row_ids.length; i++) {
uid = _row_ids[i].toString().indexOf(this.controller.dataStorePrefix) == 0 ? _row_ids[i] : this.controller.dataStorePrefix + "::" + _row_ids[i];
// Delete from internal references
this.controller.deleteRow(uid);
}
// Select & focus next row
if (next && next.id) {
this.controller._selectionMgr.setSelected(next.id, true);
this.controller._selectionMgr.setFocused(next.id, true);
}
// Update the count
var total = this.dataview.grid._total - _row_ids.length;
// This will remove the last row!
// That's OK, because grid adds one in this.controller.deleteRow()
this.dataview.grid.setTotalCount(total);
// Re-enable automatic updating
this.dataview.grid.doInvalidate = true;
this.dataview.grid.invalidate();
}
id_loop: for (var i = 0; i < _row_ids.length; i++) {
var uid = _row_ids[i].toString().indexOf(this.controller.dataStorePrefix) == 0 ? _row_ids[i] : this.controller.dataStorePrefix + "::" + _row_ids[i];

63
api/js/etemplate/et2_extension_nextmatch.ts
View File

@ -778,38 +778,41 @@ export class et2_nextmatch extends et2_DOMWidget implements et2_IResizeable, et2
// Record current & next index
var uid = _row_ids[0].toString().indexOf(this.controller.dataStorePrefix) == 0 ? _row_ids[0] : this.controller.dataStorePrefix + "::" + _row_ids[0];
const entry = this.controller._selectionMgr._getRegisteredRowsEntry(uid);
let next = (entry.ao ? entry.ao.getNext(_row_ids.length) : null);
if(next == null || !next.id || next.id == uid)
if(entry && entry.idx !== null)
{
// No next, select previous
next = (entry.ao?entry.ao.getPrevious(1):null);
let next = (entry.ao ? entry.ao.getNext(_row_ids.length) : null);
if(next == null || !next.id || next.id == uid)
{
// No next, select previous
next = (entry.ao ? entry.ao.getPrevious(1) : null);
}
// Stop automatic updating
this.dataview.grid.doInvalidate = false;
for (var i = 0; i < _row_ids.length; i++)
{
uid = _row_ids[i].toString().indexOf(this.controller.dataStorePrefix) == 0 ? _row_ids[i] : this.controller.dataStorePrefix + "::" + _row_ids[i];
// Delete from internal references
this.controller.deleteRow(uid);
}
// Select & focus next row
if(next && next.id)
{
this.controller._selectionMgr.setSelected(next.id, true);
this.controller._selectionMgr.setFocused(next.id, true);
}
// Update the count
const total = this.dataview.grid._total - _row_ids.length;
// This will remove the last row!
// That's OK, because grid adds one in this.controller.deleteRow()
this.dataview.grid.setTotalCount(total);
// Re-enable automatic updating
this.dataview.grid.doInvalidate = true;
this.dataview.grid.invalidate();
}
// Stop automatic updating
this.dataview.grid.doInvalidate = false;
for(var i = 0; i < _row_ids.length; i++)
{
uid = _row_ids[i].toString().indexOf(this.controller.dataStorePrefix) == 0 ? _row_ids[i] : this.controller.dataStorePrefix + "::" + _row_ids[i];
// Delete from internal references
this.controller.deleteRow(uid);
}
// Select & focus next row
if(next && next.id)
{
this.controller._selectionMgr.setSelected(next.id,true);
this.controller._selectionMgr.setFocused(next.id,true);
}
// Update the count
const total = this.dataview.grid._total - _row_ids.length;
// This will remove the last row!
// That's OK, because grid adds one in this.controller.deleteRow()
this.dataview.grid.setTotalCount(total);
// Re-enable automatic updating
this.dataview.grid.doInvalidate = true;
this.dataview.grid.invalidate();
}
id_loop: