Ownership of a new view

From: McAjvar <mcajvar(at)gmail(dot)com>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: Ownership of a new view
Date: 2011-06-15 15:34:39
Message-ID: BANLkTikyjPCZ72qFWbVVWakB2e_+Br5rbw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

I'm resending this mail as the first one seems to have gotten stuck due to
me messing up something with my email address.
***
Hi!

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

Regards,
Miha Radej

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2011-06-15 21:29:46 pgAdmin III commit: Fix the owner combobox
Previous Message Magnus Hagander 2011-06-15 11:06:14 Re: Branch 1.14?