[FIX] account_move_line_rma_order_line: remove security group group_a…

…ccount_move_rma_order_line + clean up commented code
ForgeFlow · May 9, 2024 · 5193293 · 5193293
1 parent 3247261
commit 5193293
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 22 deletions.
diff --git a/account_move_line_rma_order_line/__manifest__.py b/account_move_line_rma_order_line/__manifest__.py
@@ -10,9 +10,7 @@
     "category": "Generic",
     "depends": ["stock_account", "rma_account"],
     "license": "AGPL-3",
-    "data": [
-        "security/account_security.xml",
-    ],
+    "data": [],
     "installable": True,
     "maintainers": ["ChisOForgeFlow"],
     "development_status": "Beta",

diff --git a/account_move_line_rma_order_line/security/account_security.xml b/account_move_line_rma_order_line/security/account_security.xml
diff --git a/account_move_line_rma_order_line/tests/test_account_move_line_rma_order_line.py b/account_move_line_rma_order_line/tests/test_account_move_line_rma_order_line.py
@@ -97,7 +97,6 @@ def _create_account(cls, acc_type, name, code, company, reconcile=False):
     @classmethod
     def _create_product(cls):
         """Create a Product."""
-        #        group_ids = [group.id for group in groups]
         product_ctg = cls.product_ctg_model.create(
             {
                 "name": "test_product_ctg",
@@ -171,16 +170,8 @@ def _create_rma(cls, products2move, partner):
                     "partner_id": move.partner_id.id,
                 }
             )
-            # data = wizard._prepare_rma_line_from_stock_move(move)
             wizard.add_lines()
 
-            # CHECK ME: this code duplicate rma lines, what is the porpourse?
-            # if move.product_id.rma_customer_operation_id:
-            #     move.product_id.rma_customer_operation_id.in_route_id = False
-            # move.product_id.categ_id.rma_customer_operation_id = False
-            # move.product_id.rma_customer_operation_id = False
-            # wizard._prepare_rma_line_from_stock_move(move)
-            # cls.line = cls.rma_line_model.create(data)
         return rma_id
 
     def _get_balance(self, domain):