From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | McAjvar <mcajvar(at)gmail(dot)com> |
Cc: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Ownership of a new view |
Date: | 2011-06-15 21:30:05 |
Message-ID: | 1308173406.1993.88.camel@laptop |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi,
On Wed, 2011-06-15 at 09:53 +0200, McAjvar wrote:
> [...]
> When creating a new view I've ran across some puzzling behaviour. I
> made a view as a superuser and assigned view ownership to another user
> who would be using it. Afterwards I was puzzled when I tried to select
> data from it and it wouldn't let me, saying the user does not have
> permission to do so. The view properties showed that indeed the
> superuser was the view owner instead of the user I chose.
>
> This happened with pgAdmin 1.10.2, 1.14.0 Beta 1 on Windows and I've
> built pgAdmin from source from a fresh checkout on Linux and observed
> the same.
>
> Upon checking I found out that when I was creating a new view that the
> group/role list does not have an empty row like other dialogs do
> (like, creating a new table, sequence, ...). So when I chose the first
> user (index 0) from the list, the generated SQL didn't reflect the
> change of user ownership, choosing any other user (index > 0) added
> the expected ALTER ... OWNER TO ... query.
>
> This change fixed my problem and brought the combo box in line with
> those in other dialogs:
>
> diff --git a/pgadmin/dlg/dlgView.cpp b/pgadmin/dlg/dlgView.cpp
> index cd82bca..78f1f5d 100644
> --- a/pgadmin/dlg/dlgView.cpp
> +++ b/pgadmin/dlg/dlgView.cpp
> @@ -57,6 +57,8 @@ pgObject *dlgView::GetObject()
>
> int dlgView::Go(bool modal)
> {
> + if (!view)
> + cbOwner->Append(wxEmptyString);
> AddGroups(cbOwner);
> AddUsers(cbOwner);
>
You're right. I've commited your fix with some changes (other objects
needed that too). Thanks a lot.
--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2011-06-15 21:38:55 | Re: Discussion - Search Objects |
Previous Message | Guillaume Lelarge | 2011-06-15 21:29:46 | pgAdmin III commit: Fix the owner combobox |