Replace the newlines in nl2br()
Personne n'a encore pris cette issue.
- Langage dominant
- C
- Étoiles
- 40.4k
- Forks
- 8.2k
- Merge moyen
- 2 j 13 h
- PR mergées (30 j)
- 96
Description
Description
The function name of nl2br() suggests that it is a "replace" operation but is actually an "insert" operation.
Would it be possible to add a 3rd parameter so the function would actually replace the newline characters?
nl2br(string $string, bool $use_xhtml = true, bool $replace = false): string
The confusion with the name and the need for the 3rd parameter is obvious in the user contributed notes (most popular one and following ones): https://www.php.net/manual/en/function.nl2br.php#49516
The advantages of the 3rd parameter (instead of changing the behavior):
- nl2br() would be idempotent when 3rd parameter is set to true. Today n2br() cannot be applied several times to a string without managing extra inserts.
- Simplify the code. We could remove the wrappers to replace the newlines from our code.
- Flexibility. Old behavior could be kept.
- Backward compatible.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par localiser l’implémentation de nl2br() et ses tests dans php-src, puis comparez le comportement actuel d’insertion avec le paramètre replace proposé. Le travail sera considéré comme terminé lorsque le troisième paramètre conservera le comportement existant par défaut et remplacera les caractères de nouvelle ligne lorsqu’il sera activé, avec une couverture des deux modes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- php
- Domaine
- backend
- Type d'issue
- Fonctionnalité
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 42/100