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

Réparation du partial GetTruncateContent #608

Merged
merged 1 commit into from
Sep 16, 2024
Merged

Réparation du partial GetTruncateContent #608

merged 1 commit into from
Sep 16, 2024

Conversation

Olivia206
Copy link
Contributor

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

On s'est rendu compte que la configuration truncate_description: 0 ne fonctionnait pas.

Cela vient du fait que nous avons dans notre partial cette manière de récupérer la donnée : {{ $length := .length | default 150 }}. En fait, 0 est interprêté par default comme une valeur nulle ! Il passe donc automatiquement à 150.

J'ai essayé avec or, mais cela fonctionne de la même façon.

J'ai donc opté pour cond, qu'on utilise à quelques endroits : {{ $length := cond (eq .length 0) 0 (default 150 .length) }}

  • Si le premier argument est vrai (.length = 0), alors ça retourne 0
  • Sinon elle retourne le dernier, et c'est à cet endroit qu'on donne le default si .length n'est pas défini

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

URL de test du site (optionnel)

IUT Bordeaux montaigne (accueil)
Mais n'importe quel site avec des blocs pages/actus, etc.

Screenshots

  1. Sans préciser de taille (défault 150)
    Capture d’écran 2024-09-16 à 15 03 10

  2. On précise une taille de 14 pour tester en config
    Capture d’écran 2024-09-16 à 15 02 52

  3. 0 en config
    Capture d’écran 2024-09-16 à 15 02 40

@Olivia206 Olivia206 added bug Something isn't working Incidence faible 😌 Risque faible de régression sur les sites labels Sep 16, 2024
@arnaudlevy arnaudlevy merged commit 5bc9c38 into main Sep 16, 2024
3 checks passed
@arnaudlevy arnaudlevy deleted the fix-truncate branch September 16, 2024 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Incidence faible 😌 Risque faible de régression sur les sites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants