Revisions should not url_encode or at least url_encode(diff("content"))
- Ngôn ngữ chính
- JavaScript
- Star
- 10.1k
- Fork
- 1.1k
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
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.
Hướng dẫn đóng góp
Hướng nghiên cứu
Xác định phần triển khai của bảng Revisions và mã liên quan đến #521 áp dụng url_encode và diff cho nội dung. Kiểm tra lý do việc mã hóa hiện đang diễn ra trước khi thực hiện diff, sau đó xác minh rằng việc thực hiện diff trên nội dung tạo ra một revision tập trung và URL encoding được áp dụng sau đó; sử dụng mọi bài kiểm thử liên quan hoặc dữ liệu revision có sẵn trong repository để xác nhận kết quả.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- javascript
- Lĩnh vực
- databases
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 42/100