Tuesday 06 July 2004 1:30:46 am
I see the new payment mechanism in pubsvn and was immediately struck with the decision to put the paypal module, tiny as it is, inside kernel/. I have two issues with this that are generic to all kernel/ modules.
1. It is impossible to override modules inside kernel/. This has been long standing issue that should be resolved soon. 2. Items like the paypal module are plugins and should not be given a full blown entry in kernel/. It cannot be overriden only replaced. Are you seriously going to start adding other plugin modules to kernel/? It will get messy very quickly... You should fix the extension system to allow modules to be overriden and develop these new features as extensions. Kernel/ should be used for generic code and not used for plugins in this way. Btw, what happened to defining handlers inside kernel/classes? Are you changing this? paul
-- http://www.visionwt.com
|