phpmyadmin / phpmyadmin/sql-parser

Fat finger prevention: identify potential user error on bulk update

Offen
#608 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
PHP
Sterne
485
Forks
119
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Das Issue nennt keine Datei, keinen Test und keinen Einstiegspunkt. Beginne damit, den Parser- oder Linter-Pfad für UPDATE-Anweisungen zu lokalisieren und zugehörige Tests zu prüfen; definiere dann, wie die durch Semikolons getrennte Bedingung erkannt werden soll und wie die Bestätigung angezeigt wird; abgeschlossen ist die Aufgabe, wenn das schädliche Muster identifiziert und eine Bestätigung angefordert wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
mysql, php, sql
Bereich
databases, tooling
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
30/100

Neue Issues direkt in Ihr Postfach

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