Column list wrapping and indentation

7 Posts
3 Users
0 Reactions
1,940 Views
Posts: 16
Topic starter
(@jpotelle)
Member
Joined: 4 years ago
[#89]

Isn't working as intended. I have setting such that all comma should be surrounded by SPACE, unwrapped, and left aligned. When there is SPACE after comma, it works. but when there is no SPACE after comma, it doesn't work.

e.g.

group by 1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16

(i.e. no space after comma) is producing 

group by
1,2
, 3,4
, 5,6
, 7,8
, 9,10
, 11,12
, 13,14
, 15,16

It should produce this:

group by
  1
, 2
, 3
, 4
, 5
, 6
, 7
, 8
, 9
, 10
, 11
, 12
, 13
, 14
, 15
, 16

6 Replies
Guido
Posts: 295
Admin
(@admin)
Member
Joined: 5 years ago

Hi @jpotelle,

thanks for your feedback.

This issue is related to the fact that in Europe we use the comma as decimal-point. Therefore the parser will not split digits with a comma "inside". Now, reading your comment, I think that I should not check the  decimalpoint-is-comma rule within "ORDER BY" and "GROUP BY". Will fix this with the next release

 

Regards

Guido


Reply
2 Replies
(@jpotelle)
Joined: 4 years ago

Member
Posts: 16

@admin that would explain why this also doesn't work right:

where mycolumn in (71,72, 81,82)

Is it feasible to check localization settings when dealing commas overall?


Reply
Guido
Admin
(@admin)
Joined: 5 years ago

Member
Posts: 295

@jpotelle yes, probably a localization parm is a good choice


Reply
Posts: 14
(@bbhulsey)
Member
Joined: 4 years ago

I just noticed this problem also with IN clauses (as @jpotelle notes above) , for example IN (1,2,3,4,5)  is not formatted correctly but IN ('1','2','3','4','5') will be formatted correctly.  I'm just commenting here to get automatically notified of updates, thanks!


Reply
1 Reply
Guido
Admin
(@admin)
Joined: 5 years ago

Member
Posts: 295

@bbhulsey thanks for your feedback.

Did you alread gave the new option "Decimal point is Comma" a try and uncheck it?

grafik

Regards

Guido


Reply
Posts: 14
(@bbhulsey)
Member
Joined: 4 years ago

Hello @admin, I did not.  I was not aware of it.  I have changed the option and all works well now.  Thank you again!


Reply
Share: