Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002366EresseaZAUBERöffentlich2017-12-05 19:48
ReporterSoltharBearbeitung durchEnno 
PrioritätnormalAuswirkungschwerer FehlerReproduzierbarnicht getestet
Status geschlossenLösungerledigt 
Produktversion3.14 
Zielversion3.14Behoben in Version3.14 
Zusammenfassung0002366: Blick in die Realität (und andere Zauber?) kaputt
Beschreibung

Ich glaube, dass der Zauber nicht mehr funktioniert wie beabsichtigt. Früher hat er die Einheiten in der Region angezeigt, jetzt nur noch die Regionsinfos. Davon sind vermutlich auch andere Zauber wie Aushorchen und Traumlesen betroffen. Mögliche Ursache ist commit 30cfe2c0 et al.

Ich habe aktuell, keine Einheit und keinen Test, kann den aber nachliefern.

Partei1wpy
SpielE2
Report1044

Eintrags-Beziehungen

verwandt mit 0002374 geschlossenEnno Seeschlange unsichtbar 

Notizen / Dateien

Enno

Enno

2017-09-28 20:20

Administrator   ~0007507

Du hast Recht, daran habe ich gearbeitet. Früher war in der Region eine unsichtbare Einheit, damit man sie auf dem seen_unit Level sehen konnte, heute ist das mit set_observer und get_observer gemacht, und scheinbar gibt es eine seen_spell Sichtbarkeit?

Relevanter Code in reports.c Zeile 1436 ff:

        if (fval(r, RF_OBSERVER)) {
            int skill = get_observer(r, f);
            if (skill >= 0) {
                add_seen_nb(f, r, seen_spell);
            }
        }
Enno

Enno

2017-09-28 21:26

Administrator   ~0007508

Kannst Du mir eine Beispielregion liefern?

Enno

Enno

2017-09-28 21:28

Administrator   ~0007509

In deinem Report 1043 kommt so eine Region nicht vor, glaube ich. Die Bugmeldung ist allerdings für Runde 1044, und die hat noch nicht stattgefunden. Bekommst Du Email aus der Zukunft?

Solthar

Solthar

2017-09-29 12:01

Entwickler   ~0007510

Das ist nicht aus dem aktuellen Spiel. Ich könnte aber versuchen das zu reproduzieren (lassen). Oder ich könnte Dir ein Datenfile schicken!

Solthar

Solthar

2017-10-08 11:34

Entwickler   ~0007533

Hm, Traumlesen scheint zumindest zu funktionieren. Jetzt muss ich das eigentliche Problem zu reproduzieren versuchen...

Enno

Enno

2017-10-10 18:47

Administrator   ~0007534

Es wird bei Blick in die Realität definitiv ein Observer gesetzt, habe das gerade mit einem Unit-Test bestätigt. Es kann jetzt natürlich sein, dass das nicht zur vollständigen Anzeige im Report reicht. Den Report zu testen ist vergleichsweise schwieriger.

Enno

Enno

2017-10-10 18:54

Administrator   ~0007535

Ich glaube, ich sollte mir mal Report 1006, Einheit enk5, Partei s19L angucken.

Enno

Enno

2017-10-10 18:58

Administrator   ~0007536

Einheit "gelber Drache (enk5)" zaubert den Spruch, aber das Ziel scheinen ganz viele Ozeane zu sein. Allerdings nicht komplett:
Einheit Verdammte der Ruhelosen (uwpp) sollte zu sehen sein, in der Ebene Varfosvinen. Schauen wir mal.

Enno

Enno

2017-10-10 19:10

Administrator   ~0007537

Im NR stehen in der Tat keine Einheiten, nur die Burg dort sieht man.

Im CR steht "unit";visibility, aber auch keine Einheiten. Dafür sieht man eine Nachricht, die im NR auch fehlt:
"3 Bauern flohen aus Furcht vor Verdammte der Ruhelosen (uwpp).";rendered

Seltsam.

Enno

Enno

2017-10-10 19:15

Administrator   ~0007538

Breakpoint condition: f->no==94953 && r->uid==1513049869

Enno

Enno

2017-10-10 19:23

Administrator   ~0007539

Fehler: gelber Drache gehört zur Partei "Orden des Schweigens (s19L)", also f->no==1308009

