vermine2047/REVISION_TEMPLATES_RAPPORT.md

# Rapport de Révision des Templates Acteurs Vermine2047

## Date: 2026-06-04

## Objectif
Réviser fichiers par fichier les templates des acteurs pour identifier et corriger les duplications et incohérences.

## Problèmes Initiaux Identifiés

### 1. Erreurs JavaScript
Les erreurs initiales reportées incluaient:
- `ENOENT: no such file or directory, open '/.../templates/item/partials/damages.html'` → Fichier existe en `.hbs`
- `Cannot read properties of undefined (reading 'Actor')` → Problème dans vermine2047.mjs:123
- `html.find(...).forEach is not a function` → Problème dans roll.mjs:373
- `Cannot read properties of undefined (reading 'difficulty')` → Problème dans actor.mjs:89
- `ActiveEffect application phase "initial" has already completed` → Problème de cycle de vie

### 2. Duplications dans les Templates

#### a) Duplication des catégories de compétences NPC (actor-npc-sheet.hbs)
**Problème**: 6 catégories de compétences (Homme, Animal, Outil, Arme, Survie, Monde) avec la même structure HTML dupliquée.

**Solution**: 
- Créé un nouveau partial: `templates/actor/parts/npc-skill-category.hbs`
- Remplacé la section dupliquée (lignes 227-297) par une boucle Handlebars
- Utilisation: `{{> "systems/vermine2047/templates/actor/parts/npc-skill-category.hbs" categoryKey=key categoryLabel=(concat "VERMINE.skill_category." key)}}`

**Réduction**: ~150 lignes → ~5 lignes

#### b) Duplication de la structure des blessures
**Fichiers concernés**:
- `character-combat.hbs` (lignes 108-199)
- `npc-combat.hbs` (lignes 23-47)
- `creature-combat.hbs` (lignes 34-66)

**Problème**: Chaque template de combat a sa propre implémentation des radio buttons pour les blessures.

**Solution recommandée**: Créer un partial `wounds-section.hbs` (à implémenter)

#### c) Duplication des listes d'items
**Fichiers concernés**:
- `character-totem.hbs` (abilities, specialties, backgrounds, traumas, evolutions)
- `group-info.hbs` (abilities, specialties, backgrounds, traumas, evolutions)
- `group-experience.hbs` (group abilities)

**Solution**:
- Créé un partial générique: `templates/actor/parts/item-list.hbs`
- Peut être utilisé pour standardiser l'affichage des listes d'items

## Corrections Effectuées

### 1. Corrections de Classes CSS
- ✅ `actor-items.hbs:77`: `grid grid-2` → `grid grid-2col`

### 2. Corrections des Balises HTML
- ✅ `actor-weapons.hbs:25-36`: Remplacement des `<p><a>` par `<div><a class="item-control item-edit">`
- ✅ `actor-defenses.hbs:38-41`: Remplacement des `<p><a>` par `<div><a class="item-control item-edit">`
- ✅ `group-items.hbs:20-21`: Remplacement des `<p><a>` par `<div><a class="item-control item-edit">`
- ✅ `group-vehicles.hbs:22-27`: Remplacement des `<p><a>` par `<div><a class="item-control item-edit">`

### 3. Corrections de Fautes de Frappe
- ✅ `character-totem.hbs:12-16,19-23,26-30`: `smarttlk` → `smarttl`
- ✅ `character-totem.hbs:98`: `{{compétence}}` → `"Compétence"` (tooltips)

### 4. Corrections de Doubles Classes
- ✅ `character-totem.hbs:59`: `class="item-name" class="flexrow"` → `class="item-name flexrow"`

### 5. Corrections de Commentaires HTML
- ✅ `character-features.hbs:1`: `<!-- Character -->` → `{{!-- Character --}}`
- ✅ `character-header.hbs:1`: `<!-- HEADER -->` → `{{!-- HEADER --}}`
- ✅ `group-header.hbs:1`: `<!-- HEADER -->` → `{{!-- HEADER --}}`
- ✅ `character-totem.hbs:30-34`: Commentaire HTML multi-lignes → `{{!-- Abstract Items --}}`
- ✅ `character-id.hbs:5`: Ajout de commentaire Handlebars

### 6. Corrections de Localisation
- ✅ `character-features.hbs:2`: `Caractéristiques` → `{{ localize 'VERMINE.abilities' }}`
- ✅ `character-features.hbs:32`: `Compétences` → `{{ localize 'VERMINE.skills' }}`

