Revisions should not url_encode or at least url_encode(diff("content"))
オープン
database/sequelize
discussion
- 主要言語
- JavaScript
- スター
- 10.1k
- フォーク
- 1.1k
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
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.
コントリビューションガイド
調査の方向性
Revisions テーブルの実装と、コンテンツに url_encode および diff を適用する #521 関連のコードを特定してください。現在なぜ diff の前にエンコードが行われているのかを確認し、その後、コンテンツに対して diff を行うと焦点の絞られたリビジョンが生成され、URL エンコードが後から適用されることを検証してください。結果を確認するために、関連するテストやリポジトリで利用可能なリビジョンデータがあれば使用してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript
- 領域
- databases
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100