Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002434EresseaATTACKIEREöffentlich2018-04-29 13:56
ReporterEnnoBearbeitung durchEnno 
PrioritätnormalAuswirkungkleinerer FehlerReproduzierbarnicht getestet
Status erledigtLösungerledigt 
Produktversion3.15.3 
Zielversion3.16Behoben in Version3.16 
Zusammenfassung0002434: Rassenbeschränkung auf Waffen/Rüstungen funktioniert nicht
Beschreibung

In E3 gibt es den Turmschild, der nur von Zwergen benutzbar sein soll, sowie die Repetierarmbrust. Außerdem dürfen Goblins nicht alle Waffen benutzen, z.B. keine Hellebarde. Schuppenpanzer können nur Zwerge und Halblinge verwenden.

Ich glaube, dass diese Beschränkung nicht gilt. Das Studium von xmlreader.c lässt das jedenfalls vermuten.

Schritte zur Reproduktion

Leider gibt es keine Tests. Ich glaube, test_goblins in tests/e3/items.lua sollte das einmal leisten, aber dort sind keinerlei assert Statements, weil es quasi unmöglich ist, das zu merken, wenn jemand mit einer Waffe kämpft, die er nicht benutzen kann.

Partei0
SpielE3
Report456

Notizen / Dateien

Enno

Enno

2018-04-29 12:47

Administrator   ~0007868

Huh. Bei der Waffe benutzen wir hier itype->mask_allow über rc_can_use. Das ist aber nicht über <modifier> gesetzt, wie ich dachte. Ist besser so, denke ich, aber wieso ist das dann beim Turmschild so <modifier type="canuse" function="mod_dwarves_only"/>?

Enno

Enno

2018-04-29 13:56

Administrator   ~0007869

Ich habe den Code übersichtlicher gemacht, und im Debugger getestet, dass goblins keinen Turmschild und keine Äxte benutzen können. Test zu schreiben ist unter den Umständen zu kompliziert (der Kampf ist katastrophaler Wurstcode).

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2018-04-29 10:40 Enno Neuer Eintrag
2018-04-29 10:40 Enno Status neu => zugewiesen
2018-04-29 10:40 Enno Bearbeitung durch => Enno
2018-04-29 12:47 Enno Notiz hinzugefügt: 0007868
2018-04-29 13:56 Enno Status zugewiesen => erledigt
2018-04-29 13:56 Enno Lösung offen => erledigt
2018-04-29 13:56 Enno Notiz hinzugefügt: 0007869
2018-04-29 13:56 Enno Behoben in Version => 3.16