Enno

Enno

2017-10-10 19:35

Administrator   ~0007540

Auffällig: r->seen.mode ist seen_spell (5), nicht seen_unit. Das hatte ich erwartet, aber der CR sagt was anderes.

Im Code steht an einer Stelle: if (r->seen.mode == seen_unit || r->seen.mode == seen_travel), und dann werden für seen_spell die Meldungen nicht gezeigt. Aha. Erster Bug gefunden.

Zu den Einheiten: Da wird cansee() aufgerufen, und das schaut sich alle Einheiten der Region an (Zeile 4309 ff), ob eine davon dem Reporthaber gehört, und wenn nicht, sagt es "Nein". Das ist natürlich auch verkehrt. Das sollte gar nicht gecheckt werden müssen, wenn der seen_mode schon >= seen_unit ist. Dafür sollte der stalth-modifier des Observers (Zauberstufe / 2) einfliessen.

Enno

Enno

2017-10-10 20:19

Administrator   ~0007541

ISt gefixt im NR, aber im CR fehlt die Einheit immer noch. :-(

Enno

Enno

2017-10-10 20:26

Administrator   ~0007542

Die Änderung ist mir aus Versehen noch in die geplante 3.14 Version gerutscht (habe vergessen, einen Branch zu machen), aber dafür sollte es dann am Sonntag in der Testauswertung schon wieder repariert sein. Auch für bereits gezauberte Zauber, muss also nicht neu gezaubert werden.

Enno

Enno

2017-10-15 20:41

Administrator   ~0007564

Hier musste ich leider an dem Code, der das gefixt hat, noch einmal Veränderungen vornehmen, deshalb sollte ich das vor dem Release besser neu testen.

Enno

Enno

2017-10-16 20:10

Administrator   ~0007565

Oberflächlicher Test (Neuauswertung 1006) lässt mich vermuten, dass der Bug immer noch gefixt ist.

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2017-09-28 19:02 Solthar Neuer Eintrag
2017-09-28 20:20 Enno Notiz hinzugefügt: 0007507
2017-09-28 21:26 Enno Notiz hinzugefügt: 0007508
2017-09-28 21:28 Enno Notiz hinzugefügt: 0007509
2017-09-29 12:01 Solthar Notiz hinzugefügt: 0007510
2017-10-08 11:34 Solthar Notiz hinzugefügt: 0007533
2017-10-10 18:47 Enno Notiz hinzugefügt: 0007534
2017-10-10 18:54 Enno Notiz hinzugefügt: 0007535
2017-10-10 18:58 Enno Notiz hinzugefügt: 0007536
2017-10-10 19:10 Enno Notiz hinzugefügt: 0007537
2017-10-10 19:10 Enno Bearbeitung durch => Enno
2017-10-10 19:10 Enno Status neu => zugewiesen
2017-10-10 19:11 Enno Produktversion => 3.14
2017-10-10 19:11 Enno Zielversion => 3.15
2017-10-10 19:15 Enno Notiz hinzugefügt: 0007538
2017-10-10 19:23 Enno Notiz hinzugefügt: 0007539
2017-10-10 19:35 Enno Notiz hinzugefügt: 0007540
2017-10-10 20:19 Enno Notiz hinzugefügt: 0007541
2017-10-10 20:24 Enno Zielversion 3.15 => 3.14
2017-10-10 20:26 Enno Status zugewiesen => erledigt
2017-10-10 20:26 Enno Lösung offen => erledigt
2017-10-10 20:26 Enno Behoben in Version => 3.14
2017-10-10 20:26 Enno Notiz hinzugefügt: 0007542
2017-10-15 14:04 Enno Beziehung hinzugefügt verwandt mit 0002374
2017-10-15 20:41 Enno Status erledigt => Rückmeldung
2017-10-15 20:41 Enno Lösung erledigt => wiedereröffnet
2017-10-15 20:41 Enno Notiz hinzugefügt: 0007564
2017-10-16 20:10 Enno Status Rückmeldung => erledigt
2017-10-16 20:10 Enno Lösung wiedereröffnet => erledigt
2017-10-16 20:10 Enno Notiz hinzugefügt: 0007565
2017-12-05 19:48 Enno Status erledigt => geschlossen