search

gshank / dbix-class-resultset-recursiveupdate

DBIx::Class::ResultSet::RecursiveUpdate
4 stars 9 forks source link

Change to $rs->delete_all to trigger cascade actions #8

Closed cngarrison closed 10 years ago

abraxxa commented 10 years ago

This code change influences the modules behavior and performance too much as to accept it as is. It must be at least configurable and default to the current behavior.

cngarrison commented 10 years ago

Good morning,

On 12/02/14 at 6:23 AM -0800, Alexander Hartmaier notifications@github.com wrote:

This code change influences the modules behavior and performance too much as to accept it as is. It must be at least configurable and default to the current behavior.

I can make it configurable.

But aside from the how the DebugObject collected messages, I couldn't find any behavioural consequences of the change. The performance impact was (IMO) negligible, but of course other use cases will affect that.

Anyway, what behavioural changes should I be watching for?

Thanks, Charlie

 Make-Tracks Secure Hosting Pty Ltd  <http://www.make-tracks.com>
 PO Box 141                               <admin@make-tracks.com>
 Windsor NSW 2756  Australia                 Ph:  +61 2 9043 0613

 ** Take Control of Your Website **