Improve SQL Formatter about sequel-ace HOT 7 CLOSED

I think SP was hoping to do a built-in implementation at some point, but years ago there didn't seem to be a nice library we could use. Things have hopefully changed!

from sequel-ace.

At the very least, the formatted SQL string should be injected back into the editor.

Tried this out and all you need to do is change showashtml to replaceselection

<key>output</key>
<string>replaceselection</string>

in

from sequel-ace.

Could also look into bundling the app with python (unless we can still use system python from the sandbox?

The bundles run system php, ruby and bash somehow from the sandbox.

from sequel-ace.

Better yet, we should consider implementing local formatting of the SQL string that doesn't require access to the internet (perhaps via python) both for security purposes and for allowing offline development with a local DB.

poor-mans-t-sql-formatter-npm-cli works quite well on basic SQL I tested, but two issues with it:

1. User needs node installed, and npm
2. It's for T-SQL, so complex MySQL might not get formatted properly.

from sequel-ace.

poor man's formatter:

UPDATE IGNORE PushRegos,
	PushRegos_copy
SET `PushRegos`.`deviceToken` = `PushRegos_copy`.`deviceToken`
FROM `PushRegos_copy`
INNER JOIN PushRegos ON PushRegos_copy.deviceToken LIKE CONCAT (
		PushRegos.deviceToken,
		'%'
		)
	AND LENGTH(PushRegos.deviceToken) < 64;

INSERT IGNORE
INTO warningsHist
VALUES (
	15,
	"Strong Monsoon Signal ( 11:20 HKT - 25.06.2012 )",
	now(),
	NULL,
	NULL
	);

sqlformat.org formatter:

UPDATE
IGNORE PushRegos,
       PushRegos_copy
SET `PushRegos`.`deviceToken` = `PushRegos_copy`.`deviceToken`
FROM `PushRegos_copy`
INNER JOIN PushRegos ON PushRegos_copy.deviceToken LIKE CONCAT(PushRegos.deviceToken, '%')
and LENGTH(PushRegos.deviceToken) < 64;

INSERT
IGNORE INTO warningsHist
values (15,"Strong Monsoon Signal ( 11:20 HKT - 25.06.2012 )", now(), NULL, NULL)

I think I prefer sqlformat.org.

from sequel-ace.

I prefer sqlformatter.org too!

Could also look into bundling the app with python (unless we can still use system python from the sandbox? That would be better) and using a package to parse locally, like mentioned here. sequelpro/sequelpro#1988 (comment)

from sequel-ace.

At the very least, the formatted SQL string should be injected back into the editor.

Tried this out and all you need to do is change showashtml to replaceselection

<key>output</key>
<string>replaceselection</string>

in

Sequel-Ace/SharedSupport/Default Bundles/Format SQL.spBundle/command.plist

Lines 45 to 46 in 8258485

	<key>output</key>
	<string>showashtml</string>

Cool!
There may be an issue with the default bundle updater too because when I made this change in the default bundle, for me at least it wasn't automatically copied over into my user's copy of the bundle.

See #26

from sequel-ace.