Zwergenhort #4

Merged
Byroks merged 11 commits from lefrique into develop 2024-01-07 14:31:21 +01:00
Member

Mein Geschenk zu Weihnachten:

Implementierung des Zwergenhort gem. KOD S. 167 mit Angabe aktueller Mindesthort für aktuellen Grad und benötigtem Hort für nächsten Grad bei Zwergen. Der Hort erscheint nur bei ausgewählter Rasse Zwerg und wird bei allen anderen Rassen versteckt. Die Anrechnung von Gold oder Gegenständen wie auch im KOD erwähnt erfordert die Implementierung eines Goldmanagements und am besten auch eine Neustrukturierung des Charakterbogens (s. weitere Issues).

Frohe Weihnachten ;)

Mein Geschenk zu Weihnachten: Implementierung des Zwergenhort gem. KOD S. 167 mit Angabe aktueller Mindesthort für aktuellen Grad und benötigtem Hort für nächsten Grad bei Zwergen. Der Hort erscheint nur bei ausgewählter Rasse Zwerg und wird bei allen anderen Rassen versteckt. Die Anrechnung von Gold oder Gegenständen wie auch im KOD erwähnt erfordert die Implementierung eines Goldmanagements und am besten auch eine Neustrukturierung des Charakterbogens (s. weitere Issues). Frohe Weihnachten ;)
Le-Frique added 1 commit 2023-12-23 13:45:08 +01:00
Byroks requested changes 2023-12-27 22:02:57 +01:00
Byroks left a comment
Owner

Wir müssen den Hort einstellen können und den nicht aus den ES berechnen, da ist noch ein kleiner Fehler drin.
Wir haben zwei Möglichkeiten den Hort zu berechnen, entweder wir erstellen einen Toggle (Checkbox) für Items um zu bestimmen das diese in den Hort gehören und dann rechnen wir alle Werte von dort zusammen.
Alternative wäre auch einfach einen Werteintrag wie für Gold zu haben der den Hort bestimmt. Das wäre auf jeden fall leichter und schneller zu Implementieren. Man kann auch beide Variationen verknüpfen.

Wir müssen den Hort einstellen können und den nicht aus den ES berechnen, da ist noch ein kleiner Fehler drin. Wir haben zwei Möglichkeiten den Hort zu berechnen, entweder wir erstellen einen Toggle (Checkbox) für Items um zu bestimmen das diese in den Hort gehören und dann rechnen wir alle Werte von dort zusammen. Alternative wäre auch einfach einen Werteintrag wie für Gold zu haben der den Hort bestimmt. Das wäre auf jeden fall leichter und schneller zu Implementieren. Man kann auch beide Variationen verknüpfen.
lang/de.json Outdated
@ -95,2 +95,4 @@
"midgard5.currency-silver": "Silber",
"midgard5.currency-copper": "Kupfer",
"midgard5.hoard": "Hort",
"midgard5.hoardNext": "Hort für nächsten Grad",
Owner

Bitte im Style von den anderen localizations halten (hoard-next)

Bitte im Style von den anderen localizations halten (hoard-next)
Le-Frique marked this conversation as resolved
@ -144,3 +145,3 @@
ret.stats.drinking = this.modResult(0);
ret.stats.drinkingFW = Math.floor(ret.attributes.ko.value / 10);
ret.stats.hoard = M5Character.hoardFromExp(data.es);
Owner

Das funktioniert so leider nicht ist aber ein guter Anfang. Wir können den wert aber als Eingabe neben den anderen Goldeinträgen hinzufügen um damit die Hortgröße zu bestimmen.

Das funktioniert so leider nicht ist aber ein guter Anfang. Wir können den wert aber als Eingabe neben den anderen Goldeinträgen hinzufügen um damit die Hortgröße zu bestimmen.
Author
Member

Hab ich jetzt so ähnlich gemacht, ziehe aber den Hort aus dem levelTreshold direkt.

