Re: [pgadmin-hackers] pgAdmin III commit: Support the new check_option parameter of views
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. 140709_pgview.patch Description: Binary data -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers
Re: [pgadmin-hackers] pgAdmin III commit: Support the new check_option parameter of views
Hi, 2014-07-09 17:08 GMT+02:00 J.F. Oster jinfros...@mail.ru: 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. You're right. Thanks for the patch, I commited and pushed 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. -- Guillaume. http://blog.guillaume.lelarge.info http://www.dalibo.com
[pgadmin-hackers] pgAdmin III commit: Support the new check_option parameter of views
Support the new check_option parameter of views Branch -- master Details --- http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=178f27be48c6c1af970a104d590e4d3abaccdcc4 Modified Files -- CHANGELOG |1 + pgadmin/dlg/dlgView.cpp | 50 +++ pgadmin/include/schema/pgView.h | 11 - pgadmin/schema/pgView.cpp | 23 +- pgadmin/ui/dlgView.xrc | 23 -- 5 files changed, 99 insertions(+), 9 deletions(-) -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers