Feature 1865 leere unterhaltspflichtige Gebäude #21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Gebäude ohne Besitzer werden automatisch vom Regionsbesitzer übernommen (Die Einheit wird sozusagen der neue Gebäudebesitzer). Betreten dann nur noch wenn Helfe Bewache vom Regionsbesitzer. Wichtig: <param name="rules.region_owners" value="1"/>, ist Voraussetzung, es geht also nicht in E2. Es kann für beliebige Gebäude eingeschaltet werden. Example config : <param name="rules.region_owner_pay_building" value="market harbour lighthouse"/>
Mit rules.region_owner_pay_building ein kann es vorkommen, das wegen einem leeren Gebäude die größte Burg in der Region gesucht wird. Wenn es aber keine Burg in der Region gibt, wird building_owner mit NULL aufgerufen. Das musste abgefangen werden.
Neu Funktion check_param die überprüft ob ein übergebener Wert in dem Wert des entsprechenden Parameters steht.
Der Burgenbesitzer sieht nur noch etwas wenn er auch Leuchtturmbesitzer ist. Das geht jetzt auch für E2 ohne region_owner rule.
Die Abfrage der Option war ohne Default, so das sie hier immer an war. In E2 führ das dann wegen cmp_taxes zur assertion. Das wars, jetzt sollte alles rund sein.
|
complicated merge, and I ended up with a missing closing brace afterwards. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Gebäude ohne Besitzer werden automatisch vom Regionsbesitzer übernommen
(Die Einheit wird sozusagen der neue Gebäudebesitzer). Betreten dann nur
noch wenn Helfe Bewache vom Regionsbesitzer.
Wichtig:
param name="rules.region_owners" value="1" ist Voraussetzung, es
geht also nicht in E2.
Es kann für beliebige Gebäude eingeschaltet werden.
Beispiel :
param name="rules.region_owner_pay_building" value="market harbour
lighthouse"