Re: pgAdmin III commit: Support the new check_option parameter of views

From: "J(dot)F(dot) Oster" <jinfroster(at)mail(dot)ru>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: pgAdmin III commit: Support the new check_option parameter of views
Date: 2014-07-09 15:08:44
Message-ID: 1746697466.20140709190844@mail.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hello Guillaume,

I've noticed that views for which I had never specified check_option
or security_barrier (in pgsql 9.3) have got incorrect DDL generated -
the "WITH (options)" clause is empty:

CREATE OR REPLACE VIEW account_v WITH () AS
SELECT acc.id,
...

Once I set security_barrier on and back off for a view, it starts
showing "WITH (security_barrier=FALSE)", not an empty "WITH ()" until
I recreate a view. Probably that is the reason why the issue went
unnoticed during development.

Attached patch should fix it.

Wednesday, July 2, 2014, 7:47:42 PM, you wrote:

GL> Support the new check_option parameter of views

GL> Branch
GL> ------
GL> master

GL> Details
GL> -------
GL> http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=178f27be48c6c1af970a104d590e4d3abaccdcc4

GL> Modified Files
GL> --------------
GL> CHANGELOG | 1 +
GL> pgadmin/dlg/dlgView.cpp | 50
GL> +++++++++++++++++++++++++++++++++++----
GL> pgadmin/include/schema/pgView.h | 11 ++++++++-
GL> pgadmin/schema/pgView.cpp | 23 +++++++++++++++++-
GL> pgadmin/ui/dlgView.xrc | 23 ++++++++++++++++--
GL> 5 files changed, 99 insertions(+), 9 deletions(-)

--
Best regards,
J.F.

Attachment Content-Type Size
140709_pgview.patch application/octet-stream 586 bytes

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message J.F. Oster 2014-07-09 16:24:56 PATCH: new feature for Query tool: Favourites injection
Previous Message Dave Page 2014-07-09 11:16:57 Re: PATCH: More search in object's definitions