PoorMansTSqlFormatter icon indicating copy to clipboard operation
PoorMansTSqlFormatter copied to clipboard

"Enable Keyword Standardization" option should be better labelled

Open TaoK opened this issue 5 years ago • 1 comments

The "Enable Keyword Standardization" option on poorsql.com ("keywordStandardization" in the library) is misleadingly-named: It should more clearly reflect:

  • The fact that it is only safe for T-Sql, not for other variants (eg MySql)
  • The fact that it "standardizes" to more common forms of keywords in T-Sql, rather than more standard/compatible forms. For example, "NVarChar" is more commonly used in T-Sql, but "National Character Varying" is more standard / more cross-compatible.

Furthermore, given the reasonably wide demographic of users, it does not make sense to enable this option by default, as it can "damage" MySql code. The the option should be disabled by default on PoorSql.com, as it is in the library.

TaoK avatar Apr 12 '19 17:04 TaoK

The default for poorsql.com has been changed, updating description to target naming primarily.

TaoK avatar Apr 23 '19 08:04 TaoK