Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nom de la catégorie remplacé par "Autre matériel" #319

Closed
Tuxem opened this issue Jan 5, 2022 · 6 comments · Fixed by #335
Closed

Nom de la catégorie remplacé par "Autre matériel" #319

Tuxem opened this issue Jan 5, 2022 · 6 comments · Fixed by #335
Assignees
Labels
Bug Tickets relatifs à une fonctionnalité qui ne fonctionne pas comme prévu
Milestone

Comments

@Tuxem
Copy link
Contributor

Tuxem commented Jan 5, 2022

Bonjour à tous,

Mise à jour vers la 0.17.

J'ai voulu tester l'affichage des catégories lors de l'impression de la liste. J'ai bien mes items qui sont triés par catégorie, mais l'intitulé de chaques catégories est "Autre matériel".

J'ai que des catégories. Je n'ai pas de sous-catégories

Est-ce normal ?

En tout cas, encore merci, super taf

@polosson
Copy link
Member

polosson commented Jan 6, 2022

En effet, en l'absence de sous-catégorie, le système affiche "autre matériel"... Mais cela ne fait sens que quand il existe des sous-catégories à afficher.

Il est possible de supprimer cette mention "Autre matériel" dans le template du PDF, ça se trouve à la ligne 139 de src/views/pdf/event-summary-default.twig.

Cela dit, on pourrait ajouter un check qui ajoute cette mention uniquement s'il existe plusieurs sous-catégories différentes dans la liste... À voir.

@Tuxem
Copy link
Contributor Author

Tuxem commented Jan 6, 2022

Okay, du coup je fais un peu de clean dans mon inventaire et rajoute des sous catégories, ça fera pas de mal ^^

Par contre chose assez étonante, en regardant le twig, il aurait du me mettre "Ma categorie - Autre matériel" alors que là il me méttait juste "Autre matériel". Et maintenant que j'ai mes sous-catégories, il me met juste le nom des sous-catgories.

Du coup c'est pas vraiment un problème mais plus une info donc si vous voulez on peut cloturer l'issue

@polosson
Copy link
Member

polosson commented Jan 6, 2022

Ah, je pense savoir pourquoi ça se comporte ainsi : c'est parce que le tri est basé sur les sous-catégories directement.

Tu peux essayer de choisir "catégories" dans la page des paramètres globaux de l'appli, sous l'onglet "Fiches de sorties", le sélecteur "Mode de présentation", pour voir ce que ça donne ? Je pense que ça correspondrait plus à ta façon d'organiser le matériel du coup... Dis moi !

@Tuxem
Copy link
Contributor Author

Tuxem commented Jan 6, 2022

C'était déjà le paramètre sur lequel j'étais. J'ai essayé les deux, ça ne me met que la sous-catégorie, pas de catégorie

@polosson
Copy link
Member

polosson commented Jan 6, 2022

Hmmm bizarre, il y a donc sûrement un bug à ce niveau là, on va devoir creuser. Merci @Tuxem !

@Tuxem
Copy link
Contributor Author

Tuxem commented Jan 6, 2022

Comme d'habitude, si vous avez besoin de mon dump SQL ou autre chose n'hésitez pas ;)

@Donov4n Donov4n added the Bug Tickets relatifs à une fonctionnalité qui ne fonctionne pas comme prévu label Jan 6, 2022
@polosson polosson added this to the 0.18.0 milestone Feb 16, 2022
@Donov4n Donov4n linked a pull request Mar 12, 2022 that will close this issue
polosson added a commit that referenced this issue Mar 14, 2022
* Enlève "Autre matériel" des fiches de sortie si pas de sous-categ (#319)

* Corrige selon review

* Corrige les tests unitaires
Donov4n pushed a commit that referenced this issue Nov 5, 2023
* Ajoute le back-end des emplacements du matériel dans les parcs (#294)

* Ajoute le front-end des emplacements du matériel dans les parcs (#294)

* Affiche les emplacements dans les fiches de sortie (+ corrections)

* Affiche les emplacements dans les inventaires de retour (#294)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Tickets relatifs à une fonctionnalité qui ne fonctionne pas comme prévu
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants