Le 31/10/2010 00:39, Dave Page a écrit : > On Sun, Oct 31, 2010 at 1:56 AM, Guillaume Lelarge > <[email protected]> wrote: >> Le 30/10/2010 10:25, Dave Page a écrit : >> >>> Yeah, that's really nasty. I guess we need split the commands at ;. >> >> Yeah. If it's not between quotes. I don't like it at all, but I don't >> see another way of doing it. >> >>> I guess we should pass a flag down somehow to tell the function that >>> executes the query to do that and then we could also potentially get >>> rid of the double SQL boxes. I'm not looking at the code, but I >>> suspect that'll be nasty. >>> >> >> We actually aren't required to add such a flag. We can check if the >> query contains "ALTER TYPE", and "ADD AFTER" or "ADD BEFORE". > > That's knowledge I'd rather avoid hardwiring into the lower level > machinery here. >
So do I. I tried a few things yesterday. Changing the apply() and GetSql() parameters imply to change all GetSql for all dlg* source code. That will be quite an invasive patch. -- Guillaume http://www.postgresql.fr http://dalibo.com -- Sent via pgadmin-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers
