Tryton - Issues

 

Message40997

Author ced
Recipients pokoli, reviewbot
Date 2018-05-24.11:15:04
Content
On 2018-05-24 10:50, Sergi Almacellas Abellana wrote:
> > I think such domain should never depends on the context, so I would
> > also update other modules to follow this rule.
> 
> Current situation is that default values are not loaded if we do not
> depend on the context. 

In which case? For SaleLine, the _parent_sale.company should always be
available as sale is required. Idem for PurchaseLine. For InvoiceLine, I
think it should use first 'company' and if not set,
'_parent_invoice.company', one of those should always be set.

> I'm wondering if we should not remove the context dependency with
> issue4080 as we have to much dependency on the context now that is
> hard to test the code when the context is not set (as currently you do
> not have access to any company).

issue4080 will take a lot of time to be solved. And indeed I think we
should go progressively and this issue is one example of small steps
that will ease to fix issue4080.
History
Date User Action Args
2018-05-24 11:15:05cedsetrecipients: + pokoli, reviewbot
2018-05-24 11:15:05cedlinkissue7198 messages
2018-05-24 11:15:04cedcreate

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