Fixed pre-commit issues

README.md

@@ -22,6 +22,7 @@ Available addons
 addon | version | maintainers | summary
 --- | --- | --- | ---
 [g2p_bank](g2p_bank/) | 17.0.1.0.0 |  | G2P Registry: Bank Details
+[g2p_profile_image](g2p_profile_image/) | 17.0.1.0.0 |  | OpenG2P Profile Image
 [g2p_registry_addl_info](g2p_registry_addl_info/) | 17.0.1.0.0 |  | G2P Registry: Additional Info
 [g2p_registry_base](g2p_registry_base/) | 17.0.1.0.0 |  | G2P Registry: Base
 [g2p_registry_group](g2p_registry_group/) | 17.0.1.0.0 |  | G2P Registry: Groups

g2p_profile_image/README.rst

@@ -0,0 +1,52 @@
+=====================
+OpenG2P Profile Image
+=====================
+
+.. 
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! This file is generated by oca-gen-addon-readme !!
+   !! changes will be overwritten.                   !!
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! source digest: sha256:7bdf68b1fba30669d416a0928115edd95a9cfb1ee7091d4b593a0506ac530e20
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+
+.. |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/github-OpenG2P%2Fopeng2p--registry-lightgray.png?logo=github
+    :target: https://github.com/OpenG2P/openg2p-registry/tree/17.0-develop/g2p_profile_image
+    :alt: OpenG2P/openg2p-registry
+
+|badge1| |badge2|
+
+G2P Profile Image
+
+**Table of contents**
+
+.. contents::
+   :local:
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues <https://github.com/OpenG2P/openg2p-registry/issues>`_.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+`feedback <https://github.com/OpenG2P/openg2p-registry/issues/new?body=module:%20g2p_profile_image%0Aversion:%2017.0-develop%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
+~~~~~~~
+
+* OpenG2P
+
+Maintainers
+~~~~~~~~~~~
+
+This module is part of the `OpenG2P/openg2p-registry <https://github.com/OpenG2P/openg2p-registry/tree/17.0-develop/g2p_profile_image>`_ project on GitHub.
+
+You are welcome to contribute.

g2p_profile_image/__init__.py

@@ -1 +1 @@
-from . import models
+from . import models

g2p_profile_image/__manifest__.py

@@ -8,13 +8,13 @@
     "website": "https://openg2p.org",
     "license": "Other OSI approved licence",
     "depends": [
-        "g2p_social_registry",
+        "g2p_registry_base",
         "g2p_documents",
     ],
-    "external_dependencies": {},
-    "data": [
-       
-        ],
+    "external_dependencies": {
+        "python": ["pillow==9.0.1"],
+    },
+    "data": [],
     "assets": {},
     "demo": [],
     "images": [],

g2p_profile_image/models/__init__.py

@@ -1 +1 @@
-from . import profile_image,document_file
+from . import profile_image, document_file

g2p_profile_image/models/document_file.py

@@ -1,32 +1,36 @@
-from odoo import api, fields, models
+from odoo import _, models
 from odoo.exceptions import UserError
 
 
 class G2PDocumentFile(models.Model):
     _inherit = "storage.file"
-    def create(self,vals):
-        if type(vals) is dict:
+
+    def create(self, vals):
+        if isinstance(vals, dict):
             self._check_profile_tag(vals)
-        elif type(vals) is list:
+        elif isinstance(vals, list):
             for i in range(len(vals)):
                 vals_obj = vals[i]
                 self._check_profile_tag(vals_obj)
 
-
-
-        return super(G2PDocumentFile,self).create(vals)
+        return super().create(vals)
 
     def _check_profile_tag(self, vals_obj):
-        profile_tag = self.env['g2p.document.tag'].get_tag_by_name('Profile Image')
+        profile_tag = self.env["g2p.document.tag"].get_tag_by_name("Profile Image")
         profile_tag_id = profile_tag.id
-        tags_ids = vals_obj.get('tags_ids', [])
+        tags_ids = vals_obj.get("tags_ids", [])
         has_profile_tag = any(tag[1] == profile_tag_id for tag in tags_ids)
 
         if has_profile_tag:
-            existing_file = self.search([
-                ('registrant_id', '=', vals_obj.get('registrant_id')),
-                ('tags_ids', 'in', [profile_tag.id])
-            ], limit=1)
-            
+            existing_file = self.search(
+                [("registrant_id", "=", vals_obj.get("registrant_id")), ("tags_ids", "in", [profile_tag.id])],
+                limit=1,
+            )
+
             if existing_file:
-                raise UserError("Profile image already exists. To change your profile picture, hover over your current image and click the edit button.")
+                raise UserError(
+                    _(
+                        "Profile image already exists. To change your profile picture,"
+                        "hover over your current image and click the edit button."
+                    )
+                )

