Tryton - Issues

 

Issue7178

Title Allow pool registration depending on activated modules
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type feature request Components trytond
Assigned To ced Keywords review
Reviews 37051002
View: 37051002

Created on 2018-02-28.16:11:51 by ced, last changed by roundup-bot.

Messages
New changeset 12ec4d123624 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/stock_shipment_measurements/rev/12ec4d123624
New changeset e64c35ee094f by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/stock_product_location/rev/e64c35ee094f
New changeset b27ca720c1dc by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/stock_location_move/rev/b27ca720c1dc
New changeset 59c738be341f by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/stock_consignment/rev/59c738be341f
New changeset 21445b6ae6d2 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/sale_shipment_cost/rev/21445b6ae6d2
New changeset e5a9223ad844 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/project_revenue/rev/e5a9223ad844
New changeset 619d85609398 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/product_price_list_dates/rev/619d85609398
New changeset 8418abb76f55 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/commission/rev/8418abb76f55
New changeset c3a28bb144a8 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/carrier_weight/rev/c3a28bb144a8
New changeset 4d9e163a9a8e by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/carrier_percentage/rev/4d9e163a9a8e
New changeset 74a8e32163f1 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/analytic_invoice/rev/74a8e32163f1
New changeset 94ec5418eb3c by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/account_tax_rule_country/rev/94ec5418eb3c
New changeset 6060afed2d25 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/account_payment_clearing/rev/6060afed2d25
New changeset cd51122c4b8e by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/account_payment/rev/cd51122c4b8e
New changeset cf68f2b90542 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/account_dunning_fee/rev/cf68f2b90542
New changeset 1876121918f8 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/account_credit_limit/rev/1876121918f8
New changeset 9e10bd1b584d by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/modules/account_asset/rev/9e10bd1b584d
New changeset 3d5aafbee665 by Cédric Krier in branch 'default':
Add depends on Pool.register
http://hg.tryton.org/trytond/rev/3d5aafbee665
review37051002 updated at https://codereview.tryton.org/37051002/#ps20001
New review37051002 at https://codereview.tryton.org/37051002/#ps1
msg38668 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-02-28.16:11:51
For now, we have the option to register mixin for a model that is defined in an extras_depend. For this case, we just ignore it silently.
But this is not enough to support all cases. Sometimes we want to be able to add a field on a model only if a module is activated because the field needs this module (ex: the target of a many2one).

So the proposal is to allow to register classes in the pool under a depends condition.
This way, we could remove the silent ignore of mixin registered without proper class. This will also make the registration process more explicit about the dependency.
History
Date User Action Args
2018-03-20 17:32:40roundup-botsetmessages: + msg39145
2018-03-20 17:32:23roundup-botsetmessages: + msg39144
2018-03-20 17:32:07roundup-botsetmessages: + msg39143
2018-03-20 17:31:48roundup-botsetmessages: + msg39142
2018-03-20 17:31:30roundup-botsetmessages: + msg39141
2018-03-20 17:31:12roundup-botsetmessages: + msg39140
2018-03-20 17:30:55roundup-botsetmessages: + msg39139
2018-03-20 17:30:38roundup-botsetmessages: + msg39138
2018-03-20 17:30:13roundup-botsetmessages: + msg39137
2018-03-20 17:29:58roundup-botsetmessages: + msg39136

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