I really wish people would understand I didn't change anything here. I tried really hard to change nothing except allow choosing a partner instead of a pricelist in what was already existing functionality, but it seems community merges are impossible.
On Tue, Feb 11, 2014 at 12:41 AM, Graeme Gellatly <[email protected]>wrote: > Sorry, I correct myself, it is not even a functional field just a dummy > field. It adds precisely nothing to product model, > > > On Tue, Feb 11, 2014 at 12:38 AM, Graeme Gellatly <[email protected]>wrote: > >> it's just a functional field no different to pricelist which is in core >> of product. How is it different to core, I fail to see any distinction at >> all. >> >> >> On Mon, Feb 10, 2014 at 10:50 PM, Markus Schneider < >> [email protected]> wrote: >> >>> Hi Graeme, >>> >>> thank you for your replay. My suggestion was only that we think about >>> this topic and what was our solution. >>> >>> You are right that we can only see information about a product one by >>> one. So it can be helpfull to have a full list. Okay you can export a >>> complete pricelist as pdf. So the goal is to have a interactive list. >>> >>> Here is the point thinking about this solution i don't like. I try to >>> explain. >>> >>> Adding partner_id to the product is bad, becouse it adds context to the >>> product. In my understanding of good programming a basic modle like >>> product.product should be context free. So i suggest instead of having a >>> extension of product.product it would be better to add a reporting model >>> (which inherits from product.product) and than you can add the context >>> there like partner. >>> This has also a second benefied. As you see in practice customer >>> specific price changes on time. And you will have soon the problem that you >>> want to know, what was the price last year, or what will be the price next >>> year. That it is easyer to extend a reporting model instead adding this >>> context to the product.product model. >>> -- >>> >>> https://code.launchpad.net/~gdgellatly/openerp-product-attributes/partner-pricelist-7/+merge/202985 >>> You are the owner of >>> lp:~gdgellatly/openerp-product-attributes/partner-pricelist-7. >>> >> >> > -- https://code.launchpad.net/~gdgellatly/openerp-product-attributes/partner-pricelist-7/+merge/202985 Your team OpenERP Community is subscribed to branch lp:openerp-product-attributes. _______________________________________________ Mailing list: https://launchpad.net/~openerp-community Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-community More help : https://help.launchpad.net/ListHelp
