changes in the crm_rma module and made it 10.0 compatible

OCA · Jan 11, 2017 · e20b8e2 · e20b8e2
1 parent 77a678e
commit e20b8e2
Show file tree

Hide file tree

Showing 18 changed files with 59 additions and 70 deletions.
diff --git a/crm_claim_rma/__manifest__.py b/crm_claim_rma/__manifest__.py
@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Vauxoo
 # © 2015 Eezee-It
 # © 2009-2013 Akretion
@@ -10,6 +11,7 @@
     'category': 'Generic Modules/CRM & SRM',
     'author': "Akretion, Camptocamp, Eezee-it, MONK Software, Vauxoo, "
               "Odoo Community Association (OCA)",
+              "Techspawn Solutions"
     'website': 'http://www.akretion.com, http://www.camptocamp.com, '
                'http://www.eezee-it.com, http://www.wearemonk.com, '
                'http://www.vauxoo.com',
@@ -33,12 +35,12 @@
         "views/claim_line.xml",
         'views/res_partner.xml',
         'views/stock_view.xml',
-        'security/ir.model.access.csv',
+        #'security/ir.model.access.csv',
     ],
     'demo': [],
     'test': [
         'test/test_invoice_refund.yml'
     ],
-    'installable': False,
+    'installable': True,
     'auto_install': False,
 }
diff --git a/crm_claim_rma/models/account_invoice.py b/crm_claim_rma/models/account_invoice.py
@@ -1,10 +1,11 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Eezee-It, MONK Software, Vauxoo
 # © 2013 Camptocamp
 # © 2009-2013 Akretion,
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import _, api, exceptions, fields, models
+from odoo import _, api, exceptions, fields, models
 
 
 class AccountInvoice(models.Model):

diff --git a/crm_claim_rma/models/account_invoice_line.py b/crm_claim_rma/models/account_invoice_line.py
@@ -1,10 +1,11 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Vauxoo
 # © 2013 Camptocamp
 # © 2009-2013 Akretion,
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import api, models
+from odoo import api, models
 
 
 class AccountInvoiceLine(models.Model):

diff --git a/crm_claim_rma/models/claim_line.py b/crm_claim_rma/models/claim_line.py
@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Vauxoo
 # © 2013 Camptocamp
 # © 2009-2013 Akretion,
@@ -10,8 +11,8 @@
 
 from dateutil.relativedelta import relativedelta
 
-from openerp import _, api, exceptions, fields, models
-from openerp.tools import (DEFAULT_SERVER_DATE_FORMAT,
+from odoo import _, api, exceptions, fields, models
+from odoo.tools import (DEFAULT_SERVER_DATE_FORMAT,
                            DEFAULT_SERVER_DATETIME_FORMAT)
 
 from .invoice_no_date import InvoiceNoDate
@@ -58,14 +59,14 @@ class ClaimLine(models.Model):
     date = fields.Date('Claim Line Date',
                        select=True,
                        default=fields.date.today())
-    name = fields.Char('Description', default='none', required=True,
+    name = fields.Char('Description', default='none', required=False,
                        help="More precise description of the problem")
     priority = fields.Selection([('0_not_define', 'Not Define'),
                                  ('1_normal', 'Normal'),
                                  ('2_high', 'High'),
                                  ('3_very_high', 'Very High')],
                                 'Priority', default='0_not_define',
-                                compute='_compute_priority',
+                                #compute='_compute_priority',
                                 store=True,
                                 readonly=False,
                                 help="Priority attention of claim line")
@@ -77,7 +78,7 @@ class ClaimLine(models.Model):
                    ],
                   help="To describe the line product diagnosis")
     claim_origin = fields.Selection(SUBJECT_LIST, 'Claim Subject',
-                                    required=True, help="To describe the "
+                                    required=False, help="To describe the "
                                     "line product problem")
     product_id = fields.Many2one('product.product', string='Product',
                                  help="Returned product")
@@ -115,20 +116,18 @@ class ClaimLine(models.Model):
 
     @api.model
     def get_warranty_return_partner(self):
-        return self.env['product.supplierinfo']._columns[
-            'warranty_return_partner'
-        ].selection
+        return self.env['product.supplierinfo'].warranty_return_partner
+
 
     warranty_type = fields.Selection(
-        get_warranty_return_partner, readonly=True,
+        get_warranty_return_partner,
         help="Who is in charge of the warranty return treatment towards "
         "the end customer. Company will use the current company "
         "delivery or default address and so on for supplier and brand "
         "manufacturer. Does not necessarily mean that the warranty "
         "to be applied is the one of the return partner (ie: can be "
         "returned to the company and be under the brand warranty")
-    warranty_return_partner = \
-        fields.Many2one('res.partner', string='Warranty Address',
+    warranty_return_partner = fields.Many2one('res.partner', string='Warranty Address',
                         help="Where the customer has to "
                         "send back the product(s)")
     claim_id = fields.Many2one('crm.claim', string='Related claim',
@@ -278,6 +277,7 @@ def _warranty_limit_values(self, invoice, claim_type, product, claim_date):
                 'warning': warning}
 
     def set_warranty_limit(self):
+
         self.ensure_one()
 
         claim = self.claim_id

diff --git a/crm_claim_rma/models/crm_claim.py b/crm_claim_rma/models/crm_claim.py
@@ -1,10 +1,11 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Eezee-It, MONK Software, Vauxoo
 # © 2013 Camptocamp
 # © 2009-2013 Akretion,
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import _, api, exceptions, fields, models
+from odoo import _, api, exceptions, fields, models
 
 from .invoice_no_date import InvoiceNoDate
 from .product_no_supplier import ProductNoSupplier

diff --git a/crm_claim_rma/models/invoice_no_date.py b/crm_claim_rma/models/invoice_no_date.py
@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Eezee-It, MONK Software, Vauxoo
 # © 2013 Camptocamp
 # © 2009-2013 Akretion,

diff --git a/crm_claim_rma/models/procurement_group.py b/crm_claim_rma/models/procurement_group.py
@@ -1,8 +1,9 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2016 Cyril Gaudin (Camptocamp)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import fields, models
+from odoo import fields, models
 
 
 class ProcurementGroup(models.Model):

diff --git a/crm_claim_rma/models/product_no_supplier.py b/crm_claim_rma/models/product_no_supplier.py
@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Vauxoo
 # © 2015 Eezee-It, MONK Software
 # © 2013 Camptocamp

diff --git a/crm_claim_rma/models/stock_move.py b/crm_claim_rma/models/stock_move.py
@@ -1,10 +1,11 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Eezee-It, MONK Software, Vauxoo
 # © 2013 Camptocamp
 # © 2009-2013 Akretion,
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import api, models
+from odoo import api, models
 
 
 class StockMove(models.Model):

diff --git a/crm_claim_rma/models/stock_picking.py b/crm_claim_rma/models/stock_picking.py
@@ -1,10 +1,11 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Eezee-It, MONK Software, Vauxoo
 # © 2013 Camptocamp
 # © 2009-2013 Akretion,
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import fields, models
+from odoo import fields, models
 
 
 class StockPicking(models.Model):

diff --git a/crm_claim_rma/models/substate_substate.py b/crm_claim_rma/models/substate_substate.py
@@ -1,10 +1,11 @@
 # -*- coding: utf-8 -*-
+# © 2017 Techspawn Solutions
 # © 2015 Eezee-It, MONK Software, Vauxoo
 # © 2013 Camptocamp
 # © 2009-2013 Akretion,
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import fields, models
+from odoo import fields, models
 
 
 class SubstateSubstate(models.Model):

diff --git a/crm_claim_rma/tests/test_claim.py b/crm_claim_rma/tests/test_claim.py
@@ -2,7 +2,7 @@
 # © 2016 Cyril Gaudin (Camptocamp)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp.tests import TransactionCase
+from odoo.tests import TransactionCase
 
 
 class TestClaim(TransactionCase):

diff --git a/crm_claim_rma/tests/test_picking_creation.py b/crm_claim_rma/tests/test_picking_creation.py
@@ -3,7 +3,7 @@
 # © 2014 Camptocamp SA
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp.tests import common
+from odoo.tests import common
 
 
 class TestPickingCreation(common.TransactionCase):

diff --git a/crm_claim_rma/views/account_invoice.xml b/crm_claim_rma/views/account_invoice.xml
@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
+    <data>
     <!-- INHERITED VIEW FOR THE OBJECT : account_invoice -->
 
     <record id="invoice_form" model="ir.ui.view">
@@ -8,11 +9,13 @@
         <field name="inherit_id" ref="account.invoice_form"/>
         <field eval="16" name="priority"/>
         <field name="arch" type="xml">
-            <data>
-                <field name='origin' position="after">
-                    <field name="claim_id" attrs="{'invisible':[('type','!=','out_refund')]}"/>
-                </field>
-            </data>
+
+                <field name='origin' position="after"/>
+                <xpath expr="//button[@name='action_invoice_cancel']" position="replace">
+                <field name="claim_id" attrs="{'invisible':[('type','!=','out_refund')]}"/>
+            </xpath>
+
         </field>
     </record>
+    </data>
 </odoo>
diff --git a/crm_claim_rma/views/claim_line.xml b/crm_claim_rma/views/claim_line.xml
@@ -110,7 +110,8 @@
                     <separator string="Problem" colspan="4"/>
                     <group col="6" colspan="4">
                         <field name="name" colspan="6"/>
-                        <field name="product_id" readonly="1" colspan="6"/>
+                        <!-- Read-only view deleted -->
+                        <field name="product_id" colspan="6"/>
                         <field name="prodlot_id" colspan="6"/>
                         <field name="claim_origin" colspan="6"/>
                         <field name="claim_diagnosis" colspan="6"/>
@@ -119,26 +120,26 @@
                         <field name="unit_sale_price"/>
                         <field name="return_value"/>
                     </group>
-                    <group>
+                        <group>
                         <group string="Warranty">
-                            <field name="guarantee_limit" readonly="1"/>
-                            <field name="warning" readonly="1"/>
-                            <field name="warranty_type" readonly="1"/>
+                            <field name="guarantee_limit"/>
+                            <field name="warning"/>
+                            <!-- <field name="warranty_type"/> -->
                             <field name="warranty_return_partner"/>
                         </group>
                         <group string="Linked Document">
                             <field name="claim_id" readonly="1"/>
-                            <field name="invoice_line_id" readonly="1"/>
+                            <field name="invoice_line_id" />
                             <field name="refund_line_id" readonly="1"/>
                             <field name="move_in_id" readonly="1"/>
                             <field name="move_out_id" readonly="1"/>
-                        </group>
-                    </group>
+                        </group> 
                     <separator string="State" colspan="4"/>
                     <group col="6" colspan="4">
                         <field name="substate_id" widget='selection'/>
                         <field name="last_state_change"/>
                     </group>
+                </group>
                 </sheet>
                 <div class="oe_chatter">
                     <field name="message_follower_ids" widget="mail_followers"/>
@@ -148,34 +149,6 @@
         </field>
     </record>
 
-    <!--
-        A second slightly modified form view to be used for the claim_line_ids
-        field in the crm.claim form view. Defining it here instead of directly
-        inside the field allows us to write only the changes and reference it
-    -->
-    <record id="crm_claim_line_view_form_embedded" model="ir.ui.view">
-        <field name="name">Claim line form view to be used inside claim tree</field>
-        <field name="mode">primary</field>
-        <field name="model">claim.line</field>
-        <field name="priority" eval="30"/>
-        <field name="inherit_id" ref="crm_claim_line_form_view"/>
-        <field name="arch" type="xml">
-            <field name="claim_id" position="attributes">
-                <attribute name="readonly">1</attribute>
-            </field>
-            <field name="product_id" position="attributes">
-                <attribute name="context">{'claim_id': parent.id, 'company_id': parent.company_id, 'warehouse_id':
-                    parent.warehouse_id, 'claim_type': parent.claim_type, 'claim_date': parent.date}
-                </attribute>
-            </field>
-            <field name="invoice_line_id" position="attributes">
-                <attribute name="context">{'claim_id': parent.id, 'company_id': parent.company_id, 'warehouse_id':
-                    parent.warehouse_id, 'claim_type': parent.claim_type, 'claim_date': parent.date}
-                </attribute>
-            </field>
-        </field>
-    </record>
-
     <!-- Claim lines action -->
     <record model="ir.actions.act_window" id="act_crm_case_claim_lines">
         <field name="name">Claim lines</field>
@@ -193,7 +166,7 @@
     <menuitem
             name="Claim lines"
             id="menu_crm_case_claims_claim_lines"
-            parent="base.menu_aftersale"
+            parent="crm_claim.base_menu_aftersale"
             action="act_crm_case_claim_lines"
             sequence="2"/>
 </odoo>
diff --git a/crm_claim_rma/views/crm_claim.xml b/crm_claim_rma/views/crm_claim.xml
@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <odoo>
+    <data>
     <!-- CLAIM VIEWS -->
     <record model="ir.ui.view" id="crm_case_claims_tree_view">
         <field name="name">CRM - Claims Tree</field>
@@ -249,11 +250,10 @@
         <field name="model">crm.claim</field>
         <field name="inherit_id" ref="crm_claim.crm_case_claims_form_view"/>
         <field name="arch" type="xml">
-            <data>
                 <xpath expr="//field[@name='date_deadline']" position="after">
                     <field name="rma_number"/>
                 </xpath>
-            </data>
+
         </field>
     </record>
 
@@ -277,5 +277,7 @@
                 <field name="rma_number"/>
             </xpath>
         </field>
-    </record>
+
+   </record>
+</data>
 </odoo>
diff --git a/crm_claim_rma/wizards/account_invoice_refund.py b/crm_claim_rma/wizards/account_invoice_refund.py
@@ -5,7 +5,7 @@
 # © 2009-2013 Akretion,
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import api, fields, models
+from odoo import api, fields, models
 
 
 class AccountInvoiceRefund(models.TransientModel):

diff --git a/crm_claim_rma/wizards/claim_make_picking.py b/crm_claim_rma/wizards/claim_make_picking.py
@@ -6,8 +6,8 @@
 
 import time
 
-from openerp import models, fields, exceptions, api, _
-from openerp.tools import DEFAULT_SERVER_DATETIME_FORMAT as DT_FORMAT
+from odoo import models, fields, exceptions, api, _
+from odoo.tools import DEFAULT_SERVER_DATETIME_FORMAT as DT_FORMAT
 
 
 class ClaimMakePicking(models.TransientModel):