Message 22074

Author
albertca
Date
2015-08-05.15:38:33
Message id
22074

Content

Regarding the fact that multi-company should be a rare case: I see it is very frequent to have several companies. I've just checked with real cases and from the latest 14 customers we've got, 7 have more than one company. Of those 7, 2 of them have 2 companies just for fiscal reasons. In the other 4 cases there are:

- In two cases there are 2 companies doing the same activity
- In the other three cases there are +4 companies doing the same actiity

I can understand that we should try not to make things more complex if only one company is envolved, but I'd like to remark that I don't see it as such a strange case. Also, personally I don't think the o2m on party with a tab on "Company dependent fields" make things much more complex for users as the first company in form-view can be shown by default (just like addresses in party).

Regarding the API I must say that like Raimon I also see it a little bit complex to have a class per field. After all, in the Party model we'll have several fields that depend on company but probably no fields that will depend on warehouse or other parameters.

Also, probably working on the party module is not a very good example because IMHO both properties should be replaced by simple Many2One fields. I find myself thinking that the patch tries to plan that another module will extend the lang field and allows overriding the default value by adding a pattern but I think we cannot pretend that core modules are ready for those customizations out of the box. I mean, another module could make the default country company-dependent, for example, and we don't want to have to modify the core party module for that.

Could you update the patch including the account module (that includes company too)? Maybe that would make some things clearer for me...
History
Date User Action Args
2015-08-05 15:38:34albertcasetmessageid: <1438781914.96.0.402535549698.issue2349@tryton.org>
2015-08-05 15:38:34albertcasetrecipients: + ced, ohuisman, eric, resteve, pokoli, angel, guillemNaN, unicode2013
2015-08-05 15:38:34albertcalinkissue2349 messages
2015-08-05 15:38:33albertcacreate

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