Add red portal use in Summoner + minor fix

[5noop]

Summoner.js, Duriel.js, Loader.js

• If Duriel script is run after Summoner, use red portal instead of going to town

Andariel.js, Duriel.js

• Fix a classic issue with ClassAttack.doCast, c.f. https://blizzhackers.discourse.group/t/sorc-wont-static-duriel/116/

Town.js, Pather.js

• If bot starts in act2, use Warriv to go to act 1 instead of walking across Lut Gholein. Implemented in useWaypoint and doChores

[5noop]

There is one bug left with Warriv:
In single player in the first game after a profile starts, openMenu() on Warriv fails so the bot cannot move to a1 and starts walking to the waypoint when it's done trying to talk to Warriv. There is no problem in the following games or on battle.net.
Let me know if you want me to remove this feature.

[noah-]

There is one bug left with Warriv:
In single player in the first game after a profile starts, openMenu() on Warriv fails so the bot cannot move to a1 and starts walking to the waypoint when it's done trying to talk to Warriv. There is no problem in the following games or on battle.net.
Let me know if you want me to remove this feature.

Hmm this is odd, I'll look into it. The spaces you had previously were correct, feel free to put them back.

[imbalanced]

Please use NPC.Warriv variable for npc name to ensure compatibility with different locales.

[imbalanced]

npc isn't defined here.

[imbalanced]

While using warriv to travel to A1 for chores may save time in common cases, it changes behavior for bots that may expect to be on current act for other tasks. Most definitely this isn't desired behavior for bots on normal difficulty because pots are lower tiered in a1.

[noah-]

consider the following refactor:

		for (s in Scripts) {
			if (Scripts[s]) {
				scriptList.push(s);
			}
		}

		for (this.scriptIndex = 0; this.scriptIndex < charScripts.length; this.scriptIndex++) {
			script = this.scriptList[this.scriptIndex];

			if (this.fileList.indexOf(script) < 0) {
				Misc.errorReport("ÿc1Script " + script + " doesn't exist.");
				continue;
			}

			if (!include("bots/" + script + ".js")) {
				Misc.errorReport("Failed to include script: " + script);
				continue;
			}

			if (isIncluded("bots/" + script + ".js")) {
				try {
					if (typeof (global[script]) !== "function") {
						throw new Error("Invalid script function name");
					}

					if (this.skipTown.indexOf(script) > -1 || Town.goToTown()) {
						print("ÿc2Starting script: ÿc9" + script);
						//scriptBroadcast(JSON.stringify({currScript: script}));
						Messaging.sendToScript("tools/toolsthread.js", JSON.stringify({currScript: script}));

						reconfiguration = typeof Scripts[script] === 'object';

						if (reconfiguration) {
							print("ÿc2Copying Config properties from " + script + " object.");
							this.copy(Scripts[script], Config);
						}

						global[script]();

						if (reconfiguration) {
							print("ÿc2Reverting back unmodified config properties.");
							this.copy(unmodifiedConfig, Config);
						}
					}
				} catch (error) {
					Misc.errorReport(error, script);
				}
			}
		}

change existing scriptList to fileList
move charScripts from scriptName to main function, rename charScripts to scriptList
change the loader loop from what it is to for loop using new scriptList you have created

[5noop]

looks good, modifications done

[noah-]

Looks good to me. Thanks.