hackmdio / hackmdio/codimd

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

Offen
#522 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
database/sequelize discussion
Vorherrschende Sprache
JavaScript
Sterne
10.1k
Forks
1.1k
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.