Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002274EresseaGeneralöffentlich2019-10-03 21:55
ReporterSchuppengnom Bearbeitung durchEnno  
PrioritätniedrigSchweregradTrivialReproduzierbarnicht getestet
Status geschlossenLösungerledigt 
Zielversion3.11Behoben in Version3.11 
Zusammenfassung0002274: 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.

TagsKeine Tags zugeordnet.
Parteiorkz
SpielDeveron
Report140

Notizen / Dateien

Enno

Enno

2017-01-20 16:02

Administrator   ~0006903

Notiz für mich: Das ist error147

Xolgrim

Xolgrim

2017-01-20 19:10

Tester   ~0006904

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.

Enno

Enno

2017-01-20 21:39

Administrator   ~0006905

Kann das Problem nicht reproduzieren. Sicher, dass da in Report 140 war? Das ist ja schon zwei Runden her.

Enno

Enno

2017-01-20 21:40

Administrator   ~0006906

Aha! Report Nummer 141 ist es.

Enno

Enno

2017-01-20 22:03

Administrator   ~0006907

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.

Xolgrim

Xolgrim

2017-01-21 09:21

Tester   ~0006911

Und die Abfrage nach der größten BESETZTEN Burg der Region, nicht nach der größten Burg.

Enno

Enno

2017-01-21 10:45

Administrator   ~0006912

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.

Xolgrim

Xolgrim

2017-01-21 11:19

Tester   ~0006913

Regionsbesitz gibt es in E2 nicht.

CTD

CTD

2017-01-21 12:29

Entwickler   ~0006914

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.

Enno

Enno

2017-01-23 15:25

Administrator   ~0006916

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.

Enno

Enno

2017-01-23 15:27

Administrator   ~0006917

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.

Enno

Enno

2017-01-24 17:36

Administrator   ~0006921

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.

CTD

CTD

2017-01-25 13:37

Entwickler   ~0006930

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.

Enno

Enno

2017-01-25 21:14

Administrator   ~0006931

PR https://github.com/eressea/server/pull/640

Enno

Enno

2017-05-26 06:33

Administrator   ~0007227

Die Version 3.11 ist historisch, alle gefixten Bugs scheinen keine Probleme zu haben.

Eintrags-Historie

Ä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