PB 85: PAE V Patch 4 - ab geht das hier!

[Ramkhamhaeng]

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

[Flunky]

Was hast du da drin? Auch schon eine der Händler-Varianten?

[Ramkhamhaeng]

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.)

Was hast du da drin? Auch schon eine der Händler-Varianten?

Enthalten sind folgende Änderungen:
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.

[Rob Anybody]

Patchen klappt und es ist auch ein Söldnerbutton nach Rundenende zu sehen. Genauer habe ich das aber nicht getestet.

[Flunky]

Der Händler-Patch ist mMn falsch.

Vorher:

            _doResearchPush(iBuyer, iGewinnWissen)
            _doResearchPush(iSeller, iGewinnWissen) 

Nachher:

            if iSeller != iBuyer: 
              _doResearchPush(iSeller, iGewinnWissen) 
            elif iSeller != iOriginCiv: 
              _doResearchPush(iOriginCiv, 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:

            if iSeller != iBuyer: 
              _doResearchPush(iSeller, iGewinnWissen) 
            _doResearchPush(iBuyer, iGewinnWissen) 

[Ramkhamhaeng]

Der Händler-Patch ist mMn falsch.

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.

[Flunky]

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:

# 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) --- 

Nachher:

# 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) --- 

Auskommentiert sind da noch 2 Varianten drin, die in der BtS-Handelswegberechnung berücksichtigt werden.

[Suite]

Soll ich denn nun ziehen?

[Ramkhamhaeng]

Inland gibt es diesen Bonus eh nicht, der Block beginnt mit "if iOriginCiv != iBuyer"

Stimmt

Soll ich denn nun ziehen?

Ja, Feuer frei! Bleibt alles wie es ist.

[Flunky]

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.

[Suite]

Bleibt alles wie es ist.

Das Kinderzimmer bleibt auch so, wie es ist! Es wird sich hier kein bisschen ändern!!

Edit: Da war irgendwas schreibgeschützt...

[Ramkhamhaeng]

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.

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

[Ramkhamhaeng]

@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.

[TeeRohr]

Link?

[Flunky]

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.