Re: pgAdmin III commit: Beautify the query shown in the SQL pane for the fu

From: "J(dot)F(dot) Oster" <jinfroster(at)mail(dot)ru>
To: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: pgAdmin III commit: Beautify the query shown in the SQL pane for the fu
Date: 2014-10-28 16:28:13
Message-ID: 1069775154.20141028192813@mail.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hello Ashesh,

Right, bringing related logic close together is much better. But there
is a small difference in your version. Per discussion held on
pgadmin-support,

>> ...
>> vertical formatting shouldn't be applied where
>> number of parameters is less than two.

DP> Right.

So we expect:

CREATE OR REPLACE FUNCTION simple_function(param1 integer)
RETURNS integer AS
$BODY$
select 1;
$BODY$
LANGUAGE sql ...

but get:

CREATE OR REPLACE FUNCTION simple_function(
param1 integer)
RETURNS integer AS ...

I suggest changing line 518 from:
else if (multiline)
to
else if (multiline && argTypesArray.Count() > 1)

Thanks.

Tuesday, October 28, 2014, 2:35:50 PM, you wrote:

AV> Beautify the query shown in the SQL pane for the functions having multiple
AV> arguments.

AV> Reviewed by Sanket Mehta and small modification done by Ashesh Vashi.

AV> Branch
AV> ------
AV> master

AV> Details
AV> -------
AV> http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=ea803ad68290bb506fd2148ee8ea8f4176e34197
AV> Author: J.F. Oster <jinfroster(at)mail(dot)ru>

AV> Modified Files
AV> --------------
AV> CHANGELOG | 3 +++
AV> pgadmin/include/schema/pgFunction.h | 2 +-
AV> pgadmin/schema/pgFunction.cpp | 12 +++++++++---
AV> 3 files changed, 13 insertions(+), 4 deletions(-)

--
Best regards,
J.F.

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Sanket Mehta 2014-10-28 16:29:34 Re: PATCH: Clear undo/redo history when a file is opened in Query editor
Previous Message J.F. Oster 2014-10-28 16:08:48 Re: PATCH: Clear undo/redo history when a file is opened in Query editor