Conversation
only owner can change name fixed NAME FOREIGN building caught some missing cases NAME GROUP works DISPLAY FACTION works (just like BANNER) added more test cases
|
Die Vereinheitlichung von |
|
Ich hab eine etwas einfachere Version gemacht. Ich verstehe, dass ein Flag, ob ein Ding benannt ist einfacher wäre, aber wie man um eine Fallunterscheidung herumkommt sehe ich nicht, weil man ja immer Gebäude, Schiffe, Einheiten, Parteien unterscheiden muss. Und dann auch noch Name und Beschreibung getrennt behandeln. Aber wenn du das machst, kann ich gerne schauen, welche Testfälle noch fehlen. Ich bin ein bisschen perplex, wievel komplizierter Code für ein eigentlich simplex Konzept "Namen ändern" hier nötig ist. |
3dfa0c4 to
5730779
Compare
|
Was kann ich tun, damit das hier abgeschlossen wirdß |
Fix für https://bugs.eressea.de/view.php?id=2747 und mehr.
Habe einige Tests hinzugefügt und die ganzen Fälle vereinheitlicht. War die Hölle ohne Polymorphie. Ist immer noch hässlich, aber besser kriege ich es nicht hin.
Sichtbare Änderungen: