hackmdio / hackmdio/codimd

Revisions should not url_encode or at least url_encode(diff("content"))

Ouverte
#522 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
database/sequelize discussion
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.