search

jcberquist / commandbox-cfformat

A CommandBox module for formatting CFML component files.
MIT License
22 stars 10 forks source link

Qb statements getting split #98

Closed Daemach closed 3 years ago

Daemach commented 3 years ago

I really love this tool. Can you tell me what setting would keep the querybuilder statements (in red) from getting split? I know they work as is, but the red is glaring :) I would prefer to see the following if possible: https://www.screencast.com/t/FXsPSODj

image

elpete commented 3 years ago

It's because query is a tag-in-script word. CFFormat splits the chain on to separate lines and then the parser wonders if you were using the tag-in-script query. Using a different variable name like qb instead of query fixes the red.

On Thu, Jan 7, 2021, 7:36 AM Daemach notifications@github.com wrote:

I really love this tool. Can you tell me what setting would keep the querybuilder statements (in red) from getting split? I know they work as is, but the red is glaring :) I would prefer to see the following if possible: https://www.screencast.com/t/FXsPSODj

[image: image] https://user-images.githubusercontent.com/12553953/103904624-5bd4b700-50b2-11eb-9f1b-a18b36a0188d.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jcberquist/commandbox-cfformat/issues/98, or unsubscribe https://github.com/notifications/unsubscribe-auth/AATWYXSOI4HPCW554IWSYEDSYXBINANCNFSM4VZCBYEQ .

jcberquist commented 3 years ago

@Daemach: what @elpete said :) - this is a limitation of the syntax highlighting in VS Code. VS Code (and others) typically do their highlighting line by line - so query on a line by itself is ambiguous - it could be a variable, or it could be the start of a tag in script. The syntax highlighting chooses a tag in script in such situations - so for now, the thing to do is what @elpete said and use a different variable name.