Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.

Open erp v7 contacts issue


Published on

  • Be the first to comment

  • Be the first to like this

Open erp v7 contacts issue

  1. 1. What is wrong withcontacts management inOpenERP v7?(as of April 2013, 4 months after the release, hopefullywe may have fixed it when you will read that later)v 1.1by Raphaël Valyi
  2. 2. reminder: conceptual changein v7now res.partner and res.partner.address (contacts oraddresses) records all live in the res.partner table.Only spec published about this was: that we will use:● company: what was a res.partner is 6.1 (customer orsupplier commercial entity),● contacts: addresses or contacts of these commercialentities.
  3. 3. reminder: conceptual changein v7The main idea of putting everything in the same table was:● to make B2C easier● simplify menus and view definitions● be able to use both companies and contacts with thesame field/key when we can: ex to send a mail, selectthe contact of an invoice etc...Moving things in the same table was certainly good, theproblem is not the intent, but the execution that was onlyhalf simplify, we wont talk about physical persons for whichthere is little problem in v7 and focus on the issues withcompanies.
  4. 4. so whats the problem?the mistake originated because OpenERP SA thought thatbecause things are now in the same table, then only acontact field on business documents would be several many2one fields that were previously pointing toa company in 6.1 now accept a company OR a contact in7.For instance: partner_id on:an exhaustive list is being made here● invoice● purchase order● stock picking● crm claim● crm opportunity● project task...
  5. 5. 1) The problem of the contactrecord informationBut when you invoice a contact (on purpose or becauseOpenERP does it for you as when you use the CRM), theaccount module code will try to read fiscal position, partnerpayment term, credit limit and other such fields all from thecontact.This information is not expected to be maintained on everycontacts of a company of course!This problem also happens when writing data.This problem happens in many more modules/localizationsetc...
  6. 6. 1) proposed solution byOpenERP SAautomatically hard copy these data from the company toits contacts records with an insane duplication method.● data duplication is never a good thing● even properties (like invoice account) are duplicated,that means even DUPLICATING these propertyrecords...● this is totally incompatible with extensions likebase_contact where a single contact could belong toSEVERAL companies● OpenERP SA claims that its expected you have to fillpricelist and some other fields right on every contacts soyou have an idea what to expect "its user fault"...
  7. 7. 2) Reporting is deadwanted to compare● total invoiced by customer?● total purchased by supplier?● compare supplier product prices?● do your "intrastat" reports to declare where you soldyour goods?PROBLEMS:All these reports that were doing group_by and join overpartner_id as a company will now do it on mere contactsinstead. Unless you add a second company key andchange all these reports definitions, you cannot have suchreports in V7.
  8. 8. 2) proposed solution byOpenERP SAunf*ck just the invoice reporting by effectively adding a new"commercial_entity_id" key pointing again to the thecompanies, in a new module.PROBLEMS:● As for the other objects, dream on.● modules needing such a key may add their own andhack reports definitions/filters in overlappingincompatible ways.● hell we already had such a key before and it was calledpartner_id!
  9. 9. 3) the SQL cardinalitynightmareeverywhere we have some foreign key pointing to res.partner, it may be wrong! Fear not...3) a) domains nightmarenightmaresearch with (partner_id, =, some_id)some_id may now be the contact on a company while thecode was expecting to find a company
  10. 10. 3) a) domains nightmaresearch with (partner_id, =, some_id)for instance the code may create a new invoice if no invoiceis found for record with partner_id == some_id.PROBLEMS:● In fact an invoice may already exist for that companybut not with the same contact as some_id...● To fix that, we would need a NEW KEY to thecompany and then use a child_of operator.● shocking: we already had that company key and it wascalled partner_id
  11. 11. 3) a) domains nightmarePROBLEMS● that forces us to change the code in all official andcommunity addons to change such domains and usechild_of instead● and to use the parent company of the contact if some_idis a contact.● That also means slower code when the code had onlysome_id without browsing the object, like in anon_change function for instance.
  12. 12. 3) b) one2many andmany2many nightmarea code doingfor record in partner.related_records_ids:do_something_critical(...)PROBLEM:well, now some records that would have been included inthe loop in 6.1 will unexpectedly be missing in 7 if● their partner_id was set to a different contact of thesame company● OR if they point to the company while the partner is acontact of it.
  13. 13. 3) b) one2many andmany2many nightmareunf*cking that would imply getting partner parentcompany (ideally using a field pointing to it) anditerating over each of the records related to each of itscontacts and also the ones related to the company...Not really making the code any simpler...And it also means changing all the official and communityaddons code.shocking: as for a field pointing to the company of acontact, we had it already, it was called partner_id.
  14. 14. 3) c) many2one nightmaresuppose you deal with Return Material Authorization(RMA) in OpenERP user interface.You create a new return picking. You now want to relatethat picking to a possibly existing claim ticket of thatpartner or else create a new one.Typically, the ticket_id field will have a domain such as(partner_id, =, partner_id) in order to filter only thepossible tickets to relate.
  15. 15. 3) c) many2one nightmarePROBLEM:a claim ticket may already exist for the company but not thecontact you selected in your picking in partner_id.To fix it we would need a stored key to the companyand filter with a child_ofshocking: we already had such a key called partner_idOpenERP SAs answer to that is generally: the contact is theright "granularity"! Oh is it? Tell me, for what granularitywas the module built before when partner_id was acompany?
  16. 16. 3) d) unexpected: morecomplex access rulesimagine a rule like:sales users will see only the opportunities and orders oftheir own portfolio?PROBLEM:Well now you will need to make sure that every time a newcontact of customer is created, it will be related to the rightsalesman or else write much more complex rules.That is, its more complex to set up or else they will missdocuments unexpectedly
  17. 17. "we still need to make a few adjustments, buthey, look how gorgeous is our revolutionarymodel!"
  18. 18. CONCLUSION 11. ERP documents CANNOT make it without anSQL key pointing to the company. Despite one canusually infer the company from the contact, only thecontact isnt enough in practise.2. to be able to filter with that key before saving a record, itneeds to be set with an on_change, that is solutioninvades even view definitions.3. it cannot be a fields.related because it would be set onlywhen document is saved and isnt compatible with thecase where a contact could belong to several companies.
  19. 19. CONCLUSION 21. They arent telling you the truth when they denydocuments need a key to the company2. They arent telling you the truth if they finally admitsome documents like invoice may in fact need such key,but may be not all kind of documents.3. We already had that key: it was calledpartner_id4. During 8 years, hundreds of developers used partner_idas a key to the company, this is exactly what hundreds ofmodules are expecting.
  20. 20. CONCLUSION 41. Its not acceptable that we should live test in productionall modules designed for companies which now canreceive contacts randomly.2. Its not acceptable we should change all the semantic ofpartner_id everywhere when bugs will be discoveredand make it a pointer to a contact and thenprogressively adding a new key again pointing to thecompany again (that is swaping the semantic)??!??!??3. how many years will it take to fix all these hiddenfunctional bugs when it usually takes months to get atrivial regression fix on the official branches even whena patch is provided?
  21. 21. CONCLUSION 5what kind of chaos will we have with hundreds of modules:1. adding their own new keys to a company in overlappingincompatible on_changes (incompatibles viewdefinitions)2. missing the change and doing buggy things3. rejecting the change and assuming partner_id is still acompany and that the contact is carried by another key?
  22. 22. CONCLUSION 6A SOLUTION EXISTS!1. It consists in automatically adding a new fieldcontact_id to objects having a partner_id. Thenpartner_id is also automatically hidden in forms andonly contact_id is shown.2. User cannot see any visual difference with current v73. When contact_id is set, an on_change properly setspartner_id to the right company or physical person id.So partner_id is exactly what the code has always beenmade for.4. optionally or if user belongs to "advanced contactgroup", the user may edit both contact_id andpartner_id fields so that he can pick a company thatisnt the usual company of the contact, when a contactbelongs to several companies.
  23. 23. CONCLUSION 7BUTover the last month, OpenERP SA repeatedly rejected thatidea, even after 130 messages from the most experiencedOpenERP integrators in the world ALL rejecting OpenERPSA model and ALL agreeing on having two keys partner_idand contact_id instead. its now around an insurrection of around 200posts...
  24. 24. CONCLUSION 8Yeah, I think they actually f*~k her!But eventually we work out an unf*ck solution.See these links:●●●
  25. 25. CONCLUSION 9"Wait, after 130+ posts, we got the message, in fact wemight need a second field on an invoice":commercial_entity_id well, this would give us:* #sorrysapinvoice object contact companyv6.1 address_invoice_id partner_idv7 partner_id commercial_entity_id
  26. 26. ô rage, ô desespoir...
  27. 27. I tell you, "we simplified everything!"
  28. 28. and oh, fear not! the samecode is going to make it!our code, your code... Fear not, feel thepresence...
  29. 29. EPILOGUEIt took us ~130 posts to convince them the invoice neededtwo fields, may be they also understand in a few years wealso need that second field to the company on:● purchase orders● picking● analytic accounts● CRM opportunities● CRM claims● project tasks● purchase requisitions...So we may enjoy the field inversion for lots of businessobjects and all official and third party code using them...
  30. 30. But hey, dont worry, we will make a script...
  31. 31. And who said our offshore forces cannot do in a few months,what hundreds of experts did in 8 years...
  32. 32. "Raging Bull" - 1980Robert de Nirodirector: Martin Scorsese