Eintragsdetails ansehen
ID | Projekt | Kategorie | Sichtbarkeit | Meldungsdatum | Zuletzt aktualisiert |
---|---|---|---|---|---|
0002274 | Eressea | General | öffentlich | 2017-01-20 15:28 | 2019-10-03 21:55 |
Reporter | Schuppengnom | Bearbeitung durch | Enno | ||
Priorität | niedrig | Schweregrad | Trivial | Reproduzierbar | nicht getestet |
Status | geschlossen | Lösung | erledigt | ||
Zielversion | 3.11 | Behoben in Version | 3.11 | ||
Zusammenfassung | 0002274: Beschreibe Region funktioniert nicht bei Regionsbesitzer | ||||
Beschreibung | Im Sumpf Cutes gibt es zwei Burgen, eine Wache (Größe 10) und eine Befestigung (Größe 25). Bei gehören meiner Partei orkz. Diese Woche sollte der Besitzer der Befestigung die Beschreibung Region der Region ändern. Als Fehlermeldung erhielt ich zurück: "Maurer (waLL) in Cutes (-5,10): [...] Die Einheit ist nicht Burgherr der größten Burg in der Region." Merke: Die gleiche Einheit hat erfolgreich Steuern eingetrieben! Erwartetes Verhalten wäre gewesen, dass die Umbenennung klappt. | ||||
Schritte zur Reproduktion | Ich probiere es diese Woche noch einmal mit der Einheit, der die Wache gehört. | ||||
Tags | Keine Tags zugeordnet. | ||||
Partei | orkz | ||||
Spiel | Deveron | ||||
Report | 140 | ||||
Notiz für mich: Das ist error147 |
|
Wie eben besprochen, hier noch eine Anmerkung zu einem ähnlich gelagertem Fall aus E2, welchen ich leider gerade nicht detailiert zur hand habe. In der Region eine 1250er Festung (leer) und eine 50er Burg (in meinem Besitz). Das umbenennen Reg Region funktionierte nicht "...Die Einheit ist nicht Burgherr der größten Burg in der Region." Was dem Wortlaut nach ja auch richtig ist, aber auch nicht umbedingt das erwartete Verhalten darstellt. Da die größte besetzte Burg der Region mir gehörte. |
|
Kann das Problem nicht reproduzieren. Sicher, dass da in Report 140 war? Das ist ja schon zwei Runden her. |
|
Aha! Report Nummer 141 ist es. |
|
Das Problem ist, dass der Code in cmp_current_owner die Funktion bt_effsize benutzt, welche die Ausbaustufe der Gebäude vergleicht, nicht die Kapazität. Eine Wache Größe 10 ist Stufe 2, eine Burg Größe 25 ist nur Stufe 1. Ich bin sicher, der Code ist falsch. Was bei einer Reparatur zu beachten sind, sind glaube ich nur die Zwergenburgen, eine Sonderregel von E3/E4. |
|
Und die Abfrage nach der größten BESETZTEN Burg der Region, nicht nach der größten Burg. |
|
Vielleicht wäre es besser, zu fragen, ob die Einheit zur Partei des Regionsbesitzers gehört. Das ist auf jeden Fall einleuchtender und weniger anfällig für Fehler. |
|
Regionsbesitz gibt es in E2 nicht. |
|
Regionsbesitzer könnte man doch auch in E2 einführen. Muss ja erstaml nichts regeltechnisches für den Spieler dran hängen, sondern einfach nur die Ausgabe wer Regionsbesitzer ist (Wie in E3). Damit könnte man dann einige Sachen im Code vereinheitlichen und hätte es bei zukünftigen Änderungen einfacher. |
|
Oh, falsches Thema, letzten Kommentar gelöscht. Mein Kommentar zum Regionsbesitzer bezog sich nicht auf E2, sondern auf den hier gemeldeten Bug, der in E4 auftritt, wo es das Konzept gibt, und ein Vergleich mit einer Einheit gemacht wird, nicht mit der Partei. |
|
Zum Vorschlag, Regionsbesitz in E2 einzuführen: Das braucht es nicht, um diesen Bug zu lösen. In der Tat kann die Einführung von weiteren Regeln in E2 so gut wie NIE einen Bug in E4 reparieren. |
|
Okay, nachdem ich den Code ein wenig genauer angeschaut habe: Wenn get_region_owner() in E2 den Besitzer der größten Burg angeben würde, wäre der Code in der Tat einfacher zu ändern. |
|
Das war auch der Gedanke dahinter. Wenn man an einer Stelle im Code rumbaut, und man kann unterschiedliche Behandlungen zwischen E2 und E3 sinvoll und mit vertretbarem Aufwand für den jeweiligen Progarmierer (muss jeder selbst wissen) auflösen, sollte man das tun. |
|
Die Version 3.11 ist historisch, alle gefixten Bugs scheinen keine Probleme zu haben. |
|
Änderungsdatum | Benutzername | Feld | Änderung |
---|---|---|---|
2017-01-20 15:28 | Schuppengnom | Neuer Eintrag | |
2017-01-20 16:02 | Enno | Notiz hinzugefügt: 0006903 | |
2017-01-20 19:10 | Xolgrim | Notiz hinzugefügt: 0006904 | |
2017-01-20 21:39 | Enno | Notiz hinzugefügt: 0006905 | |
2017-01-20 21:40 | Enno | Notiz hinzugefügt: 0006906 | |
2017-01-20 22:03 | Enno | Notiz hinzugefügt: 0006907 | |
2017-01-21 09:21 | Xolgrim | Notiz hinzugefügt: 0006911 | |
2017-01-21 10:45 | Enno | Notiz hinzugefügt: 0006912 | |
2017-01-21 11:19 | Xolgrim | Notiz hinzugefügt: 0006913 | |
2017-01-21 12:29 | CTD | Notiz hinzugefügt: 0006914 | |
2017-01-23 15:25 | Enno | Notiz hinzugefügt: 0006916 | |
2017-01-23 15:27 | Enno | Notiz hinzugefügt: 0006917 | |
2017-01-24 17:36 | Enno | Notiz hinzugefügt: 0006921 | |
2017-01-24 17:36 | Enno | Bearbeitung durch | => Enno |
2017-01-24 17:36 | Enno | Status | neu => zugewiesen |
2017-01-25 13:37 | CTD | Notiz hinzugefügt: 0006930 | |
2017-01-25 21:14 | Enno | Status | zugewiesen => erledigt |
2017-01-25 21:14 | Enno | Lösung | offen => erledigt |
2017-01-25 21:14 | Enno | Behoben in Version | => 3.11 |
2017-01-25 21:14 | Enno | Notiz hinzugefügt: 0006931 | |
2017-02-27 11:02 | Enno | Zielversion | => 3.11 |
2017-05-26 06:33 | Enno | Notiz hinzugefügt: 0007227 | |
2017-05-26 06:33 | Enno | Status | erledigt => geschlossen |
2019-10-03 21:55 | Enno | Kategorie | Featurewunsch => General |