Re: SX event handler behaviour (was SX Projection Report/SX enable/disable)

2007-02-05 Thread Josh Sled
On Mon, 2007-02-05 at 01:42 -0500, Peter McAlpine wrote: > Please find the attached patch: > > - SX instance models properly handle SX update/add/remove events with/ > without enabled transactions > - New SX editors now show non-enabled transactions Looks good; r15510. It'd be great if you coul

Re: SX event handler behaviour (was SX Projection Report/SX enable/disable)

2007-02-04 Thread Peter McAlpine
Please find the attached patch: - SX instance models properly handle SX update/add/remove events with/ without enabled transactions - New SX editors now show non-enabled transactions Still on my 'enabled cleanup' todo list: - Add an 'enabled' checkbox column to the SX list dialog - Don't show

Re: SX event handler behaviour (was SX Projection Report/SX enable/disable)

2007-02-03 Thread Josh Sled
On Fri, 2007-02-02 at 22:54 -0500, Peter McAlpine wrote: > On 31-Jan-07, at 10:39 PM, Josh Sled wrote: > > > > '_gnc_sx_instance_event_handler' should be modified to ignore > > non-enabled SXes that it receives updates about; I've added a fixme in > > the code about it. > > Am I correct in that t

SX event handler behaviour (was SX Projection Report/SX enable/disable)

2007-02-02 Thread Peter McAlpine
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 We discussed some things via IRC but a couple questions remain... On 31-Jan-07, at 10:39 PM, Josh Sled wrote: > > '_gnc_sx_instance_event_handler' should be modified to ignore > non-enabled SXes that it receives updates about; I've added a fixme in >