phpmyadmin / phpmyadmin/sql-parser

Fat finger prevention: identify potential user error on bulk update

Đang mở
#608 1 bình luận 1 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Ngôn ngữ chính
PHP
Star
485
Fork
119
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

Hi there,

Yesterday by copying & pasting, I ended up in this situation:

UPDATE table
SET field = 1
WHERE condition1;
AND condition 2;

Notice the ";" in after condition1. You already know how this ended.

The request is to identify this potential harmful mistakes and prompt a confirmation.

Thanks for reading.

Cheers,
J.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Issue không nêu file, test hay entry point nào. Hãy bắt đầu bằng cách xác định đường dẫn parser hoặc linter cho các câu lệnh UPDATE và xem xét các test liên quan, sau đó xác định điều kiện được phân tách bằng dấu chấm phẩy sẽ được phát hiện như thế nào và confirmation được hiển thị ra sao; công việc được xem là hoàn tất khi pattern gây hại được nhận diện và một confirmation được yêu cầu.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
mysql, php, sql
Lĩnh vực
databases, tooling
Loại issue
Tính năng
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
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
30/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.