Re: [gnome-db] "ALTER USER", etc, with GdaSqlBuilder?

Murray Cumming <[email protected]>
Newsgroups gmane.comp.gnome.db
Message-ID <1274449422.31258.3.camel@murrayc-desktop>
On Thu, 2010-05-20 at 22:48 +0200, Vivien Malerba wrote:
> On 20 May 2010 21:42, Vivien Malerba <[email protected]> wrote:
> > On 20 May 2010 21:33, Murray Cumming <[email protected]> wrote:
> >> On Thu, 2010-05-20 at 17:40 +0200, Vivien Malerba wrote:
> >>> On 20 May 2010 13:52, Murray Cumming <[email protected]> wrote:
> >>> > Is there any way to use GdaSQlBuilder to do thinks like this, which Glom
> >>> > does with PostgreSQL?:
> >>> >
> >>> > ALTER USER "bob" PASSWORD "secret"
> >>> > CREATE USER "bob" PASSWORD "secret"
> >>> > DROP USER
> >>> > REVOKE ALL PRIVILEGES ON "invoices" FROM "bob"
> >>>
> >>> No. For this kind of statement (other than SELECT, UPDATE, INSERT or
> >>> DELETE) you can either directly use SQL
> >>
> >> Yes, we do that now.
> >>
> >>>  or use the GdaServerOperation
> >>> (though at the moment the GdaServerOperation does not support doing
> >>> any user manipulation, and it should be added, tell me if you would
> >>> like to have this in the 4.2).
> >>
> >> Thanks. Of course I'd like to have it, but it's not particularly urgent.
> >> It's just the last SQL-building code in Glom that we have not replaced
> >> with GdaSqlBuilder or GdaServerOperation. I guess that it would
> >> unnecessarily delay 4.2.
> >>
> >
> > Adding it is only a matter or declaring it in the
> > dta-server-operation.h file and implementing it for the PostgreSQL
> > provider (which is the one used by Glom), with a little documentation
> > along the way... should not be that hard.
> 
> Here is a patch implementing the CREATE USER operation for PostgreSQL
> (done in 30'). If you find it usefull I'll add the other operations
> for users as well before a 4.1.6. You can test it with the
> libgda-ui/demos/gdaui-demo-4.0 and the "DDL queries" demo item.

I really don't think I'll have time to test it quickly. Please do a
release, and then commit that to master. Then I'll try it out.

-- 
[email protected]
www.murrayc.com
www.openismus.com
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.