https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32613

Jacob O'Mara <jacob.om...@ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |ASSIGNED

--- Comment #5 from Jacob O'Mara <jacob.om...@ptfs-europe.com> ---
Hi Lucas,
I can remove INSERT, DELETE etc. from the sql.js keywords. This would also
remove the syntax highlighting for these in the codemirror instance but that
should not be an issue as they should not be being used anyway.

In terms of column autocompletion the columns only appear when referencing the
column in the format 'table.column' eg:

SELECT alert.alertid FROM alert; 

would give you autocompletion for the column. While this is more typing than is
typical it does reduce the likelihood of an end user requesting an incorrect
column. I'm open to suggestions as I can modify the parsing to always show all
options if that is deemed better. (should have been written above so apologies)
additionally, I very stupidly added the . key to the list of exception keys
which will be amended in a patch I will upload later. Taking this off of 'needs
signoff' in the meantime

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to