Personal_mirrors/bitburner-src
1
0
Fork 0
mirror of https://github.com/bitburner-official/bitburner-src.git synced 2024-12-18 12:15:44 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix bug where sometimes Active Scripts UI wouldnt fully update after an Augmentation reset. This most likely happened because the player has too many scripts running, and they all get added to task queue to get deleted

Browse Source
This commit is contained in:
danielyxie 2018-05-11 20:54:59 -05:00
parent 6902ece765
commit 6b9dc76f41
4 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
dist/engine.bundle.js vendored
View File

@ -18521,7 +18521,7 @@ function prestigeWorkerScripts() {
Object(_ActiveScriptsUI_js__WEBPACK_IMPORTED_MODULE_0__["deleteActiveScriptsItem"])(workerScripts[i]); Object(_ActiveScriptsUI_js__WEBPACK_IMPORTED_MODULE_0__["deleteActiveScriptsItem"])(workerScripts[i]);
workerScripts[i].env.stopFlag = true; workerScripts[i].env.stopFlag = true;
} }
Object(_ActiveScriptsUI_js__WEBPACK_IMPORTED_MODULE_0__["updateActiveScriptsItems"])(); //Force UI to update Object(_ActiveScriptsUI_js__WEBPACK_IMPORTED_MODULE_0__["updateActiveScriptsItems"])(5000); //Force UI to update
workerScripts.length = 0; workerScripts.length = 0;
} }
@ -55109,10 +55109,10 @@ function deleteActiveScriptsItem(workerscript) {
} }
//Update the ActiveScriptsItems array //Update the ActiveScriptsItems array
function updateActiveScriptsItems() { function updateActiveScriptsItems(maxTasks=100) {
//Run tasks that need to be done sequentially (adding items, creating/deleting server panels) //Run tasks that need to be done sequentially (adding items, creating/deleting server panels)
//We'll limit this to 50 at a time in case someone decides to start a bunch of scripts all at once... //We'll limit this to 50 at a time in case someone decides to start a bunch of scripts all at once...
let numTasks = Math.min(100, ActiveScriptsTasks.length); let numTasks = Math.min(maxTasks, ActiveScriptsTasks.length);
for (let i = 0; i < numTasks; ++i) { for (let i = 0; i < numTasks; ++i) {
let task = ActiveScriptsTasks.shift(); let task = ActiveScriptsTasks.shift();
try { try {

6
dist/tests.bundle.js vendored
View File

@ -18521,7 +18521,7 @@ function prestigeWorkerScripts() {
Object(_ActiveScriptsUI_js__WEBPACK_IMPORTED_MODULE_0__["deleteActiveScriptsItem"])(workerScripts[i]); Object(_ActiveScriptsUI_js__WEBPACK_IMPORTED_MODULE_0__["deleteActiveScriptsItem"])(workerScripts[i]);
workerScripts[i].env.stopFlag = true; workerScripts[i].env.stopFlag = true;
} }
Object(_ActiveScriptsUI_js__WEBPACK_IMPORTED_MODULE_0__["updateActiveScriptsItems"])(); //Force UI to update Object(_ActiveScriptsUI_js__WEBPACK_IMPORTED_MODULE_0__["updateActiveScriptsItems"])(5000); //Force UI to update
workerScripts.length = 0; workerScripts.length = 0;
} }
@ -55109,10 +55109,10 @@ function deleteActiveScriptsItem(workerscript) {
} }
//Update the ActiveScriptsItems array //Update the ActiveScriptsItems array
function updateActiveScriptsItems() { function updateActiveScriptsItems(maxTasks=100) {
//Run tasks that need to be done sequentially (adding items, creating/deleting server panels) //Run tasks that need to be done sequentially (adding items, creating/deleting server panels)
//We'll limit this to 50 at a time in case someone decides to start a bunch of scripts all at once... //We'll limit this to 50 at a time in case someone decides to start a bunch of scripts all at once...
let numTasks = Math.min(100, ActiveScriptsTasks.length); let numTasks = Math.min(maxTasks, ActiveScriptsTasks.length);
for (let i = 0; i < numTasks; ++i) { for (let i = 0; i < numTasks; ++i) {
let task = ActiveScriptsTasks.shift(); let task = ActiveScriptsTasks.shift();
try { try {

4
src/ActiveScriptsUI.js
View File

@ -182,10 +182,10 @@ function deleteActiveScriptsItem(workerscript) {
} }
//Update the ActiveScriptsItems array //Update the ActiveScriptsItems array
function updateActiveScriptsItems() { function updateActiveScriptsItems(maxTasks=100) {
//Run tasks that need to be done sequentially (adding items, creating/deleting server panels) //Run tasks that need to be done sequentially (adding items, creating/deleting server panels)
//We'll limit this to 50 at a time in case someone decides to start a bunch of scripts all at once... //We'll limit this to 50 at a time in case someone decides to start a bunch of scripts all at once...
let numTasks = Math.min(100, ActiveScriptsTasks.length); let numTasks = Math.min(maxTasks, ActiveScriptsTasks.length);
for (let i = 0; i < numTasks; ++i) { for (let i = 0; i < numTasks; ++i) {
let task = ActiveScriptsTasks.shift(); let task = ActiveScriptsTasks.shift();
try { try {

2
src/NetscriptWorker.js
View File

@ -53,7 +53,7 @@ function prestigeWorkerScripts() {
deleteActiveScriptsItem(workerScripts[i]); deleteActiveScriptsItem(workerScripts[i]);
workerScripts[i].env.stopFlag = true; workerScripts[i].env.stopFlag = true;
} }
updateActiveScriptsItems(); //Force UI to update updateActiveScriptsItems(5000); //Force UI to update
workerScripts.length = 0; workerScripts.length = 0;
} }