[11.0][MIG] mrp_repair_refurbish

mrp_repair_refurbish/README.rst

@@ -1,25 +1,46 @@
-.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
-   :target: https://www.gnu.org/licenses/agpl-3.0-standalone.html
-   :alt: License: AGPL-3
-
 ====================
 MRP Repair Refurbish
 ====================
 
+.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! This file is generated by oca-gen-addon-readme !!
+   !! changes will be overwritten.                   !!
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+
+.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
+    :target: https://odoo-community.org/page/development-status
+    :alt: Beta
+.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
+    :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+    :alt: License: AGPL-3
+.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fmanufacture-lightgray.png?logo=github
+    :target: https://github.com/OCA/manufacture/tree/11.0/mrp_repair_refurbish
+    :alt: OCA/manufacture
+.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
+    :target: https://translation.odoo-community.org/projects/manufacture-11-0/manufacture-11-0-mrp_repair_refurbish
+    :alt: Translate me on Weblate
+.. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png
+    :target: https://runbot.odoo-community.org/runbot/129/11.0
+    :alt: Try me on Runbot
+
+|badge1| |badge2| |badge3| |badge4| |badge5| 
+
 This module adds the ability to obtain refurbished product as a consequence
 of the repair of a product that was defective.
 
+**Table of contents**
+
+.. contents::
+   :local:
 
 Configuration
 =============
 
 #. For each product that can potentially be refurbished, define the
    refurbished version in the product form.
-
 #. A default 'Refurbished' virtual location is created during module install,
    and proposed by default on each product.
 
-
 Usage
 =====
 
@@ -36,43 +57,50 @@ destination location.
 The components that were added to the repair order will be moved to the
 'Refurbish' location.
 
-.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
-   :alt: Try me on Runbot
-   :target: https://runbot.odoo-community.org/runbot/129/9.0
-
-
 Bug Tracker
 ===========
 
-Bugs are tracked on `GitHub Issues
-<https://github.com/OCA/manufacture/issues>`_. In case of trouble, please
-check there if your issue has already been reported. If you spotted it first,
-help us smashing it by providing a detailed and welcomed `feedback
-<https://github.com/OCA/
-manufacture/issues/new?body=module:%20
-mrp_bom_dismantling%0Aversion:%20
-9.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+Bugs are tracked on `GitHub Issues <https://github.com/OCA/manufacture/issues>`_.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us smashing it by providing a detailed and welcomed
+`feedback <https://github.com/OCA/manufacture/issues/new?body=module:%20mrp_repair_refurbish%0Aversion:%2011.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+
+Do not contact contributors directly about support or help with technical issues.
 
 Credits
 =======
 
+Authors
+~~~~~~~
+
+* Eficent
+
 Contributors
-------------
+~~~~~~~~~~~~
 
 * Jordi Ballester Alomar <jordi.ballester@eficent.com>
 * Lois Rilo <lois.rilo@eficent.com>
 
-Maintainer
-----------
+Other credits
+~~~~~~~~~~~~~
+
+The initial development of this module has been financially supported by:
+
+* Aleph Objects, Inc.
+
+Maintainers
+~~~~~~~~~~~
+
+This module is maintained by the OCA.
 
 .. image:: https://odoo-community.org/logo.png
    :alt: Odoo Community Association
    :target: https://odoo-community.org
 
-This module is maintained by the OCA.
-
 OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
-To contribute to this module, please visit https://odoo-community.org.
+This module is part of the `OCA/manufacture <https://github.com/OCA/manufacture/tree/11.0/mrp_repair_refurbish>`_ project on GitHub.
+
+You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

mrp_repair_refurbish/__init__.py

@@ -1,5 +1 @@
-# -*- coding: utf-8 -*-
-# © 2016 Cyril Gaudin (Camptocamp)
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
-
 from . import models

mrp_repair_refurbish/__manifest__.py