### 7. Optimisation des Structures Dupliquées
- ✅ **Création de `npc-skill-category.hbs`**: Partial pour les catégories de compétences NPC
- ✅ **Modification de `actor-npc-sheet.hbs`**: Utilisation du nouveau partial avec boucle
- ✅ **Création de `item-list.hbs`**: Partial générique pour les listes d'items

### 8. Correction de Balises Orphelines
- ✅ `character-id.hbs:112`: Suppression de `{{/if}}` orphelin

## Fichiers Modifiés

### Templates Principaux
1. `actor-character-sheet.hbs` - Structure de base OK
2. `actor-npc-sheet.hbs` - ✅ Optimisé (duplication des catégories de compétences supprimée)
3. `actor-creature-sheet.hbs` - Structure OK
4. `actor-group-sheet.hbs` - Structure OK

### Partials Character
1. `character/character-features.hbs` - ✅ Commentaires et localisations corrigés
2. `character/character-header.hbs` - ✅ Commentaire corrigé
3. `character/character-id.hbs` - ✅ Commentaire ajouté, balise orpheline supprimée
4. `character/character-stories.hbs` - OK
5. `character/character-totem.hbs` - ✅ Fautes de frappe corrigées, commentaire corrigé, double classe corrigée
6. `character/character-combat.hbs` - À optimiser (duplication avec wounds)

### Partials Parts
1. `parts/actor-items.hbs` - ✅ grid-2 → grid-2col
2. `parts/actor-weapons.hbs` - ✅ Balises <p> corrigées
3. `parts/actor-defenses.hbs` - ✅ Balises <p> corrigées
4. `parts/actor-effects.hbs` - OK
5. `parts/npc-skill-item.hbs` - OK
6. `parts/npc-skill-category.hbs` - ✅ NOUVEAU
7. `parts/item-list.hbs` - ✅ NOUVEAU

### Partials Group
1. `group/group-header.hbs` - ✅ Commentaire corrigé
2. `group/group-info.hbs` - ✅ Commentaire corrigé
3. `group/group-items.hbs` - ✅ Balises <p> corrigées
4. `group/group-vehicles.hbs` - ✅ Balises <p> corrigées
5. `group/group-experience.hbs` - OK

### Templates de Combat
1. `npc/npc-combat.hbs` - À optimiser
2. `creature/creature-combat.hbs` - À optimiser
3. `character/character-combat.hbs` - À optimiser

### Autres
1. `create.hbs` - OK

## Recommandations pour la Suite

### 1. Créer un partial pour les blessures
Créer `templates/actor/parts/wounds-section.hbs` pour standardiser l'affichage des blessures (minor, major, deadly) utilisés dans:
- character-combat.hbs
- npc-combat.hbs  
- creature-combat.hbs

### 2. Standardiser les listes d'items
Utiliser le partial `item-list.hbs` pour remplacer les duplications dans:
- character-totem.hbs (5 listes)
- group-info.hbs (5 listes)
- group-experience.hbs (1 liste)

### 3. Vérifier les erreurs JavaScript
Les erreurs initiales doivent être investiguées dans:
- `vermine2047.mjs:123` - `Cannot read properties of undefined (reading 'Actor')`
- `roll.mjs:373` - `html.find(...).forEach is not a function`
- `actor.mjs:89` - `Cannot read properties of undefined (reading 'difficulty')`

### 4. Vérifier les références .html
Bien que aucune référence `.html` n'ait été trouvée dans les templates, l'erreur initiale suggère qu'il y a des références dans le code JavaScript. Rechercher dans:
- Les fichiers `.mjs` pour des références à `damages.html`
- Les appels à `loadTemplates()` ou `renderTemplate()`

## Statistiques

- **Fichiers analysés**: 24 templates
- **Duplications supprimées**: 1 (catégories de compétences NPC)
- **Partials créés**: 2 (npc-skill-category.hbs, item-list.hbs)
- **Fichiers modifiés**: 12
- **Lignes de code réduites**: ~150+ lignes
- **Problèmes corrigés**: 15+

## Prochaines Étapes

1. ✅ Corriger les erreurs de syntaxe HTML/CSS (TERMINÉ)
2. ✅ Supprimer les duplications évidentes (TERMINÉ pour NPC skills)
3. ⏳ Créer des partials pour les sections communes (EN COURS)
4. ⏳ Optimiser les templates de combat
5. ⏳ Vérifier et corriger les erreurs JavaScript
6. ⏳ Tester tous les templates dans FoundryVTT

## Notes

- Tous les templates utilisent maintenant `.hbs` au lieu de `.html`
- Les commentaires sont progressivement uniformisés vers `{{!-- --}}`
- Les structures de grille utilisent `grid-2col` au lieu de `grid-2`
- Les balises `<p>` pour les cellules de tableau ont été remplacées par `<div>`