{"action":"created","issue":{"url":"https://api.github.com/repos/OCA/partner-contact/issues/154","labels_url":"https://api.github.com/repos/OCA/partner-contact/issues/154/labels{/name}","comments_url":"https://api.github.com/repos/OCA/partner-contact/issues/154/comments","events_url":"https://api.github.com/repos/OCA/partner-contact/issues/154/events","html_url":"https://github.com/OCA/partner-contact/issues/154","id":100361703,"number":154,"title":"[8.0] partner_firstname - Error when deleting a res.partner","user":{"login":"rafaelbn","id":8736623,"avatar_url":"https://avatars.githubusercontent.com/u/8736623?v=3","gravatar_id":"","url":"https://api.github.com/users/rafaelbn","html_url":"https://github.com/rafaelbn","followers_url":"https://api.github.com/users/rafaelbn/followers","following_url":"https://api.github.com/users/rafaelbn/following{/other_user}","gists_url":"https://api.github.com/users/rafaelbn/gists{/gist_id}","starred_url":"https://api.github.com/users/rafaelbn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rafaelbn/subscriptions","organizations_url":"https://api.github.com/users/rafaelbn/orgs","repos_url":"https://api.github.com/users/rafaelbn/repos","events_url":"https://api.github.com/users/rafaelbn/events{/privacy}","received_events_url":"https://api.github.com/users/rafaelbn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2015-08-11T17:14:39Z","updated_at":"2015-09-08T11:47:38Z","closed_at":null,"body":"Tested in OCA's runbot\r\nhttps://runbot.odoo-community.org/runbot/repo/github-com-oca-partner-contact-134\r\n\r\n* Create a partner\r\n* Don't select \"is a company\"\r\n* Filled \"firstname\" and \"lastname\"\r\n* Save partner\r\n* Delete the partner we have created\r\n* You get this error:\r\n\r\n> AccessError\r\n> No value found for res.partner(75,).birthdate_date\r\n\r\nI have tested in Odoo's runbot and you can delete any res.partner if it hasn't any other relation with other objects.\r\n\r\nMaybe this bug it not from this module partner_contact_birthdate\r\n\r\n@pedrobaeza I think that any module in this project must at least pass test of creating, modifying and deleting a res.partner, if not shouldn't be merged.\r\n\r\ncc @sysadminmatmoz @Yajo \r\n"},"comment":{"url":"https://api.github.com/repos/OCA/partner-contact/issues/comments/138529692","html_url":"https://github.com/OCA/partner-contact/issues/154#issuecomment-138529692","issue_url":"https://api.github.com/repos/OCA/partner-contact/issues/154","id":138529692,"user":{"login":"Yajo","id":973709,"avatar_url":"https://avatars.githubusercontent.com/u/973709?v=3","gravatar_id":"","url":"https://api.github.com/users/Yajo","html_url":"https://github.com/Yajo","followers_url":"https://api.github.com/users/Yajo/followers","following_url":"https://api.github.com/users/Yajo/following{/other_user}","gists_url":"https://api.github.com/users/Yajo/gists{/gist_id}","starred_url":"https://api.github.com/users/Yajo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Yajo/subscriptions","organizations_url":"https://api.github.com/users/Yajo/orgs","repos_url":"https://api.github.com/users/Yajo/repos","events_url":"https://api.github.com/users/Yajo/events{/privacy}","received_events_url":"https://api.github.com/users/Yajo/received_events","type":"User","site_admin":false},"created_at":"2015-09-08T11:47:38Z","updated_at":"2015-09-08T11:47:38Z","body":"The point is that `partner_firstname` and `partner_contact_birthdate` work flawlessly unless `partner_relations` is installed. It fails when there's a computed field trying to set a value in the model. This is expected behavior, that raises a `MissingError`, which gets skipped by the ORM. But when `partner_relations` is installed, computed fields raise `AccessError`, which bubbles to the user and prevents deletion.\r\n\r\nLet me guess that any module adding a computed field to `res.partner` will fail.\r\n\r\nI'm *fixing* `partner_firstname` and `partner_contact_birthdate`, but I'm pretty sure the real bug is in `partner_relations`.\r\n\r\nDoes anybody know what is `partner_relations` doing? It's a very low-level module and I think it does something wrong, but after debugging for awhile I cannot find it."}}