fix: resize group images (#8273)

* fix: resize group images

* completing resize group banner image

Co-authored-by: maze-runner <[email protected]>
Co-authored-by: Sundaram Dubey <[email protected]>

Author: progmatic-99

app/api/groups.py

@@ -37,6 +37,10 @@ def before_create_object(self, data, view_kwargs):
             if not has_access('is_owner', event_id=event):
                 raise ForbiddenError({'source': ''}, "Event owner access required")
 
+    def after_create_object(self, group, data, view_kwargs):
+        if data.get('banner_url'):
+            start_image_resizing_tasks(group, data['banner_url'])
+
     schema = GroupSchema
     decorators = (jwt_required,)
     methods = [
@@ -47,6 +51,7 @@ def before_create_object(self, data, view_kwargs):
         'model': Group,
         'methods': {
             'before_create_object': before_create_object,
+            'after_create_object': after_create_object,
         },
     }
 
@@ -125,6 +130,12 @@ def before_update_object(self, group, data, view_kwargs):
             if not has_access('is_owner', event_id=event):
                 raise ForbiddenError({'source': ''}, "Event owner access required")
 
+        if (
+            data.get('banner_url')
+        ):
+            start_image_resizing_tasks(group, data['banner_url'])
+        
+
     decorators = (
         api.has_permission(
             'is_user_itself', methods="PATCH,DELETE", fetch="user_id", model=Group
@@ -158,3 +169,10 @@ class GroupRelationship(ResourceRelationship):
         'session': db.session,
         'model': Group,
     }
+
+
+def start_image_resizing_tasks(group, banner_url):
+    group_id = str(group.id)
+    from .helpers.tasks import resize_group_images_task
+
+    resize_group_images_task.delay(group_id, banner_url)

app/api/helpers/tasks.py

@@ -43,6 +43,7 @@
 from app.models.discount_code import DiscountCode
 from app.models.event import Event
 from app.models.exhibitor import Exhibitor
+from app.models.group import Group
 from app.models.order import Order
 from app.models.session import Session
 from app.models.speaker import Speaker
@@ -214,7 +215,6 @@ def resize_exhibitor_images_task(self, exhibitor_id, photo_url):
             photo_url, 'event-image', exhibitor_id, folder='exhibitors'
         )
         exhibitor.thumbnail_image_url = uploaded_images['thumbnail_image_url']
-        exhibitor.banner_url = uploaded_images['large_image_url']
         save_to_db(exhibitor)
         logging.info(
             f'Resized images saved successfully for exhibitor with id: {exhibitor_id}'
@@ -227,6 +227,30 @@ def resize_exhibitor_images_task(self, exhibitor_id, photo_url):
         )
 
 
+@celery.task(base=RequestContextTask, name='resize.group.images', bind=True)
+def resize_group_images_task(self, group_id, banner_url):
+    group = Group.query.get(group_id)
+    try:
+        logging.info(
+            'Group image resizing tasks started for group with id {}: {}'.format(
+                group_id, banner_url
+            )
+        )
+        uploaded_images = create_save_image_sizes(
+            banner_url, 'event-image', group.id
+        )
+
+        group.thumbnail_image_url = uploaded_images['thumbnail_image_url']
+        save_to_db(group)
+        logging.info(f'Resized images saved successfully for group with id: {group_id}')
+    except (requests.exceptions.HTTPError, requests.exceptions.InvalidURL, OSError):
+        logging.exception(
+            'Error encountered while generating resized images for group with id: {}'.format(
+                group_id
+            )
+        )
+
+
 @celery.task(base=RequestContextTask, name='resize.user.images', bind=True)
 def resize_user_images_task(self, user_id, original_image_url):
     user = safe_query(User, 'id', user_id, 'user_id')

app/api/schema/groups.py

@@ -33,6 +33,7 @@ class Meta:
     social_links = fields.Nested(GroupSocialLinkSchema, many=True)
     logo_url = fields.Url(allow_none=True)
     banner_url = fields.Url(allow_none=True)
+    thumbnail_image_url = fields.Url(dump_only=True)
     follower_count = fields.Integer(dump_only=True)
     about = fields.Str(allow_none=True)
 

app/models/group.py

@@ -21,6 +21,7 @@ class Group(SoftDeletionModel):
     social_links = db.Column(db.JSON)
     logo_url = db.Column(db.String)
     banner_url = db.Column(db.String)
+    thumbnail_image_url = db.Column(db.String)
     about = db.Column(db.Text)
     created_at: datetime = db.Column(db.DateTime(timezone=True), default=datetime.utcnow)
     modified_at: datetime = db.Column(

migrations/versions/rev-2021-10-06-18:48:55-ed141a4b57df_.py

@@ -0,0 +1,28 @@
+"""empty message
+
+Revision ID: ed141a4b57df
+Revises: 3a14893d21f0
+Create Date: 2021-10-06 18:48:55.752566
+
+"""
+
+from alembic import op
+import sqlalchemy as sa
+import sqlalchemy_utils
+
+
+# revision identifiers, used by Alembic.
+revision = 'ed141a4b57df'
+down_revision = '3a14893d21f0'
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.add_column('groups', sa.Column('thumbnail_image_url', sa.String(), nullable=True))
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_column('groups', 'thumbnail_image_url')
+    # ### end Alembic commands ###