Eintragsdetails ansehen
ID | Projekt | Kategorie | Sichtbarkeit | Meldungsdatum | Zuletzt aktualisiert |
---|---|---|---|---|---|
0002366 | Eressea | Magie | öffentlich | 2017-09-28 19:02 | 2017-12-05 19:48 |
Reporter | Solthar | Bearbeitung durch | Enno | ||
Priorität | normal | Schweregrad | schwerer Fehler | Reproduzierbar | nicht getestet |
Status | geschlossen | Lösung | erledigt | ||
Produktversion | 3.14.0 | ||||
Zielversion | 3.14.0 | Behoben in Version | 3.14.0 | ||
Zusammenfassung | 0002366: 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. | ||||
Tags | Keine Tags zugeordnet. | ||||
Partei | 1wpy | ||||
Spiel | E2 | ||||
Report | 1044 | ||||
Du hast Recht, daran habe ich gearbeitet. Früher war in der Region eine unsichtbare Einheit, damit man sie auf dem Relevanter Code in reports.c Zeile 1436 ff: |
|
Kannst Du mir eine Beispielregion liefern? |
|
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? |
|
Das ist nicht aus dem aktuellen Spiel. Ich könnte aber versuchen das zu reproduzieren (lassen). Oder ich könnte Dir ein Datenfile schicken! |
|
Hm, Traumlesen scheint zumindest zu funktionieren. Jetzt muss ich das eigentliche Problem zu reproduzieren versuchen... |
|
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. |
|
Ich glaube, ich sollte mir mal Report 1006, Einheit enk5, Partei s19L angucken. |
|
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. |
|
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. |
|
Breakpoint condition: f->no==94953 && r->uid==1513049869 |
|
Fehler: gelber Drache gehört zur Partei "Orden des Schweigens (s19L)", also f->no==1308009 |
|
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. |
|
ISt gefixt im NR, aber im CR fehlt die Einheit immer noch. :-( |
|
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. |
|
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. |
|
Oberflächlicher Test (Neuauswertung 1006) lässt mich vermuten, dass der Bug immer noch gefixt ist. |
|
Ä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.0 |
2017-10-10 19:11 | Enno | Zielversion | => 3.15.0 |
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.0 => 3.14.0 |
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.0 |
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 |
2022-04-27 17:27 | Enno | Kategorie | ZAUBER => Magie |