Tryton - Issues

 

Message31265

Author ced
Recipients pokoli, reviewbot
Date 2017-01-13.16:00:22
Content
I think the design should be simpler.
Here it is:

- register in the pool Mixin with a list of classes
- at the end of the load_module_graph:
  - the pool is called to loop over all the classes
  - test if it is a subclass of all the classes registered with the Mixin.
  - add the Mixin to the pool class

I think it is a much simpler design and there is no need to have the Mixin applied before having loaded all the modules. And if there are cases where it is needed, then the module can just explicitly extend the needed classes with the Mixin.
History
Date User Action Args
2017-01-13 16:00:22cedsetmessageid: <1484319622.47.0.427302619923.issue4735@tryton.org>
2017-01-13 16:00:22cedsetrecipients: + pokoli, reviewbot
2017-01-13 16:00:22cedlinkissue4735 messages
2017-01-13 16:00:22cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)