Project

General

Profile

Feature #5450

DB Exception: highlight problematic text position, SQL syntax highlight

Added by Carsten Rose over 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
15.02.2018
Due date:
% Done:

0%

Estimated time:
Discuss:

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.

squiggly_underline.PNG (128 KB) squiggly_underline.PNG Elias Villiger, 10.10.2018 18:54
Beispiel doppelter hit.PNG (208 KB) Beispiel doppelter hit.PNG Elias Villiger, 10.10.2018 18:57
1745
1746

Related issues

Related to QFQ - Feature #6992: DB exception: Syntax Highlight Some day maybe2018-10-13

Associated revisions

Revision 14136c87 (diff)
Added by Elias Villiger almost 2 years ago

F5450 - MySQL Exception: underline faulty area - #refs 5450

History

#1 Updated by Carsten Rose over 2 years ago

  • Target version set to next

#2 Updated by Benjamin Baer over 2 years ago

  • Target version changed from next to 18.10.3

#3 Updated by Carsten Rose about 2 years ago

Wird mit Code Review von Benj gemacht.

#4 Updated by Carsten Rose almost 2 years ago

  • Subject changed from DB Exception: highlight problematic text position to DB Exception: highlight problematic text position, SQL syntax highlight
  • Description updated (diff)

#5 Updated by Elias Villiger almost 2 years ago

  • Status changed from New to In Progress
  • Assignee set to Elias Villiger

#6 Updated by Elias Villiger almost 2 years ago

1745

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.

#7 Updated by Elias Villiger almost 2 years ago

1746

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).

#8 Updated by Carsten Rose almost 2 years ago

#9 Updated by Carsten Rose almost 2 years ago

  • Status changed from In Progress to Closed
  • Target version changed from 18.10.3 to 18.10.2

Also available in: Atom PDF