Attributs avec des espaces après le nom

[Ulty]

Il arrive parfois que quand on copie-colle un attribut depuis la documentations, on copie un peu plus que le nom de l'attribut, et du coup l'attribut n'est pas reconnu par le script.

En principe, il me semble qu'aucun nom d'attribut ne devrait se terminer par une espace. Je vois 3 solutions pour éviter ce problème qui peut vraiment embêter un utilisateur un peu débutant (voire même expérimenté):

1. le script modifie le nom des attributs à leur création (on fait simplement nom.trim()).
2. on écrit une commande api !cof-sanitize qui va regarder tous les noms d'attributs en jeu, repérer ceux qui ont des espaces au début ou à la fin et les signaler/proposer de les corriger
3. on es saie dans le script de chercher aussi les variantes avec une espace quand on cherche un nom d'attribut particulier.

Des avis ?

[Far2Casual]

Le plus safe à mon avis c'est de trim au moment où tu utilises/cherches l'attribut. Sinon t'auras toujours un cas bizarre où ça finira par pas marcher je pense.

[stephaned68]

Le script pourrait avoir une fonction qui crée les attributs hors-fiche nécessaires à certaines capacités, genre !cof-attrib.
Tu sélectionnes un token avant de taper la commande, ou bien tu tapes !cof-attrib --charid=@{nom du perso|character_id}.
Ca t'affiche une liste de tous les attributs hors-fiche connus dans le chat avec un lien pour chaque et quand tu cliques sur l'un de ces liens ça relance la même fonction mais avec une syntaxe de type !cof-attrib --charid=xxx --name=xxxx --value=?{Valeur ?} qui va afficher un popup pour saisir la valeur de l'attribut, et le créer sous le bon nom dans la bonne fiche de perso.

[stephaned68]

Je me suis senti inspiré ce matin, et j'en suis là :

Si le profil est rentré dans la fiche de personnage, seules les capacités du profil correspondant sont affichées.

Quand on clique sur l'un des boutons, par exemple Vigilance du barbare :

Dans le chat :

Et dans la fiche de personnage :

Je gère aussi les valeurs max, bien sûr

Le truc fastidieux, c'est qu'il faut entrer les capacités dans le code du script :

Kénéné vouzenpenssé ?

[Far2Casual]

Que je comprenne bien ; l'idée c'est "d'assigner" les attributs nécessaires à certaines capacités via l'exécution d'une commande, c'est ça ?

[Ulty]

Le script pourrait avoir une fonction qui crée les attributs hors-fiche nécessaires à certaines capacités, genre !cof-attrib.
Tu sélectionnes un token avant de taper la commande, ou bien tu tapes !cof-attrib --charid=@{nom du perso|character_id}.
Ca t'affiche une liste de tous les attributs hors-fiche connus dans le chat avec un lien pour chaque et quand tu cliques sur l'un de ces liens ça relance la même fonction mais avec une syntaxe de type !cof-attrib --charid=xxx --name=xxxx --value=?{Valeur ?} qui va afficher un popup pour saisir la valeur de l'attribut, et le créer sous le bon nom dans la bonne fiche de perso.

Comme tu l'as remarqué, ça afficherait beaucoup trop d'attributs hors fiche. On pourrait aussi simplement suggérer d'utiliser !cof-set-attribute, qui existe déjà, pour rentrer les nouveaux attributs. Ça aura l'avantage de supprimer automatiquement les blancs qui trainent.

L'autre solution qui consiste à n'afficher que les attributs liés aux capacités du profil implique un gros effort d'implémentation. Quitte à faire ça, je crois que je préfèrerais une automatisation au niveau de la fiche, comme ce que tu as fait avec CoC et ton script : pré-remplissage des noms de voies quand on choisit le profil, possibilité de changer ensuite les noms de voies ou d'utiliser un profil hybride, puis quand on coche une capacité dans la liste, le script se charge de mettre les attributs qui vont bien, sans autre effort de la part de l'utilisateur.

[stephaned68]

Ça fait en effet un affichage assez imposant, mais bon, c'est une fois de temps en temps, quand on acquiert une nouvelle capacité. Du coup, j'ai creusé une autre idée, en effet basée sur les modifs de la fiche COC. Sauf qu'un sheet-worker ne peut pas accéder aux fonctions API de création d'objet. Alors, pourquoi ne pas avoir un attribut caché, qui contient un objet Javascript sérialisé ? Quand on coche une des capacités, on crée une nouvelle propriété de cet objet, dont le nom est le nom de la capacité convertie en camelCase (Même pas mal => memePasMal) et la valeur true (false quand la case est décochée). Et on re-sérialise l'objet avant de le re-stocker dans l'attribut caché. Inconvénient : on ne gère que des valeurs booléenne, et il faut modifier le script COFantasy pour récupérer ces valeurs. Une autre alternative : un section répétable pour les attributs supplémentaires, mais là aussi le script doit être modifié...

[Far2Casual]

Un Sheet Worker ne peut ajouter/créer un attribut de perso ? Je pensais que c'était déjà le cas avec la fiche actuelle non ?

Sinon c'est une bonne idée, ça fait un moment que je me disais que ce serait cool d'avoir des fiches avec des voies pré-remplies qu'il suffit de cocher.
Et pas que pour les PJs d'ailleurs...

[stephaned68]

@Far2Casual
Oui, les seuls attributs qu'un sheet-worker puisse créer ou supprimer sont les attributs d'une ligne de section répétable avec la fonction generateRowId(). Pour les autres attributs, tu dois avoir un input éventuellement de type hidden avec le nom de l'attribut pour que la fonction setAttrs() puisse le mettre à jour. On n'a pas accès au DOM de la fiche pour y créer de nouveaux champs à la volée.

[Ulty]

@Far2Casual
Oui, les seuls attributs qu'un sheet-worker puisse créer ou supprimer sont les attributs d'une ligne de section répétable avec la fonction generateRowId(). Pour les autres attributs, tu dois avoir un input éventuellement de type hidden avec le nom de l'attribut pour que la fonction setAttrs() puisse le mettre à jour. On n'a pas accès au DOM de la fiche pour y créer de nouveaux champs à la volée.

Je viens de vérifier sur une sandbox, et non, le sheet worker peut bien créer des attributs qui ne sont pas définis dans la fiche, et en plus en case sensitive, on dirait. Tu avais vu cette impossibilité dans la doc des sheets workers ?

[stephaned68]

@Ulty
Non je l'avais bêtement déduit du fait qu'on n'avait pas accès au DOM donc qu'on ne pouvait pas modifier la fiche par code, ne connaissant pas la plomberie interne de Roll20. Donc a priori, d'une session de jeu à une autre, ce sont les objets qu'on voit dans l'onglet attributes & abilties qui sont sérialisés à partir des valeurs de la fiche quand on ferme celle-ci et/ou qu'on quitte la campagne, et désérialisés puis rechargés dans les champs de la fiche quand on réouvre la campagne et la fiche.
Voila qui ouvre des horizons insoupçonnés... ;)

[stephaned68]

Wo pitain wo pitain wo pitain
Court partout en poussant des petits cris de belette en rut

[Ulty]

Et même encore mieux : on peut afficher les valeurs d'attributs non déclarés sur la fiche, avec <span name="attr_nomAttribut"></span>...

[stephaned68]

Mais comment fais-tu ça puisque tu ne peux pas modifier la fiche à la volée par code ?