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 |
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 |