Habe trotzdem mal einen Patch erstellt und werde den Server im Laufe des Nachmittages aktualisieren. D.h. es gibt dann heute irgendwann ein neues Zip, was ihr in euer Mod-Verzeichnis entpacken müsst
Habe trotzdem mal einen Patch erstellt und werde den Server im Laufe des Nachmittages aktualisieren. D.h. es gibt dann heute irgendwann ein neues Zip, was ihr in euer Mod-Verzeichnis entpacken müsst
Was hast du da drin? Auch schon eine der Händler-Varianten?
So, der Server ist gepatcht. Bitte ladet euch https://kirk.zulan.net/pb/PAE_PB_Bugfix2.zip herunter und kopiert die drei Dateien daraus in die Mod.
(Wenn ihr den enthaltenen Ordner (Assets) in euer Mod-Verzeichnis entpackt sollte euer Entpackprogramm alle Pfade richtig wählen.)
Enthalten sind folgende Änderungen:Zitat von Flunky
1. Doppeltes Popup beim nächsten Login unterdrückt.
2. Doppelten Forschungs-Ertrag beim Handeln innerhalb des eigenen Landes (Das ist der von Flunky angesprochene Patch)
3. Söldner anheuern und weitere Aktionen sind nun auch nach dem Drücken auf "Runde beenden" möglich.
Patchen klappt und es ist auch ein Söldnerbutton nach Rundenende zu sehen. Genauer habe ich das aber nicht getestet.
Aber an jenem Morgen war es Magie gewesen. Und es hörte nicht auf, Magie zu sein,
nur weil man [inzwischen] eine Erklärung dafür hatte ... (Terry Pratchett)
Brandstifter benötigen keine Streichhölzer, sie zündeln mit Worten.
Wer Brandstifter im Internet duldet und nicht wieder und wieder widerspricht,
darf sich nicht wundern, wenn auch bald sein wahres Leben brennt. (frei nach Max Frisch)
Der Händler-Patch ist mMn falsch.
Vorher:
Nachher:PHP-Code:_doResearchPush(iBuyer, iGewinnWissen)
_doResearchPush(iSeller, iGewinnWissen)
Vorher haben Käufer und Verkäufer die Forschung bekommen, jetzt Verkäufer und Ursprungsciv. Wenn die Ursprungsciv Forschung bekommen sollte, dann wenn schon beim Einkauf. Das ist aber aktuell auskommentiert. Um das Verhalten wie vorher zu erhalten, nur ohne doppelten Ertrag bei Verkauf in eigenem Land, sollte das so aussehen:PHP-Code:if iSeller != iBuyer:
_doResearchPush(iSeller, iGewinnWissen)
elif iSeller != iOriginCiv:
_doResearchPush(iOriginCiv, iGewinnWissen)
PHP-Code:if iSeller != iBuyer:
_doResearchPush(iSeller, iGewinnWissen)
_doResearchPush(iBuyer, iGewinnWissen)
Mist, dann wartet ggf. noch mit dem Patchen bis ich das ausgebessert habe.
@Flunky: Oder hast du noch eine Idee für den Inland-Handels-Ertrag? Da fehlt mir eine Idee für eine passable Lösung.
Edit: Ok, kann doch alles so bleiben wie es jetzt ist.
Geändert von Ramkhamhaeng (24. Juni 2018 um 16:35 Uhr)
Inland gibt es diesen Bonus eh nicht, der Block beginnt mit "if iOriginCiv != iBuyer"
Was die Preisberechnung angeht, hab ich da mal was vorbereitet
Vorher:
Nachher:PHP-Code:# Money player gets for selling bonus
def calculateBonusSellingPrice(pUnit, pCity):
if not pUnit.getUnitType() in L.LTradeUnits:
return -1
eBonus = CvUtil.getScriptData(pUnit, ["b"], -1)
if eBonus == -1:
return -1
iValue = getBonusValue(eBonus)
#iValue += iValue / 2 # besserer Verkaufswert fuer bessere Bonusgueter (Luxusgut)
iBuyer = pCity.getOwner()
iSeller = CvUtil.getScriptData(pUnit, ["originCiv"], pUnit.getOwner())
pBuyer = gc.getPlayer(iBuyer)
pSeller = gc.getPlayer(iSeller)
if CvUtil.hasBonusIgnoreFreeBonuses(pCity, eBonus): # allows "cancellation" of buying / Bonus direkt nach Einkauf wieder verkaufen (ohne Gewinn)
return _calculateBonusBuyingPrice(eBonus, iSeller, iBuyer) # Switch positions of seller and buyer
if pBuyer.getTeam() == pSeller.getTeam():
iModifier = 110 # doch +10%
else:
# Furious = 0, Annoyed = 1, Cautious = 2, Pleased = 3, Friendly = 4
iModifier = 100 + 10 * pSeller.AI_getAttitude(iBuyer)
# Hauptstadt
if pCity.isCapital(): iModifier += 20
#CyInterface().addMessage(iBuyer, True, 10, CyTranslator().getText("TXT_KEY_MESSAGE_TEST",("iValue",iValue)), None, 2, None, ColorTypes(10), 0, 0, False, False)
#CyInterface().addMessage(iBuyer, True, 10, CyTranslator().getText("TXT_KEY_MESSAGE_TEST",("iModifier",iModifier)), None, 2, None, ColorTypes(10), 0, 0, False, False)
#CyInterface().addMessage(iBuyer, True, 10, CyTranslator().getText("TXT_KEY_MESSAGE_TEST",("City Pop.",pCity.getPopulation())), None, 2, None, ColorTypes(10), 0, 0, False, False)
#CyInterface().addMessage(iBuyer, True, 10, CyTranslator().getText("TXT_KEY_MESSAGE_TEST",("100 + Pop * 5",(100 + pCity.getPopulation() * 5))), None, 2, None, ColorTypes(10), 0, 0, False, False)
#CyInterface().addMessage(iBuyer, True, 10, CyTranslator().getText("TXT_KEY_MESSAGE_TEST",("Gesamt",(iValue * iModifier * (100 + pCity.getPopulation() * 5)) / 10000)), None, 2, None, ColorTypes(10), 0, 0, False, False)
return (iValue * iModifier * (100 + pCity.getPopulation() * 5)) / 10000
# --- End of price stuff (trade) ---
Auskommentiert sind da noch 2 Varianten drin, die in der BtS-Handelswegberechnung berücksichtigt werden.PHP-Code:
# Money player gets for selling bonus
def calculateBonusSellingPrice(pUnit, pCity):
if not pUnit.getUnitType() in L.LTradeUnits:
return -1
eBonus = CvUtil.getScriptData(pUnit, ["b"], -1)
iSeller = CvUtil.getScriptData(pUnit, ["originCiv"], pUnit.getOwner())
x = CvUtil.getScriptData(pUnit, ["x"], -1)
y = CvUtil.getScriptData(pUnit, ["y"], -1)
if eBonus == -1:
return -1
iValue = getBonusValue(eBonus)
iBuyer = pCity.getOwner()
iModifier = 100
if not (iBuyer == iSeller and x == pUnit.getX() and y == pUnit.getY()):
pBuyer = gc.getPlayer(iBuyer)
pSeller = gc.getPlayer(iSeller)
# iModifier = pCity.totalTradeModifier()
# pOriginPlot = CyMap().plot(x,y)
# pOriginCity = None
# if pOriginPlot != None and not pOriginPlot.isNone():
# pOriginCity = pOriginPlot.getPlotCity()
# if pOriginCity != None and not pOriginCity.isNone():
# if pCity.area() != pOriginCity.area():
# iModifier += gc.getDefineINT("OVERSEAS_TRADE_MODIFIER");
if pBuyer.getTeam() == pSeller.getTeam():
iModifier += 10 # doch +10%
else:
# Furious = 0, Annoyed = 1, Cautious = 2, Pleased = 3, Friendly = 4
iModifier += 10 * pSeller.AI_getAttitude(iBuyer)
# iModifier += pCity.getForeignTradeRouteModifier();
# Hauptstadt
if pCity.isCapital():
iModifier += 20
iModifier *= (100 + pCity.getPopulation() * 5) / 100
return iValue * iModifier / 100
# --- End of price stuff (trade) ---
Geändert von Flunky (24. Juni 2018 um 16:40 Uhr)
Soll ich denn nun ziehen?
Stimmt
Ja, Feuer frei! Bleibt alles wie es ist.Soll ich denn nun ziehen?
Warum? Mit deiner Version bekommt derjenige, wo ich vor x Runden ne Ressi eingekauft hat, plötzlich Forschungspunkte, und derjenige, in dessen Stadt ich verkaufe, geht leer aus.
@Preisberechnung: da geht es darum, dass mit der aktuellen Version Bronze und andere Ressourcen, die durch Gebäude entstehen, einfach an Ort und Stelle eingekauft und gewinnbringend wiederverkauft werden können.
Das Kinderzimmer bleibt auch so, wie es ist! Es wird sich hier kein bisschen ändern!!
Edit: Da war irgendwas schreibgeschützt...
Lange Leitung bei mir heute. Hatte einen deiner vorherigen Kommentare so verstanden, dass das elif immer zu False auswertet, aber es ist ja immer True. Ok, dann ändere ich das und mit deiner anderen Codeänderung lohnt sich der Aufwand dann auch
@All: Bitte ladet den Bugfix jetzt nochmal runter, falls ihr schon gepatcht hattet. Habe die Dateien jetzt noch mal aktualisiert und die Änderungswünsche von Flunky integriert.
Link?
Im Beitrag vorhin: https://kirk.zulan.net/pb/PAE_PB_Bugfix2.zip
Damit könnt ihr jetzt auch Ressourcen gewinnbringend verkaufen, die der andere schon hat.
Berechtigungen
Zitieren
