Refactor attachment field and add product_variant_id to product i…

…mage view. Consolidate `attachment` field definition for readability in `image_mixin.py`. Add `product_variant_id` field to the tree view in `product_image_views.xml` to improve functionality.
cbusillo · Jul 25, 2024 · eea4243 · eea4243
1 parent 0ae7a09
commit eea4243
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/product_connect/mixins/image_mixin.py b/product_connect/mixins/image_mixin.py
@@ -5,10 +5,8 @@ class ImageMixin(models.AbstractModel):
     _description = "Image Mixin"
     _inherit = "image.mixin"
 
-    attachment = fields.Many2one(
-        "ir.attachment", compute="_compute_attachment", store=True
-    )
-    image_1920_file_size = fields.Integer(related="attachment.file_size", store=True)
+    attachment = fields.Many2one("ir.attachment", compute="_compute_attachment", store=True)
+    image_1920_file_size = fields.Integer(related="attachment.file_size", store=True, group_operator="sum")
 
     @api.depends("attachment")
     def _compute_attachment(self) -> None:

diff --git a/product_connect/views/product_image_views.xml b/product_connect/views/product_image_views.xml
@@ -7,6 +7,7 @@
                 <field name="name"/>
                 <field name="image_1920_file_size"/>
                 <field name="image_1920" widget="image_upload"/>
+                <field name="product_tmpl_id"/>
                 <button name="action_open_full_image" type="object" string="View Full Image" class="btn btn-primary"/>
             </tree>
         </field>