Comments (2)
I’d go a little further and say these script blocks shouldn’t exist in the cfm files either… that’s what .js files are for :-) Unfortunately, it’s a scattered legacy monolith.
My original thought was just a setting to ignore script blocks (off by default to keep the current behavior). I only expanded to the regex because I recalled cfformat doing the same to some XML and the regex approach should cover that too.
we’ll just have to be more diligent in code reviews until things get a bit cleaner in the app.
thanks for cfformt!
from commandbox-cfformat.
I agree this is a problem, and is one of the reasons my focus with this library was on component files where <script>
tags are not likely to be found (though of course it does happen). The mixture of CFML and JavaScript makes things tricky. I am not sure yet which approach to solving this is the most doable. Whatever is done is most likely always going to be imperfect, but I do think things can be improved.
from commandbox-cfformat.
Related Issues (20)
- Subsequent lines of a multiline statement should be indented. HOT 2
- cfformat check doesn't handle Unicode with BOM files HOT 1
- Forgebox installation is broken HOT 1
- DocBox commenting formatting HOT 6
- struct element alignment HOT 5
- Tag type comments converted to script comments erroneously HOT 2
- annotation with a `:` is adding a space HOT 2
- Latest version removing indendations on Comments HOT 1
- Latest version removing comments HOT 1
- CommandBox no longer reloads on module installation HOT 8
- preserve case in doc-block? HOT 2
- Indent content between cfscript tags HOT 3
- Property alignment incorrect when the number of attributes differs
- annotation with a - is adding a space
- tag-check and single tags that are not self-closing HOT 3
- abstract component modifier injects spaces
- Lines are improperly outdented
- Method Multiline formatting/highlighting when inital element is not a method HOT 1
- CSS styles
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from commandbox-cfformat.