pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FIX] *: incorrect many2many commands

Browse Source 
Those cases worked by chance until the former commit.
This commit is contained in:
Raphael Collet
2017-09-20 13:30:56 +02:00
parent 49f6a1b544
commit 8ffe241bee
5 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/hr_payroll/tests/common.py
+3 -1
View File
@@ -34,7 +34,9 @@ class TestPayslipBase(TransactionCase):
'name': 'Salary Structure for Software Developer',
'code': 'SD',
'company_id': self.ref('base.main_company'),
'rule_ids': [(4, [self.hra_rule_id, self.conv_rule_id, self.prof_tax_rule_id, self.pf_rule_id, self.mv_rule_id, self.comm_rule_id])]
'rule_ids': [(4, self.hra_rule_id), (4, self.conv_rule_id),
(4, self.prof_tax_rule_id), (4, self.pf_rule_id),
(4, self.mv_rule_id), (4, self.comm_rule_id)],
})
# I create a contract for "Richard"
addons/hr_payroll/tests/test_payslip_flow.py
+1 -1
View File
@@ -56,7 +56,7 @@ class TestPayslipFlow(TestPayslipBase):
# I create record for generating the payslip for this Payslip run.
payslip_employee = self.env['hr.payslip.employees'].create({
'employee_ids': [(4, self.richard_emp.ids)]
'employee_ids': [(4, self.richard_emp.id)]
})
# I generate the payslip by clicking on Generat button wizard.
addons/l10n_in_hr_payroll/tests/test_payment_advice_batch.py
+1 -1
View File
@@ -14,7 +14,7 @@ class TestPaymentAdviceBatch(TestPaymentAdviceBase):
# I create record for generating the Payslip for Payslip Batch.
payslip_employee = self.PayslipEmployee.create({
'employee_ids': [(4, self.rahul_emp.ids)]
'employee_ids': [(4, self.rahul_emp.id)]
})
# I generate the payslip by clicking on Generate button wizard.
addons/stock/tests/test_move.py
+5 -5
View File
@@ -1030,7 +1030,7 @@ class StockMove(TransactionCase):
})
move_stock_pack_1.write({'move_dest_ids': [(4, move_pack_cust.id, 0)]})
move_stock_pack_2.write({'move_dest_ids': [(4, move_pack_cust.id, 0)]})
move_pack_cust.write({'move_orig_ids': [(4, [move_stock_pack_1.id, move_stock_pack_2.id], 0)]})
move_pack_cust.write({'move_orig_ids': [(4, move_stock_pack_1.id, 0), (4, move_stock_pack_2.id, 0)]})
(move_stock_pack_1 + move_stock_pack_2 + move_pack_cust)._action_confirm()
@@ -1106,7 +1106,7 @@ class StockMove(TransactionCase):
})
move_stock_pack_1.write({'move_dest_ids': [(4, move_pack_cust.id, 0)]})
move_stock_pack_2.write({'move_dest_ids': [(4, move_pack_cust.id, 0)]})
move_pack_cust.write({'move_orig_ids': [(4, [move_stock_pack_1.id, move_stock_pack_2.id], 0)]})
move_pack_cust.write({'move_orig_ids': [(4, move_stock_pack_1.id, 0), (4, move_stock_pack_2.id, 0)]})
(move_stock_pack_1 + move_stock_pack_2 + move_pack_cust)._action_confirm()
@@ -1165,7 +1165,7 @@ class StockMove(TransactionCase):
'product_uom': self.uom_unit.id,
'product_uom_qty': 1.0,
})
move_stock_pack.write({'move_dest_ids': [(4, move_pack_cust_1.id, move_pack_cust_2.id, 0)]})
move_stock_pack.write({'move_dest_ids': [(4, move_pack_cust_1.id, 0), (4, move_pack_cust_2.id, 0)]})
move_pack_cust_1.write({'move_orig_ids': [(4, move_stock_pack.id, 0)]})
move_pack_cust_2.write({'move_orig_ids': [(4, move_stock_pack.id, 0)]})
@@ -1214,7 +1214,7 @@ class StockMove(TransactionCase):
'product_uom': self.uom_unit.id,
'product_uom_qty': 3.0,
})
move_stock_stock_1.write({'move_orig_ids': [(4, [move_supp_stock_1.id, move_supp_stock_2.id], 0)]})
move_stock_stock_1.write({'move_orig_ids': [(4, move_supp_stock_1.id, 0), (4, move_supp_stock_2.id, 0)]})
move_stock_stock_2 = self.env['stock.move'].create({
'name': 'test_link_assign_6_1',
'location_id': self.stock_location.id,
@@ -1223,7 +1223,7 @@ class StockMove(TransactionCase):
'product_uom': self.uom_unit.id,
'product_uom_qty': 3.0,
})
move_stock_stock_2.write({'move_orig_ids': [(4, [move_supp_stock_1.id, move_supp_stock_2.id], 0)]})
move_stock_stock_2.write({'move_orig_ids': [(4, move_supp_stock_1.id, 0), (4, move_supp_stock_2.id, 0)]})
(move_supp_stock_1 + move_supp_stock_2 + move_stock_stock_1 + move_stock_stock_2)._action_confirm()
move_supp_stock_1._action_assign()
addons/website_crm_partner_assign/security/ir_rule.xml
+2 -2
View File
@@ -4,7 +4,7 @@
<record id="assigned_lead_portal_rule_1" model="ir.rule">
<field name="name">Portal Graded Partner: read and write assigned leads</field>
<field name="model_id" ref="crm.model_crm_lead"/>
<field name="groups" eval="[(4, [ref('base.group_portal')])]"/>
<field name="groups" eval="[(4, ref('base.group_portal'))]"/>
<field name="domain_force">[('partner_assigned_id','child_of',user.commercial_partner_id.id)]</field>
<field name="perm_read" eval="True"/>
<field name="perm_write" eval="True"/>
@@ -16,7 +16,7 @@
<record id="res_partner_grade_rule_portal_public" model="ir.rule">
<field name="name">Portal/Public user: read only website published</field>
<field name="model_id" ref="website_crm_partner_assign.model_res_partner_grade"/>
<field name="groups" eval="[(4, [ref('base.group_portal')]), (4, [ref('base.group_public')])]"/>
<field name="groups" eval="[(4, ref('base.group_portal')), (4, ref('base.group_public'))]"/>
<field name="domain_force">[('website_published','=', True)]</field>
<field name="perm_read" eval="True"/>
</record>