Hab ich jetzt so ähnlich gemacht, ziehe aber den Hort aus dem levelTreshold direkt.
Le-Frique marked this conversation as resolved
@ -371,7 +374,21 @@ export class M5Character extends Actor {
const ret = M5Character.levelThreshold.findIndex((val) => val > exp);
Owner

Wir sollten hier überprüfen was geringer ist, EXP oder Hort und entsprechend den Grad bestimmen

Wir sollten hier überprüfen was geringer ist, EXP oder Hort und entsprechend den Grad bestimmen
Author
Member

Komplett verworfen und mit Mindesthort und Hort nächster Grad aus levelTreshold gelöst.

Komplett verworfen und mit Mindesthort und Hort nächster Grad aus levelTreshold gelöst.
Le-Frique marked this conversation as resolved
@ -372,2 +375,4 @@
return ret === -1 ? M5Character.levelThreshold.length : ret;
}
static hoardFromExp(exp: number): number {
Owner

Sollte nicht benötigt werden außer ich verstehe den gewollten nutzen falsch

Sollte nicht benötigt werden außer ich verstehe den gewollten nutzen falsch
Le-Frique marked this conversation as resolved
@ -7,3 +7,3 @@
<td></td>
<td colspan="5" style="font-size: x-large; font-weigh: bold;"><input name="name" type="text" value="{{actor.name}}" data-dtype="String" /></td>
<td style="font-size: x-large; font-weigh: bold; text-align: center; text-align-vertical: middle;" ><div style="border: 2px solid black; border-radius: 10px; background-color: white;">{{data.calc.level}}</div></td>
{{#if (eq data.info.race "Zwerg")}}
Owner

Bricht leider den abstand bei den Prüfwürfen, allgemein finde ich das etwas sehr groß.
Ist das vielleicht bei den Berechneten Werten unten besser aufgehoben? Dort steht gerade eh 2 mal Gifttolleranz. Man könnte es dann entweder als:

Hort: x von y --> x der Aktuelle Hort und y der ES/2
Hort: x benötigt für nächsten Grad --> x wäre in dem fall der noch benötigte aka rest von ES/2 - Aktueller Hort
oder
(Gesamt)Hort für nächsten Grad: x --> Einfach nur ES/2

Und oben dann etwas kleiner nur den aktuellen Hort anzeigen

Bricht leider den abstand bei den Prüfwürfen, allgemein finde ich das etwas sehr groß. Ist das vielleicht bei den Berechneten Werten unten besser aufgehoben? Dort steht gerade eh 2 mal Gifttolleranz. Man könnte es dann entweder als: `Hort: x von y` --> x der Aktuelle Hort und y der ES/2 `Hort: x benötigt für nächsten Grad` --> x wäre in dem fall der noch benötigte aka rest von ES/2 - Aktueller Hort oder `(Gesamt)Hort für nächsten Grad: x` --> Einfach nur ES/2 Und oben dann etwas kleiner nur den aktuellen Hort anzeigen
Author
Member

So umgesetzt und Vermögensmanagment aktuellen Hort dort auch platziert, es waren auch Resi doppelt.

So umgesetzt und Vermögensmanagment aktuellen Hort dort auch platziert, es waren auch Resi doppelt.
Le-Frique marked this conversation as resolved
Owner

Bitte #5 in diesem Pullrequest mit abarbeiten, macht es leichter alles zu testen und übersichtlicher

Bitte #5 in diesem Pullrequest mit abarbeiten, macht es leichter alles zu testen und übersichtlicher
Author
Member

Bitte #5 in diesem Pullrequest mit abarbeiten, macht es leichter alles zu testen und übersichtlicher

Da beiß ich mir grad die Zähne etwas dran aus, habe Vermögen und verfügbares Vermögen dafür angelegt, bekomme die aber net aus den Gold, Silber und Kupferwerten berechnet. Muss Gold, Kupfer und Silber noch iwie als Stat oder so angelegt werden, um auf die Werte dann zugreifen zu können...?

> Bitte #5 in diesem Pullrequest mit abarbeiten, macht es leichter alles zu testen und übersichtlicher Da beiß ich mir grad die Zähne etwas dran aus, habe Vermögen und verfügbares Vermögen dafür angelegt, bekomme die aber net aus den Gold, Silber und Kupferwerten berechnet. Muss Gold, Kupfer und Silber noch iwie als Stat oder so angelegt werden, um auf die Werte dann zugreifen zu können...?
Le-Frique added 4 commits 2023-12-30 19:26:40 +01:00
Le-Frique added 1 commit 2023-12-30 19:39:32 +01:00
Le-Frique added a new dependency 2023-12-30 19:57:11 +01:00
Le-Frique added 1 commit 2024-01-02 16:45:18 +01:00
Le-Frique added 1 commit 2024-01-03 18:04:24 +01:00
Le-Frique added 1 commit 2024-01-03 18:49:30 +01:00
65179400a3 Zwergenhort
Zwergenhort endlich implementiert
Le-Frique added 1 commit 2024-01-03 19:33:44 +01:00
56887873e2 Kein $ und €
€ und $ entfernt
Byroks added 1 commit 2024-01-07 14:30:48 +01:00
Byroks merged commit b8e873537c into develop 2024-01-07 14:31:21 +01:00
Byroks deleted branch lefrique 2024-01-07 14:31:21 +01:00
Byroks referenced this issue from a commit 2024-01-07 14:31:22 +01:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Blocks
#5 Goldmanagement
MidgardVTT-Entwicklung/foundry-vtt-system-midgard5
Reference: MidgardVTT-Entwicklung/foundry-vtt-system-midgard5#4
No description provided.