Skip to content
Snippets Groups Projects
Commit b94cba8c authored by Eric Oulashin's avatar Eric Oulashin
Browse files

DD message area chooser: Updated to try to prevent the error...

DD message area chooser: Updated to try to prevent the error "this.subBoardListPrintfInfo[pGrpIdx] is undefined" when only choosing a sub-board within the user's current message group.
parent cdafa7ae
No related branches found
No related tags found
2 merge requests!463MRC mods by Codefenix (2024-10-20),!184DD message area chooser: Updated to try to prevent an "undefined" error when only choosing a sub-board within the user's current message group
Pipeline #3112 passed
...@@ -34,7 +34,7 @@ ...@@ -34,7 +34,7 @@
* the sub-board list when using the lightbar menu * the sub-board list when using the lightbar menu
* 2022-06-11 Eric Oulashin Version 1.26 * 2022-06-11 Eric Oulashin Version 1.26
* Updated to try to prevent the error "this.subBoardListPrintfInfo[pGrpIdx] is undefined" * Updated to try to prevent the error "this.subBoardListPrintfInfo[pGrpIdx] is undefined"
* in CreateLightbarSubBoardMenu() * when only choosing a sub-board within the user's current message group.
*/ */
// TODO: In the area list, the 10,000ths digit (for # posts) is in a different color) // TODO: In the area list, the 10,000ths digit (for # posts) is in a different color)
...@@ -155,7 +155,9 @@ if (executeThisScript) ...@@ -155,7 +155,9 @@ if (executeThisScript)
// here just in case, and change the user's message area // here just in case, and change the user's message area
// here. Otherwise, if choosing the message group first, // here. Otherwise, if choosing the message group first,
// SelectMsgArea() will change the user's sub-board. // SelectMsgArea() will change the user's sub-board.
var msgGroupIdx = (gChooseMsgGrpOnStartup ? 0/*null*/ : bbs.curgrp); var msgGroupIdx = (gChooseMsgGrpOnStartup ? 0/*null*/ : +bbs.curgrp);
if (!gChooseMsgGrpOnStartup)
msgAreaChooser.BuildSubBoardPrintfInfoForGrp(msgGroupIdx);
var chosenIdx = msgAreaChooser.SelectMsgArea(gChooseMsgGrpOnStartup, msgGroupIdx); var chosenIdx = msgAreaChooser.SelectMsgArea(gChooseMsgGrpOnStartup, msgGroupIdx);
if (!gChooseMsgGrpOnStartup && (typeof(chosenIdx) === "number")) if (!gChooseMsgGrpOnStartup && (typeof(chosenIdx) === "number"))
bbs.cursub_code = msg_area.grp_list[bbs.curgrp].sub_list[chosenIdx].code; bbs.cursub_code = msg_area.grp_list[bbs.curgrp].sub_list[chosenIdx].code;
...@@ -2057,9 +2059,6 @@ function DDMsgAreaChooser_GetMsgSubBrdLine(pGrpIndex, pSubIndex, pHighlight) ...@@ -2057,9 +2059,6 @@ function DDMsgAreaChooser_GetMsgSubBrdLine(pGrpIndex, pSubIndex, pHighlight)
numMsgs); numMsgs);
} }
msgBase.close(); msgBase.close();
// Free some memory?
delete msgBase;
} }
return subBoardLine; return subBoardLine;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment