ecirpnaes / SimpleDataScripter

Azure Data Studio Extension: Scripts table data
MIT License
69 stars 15 forks source link

feat (Insert): Prefix unicode fields with N. Closes #13 #15

Closed jafin closed 3 years ago

jafin commented 3 years ago

Prefix unicode fields with N.

ecirpnaes commented 3 years ago

I actually have a fix for this issue in a local branch. Just haven't gotten around to pushing and merging into the main dev branch. Your fix is fine with the exception that you missed rolling the "char" datatype up in the case statement with the other varchar datatypes. (char != nchar)

jafin commented 3 years ago

@ecirpnaes No worries, I needed it locally so I did the PR. but if you've already got it baked all good!