Merge lp:~camptocamp/margin-analysis/7.0-product_price_history-index-1301847 into lp:~margin-analysis-core-editors/margin-analysis/7.0
Proposed by
Guewen Baconnier @ Camptocamp
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 63 | ||||
Proposed branch: | lp:~camptocamp/margin-analysis/7.0-product_price_history-index-1301847 | ||||
Merge into: | lp:~margin-analysis-core-editors/margin-analysis/7.0 | ||||
Diff against target: |
26 lines (+12/-1) 1 file modified
product_price_history/product_price_history.py (+12/-1) |
||||
To merge this branch: | bzr merge lp:~camptocamp/margin-analysis/7.0-product_price_history-index-1301847 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Yannick Vaucher @ Camptocamp | code review, no test | Approve | |
Pedro Manuel Baeza | code review | Approve | |
Review via email: mp+214001@code.launchpad.net |
Commit message
Add index on the columns that are used in the SELECT everytime a price is fetched
Description of the change
fix lp:1301847
Details on the bug report.
That also removes a TODO!
To post a comment you must log in.
Can't this be done with select=1 attribute?
Regards.