g2p_profile_image/models/profile_image.py

@@ -1,60 +1,48 @@
-from odoo import api, fields, models
 import base64
 import io
-import logging
+
 from PIL import Image
+
+from odoo import models
 from odoo.exceptions import ValidationError
-import time
 
-_logger = logging.getLogger(__name__)  # Set up a logger for your model
 
 class G2PImageStorage(models.Model):
     _inherit = "res.partner"
 
     def write(self, values):
-        profile_tag = self.env['g2p.document.tag'].get_tag_by_name('Profile Image')
+        profile_tag = self.env["g2p.document.tag"].get_tag_by_name("Profile Image")
         if not profile_tag:
-            profile_tag = self.env['g2p.document.tag'].create({'name': 'Profile Image'})
-        storage_file = self.env["storage.file"].search([
-            ('registrant_id', '=', self.id), 
-            ('tags_ids', 'in', [profile_tag.id])
-        ], limit=1)
-
-        backend_id = self.env["storage.backend"].search([('name', '=', 'Default S3 Document Store')], limit=1)
+            profile_tag = self.env["g2p.document.tag"].create({"name": "Profile Image"})
+        storage_file = self.env["storage.file"].search(
+            [("registrant_id", "=", self.id), ("tags_ids", "in", [profile_tag.id])], limit=1
+        )
 
+        backend_id = self.env["storage.backend"].search([("name", "=", "Default S3 Document Store")], limit=1)
 
         if values and values.get("image_1920") is not False:
             img = values.get("image_1920", False)
             if img:
                 binary_image = base64.b64decode(img)
                 image_size = len(binary_image)
-                _logger.info(f"Decoded image size: {image_size} bytes")
-
+
                 if image_size > 1 * 1024 * 1024:  # Image is > 1 MB
                     try:
-                        resize_start = time.time()
                         # Scale down the image to be less than 1 MB
                         scaled_down_image = self._resize_image(binary_image)
-                        resize_end = time.time()
-                        _logger.info(f"Image resize took: {resize_end - resize_start:.2f} seconds")
-
                         values["image_1920"] = scaled_down_image
-                        scaled_down_size = len(base64.b64decode(scaled_down_image))
-                        _logger.info(f"Scaled down image size: {scaled_down_size} bytes")
-
-
 
                     except Exception as e:
-                        raise ValidationError(f"Error: {e}")
-                    
+                        raise ValidationError(f"Error: {e}") from e
+
                     storage_file_vals = {
-                            "data": img,
-                            "file_size": image_size,
-                            "name": "Profile Image",
-                            "tags_ids": [(4, profile_tag.id)],
-                            "registrant_id": self.id,
-                            "backend_id": backend_id.id, 
-                        }
+                        "data": img,
+                        "file_size": image_size,
+                        "name": "Profile Image",
+                        "tags_ids": [(4, profile_tag.id)],
+                        "registrant_id": self.id,
+                        "backend_id": backend_id.id,
+                    }
 
                     if storage_file:
                         storage_file.unlink()
@@ -69,11 +57,10 @@ def write(self, values):
             if values and storage_file:
                 storage_file.unlink()
 
-        result = super(G2PImageStorage, self).write(values)
+        result = super().write(values)
         return result
 
     def _resize_image(self, binary_image):
-        start_time = time.time()
         image_stream = io.BytesIO(binary_image)
         image = Image.open(image_stream)
         image_format = image.format
@@ -82,23 +69,17 @@ def _resize_image(self, binary_image):
         # resizing with thumbnail
         image.thumbnail((1024, 1024), Image.ANTIALIAS)
 
-
         buffered = io.BytesIO()
         image.save(buffered, format=image_format, optimize=True, quality=85)
-        
+
         # size of the buffered image
         if buffered.tell() <= max_size:
-            end_time = time.time()
-            _logger.info(f"Total resize operation took: {end_time - start_time:.2f} seconds")
             return base64.b64encode(buffered.getvalue())
 
-
         for quality in range(80, 20, -10):
             buffered = io.BytesIO()
             image.save(buffered, format=image_format, optimize=True, quality=quality)
             if buffered.tell() <= max_size:
-                end_time = time.time()
-                _logger.info(f"Total resize operation took: {end_time - start_time:.2f} seconds")
                 return base64.b64encode(buffered.getvalue())
 
         # If quality reduction is not enough, reduce dimensions further
@@ -113,6 +94,4 @@ def _resize_image(self, binary_image):
             if buffered.tell() <= max_size:
                 break
 
-        end_time = time.time()
-        _logger.info(f"Total resize operation took: {end_time - start_time:.2f} seconds")
         return base64.b64encode(buffered.getvalue())

g2p_profile_image/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

g2p_profile_image/readme/DESCRIPTION.rst

@@ -0,0 +1 @@
+G2P Profile Image