hankinsoft / SQLPro

SQLPro bug & features tracking.
106 stars 28 forks source link

Should escape table/columns for default query in SQLPro for Postgres #973

Closed lifeiscontent closed 6 months ago

lifeiscontent commented 7 months ago

Describe the bug in a DB that has a database such as #tmp_1st_week the default query should be valid

To Reproduce Steps to reproduce the behavior:

  1. Create table with special chars
  2. double click on table
  3. see invalid SQL

Expected behavior

SELECT * FROM public."#tmp_1st_week" LIMIT 1000;

Actual behavior

SELECT * FROM public.#tmp_1st_week LIMIT 1000;

Screenshots

Screenshot 2024-04-15 at 10 11 49 AM

Environment details (please complete the following information):

Additional context Add any other context about the problem here.

Issues will be closed until environmental details are provided.

If the above template is not completed, issues with be closed with the statement Issue has been closed. Please edit the initial post (or create a new issue) and follow the template. Once completed, the issue may be reopened.

hankinsoft commented 7 months ago

Ahhh, thanks for those details. I've got this reproduced and fixed. A build has been submitted to apple. It usually takes about 24 hours for it to be approved.

hankinsoft commented 7 months ago

Latest build is up and has this all sorted. If you get the chance please let me know if everything is working as expected.

lifeiscontent commented 7 months ago

@hankinsoft thanks again for a very quick turn around 🎉