Berater » [BUG] Stacking event handlers

My Free Farm Berater / Bash Bot � Allgemeine Diskussion � Berater � [BUG] Stacking event handlers

Themen-Einstellungen

Bereich wechseln

Skripte Ver�ffentlichungen / Publications Allgemeine Diskussion Berater Automat andere Skripte Bash Bot Entwicklung (Development) [geschlossen/closed] General Discussion Adviser Automaton other Scripts Off-Topic Off-Topic

#1 | [BUG] Stacking event handlers datum

26.07.2021 19:47

Harry

Bot-Schrauber

Informationen anzeigen

Beitr�ge:	591
Registriert seit:	14.07.2015

moin linus--tux,

bei nicht-premium farmen kommen bei den quest knöppen bei jedem update request ein click handler für die forsthütte und die picknick area dazu. der alte wird dabei nicht entfernt.
das äussert sich dann so, wenn man sich länger auf einer solchen farm aufhält und dann z.b. den forsthütten-questknopp drückt, dass jeder der alten click handler abgearbeitet wird und es so aussieht, als würd der Berater hängen.
too-many-listeners.PNG - Bild entfernt (keine Rechte)

beim picknickgebiet stört mich das nicht weiter.. die quests hab ich alle durch.
für die bäumerei reiche ich folgenden verbesserungsvorschlag als diff ein:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
$ diff -Naur MyFreeFarm_Berater.user.js.bak MyFreeFarm_Berater.user.js
--- MyFreeFarm_Berater.user.js.bak      2021-05-15 12:32:57.774196600 +0200
+++ MyFreeFarm_Berater.user.js  2021-07-26 19:18:33.786883500 +0200
@@ -3217,6 +3217,23 @@
     if(DEVMODE_FUNCTION){ tracking.end("berater",trackingHandle); }
 }catch(err){ GM_logError("calcProdMinRackInit","","type="+type+" prod="+prod,err); }
 }
+
+function campaignHandler() {
+    if (gameLocation.check("city", 2)) {
+        unsafeWindow.close_page();
+        unsafeWindow.showDiv("transp3");
+        unsafeWindow.initCampaigns();
+    } else {
+        document.addEventListener("gameCity2", function() {
+            document.removeEventListener("gameCity2", arguments.callee, false);
+            unsafeWindow.close_page();
+            unsafeWindow.showDiv("transp3");
+            unsafeWindow.initCampaigns();
+        }, false);
+        click($("speedlink_city2"));
+     }
+}
+
 function calcProdMinRack(caller){
 try{
     GM_logInfo("calcProdMinRack","caller="+caller,"","Begin",1);
@@ -3621,21 +3638,8 @@
                         switch (type) {
                             case "forestry":
                                 {
-                                    div.addEventListener("click", function() {
-                                        if (gameLocation.check("city", 2)) {
-                                            unsafeWindow.close_page();
-                                            unsafeWindow.showDiv("transp3");
-                                            unsafeWindow.initCampaigns();
-                                        } else {
-                                            document.addEventListener("gameCity2", function() {
-                                                document.removeEventListener("gameCity2", arguments.callee, false);
-                                                unsafeWindow.close_page();
-                                                unsafeWindow.showDiv("transp3");
-                                                unsafeWindow.initCampaigns();
-                                            }, false);
-                                            click($("speedlink_city2"));
-                                        }
-                                    }, false);
+                                    div.removeEventListener("click", campaignHandler, false);
+                                    div.addEventListener("click", campaignHandler, false);
                                     break;
                                 }
                             case "foodworld":
 

's wär nett, wenn Du das einbauen könntest :)

bitte keine supportanfragen via PN - stellt eure fragen im forum
please do not ask for support via PN - use the forum for that

Antworten

#2 | RE: [BUG] Stacking event handlers datum

20.11.2021 10:18

wie ist die konsequente stille zu verstehen?

"sehe ich nicht als so wichtig an..." [ ]
"kann momentan nicht umgesetzt werden." [ ]
"behalt' deinen schei** code für dich!" [ ]

zutreffendes ankreuzen ;)

bitte keine supportanfragen via PN - stellt eure fragen im forum
please do not ask for support via PN - use the forum for that

Antworten

#3 | RE: [BUG] Stacking event handlers datum

20.11.2021 12:00

Harry, vielleicht isser ja auf der "Insel"? 😜😁
(hatte vor längerer Zeit schon mal so etwas vor dem Wochenende geschrieben.) 😊

Antworten

#4 | RE: [BUG] Stacking event handlers datum

20.11.2021 17:06

Bei Gelegenheit werde ich deinen Vorschlag einpflegen. Bis dahin bitte ich dich darum, dass du deinen Code nach jedem Update selbst per Copy & Paste in den Berater einfügst.

Viele Grüße vom Meer für dich und allen anderen.

Antworten

Neuer PC � � Weinberg