[FIX] get resource sale unit price from pricelist when get from default

[cialuo]

[FIX] get resource sale unit price from pricelist when get from default

•  File: business_requirement_deliverable/models/project.py:L15-25
• <img border=0 src='https://avatars.githubusercontent.com/u/7600613?v=3' height=16 width=16'> not sure this is enough.
  Rule should be for the pricelist:
  If pricelist in master project then use project.pricelist
  else if pricelist is project parent then use parent_project.pricelist
  else use partner.pricelist
• <img border=0 src='https://avatars.githubusercontent.com/u/7767792?v=3' height=16 width=16'> @elicoidal Seb told me that the rule had not been confirmed, So just let me keep the function and in other model can use this function.
• <img border=0 src='https://avatars.githubusercontent.com/u/7600613?v=3' height=16 width=16'> @seb-elico your input ?
•  File: business_requirement_deliverable_cost/models/business.py:L42-55
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Wrong. It should use get_estimation_pricelist (of the BR)
•  File: business_requirement_deliverable_cost/models/business.py:L42-55
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Move this before the if to use it in the condition. It should use get_estimation_pricelist (of the BR)
•  File: business_requirement_deliverable_cost/models/business.py:L65-78
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Same problems here
•  File: business_requirement_deliverable_default/models/business_requirement_deliverable_default.py:L8-28
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'>

`````` self.get_estimation_pricelist()```

•  File: business_requirement_deliverable/models/business.py:L239-256
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Should be br.get_estimation_pricelist()
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Should be br.get_estimation_pricelist()
•  File: business_requirement_deliverable/models/business.py:L184-198
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Should be br.get_estimation_pricelist()
•  File: business_requirement_deliverable/models/business.py:L184-198
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Move before the if to use in the condition
•  File: business_requirement_deliverable/models/business.py:L205-219
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Same here
•  General Comment
• <img border=0 src='https://avatars.githubusercontent.com/u/10879075?v=3' height=16 width=16'> Make sure that everywhere in the BR module(s), when you need the pricelist of a BR, you systematically use br.get_estimation_pricelist()

  

[elicoidal]

not sure this is enough.
Rule should be for the pricelist:
If pricelist in master project then use project.pricelist
else if pricelist is project parent then use parent_project.pricelist
else use partner.pricelist

[cialuo]

@elicoidal Seb told me that the rule had not been confirmed, So just let me keep the function and in other model can use this function.

[elicoidal]

@seb-elico your input ?

[seb-elico]

Make sure that everywhere in the BR module(s), when you need the pricelist of a BR, you systematically use br.get_estimation_pricelist()