kreuvf / roll20-character-sheets

Kreuvf's fork of the Roll20 Character sheet templates. Do not work with my branches prefixed with "pre", I rebase and force-push them all the time. Check out the YouTube channel for videos on my pull requests.
https://www.youtube.com/user/Isurandil
MIT License
7 stars 6 forks source link

Interne Benennung von "Animatio" falsch #138

Open kreuvf opened 8 months ago

kreuvf commented 8 months ago

Mal mit, mal ohne zweitem "i", berichtet von @Triqueon in https://github.com/kreuvf/roll20-character-sheets/pull/136#issuecomment-1998676375.

grep -E -i 'animati?o' dsa4_1_character_sheet_roll20.html
                <input class="talent-zauber-animato default-hide" type="checkbox" name="attr_zauber_Animato" value="1">
                    <button type="action" name="act_z_animatio-action" class="default-hide"></button>
                    <input type="hidden" name="attr_z_animatio_action" value="">
                        <button type="roll" name="roll_z_animatio" value="@{z_animatio_action}"> <span>Animatio Stummer Diener</span></button>
                    <input type="text" class="talent-spez" name="attr_Spez_Animato">
                    <input type="number" class="talent-taw" name="attr_ZfW_Animato" min="0" value="0">
                    <button name="AnimatoSpez" type="roll" value="@{gm_roll_opt} &{template:DSA-Zauber} {{name=Animatio Stummer Diener (@{spez_Animato})}} {{mod=[[?{Erleichterung (−) oder Erschwernis (+)|0}d1cs0cf2]]}} {{taw=[[@{ZfW_Animato} + 2]]}} {{stat1=[[@{KL}]]}} {{stat2=[[@{FF}]]}} {{stat3=[[@{GE}]]}} {{Talent=[[ { [[{0d1 + @{ZfW_Animato} + 2 - (?{Erleichterung (−) oder Erschwernis (+)|0}), 0d1}kh1]] - {1d20cs1cf20 + [[{0d1 + (?{Erleichterung (−) oder Erschwernis (+)|0}) - (@{ZfW_Animato} + 2), 0d1}kh1]] - @{KL}, 0d1}kh1 - {1d20cs1cf20 + [[{0d1 + (?{Erleichterung (−) oder Erschwernis (+)|0}) - (@{ZfW_Animato} + 2), 0d1}kh1]] - @{FF}, 0d1}kh1 - {1d20cs1cf20 + [[{0d1 + (?{Erleichterung (−) oder Erschwernis (+)|0}) - (@{ZfW_Animato} + 2), 0d1}kh1]] - @{GE}, 0d1}kh1, 0d1 + @{ZfW_Animato} + 2}dh1]]}}">Spez</button>
                                        <label><input type="checkbox" name="attr_zauber_Animato" value="1">Animatio Stummer Diener</label>
            <button name="Animato" type="roll" value="@{z_animatio_action}"> <span>Animatio Stummer Diener</span></button>
    'z_animatio',
    'z_animatio': {'internal': "animato", 'ui': "Animatio Stummer Diener", 'stats': ['KL', 'FF', 'GE']},

Funktional läuft es, aber ist unsauber. Behebung nur mit Migration möglich.

Triqueon commented 7 months ago

Ok, ein bisschen tiefere Recherche: während der Animatio auf einem Tippfehler zu basieren scheint, hat etwa ein drittel aller zauber einen anderen "internal" Name als das, was als Triggerbutton verwendet wird. So hat "Fluch der Pestilenz" als Rollbutton bsp "roll_z_fluchderpestilenz", und als Spezialisierungsfeld "attr_Spez_fluch". Das ist ungünstig, aber nicht wirklich kaputt. Der Animatio fällt nur besonders auf, weil der internal name halt einen Tippfehler hat.

Würde ich jetzt für den Moment so lassen und hoffen, dass das Problem im Rahmen der evtl geplanten Änderung von Zaubern hin zu repeating sections (siehe Diskussion aus #139) von selbst weggeht.

kreuvf commented 6 months ago

Ok, ein bisschen tiefere Recherche: während der Animatio auf einem Tippfehler zu basieren scheint, hat etwa ein drittel aller zauber einen anderen "internal" Name als das, was als Triggerbutton verwendet wird. So hat "Fluch der Pestilenz" als Rollbutton bsp "roll_z_fluchderpestilenz", und als Spezialisierungsfeld "attr_Spez_fluch". Das ist ungünstig, aber nicht wirklich kaputt. Der Animatio fällt nur besonders auf, weil der internal name halt einen Tippfehler hat.

Diese Auffälligkeit ist das Ergebnis mehrerer Köche. Ich habe alle alten Attribute erst mal gelassen und ändere nur etwas, wenn ich muss. Zum Thema Spezialisierungsknöpfe: Die sollen irgendwann™ sowieso verschwinden.