Feature #5450
closedDB Exception: highlight problematic text position, SQL syntax highlight
0%
Description
1) In der Exception koennte der Teil, den MySQL als problematisch ansieht, automatische markiert werden.
Message [ mysqli: 1064 ] You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '/ 100) WHERE gr_id=182953 AND gr_id2=698 /* Uebunsgleiter */ AND x_id3=0 /* Korr' at line 1
Am besten in 'DBException' oder 'AbstractException' (html sanatizing?), check ob die Message ein 'mysqli' Fehler ist, dann nach 'server version for the right syntax to use near' suchen und den Rest (bis auf 'at line 1') als Pattern suchen in SQL Raw markieren.
2) Evtl. koennte auch der gesamte SQL Text mit einem Syntax Hightlight ausgestattet werden.
Files
Related issues
Updated by Benjamin Baer almost 6 years ago
- Target version changed from 55 to 18.10.3
Updated by Carsten Rose over 5 years ago
- Subject changed from DB Exception: highlight problematic text position to DB Exception: highlight problematic text position, SQL syntax highlight
- Description updated (diff)
Updated by Elias Villiger over 5 years ago
- Status changed from New to In Progress
- Assignee set to Elias Villiger
Updated by Elias Villiger over 5 years ago
- File squiggly_underline.PNG squiggly_underline.PNG added
Vorschlag: siehe Anhang.
- Nicht bis Ende markiert, da der match theoretisch mehrere Male auftreten könnte und dann das Highlight verwirrend sein könnte. Ist aber sicher ein edge case.
- Allgemeines SQL highlighting würde man dann wohl mit einem Tool wie https://highlightjs.org/usage/ machen? Ist das gewünscht?
Updated by Elias Villiger over 5 years ago
Beispiel doppelter Hit im Anhang: wenn alles ab dem ersten Hit markiert wird, verschleiert das den Fehler. Könnte man allerdings auch teilweise lösen, indem jeweils das letzte Vorkommen markiert wird (bei früheren Vorkommen ist der match string normalerweise lang genug, dass es keine doppelten Hits gibt).
Updated by Carsten Rose over 5 years ago
- Related to Feature #6992: DB exception: Syntax Highlight added
Updated by Carsten Rose over 5 years ago
- Status changed from In Progress to Closed
- Target version changed from 18.10.3 to 18.10.2