OpenERP Community Backports (Addons)

Merge lp://staging/~numerigraphe-team/ocb-addons/7.0-filter-zero-inventory-analysis into lp://staging/ocb-addons

  1. 7.0-filter-zero-inventory-analysis
  2. Merge into 7.0
Proposed by Lionel Sausin - Initiatives/Numérigraphe
Status: Work in progress
Proposed branch: lp://staging/~numerigraphe-team/ocb-addons/7.0-filter-zero-inventory-analysis
Merge into: lp://staging/ocb-addons
Diff against target: 24 lines (+5/-2)
1 file modified
stock/report/report_stock_move_view.xml (+5/-2)
To merge this branch: bzr merge lp://staging/~numerigraphe-team/ocb-addons/7.0-filter-zero-inventory-analysis
Reviewer Review Type Date Requested Status
Raphaël Valyi - http://www.akretion.com Needs Information
Alexandre Fayolle - camptocamp Needs Fixing
Holger Brunn (Therp) code review Approve
Review via email: mp+210216@code.staging.launchpad.net

This proposal supersedes a proposal from 2014-03-07.

Description of the change

This branch adds 3 filters to the inventory analysis : available products, products out of stock, the products with negative stock
The branch targets v7.0 and another fix should be developed upstream in the new WMS code for v8.

To post a comment you must log in.
Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote : Posted in a previous version of this proposal

The help texts seem a bit weird to me. How about

'Products with available quantity greater than zero'

etc.?

review: Needs Fixing (code review)
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve (code review)
Reply
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

LGTM, but the corresponding MP on official addons is missing.

review: Needs Fixing
Reply
Revision history for this message
Lionel Sausin - Initiatives/Numérigraphe (ls-initiatives) wrote :

I'm not making an MP for v7.0 because it's an improvement rather than a bugfix.
I think I remember this was factored in in tunk-wms but I'll check again. If it isn't I'll withdraw this proposal and manage the branch on my own.

Reply
Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) wrote :

I like the improvement, but before approving, I would like to read an explanation what it becomes in v8 as I couldn't even find a report.stock.inventory on trunk-wms.

review: Needs Information
Reply

Unmerged revisions

8592. By Numérigraphe

[IMP] simplify wording of help texts

8591. By Numérigraphe

[IMP] Add search filters for products available, out of stock, and with negative stock in inventory analysis

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.