Trunk : Total Injustice to the company 'name' !

[Serpent Consulting Services]

Guys,

I am not sure how critical this would be, but we faced this live on an implementation and was really a shocker.

There seems to be 2 issues, both the issues have same root!

Case 1. Try to duplicate the company record, although this should never go as a real case I guess. System would give you an error of not null constrain for the field 'name'. The field name is a fields.related and can usually fail in this case.

Proposed solution : copy() should be overridden and it can manage the rest.

Case 2. Keeping the field as related is fine, but injustice happens when the same name(from Partner model,feeling like a rich) when goes to name of the company, feels like filthy. In technical terms, the size of the name for partner_id appears to be 128 chars and name for company stands as 64 chars long. This results in "DataError: value too long for type character varying(64)" when the name of the partner doesn't really fit company name.

Proposed Solution : size for name field of company should be reset to 128.

Attached merges would pave the way for the solution.

Hope this helps.

Thanks,
Serpent Consulting Services.

[Serpent Consulting Services (serpent-consulting-services)]

To update, applying this proposal, would solve both the issues!

[Amit Parik (amit-parik)]

Hello,

I am confirming this issue for case#1 which is related to Integrity Error.

For case#2, I am not sure about the increasing the Size of the name related filed on res.comapny object because in res.partner object I have seen the same size, So I think it don't creates any problem.

@Framework Team:
You can take appropriate decision on case#2.

@Sepent Consulting Service:
I didn't agree with your comment#1 that your fixes fixed the both problem.
Correct me If I am wrong!.

Thank you!

[Vo Minh Thu (thu)]

Thanks for the bug report, case #2 is fixed in trunk:
revno: 3937
revision-id: <email address hidden>

[Serpent Consulting Services (serpent-consulting-services)]

Vo Minh Thu,

Thank you for the merge.