Dave Page a écrit :
> Bullet pointed tips, purely because that's how they escaped from my
> brain :-)
> - This should be on the Tools menu, not the View menu.
I don't understand what you mean by this. I put them on the Tools
contextual menu but I don't see a View contextual menu. Can you give me
more details ?
> - Use a single factory, and a check/uncheck menu option. See the changes
> I made to the pg_stattuple stuff as an example of a check option.
> - The menu option should probably become something like 'Trigger enabled?'
> - pgTrigger::Enable/DisableTrigger should manage the state of the
> enabled flag themselves - you should not have to change that from the
OK. I discarded pgTrigger::Enable/DisableTrigger to use
pgTrigger::iSetEnabled. Everything takes place in this method.
> - If Enable/Disable trigger don't work for some reason, StartDialog
> should exit without doing anything else as a general rule.
> Otherwise it looks OK though - certainly looks like you've got the hang
> of the factories :-)
I added some code to make use of the "ENABLE/DISABLE TRIGGER ALL" on a
You'll find my new patch attached.
In response to
pgadmin-hackers by date
|Next:||From: svn||Date: 2007-01-19 08:51:39|
|Subject: SVN Commit by dpage: r5876 - trunk/www/download|
|Previous:||From: svn||Date: 2007-01-18 08:42:57|
|Subject: SVN Commit by dpage: r5875 - branches/REL-1_6_0_PATCHES/pgadmin3/src/include|