{"comment":{"url":"https://api.github.com/repos/alfredoavanzosc/odoomrp-wip-1/comments/11895121","html_url":"https://github.com/alfredoavanzosc/odoomrp-wip-1/commit/9848def3e3ed3518ccf69e9510e1d9b588fe7474#commitcomment-11895121","id":11895121,"user":{"login":"pedrobaeza","id":7165771,"avatar_url":"https://avatars.githubusercontent.com/u/7165771?v=3","gravatar_id":"","url":"https://api.github.com/users/pedrobaeza","html_url":"https://github.com/pedrobaeza","followers_url":"https://api.github.com/users/pedrobaeza/followers","following_url":"https://api.github.com/users/pedrobaeza/following{/other_user}","gists_url":"https://api.github.com/users/pedrobaeza/gists{/gist_id}","starred_url":"https://api.github.com/users/pedrobaeza/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pedrobaeza/subscriptions","organizations_url":"https://api.github.com/users/pedrobaeza/orgs","repos_url":"https://api.github.com/users/pedrobaeza/repos","events_url":"https://api.github.com/users/pedrobaeza/events{/privacy}","received_events_url":"https://api.github.com/users/pedrobaeza/received_events","type":"User","site_admin":false},"position":8,"line":9,"path":"sale_promotion_gift/models/gift_product.py","commit_id":"9848def3e3ed3518ccf69e9510e1d9b588fe7474","created_at":"2015-06-27T10:43:35Z","updated_at":"2015-06-27T10:43:35Z","body":"Alfredo, una cosa son las etiquetas del interfaz, y otra cosa los nombres técnicos de los objetos. Yo me refería únicamente a la palabra promotion en las etiquetas, y cuando he visto que la has cambiado también en el modelo, ya me he dado cuenta que el nombre técnico no seguía las reglas habituales de área + modelo + función (aunque aquí es más difícil de definir esa \"función\"). El tema es que no tengamos objetos con nombres cortos sueltos y sin relación con dónde se utiliza. En el propio Odoo tenemos ya un ejemplo de mal uso, con los objetos payment.order, payment.mode y demás. A ti qué te dice más, promotion, o sale.promotion? Y qué pasaría si utilizas promotion y mañana tienes promociones de compras? Por eso te decía de renombrar.\r\n\r\nNo es que cambie de opinión, es un refinamiento sucesivo conforme me doy cuenta de las cosas, aunque me gustaría que todo esto siempre lo tuvieras en cuenta de partida, porque en la revisión se me pasan cosas."}}