Merge pull request #259 from RubenPX/fix-indeterminated-checkbox
fix indeterminated checkbox on select lots
This commit is contained in:
commit
6a8a38eaab
|
@ -111,6 +111,8 @@ window.addEventListener("DOMContentLoaded", () => {
|
||||||
} else {
|
} else {
|
||||||
btnSelectAll.disabled = false;
|
btnSelectAll.disabled = false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
get_device_list();
|
||||||
}
|
}
|
||||||
|
|
||||||
TableController.getAllDevices().forEach(item => {
|
TableController.getAllDevices().forEach(item => {
|
||||||
|
@ -324,19 +326,30 @@ async function processSelectedDevices() {
|
||||||
const lotID = lot.id;
|
const lotID = lot.id;
|
||||||
const srcElement = event.srcElement.parentElement.children[0]
|
const srcElement = event.srcElement.parentElement.children[0]
|
||||||
const checked = !srcElement.checked;
|
const checked = !srcElement.checked;
|
||||||
|
const { indeterminate } = srcElement
|
||||||
|
|
||||||
const found = this.list.filter(list => list.lot.id == lotID)[0];
|
const found = this.list.filter(list => list.lot.id == lotID)[0];
|
||||||
|
|
||||||
if (checked) {
|
if (checked) {
|
||||||
if (found && found.type == "Remove") {
|
if (found && found.type == "Remove") {
|
||||||
found.type = "Add";
|
const affectedDevices = found.devices.filter(dev => found.lot.devices.includes(dev.id))
|
||||||
|
if (affectedDevices.length > 0 && found.indeterminate == false) { // Remove action from list
|
||||||
|
actions.list = actions.list.filter(x => x.lot.id != found.lot.id)
|
||||||
} else {
|
} else {
|
||||||
this.list.push({ type: "Add", lot, devices: selectedDevices });
|
found.type = "Add";
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
this.list.push({ type: "Add", lot, devices: selectedDevices, indeterminate });
|
||||||
}
|
}
|
||||||
} else if (found && found.type == "Add") {
|
} else if (found && found.type == "Add") {
|
||||||
found.type = "Remove";
|
const affectedDevices = found.devices.filter(dev => !found.lot.devices.includes(dev.id))
|
||||||
|
if (affectedDevices.length > 0 && found.indeterminate == false) { // Remove action from list
|
||||||
|
actions.list = actions.list.filter(x => x.lot.id != found.lot.id)
|
||||||
} else {
|
} else {
|
||||||
this.list.push({ type: "Remove", lot, devices: selectedDevices });
|
found.type = "Remove";
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
this.list.push({ type: "Remove", lot, devices: selectedDevices, indeterminate });
|
||||||
}
|
}
|
||||||
|
|
||||||
if (this.list.length > 0) {
|
if (this.list.length > 0) {
|
||||||
|
|
Reference in New Issue