Beautifier and cursors

[mbo]

Hi,

Beautifier doesn't format cursors correctly.
You can have 500 characters (or more) on the line.

Is this a mistake on my config or a bug to be corrected ?

Declare
Cursor c_lgu(param1 In Number, param2 In Number, param3 In Number, param4 In Number, param5 In Number) Is
Select Null
From dual;
Begin
Null;
End;

I wish to obtain something like
Declare
Cursor c_lgu(
param1 In Number,
param2 In Number,
param3 In Number,
param4 In Number,
param5 In Number) Is
Select Null
From dual;
Begin
Null;
End;

regards
Mbo

 

[Marco Kalter]

That doesn't seem right. We'll fix it.

 

[Marco van der Linden]

In addition to that, would it be possible to have an allignment option for column aliases?

e.g.

instead of

select column1 column1_alias
      ,other_column column2_alias
....

have it look like

select column1      column1_alias
      ,other_column column2_alias
....

 

[Marco Kalter]

This option already exists, but only works if all columns have an alias.

 

[Marco van der Linden]

Marco, you are right, and not. The alignment of column aliases only works on 'singular' columns.

select invoice_num   invoice_num
,      invoice_currency_code    invoice_currency_code

becomes

select invoice_num           invoice_num
      ,invoice_currency_code invoice_currency_code

BUT

select invoice_num   invoice_num
,      upper(invoice_currency_code)  invoice_currency_code

becomes

select invoice_num invoice_num
      ,upper(invoice_currency_code) invoice_currency_code

As I frequently use more 'complex' expressions like decode and case in the select statements I never get the column aliases aligned

 

[Marco Kalter]

That is indeed a known restriction. Perhaps we can do something about it.

 

[Henk Jan]

In addition: the following construct does not beautify well:

cursor c_avy(b_id in number
                ,b_startdate in date
                ,b_enddate in date)
    is
      select end_date
      from   availability
      where  id = b_id
      and    (b_startdate between start_date and end_date or
              b_enddate between start_date and end_date)
    ;

results in:

cursor c_avy(b_id in number, b_startdate in date, b_enddate in date) is
      select end_date
      from   availability
      where  id = b_id
      and    (b_startdate between start_date and end_date or
            b_enddate between start_date and end_date);

As you can see the where clause is not done correctly.

And I would like to see some more control over de cursor statement. For instance I like to have the IS keyword be placed on a new line (by the way also for all other use of this keyword). An also use the settings for parameter lists for cursor statements. And the placement of the closing semi-column on a new line (only for cursor statements) would be a nice addition. This way the need for empty (comment) lines disappears.