@@ -1,10 +1,9 @@
-# -*- coding: utf-8 -*-
-# © 2016 Cyril Gaudin (Camptocamp)
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+# Copyright 2016-18 Eficent Business and IT Consulting Services S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
 {
     "name": "MRP Repair Refurbish",
     "summary": "Create refurbished products during repair",
-    "version": "9.0.1.0.1",
+    "version": "11.0.1.0.1",
     "category": "Manufacturing",
     "website": "https://github.com/OCA/manufacture",
     "author": "Eficent, Odoo Community Association (OCA)",

mrp_repair_refurbish/data/stock_data.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<openerp>
+<odoo>
     <data>
 
         <record id="stock_location_refurbish" model="stock.location">
@@ -11,6 +11,7 @@
         </record>
 
     </data>
+
     <data noupdate="1">
 
         <record forcecreate="True" id="property_stock_refurbish"
@@ -23,4 +24,4 @@
         </record>
 
     </data>
-</openerp>
+</odoo>

mrp_repair_refurbish/models/__init__.py

@@ -1,7 +1,3 @@
-# -*- coding: utf-8 -*-
-# © 2016 Cyril Gaudin (Camptocamp)
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
-
 from . import product_product
 from . import product_template
 from . import mrp_repair

mrp_repair_refurbish/models/mrp_repair.py

@@ -1,32 +1,13 @@
-# -*- coding: utf-8 -*-
-# © 2016 Cyril Gaudin (Camptocamp)
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+# Copyright 2017-18 Eficent Business and IT Consulting Services S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
 
-from openerp.osv import orm
-from openerp import api, fields, models
-
-
-class mrp_repair(orm.Model):
-    """To inherit using old api is needed here in order to be able to modify
-    the onchange method for `product_id`.
-    NOTE: This should be moved to new api in v10, when the standard is also
-    migrated.
-    """
-    _inherit = 'mrp.repair'
-
-    def onchange_product_id(self, cr, uid, ids, product_id=None):
-        res = super(mrp_repair, self).onchange_product_id(
-            cr, uid, ids, product_id=product_id)
-        product = self.pool['product.product'].browse(cr, uid, product_id)
-        res['value']['to_refurbish'] = True if \
-            product.refurbish_product_id else False
-        return res
+from odoo import api, fields, models
 
 
 class MrpRepair(models.Model):
     _inherit = 'mrp.repair'
 
-    to_refurbish = fields.Boolean('To refurbish')
+    to_refurbish = fields.Boolean()
     refurbish_location_dest_id = fields.Many2one(
         string='Refurbished Delivery Location', comodel_name='stock.location')
     refurbish_product_id = fields.Many2one(
@@ -35,7 +16,13 @@ class MrpRepair(models.Model):
         string='Refurbished Lot', comodel_name='stock.production.lot')
     refurbish_move_id = fields.Many2one(
         string='Refurbished Inventory Move', comodel_name='stock.move')
-    product_id = fields.Many2one(comodel_name='product.product')
+
+    @api.onchange('product_id')
+    def onchange_product_id(self):
+        res = super(MrpRepair, self).onchange_product_id()
+        self.to_refurbish = True if \
+            self.product_id.refurbish_product_id else False
+        return res
 
     @api.onchange('to_refurbish', 'product_id')
     def _onchange_to_refurbish(self):
@@ -48,49 +35,47 @@ class MrpRepair(models.Model):
             self.refurbish_product_id = False
             self.refurbish_location_dest_id = False
 
+    @api.multi
+    def _get_refurbish_stock_move_dict(self):
+        return {
+            'name': self.name,
+            'product_id': self.refurbish_product_id.id,
+            'product_uom': self.product_uom.id or
+            self.refurbish_product_id.uom_id.id,
+            'product_uom_qty': self.product_qty,
+            'partner_id': self.address_id and
+            self.address_id.id or False,
+            'location_id': self.location_dest_id.id,
+            'location_dest_id': self.refurbish_location_dest_id.id,
+            'restrict_lot_id': self.refurbish_lot_id.id,
+        }
+
     @api.multi
     def action_repair_done(self):
         res = super(MrpRepair, self).action_repair_done()
         for repair in self:
             if repair.to_refurbish:
-                move = self.env['stock.move'].create({
-                    'name': repair.name,
-                    'product_id': repair.refurbish_product_id.id,
-                    'product_uom': repair.product_uom.id or
-                    repair.refurbish_product_id.uom_id.id,
-                    'product_uom_qty': repair.product_qty,
-                    'partner_id': repair.address_id and
-                    repair.address_id.id or False,
-                    'location_id': repair.location_dest_id.id,
-                    'location_dest_id': repair.refurbish_location_dest_id.id,
-                    'restrict_lot_id': repair.refurbish_lot_id.id,
-                })
-                move.action_done()
+                move = self.env['stock.move'].create(
+                    repair._get_refurbish_stock_move_dict())
+                move.quantity_done = repair.product_qty
+                move._action_done()
                 repair.refurbish_move_id = move.id
         return res
 
 
-class mrp_repair_line(orm.Model):
-    """To inherit using old api is needed here in order to be able to modify
-    the onchange method for `type`.
-    NOTE: This should be moved to new api in v10, when the standard is also
-    migrated.
-    """
+class RepairLine(models.Model):
     _inherit = 'mrp.repair.line'
 
-    def onchange_operation_type(self, cr, uid, ids, type, guarantee_limit,
-                                company_id=False, context=None):
-        res = super(mrp_repair_line, self).onchange_operation_type(
-            cr, uid, ids, type, guarantee_limit, company_id=company_id,
-            context=context)
-
-        if (type == 'add' and 'to_refurbish' in context and
+    @api.onchange('type', 'repair_id')
+    def onchange_operation_type(self):
+        res = super(RepairLine, self).onchange_operation_type()
+        context = self.env.context
+        if (self.type == 'add' and 'to_refurbish' in context and
                 context['to_refurbish']):
-            res['value']['location_dest_id'] = context[
-                'refurbish_location_dest_id']
-        elif (type == 'add' and 'to_refurbish' in context and not
+            self.location_dest_id = context['refurbish_location_dest_id']
+        elif (self.type == 'add' and 'to_refurbish' in context and not
                 context['to_refurbish']):
-            scrap_location_ids = self.pool['stock.location'].search(cr, uid, [
-                ('usage', '=', 'customer')], context=context)
-            res['value']['location_dest_id'] = scrap_location_ids[0]
+            scrap_location_id = self.env['stock.location'].search([
+                ('usage', '=', 'customer')], limit=1)
+            self.location_dest_id = scrap_location_id
         return res

mrp_repair_refurbish/models/product_product.py

@@ -1,9 +1,7 @@
-# -*- coding: utf-8 -*-
-# © 2017 Eficent Business and IT Consulting Services S.L.
-# © 2016 Cyril Gaudin (Camptocamp)
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+# Copyright 2017-18 Eficent Business and IT Consulting Services S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
 
-from openerp import fields, models
+from odoo import fields, models
 
 
 class ProductProduct(models.Model):

mrp_repair_refurbish/models/product_template.py

@@ -1,8 +1,7 @@
-# -*- coding: utf-8 -*-
-# © 2016 Cyril Gaudin (Camptocamp)
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+# Copyright 2017-18 Eficent Business and IT Consulting Services S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
 
-from openerp import api, fields, models
+from odoo import api, fields, models
 
 
 class ProductTemplate(models.Model):
@@ -30,14 +29,13 @@ class ProductTemplate(models.Model):
         for template in unique_variants:
             template.refurbish_product_id = \
                 template.product_variant_ids.refurbish_product_id
-        for template in (self - unique_variants):
-            template.refurbish_product_id = False
 
-    @api.one
+    @api.multi
     def _set_refurbish_product(self):
-        if len(self.product_variant_ids) == 1:
-            self.product_variant_ids.refurbish_product_id = \
-                self.refurbish_product_id
+        for rec in self:
+            if len(rec.product_variant_ids) == 1:
+                rec.product_variant_ids.refurbish_product_id = \
+                    rec.refurbish_product_id
 
     def _search_refurbish_product(self, operator, value):
         products = self.env['product.product'].search([

mrp_repair_refurbish/readme/CONFIGURE.rst

@@ -0,0 +1,4 @@
+#. For each product that can potentially be refurbished, define the
+   refurbished version in the product form.
+#. A default 'Refurbished' virtual location is created during module install,
+   and proposed by default on each product.

mrp_repair_refurbish/readme/CONTRIBUTORS.rst

@@ -0,0 +1,2 @@
+* Jordi Ballester Alomar <jordi.ballester@eficent.com>
+* Lois Rilo <lois.rilo@eficent.com>

mrp_repair_refurbish/readme/CREDITS.rst

@@ -0,0 +1,3 @@
+The initial development of this module has been financially supported by:
+
+* Aleph Objects, Inc.

mrp_repair_refurbish/readme/DESCRIPTION.rst

@@ -0,0 +1,2 @@
+This module adds the ability to obtain refurbished product as a consequence
+of the repair of a product that was defective.

mrp_repair_refurbish/readme/USAGE.rst

@@ -0,0 +1,12 @@
+#. Create an MRP repair.
+#. Indicate a product to repair, If the product has a proposed refurbished
+   version, it will be proposed to be used. The destination location
+#. Add operations if needed.
+#. Complete the repair.
+
+The initial product will be moved to the 'Scrap' location, and the
+refurbished product will be moved from a 'Refurbish' location to the desired
+destination location.
+
+The components that were added to the repair order will be moved to the
+'Refurbish' location.

mrp_repair_refurbish/static/description/index.html

@@ -0,0 +1,455 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
+<head>
+<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+<meta name="generator" content="Docutils 0.14: http://docutils.sourceforge.net/" />
+<title>MRP Repair Refurbish</title>
+<style type="text/css">
+
+/*
+:Author: David Goodger (goodger@python.org)
+:Id: $Id: html4css1.css 7952 2016-07-26 18:15:59Z milde $
+:Copyright: This stylesheet has been placed in the public domain.
+
+Default cascading style sheet for the HTML output of Docutils.
+
+See http://docutils.sf.net/docs/howto/html-stylesheets.html for how to
+customize this style sheet.
+*/
+
+/* used to remove borders from tables and images */
+.borderless, table.borderless td, table.borderless th {
+  border: 0 }
+
+table.borderless td, table.borderless th {
+  /* Override padding for "table.docutils td" with "! important".
+     The right padding separates the table cells. */
+  padding: 0 0.5em 0 0 ! important }
+
+.first {
+  /* Override more specific margin styles with "! important". */
+  margin-top: 0 ! important }
+
+.last, .with-subtitle {
+  margin-bottom: 0 ! important }
+
+.hidden {
+  display: none }
+
+.subscript {
+  vertical-align: sub;
+  font-size: smaller }
+
+.superscript {
+  vertical-align: super;
+  font-size: smaller }
+
+a.toc-backref {
+  text-decoration: none ;
+  color: black }
+
+blockquote.epigraph {
+  margin: 2em 5em ; }
+
+dl.docutils dd {
+  margin-bottom: 0.5em }
+
+object[type="image/svg+xml"], object[type="application/x-shockwave-flash"] {
+  overflow: hidden;
+}
+
+/* Uncomment (and remove this text!) to get bold-faced definition list terms
+dl.docutils dt {
+  font-weight: bold }
+*/
+
+div.abstract {
+  margin: 2em 5em }
+
+div.abstract p.topic-title {
+  font-weight: bold ;
+  text-align: center }
+
+div.admonition, div.attention, div.caution, div.danger, div.error,
+div.hint, div.important, div.note, div.tip, div.warning {
+  margin: 2em ;
+  border: medium outset ;
+  padding: 1em }
+
+div.admonition p.admonition-title, div.hint p.admonition-title,
+div.important p.admonition-title, div.note p.admonition-title,
+div.tip p.admonition-title {
+  font-weight: bold ;
+  font-family: sans-serif }
+
+div.attention p.admonition-title, div.caution p.admonition-title,
+div.danger p.admonition-title, div.error p.admonition-title,
+div.warning p.admonition-title, .code .error {
+  color: red ;
+  font-weight: bold ;
+  font-family: sans-serif }
+
+/* Uncomment (and remove this text!) to get reduced vertical space in
+   compound paragraphs.
+div.compound .compound-first, div.compound .compound-middle {
+  margin-bottom: 0.5em }
+
+div.compound .compound-last, div.compound .compound-middle {
+  margin-top: 0.5em }
+*/
+
+div.dedication {
+  margin: 2em 5em ;
+  text-align: center ;
+  font-style: italic }
+
+div.dedication p.topic-title {
+  font-weight: bold ;
+  font-style: normal }
+
+div.figure {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+div.footer, div.header {
+  clear: both;
+  font-size: smaller }
+
+div.line-block {
+  display: block ;
+  margin-top: 1em ;
+  margin-bottom: 1em }
+
+div.line-block div.line-block {
+  margin-top: 0 ;
+  margin-bottom: 0 ;
+  margin-left: 1.5em }
+
+div.sidebar {
+  margin: 0 0 0.5em 1em ;
+  border: medium outset ;
+  padding: 1em ;
+  background-color: #ffffee ;
+  width: 40% ;
+  float: right ;
+  clear: right }
+
+div.sidebar p.rubric {
+  font-family: sans-serif ;
+  font-size: medium }
+
+div.system-messages {
+  margin: 5em }
+
+div.system-messages h1 {
+  color: red }
+
+div.system-message {
+  border: medium outset ;
+  padding: 1em }
+
+div.system-message p.system-message-title {
+  color: red ;
+  font-weight: bold }
+
+div.topic {
+  margin: 2em }
+
+h1.section-subtitle, h2.section-subtitle, h3.section-subtitle,
+h4.section-subtitle, h5.section-subtitle, h6.section-subtitle {
+  margin-top: 0.4em }
+
+h1.title {
+  text-align: center }
+
+h2.subtitle {
+  text-align: center }
+
+hr.docutils {
+  width: 75% }
+
+img.align-left, .figure.align-left, object.align-left, table.align-left {
+  clear: left ;
+  float: left ;
+  margin-right: 1em }
+
+img.align-right, .figure.align-right, object.align-right, table.align-right {
+  clear: right ;
+  float: right ;
+  margin-left: 1em }
+
+img.align-center, .figure.align-center, object.align-center {
+  display: block;
+  margin-left: auto;
+  margin-right: auto;
+}
+
+table.align-center {
+  margin-left: auto;
+  margin-right: auto;
+}
+
+.align-left {
+  text-align: left }
+
+.align-center {
+  clear: both ;
+  text-align: center }
+
+.align-right {
+  text-align: right }
+
+/* reset inner alignment in figures */
+div.align-right {
+  text-align: inherit }
+
+/* div.align-center * { */
+/*   text-align: left } */
+
+.align-top    {
+  vertical-align: top }
+
+.align-middle {
+  vertical-align: middle }
+
+.align-bottom {
+  vertical-align: bottom }
+
+ol.simple, ul.simple {
+  margin-bottom: 1em }
+
+ol.arabic {
+  list-style: decimal }
+
+ol.loweralpha {
+  list-style: lower-alpha }
+
+ol.upperalpha {
+  list-style: upper-alpha }
+
+ol.lowerroman {
+  list-style: lower-roman }
+
+ol.upperroman {
+  list-style: upper-roman }
+
+p.attribution {
+  text-align: right ;
+  margin-left: 50% }
+
+p.caption {
+  font-style: italic }
+
+p.credits {
+  font-style: italic ;
+  font-size: smaller }
+
+p.label {
+  white-space: nowrap }
+
+p.rubric {
+  font-weight: bold ;
+  font-size: larger ;
+  color: maroon ;
+  text-align: center }
+
+p.sidebar-title {
+  font-family: sans-serif ;
+  font-weight: bold ;
+  font-size: larger }
+
+p.sidebar-subtitle {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+p.topic-title {
+  font-weight: bold }
+
+pre.address {
+  margin-bottom: 0 ;
+  margin-top: 0 ;
+  font: inherit }
+
+pre.literal-block, pre.doctest-block, pre.math, pre.code {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+pre.code .ln { color: grey; } /* line numbers */
+pre.code, code { background-color: #eeeeee }
+pre.code .comment, code .comment { color: #5C6576 }
+pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
+pre.code .literal.string, code .literal.string { color: #0C5404 }
+pre.code .name.builtin, code .name.builtin { color: #352B84 }
+pre.code .deleted, code .deleted { background-color: #DEB0A1}
+pre.code .inserted, code .inserted { background-color: #A3D289}
+
+span.classifier {
+  font-family: sans-serif ;
+  font-style: oblique }
+
+span.classifier-delimiter {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+span.interpreted {
+  font-family: sans-serif }
+
+span.option {
+  white-space: nowrap }
+
+span.pre {
+  white-space: pre }
+
+span.problematic {
+  color: red }
+
+span.section-subtitle {
+  /* font-size relative to parent (h1..h6 element) */
+  font-size: 80% }
+
+table.citation {
+  border-left: solid 1px gray;
+  margin-left: 1px }
+
+table.docinfo {
+  margin: 2em 4em }
+
+table.docutils {
+  margin-top: 0.5em ;
+  margin-bottom: 0.5em }
+
+table.footnote {
+  border-left: solid 1px black;
+  margin-left: 1px }
+
+table.docutils td, table.docutils th,
+table.docinfo td, table.docinfo th {
+  padding-left: 0.5em ;
+  padding-right: 0.5em ;
+  vertical-align: top }
+
+table.docutils th.field-name, table.docinfo th.docinfo-name {
+  font-weight: bold ;
+  text-align: left ;
+  white-space: nowrap ;
+  padding-left: 0 }
+
+/* "booktabs" style (no vertical lines) */
+table.docutils.booktabs {
+  border: 0px;
+  border-top: 2px solid;
+  border-bottom: 2px solid;
+  border-collapse: collapse;
+}
+table.docutils.booktabs * {
+  border: 0px;
+}
+table.docutils.booktabs th {
+  border-bottom: thin solid;
+  text-align: left;
+}
+
+h1 tt.docutils, h2 tt.docutils, h3 tt.docutils,
+h4 tt.docutils, h5 tt.docutils, h6 tt.docutils {
+  font-size: 100% }
+
+ul.auto-toc {
+  list-style-type: none }
+
+</style>
+</head>
+<body>
+<div class="document" id="mrp-repair-refurbish">
+<h1 class="title">MRP Repair Refurbish</h1>
+
+<!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! This file is generated by oca-gen-addon-readme !!
+!! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
+<p><a class="reference external" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external" href="https://github.com/OCA/manufacture/tree/11.0/mrp_repair_refurbish"><img alt="OCA/manufacture" src="https://img.shields.io/badge/github-OCA%2Fmanufacture-lightgray.png?logo=github" /></a> <a class="reference external" href="https://translation.odoo-community.org/projects/manufacture-11-0/manufacture-11-0-mrp_repair_refurbish"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external" href="https://runbot.odoo-community.org/runbot/129/11.0"><img alt="Try me on Runbot" src="https://img.shields.io/badge/runbot-Try%20me-875A7B.png" /></a></p>
+<p>This module adds the ability to obtain refurbished product as a consequence
+of the repair of a product that was defective.</p>
+<p><strong>Table of contents</strong></p>
+<div class="contents local topic" id="contents">
+<ul class="simple">
+<li><a class="reference internal" href="#configuration" id="id1">Configuration</a></li>
+<li><a class="reference internal" href="#usage" id="id2">Usage</a></li>
+<li><a class="reference internal" href="#bug-tracker" id="id3">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="id4">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="id5">Authors</a></li>
+<li><a class="reference internal" href="#contributors" id="id6">Contributors</a></li>
+<li><a class="reference internal" href="#other-credits" id="id7">Other credits</a></li>
+<li><a class="reference internal" href="#maintainers" id="id8">Maintainers</a></li>
+</ul>
+</li>
+</ul>
+</div>
+<div class="section" id="configuration">
+<h1><a class="toc-backref" href="#id1">Configuration</a></h1>
+<ol class="arabic simple">
+<li>For each product that can potentially be refurbished, define the
+refurbished version in the product form.</li>
+<li>A default ‘Refurbished’ virtual location is created during module install,
+and proposed by default on each product.</li>
+</ol>
+</div>
+<div class="section" id="usage">
+<h1><a class="toc-backref" href="#id2">Usage</a></h1>
+<ol class="arabic simple">
+<li>Create an MRP repair.</li>
+<li>Indicate a product to repair, If the product has a proposed refurbished
+version, it will be proposed to be used. The destination location</li>
+<li>Add operations if needed.</li>
+<li>Complete the repair.</li>
+</ol>
+<p>The initial product will be moved to the ‘Scrap’ location, and the
+refurbished product will be moved from a ‘Refurbish’ location to the desired
+destination location.</p>
+<p>The components that were added to the repair order will be moved to the
+‘Refurbish’ location.</p>
+</div>
+<div class="section" id="bug-tracker">
+<h1><a class="toc-backref" href="#id3">Bug Tracker</a></h1>
+<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/manufacture/issues">GitHub Issues</a>.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us smashing it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/OCA/manufacture/issues/new?body=module:%20mrp_repair_refurbish%0Aversion:%2011.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<p>Do not contact contributors directly about support or help with technical issues.</p>
+</div>
+<div class="section" id="credits">
+<h1><a class="toc-backref" href="#id4">Credits</a></h1>
+<div class="section" id="authors">
+<h2><a class="toc-backref" href="#id5">Authors</a></h2>
+<ul class="simple">
+<li>Eficent</li>
+</ul>
+</div>
+<div class="section" id="contributors">
+<h2><a class="toc-backref" href="#id6">Contributors</a></h2>
+<ul class="simple">
+<li>Jordi Ballester Alomar &lt;<a class="reference external" href="mailto:jordi.ballester&#64;eficent.com">jordi.ballester&#64;eficent.com</a>&gt;</li>
+<li>Lois Rilo &lt;<a class="reference external" href="mailto:lois.rilo&#64;eficent.com">lois.rilo&#64;eficent.com</a>&gt;</li>
+</ul>
+</div>
+<div class="section" id="other-credits">
+<h2><a class="toc-backref" href="#id7">Other credits</a></h2>
+<p>The initial development of this module has been financially supported by:</p>
+<ul class="simple">
+<li>Aleph Objects, Inc.</li>
+</ul>
+</div>
+<div class="section" id="maintainers">
+<h2><a class="toc-backref" href="#id8">Maintainers</a></h2>
+<p>This module is maintained by the OCA.</p>
+<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
+<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/manufacture/tree/11.0/mrp_repair_refurbish">OCA/manufacture</a> project on GitHub.</p>
+<p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
+</div>
+</div>
+</div>
+</body>
+</html>

mrp_repair_refurbish/tests/__init__.py

@@ -0,0 +1 @@
+from . import test_mrp_repair_refurbish

mrp_repair_refurbish/tests/test_mrp_repair_refurbish.py

@@ -0,0 +1,91 @@
+# Copyright 2018 Eficent Business and IT Consulting Services S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from odoo.tests.common import TransactionCase
+
+
+class TestMrpMtoWithStock(TransactionCase):
+
+    def setUp(self, *args, **kwargs):
+        super(TestMrpMtoWithStock, self).setUp(*args, **kwargs)
+        self.repair_obj = self.env['mrp.repair']
+        self.repair_line_obj = self.env['mrp.repair.line']
+        self.product_obj = self.env['product.product']
+        self.move_obj = self.env['stock.move']
+
+        self.stock_location_stock = self.env.ref('stock.stock_location_stock')
+        self.customer_location = self.env.ref('stock.stock_location_customers')
+        self.refurbish_loc = self.env.ref(
+            'mrp_repair_refurbish.stock_location_refurbish')
+
+        self.refurbish_product = self.product_obj.create({
+            'name': 'Refurbished Awesome Screen',
+            'type': 'product',
+        })
+        self.product = self.product_obj.create({
+            'name': 'Awesome Screen',
+            'type': 'product',
+            'refurbish_product_id': self.refurbish_product.id,
+        })
+        self.material = self.product_obj.create({
+            'name': 'Materials',
+            'type': 'consu',
+        })
+        self._update_product_qty(self.product, self.stock_location_stock, 10.0)
+
+    def _update_product_qty(self, product, location, quantity):
+        product_qty = self.env['stock.change.product.qty'].create({
+            'location_id': location.id,
+            'product_id': product.id,
+            'new_quantity': quantity,
+        })
+        product_qty.change_product_qty()
+        return product_qty
+
+    def test_01_repair_refurbish(self):
+        """Tests that locations are properly set with a product to
+        refurbish, then complete repair."""
+        repair = self.repair_obj.create({
+            'product_id': self.product.id,
+            'product_qty': 3.0,
+            'product_uom': self.product.uom_id.id,
+            'location_dest_id': self.customer_location.id,
+        })
+        repair.onchange_product_id()
+        self.assertTrue(repair.to_refurbish)
+        repair._onchange_to_refurbish()
+        self.assertEqual(repair.refurbish_location_dest_id,
+                         self.customer_location)
+        self.assertEqual(repair.location_dest_id,
+                         self.product.property_stock_refurbish)
+        line = self.repair_line_obj.with_context(
+            to_refurbish=repair.to_refurbish,
+            refurbish_location_dest_id=repair.refurbish_location_dest_id,
+        ).new({
+            'name': 'consume stuff to repair',
+            'repair_id': repair.id,
+            'type': 'add',
+            'product_id': self.material.id,
+            'product_uom': self.material.uom_id.id,
+            'product_uom_qty': 1.0,
+        })
+        line.onchange_product_id()
+        line.onchange_operation_type()
+        self.assertEqual(line.location_id, repair.location_id)
+        self.assertEqual(line.location_dest_id, self.customer_location)
+        # Complete repair:
+        repair.action_validate()
+        repair.action_repair_start()
+        repair.action_repair_end()
+        moves = self.move_obj.search([('reference', '=', repair.name)])
+        self.assertEqual(len(moves), 2)
+        for m in moves:
+            self.assertEqual(m.state, 'done')
+            if m.product_id == self.product:
+                self.assertEqual(m.location_id, self.stock_location_stock)
+                self.assertEqual(m.location_dest_id, self.refurbish_loc)
+            elif m.product_id == self.refurbish_product:
+                self.assertEqual(m.location_id, self.refurbish_loc)
+                self.assertEqual(m.location_dest_id, self.customer_location)
+            else:
+                self.assertTrue(False, "Unexpected move.")