Revisions should not url_encode or at least url_encode(diff("content"))
- Langage dominant
- JavaScript
- Étoiles
- 10.1k
- Forks
- 1.1k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
In #521 it looks like the `Revisions` table does something like this:
diff(url_encode("content"))
This makes little sense, as the diffing will always yield all of the document. And those are stored in `lastContent` and `content` anyway.
This should be reversed to be
url_encode(diff("content"))
I'm not even entirely sure why the url-encode is done in the first place, but it appears to be there.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Repérez l’implémentation de la table Revisions ainsi que le code lié à #521 qui applique url_encode et diff au contenu. Vérifiez pourquoi l’encodage a actuellement lieu avant le calcul du diff, puis vérifiez que le calcul du diff du contenu produit une révision ciblée et que l’encodage d’URL est appliqué ensuite ; utilisez les tests associés ou les données de révision disponibles dans le dépôt pour confirmer le résultat.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- databases
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 42/100