Merge lp://staging/~magentoerpconnect-community/magentoerpconnect/955234 into lp://staging/magentoerpconnect/oerp6.1-oldstable

Status: Merged
Merge reported by: Guewen Baconnier @ Camptocamp
Merged at revision: not available
Proposed branch: lp://staging/~magentoerpconnect-community/magentoerpconnect/955234
Merge into: lp://staging/magentoerpconnect/oerp6.1-oldstable
Diff against target: 27 lines (+4/-2)
2 files modified
magentoerpconnect/product.py (+2/-0)
magentoerpconnect/sale.py (+2/-2)
To merge this branch: bzr merge lp://staging/~magentoerpconnect-community/magentoerpconnect/955234
Reviewer Review Type Date Requested Status
Guewen Baconnier @ Camptocamp Disapprove
Review via email: mp+97478@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Hi,

thanks for your patch proposition.

In fact, this is not a problem in product.product.export_inventory.
This method is responsible to export the stock of the input ids.

The cron and the button trigger the method export_inventory on the sale.shop and this one has the responsibility to call product.product.export_inventory with the set of ids exportable products of the shop.

So if a fix has to be done, that's on sale.shop.export_inventory, but this one is abstracted in base_sale_multichannels. So which branch do you use for base_sale_multichannels?

review: Disapprove
Revision history for this message
Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903) wrote :

Thanks for the explanation.

I use lp:on-change-suuport-extra-addons for the moment, but I will get back to lp:extra-addons once it's merged.

Will this module stay in this branch or moved to some other project?

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Btw the rest has to be merged, thanks

558. By Maxime Chambreuil (http://www.savoirfairelinux.com)

Remove previous commit

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.