Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002340EresseaMagieöffentlich2017-12-05 19:49
ReporterEnno Bearbeitung durchEnno  
PrioritätnormalSchweregradkleinerer FehlerReproduzierbarnicht getestet
Status geschlossenLösungerledigt 
Produktversion3.12.1 
Zielversion3.12.2Behoben in Version3.12.2 
Zusammenfassung0002340: Interner Fehler: Meldung 'curseinfo::stormwind' nicht definiert.
Beschreibung

Der Sturmwind sollte eigentlich nicht sichtbar sein, ist es aber scheinbar doch. Bei der Erzeugung der Reporte gibt es Warnungen, und Xolgrim sagt, er sieht sie auch.

Schritte zur Reproduktion

Im spells.c Code steht:

    /* Duration = 1, nur diese Runde */
    create_curse(mage, &sh->attribs, ct_find("stormwind"), power, 1,
        zero_effect, 0);
    /* Da der Spruch nur diese Runde wirkt wird er nie im Report
     * erscheinen */

Von wegen!

TagsKeine Tags zugeordnet.
Partei777
SpielE2
Report1030

Notizen / Dateien

Enno

Enno

2017-06-18 16:36

Administrator   ~0007269

Der Text "Internen Fehler" steht in der Meldung "missing_message".

Enno

Enno

2017-06-18 16:42

Administrator   ~0007270

Es wird mindestens ein solcher Curse diese Woche erstellt, und der altert dann scheinbar in curse_age auch wieder weg.

Enno

Enno

2017-06-18 16:48

Administrator   ~0007271

Ich kann das mit meinem aktuellen master nicht reproduzieren, evtl. ist es schon repariert, nur noch nicht aktiv?

Enno

Enno

2017-06-18 16:49

Administrator   ~0007272

Auch seltsam:

Wir hatten das eigentlich schon für Regentanz repariert. Trotzdem ist in test/reports/1030-1.nr:Interner Fehler: Meldung 'raindance_effect' nicht definiert.

Enno

Enno

2017-06-18 16:51

Administrator   ~0007273

Das mit Regentanz tritt offenbar nur im develop Branch (Testauswertung) auf. Da fehlt evtl. ein merge mit der letzten master-version?

Enno

Enno

2017-06-18 16:52

Administrator   ~0007274

@Xolgrim: Kannst Du mir sagen, wo Du diesen Fehler siehst? Kann ich nämlich gerade nicht reproduzieren. Test-AW oder richtige? Welche Partei?

Xolgrim

Xolgrim

2017-06-18 17:17

Tester   ~0007275

@Enno: Live AW auf: Treueschwur (4jwt) in besitz der Die Walgenorianer (L12a) Weihnachtsbaum K (xmas) in besitz der Iøniger (ioen) in der Test AW ist der Fehler nicht zu finden. Hast du also wohl schon gelöst.

Enno

Enno

2017-06-18 18:03

Administrator   ~0007276

Zuletzt bearbeitet: 2017-06-18 18:03

Ich sehe es:

Treueschwur (4jwt), Trireme, 2% beschädigt; 2bko.

  Die Winde scheinen dieses Schiff besonders zu beguenstigen. (z3mc)

  Interner Fehler: Meldung 'curseinfo::stormwind' nicht definiert.
Enno

Enno

2017-06-18 18:08

Administrator   ~0007277

Auf der Treueschwur liegt (schon im Datenfile) ein Sturmwind, der 2147483647 Wochen dauern wird. Das ist zu lange.

Enno

Enno

2017-06-18 18:15

Administrator   ~0007278

Zauber, die ähnliche Dauer haben: nocostbuilding, magicwalls, magicrunes, shipspeedup, holyground, magicresistance. Evtl. ist es für einige Zauber legitim, dass ihre Dauer "endlos" ist? Aber nicht für diesen, und es fragt sich, woher das kommt.

Enno

Enno

2017-06-18 18:16

Administrator   ~0007279

In curse_age haben wir diese Zeile: if (c_flags(c) & CURSE_NOAGE) { c->duration = INT_MAX; }

Enno

Enno

2017-06-18 18:18

Administrator   ~0007280

CURSE_NOAGE ist 2, und c->flags für diesen Zauber ist auch 2. Da haben wir's - wie passiert das?

Enno

Enno

2017-06-18 20:02

Administrator   ~0007282

Sekunde, alles verkehrt... Verwechselt. Auf der Treueschwur liegt im Datenfile zwar ein "endloser" Zauber, dieser ist aber nicht stormwind, sondern shipspeedup, und kommt von einem Sonnensegel.

Notiz: Das Setzen der Dauer auf INT_MAX wenn c->type->flags oder c->flags schon CURSE_NOAGE hat, ist überflüssig, und verwirrt, das mache ich weg.

Enno

Enno

2017-06-18 22:03

Administrator   ~0007286

gefixt in PR https://github.com/eressea/server/pull/700

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2017-06-18 16:32 Enno Neuer Eintrag
2017-06-18 16:32 Enno Status neu => zugewiesen
2017-06-18 16:32 Enno Bearbeitung durch => Enno
2017-06-18 16:36 Enno Notiz hinzugefügt: 0007269
2017-06-18 16:42 Enno Notiz hinzugefügt: 0007270
2017-06-18 16:48 Enno Notiz hinzugefügt: 0007271
2017-06-18 16:49 Enno Notiz hinzugefügt: 0007272
2017-06-18 16:51 Enno Notiz hinzugefügt: 0007273
2017-06-18 16:52 Enno Notiz hinzugefügt: 0007274
2017-06-18 17:17 Xolgrim Notiz hinzugefügt: 0007275
2017-06-18 18:03 Enno Notiz hinzugefügt: 0007276
2017-06-18 18:03 Enno Notiz bearbeitet: 0007276
2017-06-18 18:08 Enno Notiz hinzugefügt: 0007277
2017-06-18 18:15 Enno Notiz hinzugefügt: 0007278
2017-06-18 18:16 Enno Notiz hinzugefügt: 0007279
2017-06-18 18:18 Enno Notiz hinzugefügt: 0007280
2017-06-18 20:02 Enno Notiz hinzugefügt: 0007282
2017-06-18 22:03 Enno Status zugewiesen => erledigt
2017-06-18 22:03 Enno Lösung offen => erledigt
2017-06-18 22:03 Enno Behoben in Version => 3.12.2
2017-06-18 22:03 Enno Notiz hinzugefügt: 0007286
2017-06-18 22:15 Enno Zielversion 3.13.0 => 3.12.2
2017-12-05 19:49 Enno Status erledigt => geschlossen
2022-04-27 17:27 Enno Kategorie ZAUBER => Magie