Bug: Schiffshändler komisches Verhalten bei Neuzuordnung von Verkaufswaren + Held weint rum, wenn Hafen angegriffen wird

[Siedelwood]

Describe the bug Irgendwie funktioniert der bei mir auch nicht richtig. Wenn ich einkaufen gehe, kann ich immer einkaufen, egal ob ich bekannt oder unbekannt bin. Und wenn ich einkaufe, gehen dann die Angebote weg, also von vier auf drei und dann auf zwei und so weiter. Ich habe ihn so eingestellt wie früher auch. Ist mir leider eben erst aufgefallen, als ich bewusst Salz kaufen wollte.

Nachdem das Händlerschiff jetzt wieder neu gekommen ist, gab es wieder drei Angebote, nach Klicken (kaufen) sind sie dann verschwunden. Warum da vorher vier Angebote waren, obwohl ich nur 3 oben stehen habe, ist mir schleierhaft

To Reproduce Hafen als Handelspartner und man versucht zu handeln obwohl das Schiff momentan nicht da ist. Außerdem gehen beim Leerkaufen die Knöpfe weg.

Expected behavior Man kann immer kaufen, nicht nur, wenn das Schiff da ist...und immer gehen dann die Angebote weg, wenn man alles gekauft hat. Es ist, als ob er die Liste von oben bis unten durchspielt. Da war der alte Händer mit dem Gekreische anders. Da konntest du nur kaufen, wenn das Schiff da ist. Früher war das auch so, weiß nicht, was geändert wurde, dass das jetzt nicht mehr funktioniert

Früher war es doch nur so, dass man nur kaufen konnte beim Hafen, wenn das Schiff gekommen ist

Screenshots

Additional context

function Hafen()
    local TraderDescription = {
    PlayerID   = 7,       -- Partei des Hafen, 
    Path       = {"shipWP1", "shipWP2", "shipWP3", "shipWP4"}, -- Pfad (auch als Table möglich)
    Duration   = 240,     -- Ankerzeit in Sekunden
    Interval   = 9,       -- Monate zwischen zwei Anfarten
    OfferCount = 3,       -- Anzahl Angebote (1 bis 4)
    NoIce      = true,
    Offers = {
        -- Angebot, Menge
        {"G_Gems", 5},
        {"G_Iron", 2},
        {"G_Honeycomb", 2},
        {"G_Stone", 2},
        {"G_Olibanum", 5},
        {"G_Carcass", 2},
        {"G_Milk", 4},
        {"G_Grain", 5},
        {"G_Salt", 5},
        {"G_Wool", 2},
        {"G_RawFish", 2},
        {"G_Dye", 5},
        },
    };
    API.TravelingSalesmanCreate(TraderDescription);
end

Außerdem jammert der Held rum, wenn der Hafen angegriffen wird, dass ein Verbündeter angegriffen wird.

Zeile 33837

[totalwarANGEL1993]

Das ist kein Bug. Das ist das erwartete Verhalten. Außerdem ändert der Code nichts an der Diplomatie, weshalb das an etwas anderem liegen muss.

[totalwarANGEL1993]

Es könnte manchmal helfen, die Doku zu lesen. 😣

[Jelumar]

Im Kompatibilitätsmodus werden nun die Angebote nach abfahrt des Schiffes wieder herausgenommen.

[Siedelwood]

Ist nicht passiert: das Schiff ist wieder weg, aber man kann das noch kaufen...das andere vorher wurde gekauft...

Und noch etwas, was auffällt - 3 Plätze zum Verkauf angegeben, es werden aber 4 gezeigt OfferCount = 3, -- Anzahl Angebote (1 bis 4)

Scheint als würden die hinten angehängt werden. Der Weihrauch blieb über, dann kamen neue und es wurde auf 4 aufgestockt mit den 3, die übergeben wurden.

Wenn nichts gekauft wird, bleiben die Angebote bestehen bzw. einige wurden entfernt, andere nicht.

[Siedelwood]

Wurde als Feature im Pull Request hinzugefügt. Damit ist es optional möglich den Hafen mithilfe eines Parameters umzuschalten, sodass er nur Angebote hat, wenn das Schiff da ist. Ankündigungen, dass das Schiff ankommt, sind nicht enthalten (müssen die noch?). Das Verhalten sollte für jeden Hafen individuell setzbar sein, sodass mehrere Häfen auf der Map unterschiedlich Arbeiten können. Der Parameter ist "OldHarbour" und ist standardmäßig auf "false".

Der alte nicht funktionierende Ansatz wurde entfernt.

Ebenso wurde ein neues Logging eingeführt um nur Konsolencommands zu anzeigen zu lassen.