add user profile photo and make it editable (#125)

* add user profile photo and make it editable

Author: mittal-ishaan

home/admin.py

@@ -12,6 +12,7 @@
 from import_export.admin import ImportExportMixin, ImportExportModelAdmin
 
 from home.models import (
+    SDC,
     About,
     Allocation,
     AllocationForm,
@@ -25,6 +26,7 @@
     LeftLongRebate,
     LeftShortRebate,
     LongRebate,
+    Menu,
     Period,
     Rebate,
     Rule,
@@ -34,8 +36,6 @@
     StudentBills,
     UnregisteredStudent,
     Update,
-    Menu,
-    SDC,
 )
 from home.utils.rebate_checker import max_days_rebate
 
@@ -181,6 +181,7 @@ def generate_table(self, request, queryset):
             )
             caterer_bill.save()
 
+
 @admin.register(Menu)
 class MenuAdmin(admin.ModelAdmin):
     model = Menu
@@ -203,6 +204,7 @@ def reset_menu_urls(self, request, queryset):
             menu.sheet_url = ""
             menu.save()
 
+
 @admin.register(Form)
 class about_Admin(admin.ModelAdmin):
     model = Form
@@ -277,6 +279,7 @@ class about_Admin(ImportExportMixin, admin.ModelAdmin):
             None,
             {
                 "fields": (
+                    "photo",
                     "name",
                     "email",
                     "roll_no",
@@ -930,6 +933,7 @@ def name(self, obj):
         return obj.email.name if obj.email else ""
 
     actions = ["export_as_csv", "fix_duplicates", "shift_caterer"]
+
     def export_as_csv(self, request, queryset):
         """
         Export action available in the admin page
@@ -1207,10 +1211,16 @@ class about_Admin(admin.ModelAdmin):
 @admin.register(SDC)
 class SDCAdmin(admin.ModelAdmin):
     model = SDC
-    search_fields = ("student__name", "student__roll_no", "student__email", "position", "year")
+    search_fields = (
+        "student__name",
+        "student__roll_no",
+        "student__email",
+        "position",
+        "year",
+    )
     list_display = ("student", "name", "position", "year")
     list_filter = ("position", "year")
-    autocomplete_fields = ["student"]  
+    autocomplete_fields = ["student"]
 
     fieldsets = (
         (
@@ -1234,13 +1244,14 @@ def export_as_csv(self, request, queryset):
         Export action available in the admin page
         """
         import csv
+
         from django.http import HttpResponse
 
         response = HttpResponse(content_type="text/csv")
         response["Content-Disposition"] = 'attachment; filename="SDC.csv"'
         writer = csv.writer(response)
         writer.writerow(["Student", "Name", "Position", "Year"])
-        
+
         for obj in queryset:
             writer.writerow([obj.student, obj.name, obj.position, obj.year])
 

home/migrations/0009_student_photo.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.0.8 on 2025-03-27 10:49
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('home', '0008_sdc'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='student',
+            name='photo',
+            field=models.ImageField(blank=True, default=None, help_text='This contains the photo of the Student', null=True, upload_to='photos/', verbose_name='Photo'),
+        ),
+    ]

home/models/students.py

@@ -8,6 +8,14 @@ class Student(models.Model):
     Stores Student Details
     """
 
+    photo = models.ImageField(
+        _("Photo"),
+        upload_to="photos/",
+        null=True,
+        blank=True,
+        default=None,
+        help_text="This contains the photo of the Student",
+    )
     name = models.CharField(
         _("Name of Student"),
         max_length=50,

home/views.py

@@ -11,6 +11,7 @@
 from django.utils.timezone import now
 
 from home.models import (
+    SDC,
     About,
     Allocation,
     AllocationForm,
@@ -21,6 +22,7 @@
     Form,
     LeftShortRebate,
     LongRebate,
+    Menu,
     Period,
     Rebate,
     Rule,
@@ -30,8 +32,6 @@
     StudentBills,
     UnregisteredStudent,
     Update,
-    Menu,
-    SDC,
 )
 
 from .utils.get_rebate_bills import get_rebate_bills
@@ -97,6 +97,7 @@ def rules(request):
 #     context = {"caterer": caterer}
 #     return render(request, "caterer.html", context)
 
+
 def menu(request):
     """
     Display the menu along with caterer information on a single page.
@@ -105,15 +106,13 @@ def menu(request):
     caterers = Caterer.objects.filter(visible=True)
 
     context = {
-        "menus": list(menus), 
+        "menus": list(menus),
         "caterers": caterers,
     }
 
     return render(request, "menu.html", context)
 
 
-
-
 def links(request):
     """
     Display the Forms Page :model:`home.models.links`.
@@ -477,12 +476,25 @@ def profile(request):
 
     :template:`home/profile.html`
     """
-    text = ""
     student = Student.objects.filter(email__iexact=str(request.user.email)).last()
+    if request.method == "POST" and request.user.is_authenticated:
+        if student:
+            try:
+                logger.debug(request.FILES)
+                file = request.FILES["profile-picture"]
+                student.photo = file
+                student.save()
+            except Exception as e:
+                logger.error(e)
+        if url_has_allowed_host_and_scheme(request.path, allowed_hosts=None):
+            return redirect(request.path)
+        else:
+            return redirect("/")
+    text = ""
     socialaccount_obj = SocialAccount.objects.filter(
         provider="google", user_id=request.user.id
     )
-    picture = "not available"
+    picture = student.photo.url if student.photo else None
     allocation: Allocation | None = Allocation.objects.filter(email=student).last()
     show_allocated_enabled = False
     if allocation and allocation.period:
@@ -501,12 +513,10 @@ def profile(request):
             "Jain": "Yes" if allocation.jain else "No",
         }
     try:
-        if socialaccount_obj:
+        if not picture and socialaccount_obj:
             picture = socialaccount_obj[0].extra_data["picture"]
-        else:
-            picture = "not available"
     except (IndexError, KeyError):
-        picture = "not available"
+        logger.error("No picture found")
     semesters = Semester.objects.all()
     context = {
         "text": text,
@@ -545,13 +555,12 @@ def rebate_data(request):
     return JsonResponse(rebate_data)
 
 
-
 def sdc_list(request):
     """
     View to display the list of SDC members categorized by year and position.
     """
-    positions = ["Head", "Advisory", "Member"] 
-    years = [2024, 2025] 
+    positions = ["Head", "Advisory", "Member"]
+    years = [2024, 2025]
     sdc_members = SDC.objects.all()
 
     context = {
@@ -560,4 +569,3 @@ def sdc_list(request):
         "sdc_members": sdc_members,
     }
     return render(request, "sdc.html", context)
-

qrscan/templates/mess_card.html

@@ -16,8 +16,8 @@
         <img
           class="img-fluid rounded-4"
           src="{{ picture }}"
-          alt="User Image"
           alt="profile photo"
+          style="width: 120px; height: 120px; object-fit: cover;"
         />
       </div>
       <h6 class="card-subtitle mt-2">{{ student.name }}</h6>

qrscan/views.py

@@ -1,9 +1,12 @@
-from django.utils import timezone
-from django.shortcuts import render
+from allauth.socialaccount.models import SocialAccount
 from django.contrib.auth.decorators import login_required
+from django.shortcuts import render
+from django.utils import timezone
+
 from home.models import Student
+
 from .models import MessCard
-from allauth.socialaccount.models import SocialAccount
+
 
 @login_required
 def mess_card(request):
@@ -21,20 +24,22 @@ def mess_card(request):
     )
     allocation = student.allocation_set.last()
 
-    if(not allocation):
+    if not allocation:
         raise ValueError("Allocation not found!")
     mess_card, _ = MessCard.objects.get_or_create(student=student)
 
-    if(not mess_card.allocation):
+    if not mess_card.allocation:
         setattr(mess_card, "allocation", allocation)
         mess_card.save()
-    elif((mess_card.allocation != allocation) and mess_card.allocation.period.end_date < timezone.localtime().date()):
+    elif (
+        mess_card.allocation != allocation
+    ) and mess_card.allocation.period.end_date < timezone.localtime().date():
         setattr(mess_card, "allocation", allocation)
         mess_card.save()
 
-    picture = "not available"
+    picture = student.photo.url if student.photo else None
     try:
-        if socialaccount_obj:
+        if not picture and socialaccount_obj.exists():
             picture = socialaccount_obj[0].extra_data["picture"]
     except (IndexError, KeyError):
         picture = "not available"
@@ -47,4 +52,3 @@ def mess_card(request):
     }
 
     return render(request, "mess_card.html", context=context)
-

templates/profile.html

@@ -13,7 +13,14 @@
                 <div class="card mt-4">
                     <div class="card-body" style="padding-left: 5%;">
                         <h5 class="card-title">Your Profile</h5>
-                        <img class="img-fluid rounded-4" src="{{ picture }}" alt="User Image">
+                        <img class="img-fluid rounded-4" src="{{ picture }}" alt="User Image" style="width: 90px; height: 90px; object-fit: cover;">
+                        <form action="" method="POST" enctype="multipart/form-data" style="margin-top: 10px;">
+                            {% csrf_token %}
+                            <div style="display: flex; align-items: center; gap: 10px;">
+                                <input type="file" name="profile-picture" id="profile-picture" class="form-control" accept="image/*" required>
+                                <button type="submit" class="btn btn-primary" style="background-color: rgb(30, 55, 146)">Upload</button>
+                            </div>
+                        </form>
                         <h6 class="card-subtitle mt-2">{{ student.name }}</h6>
                         <span class="card-text">Roll Number: {{ student.roll_no }}</span><br>
                         <span class="card-text">Email: {{ user.email }}</span><br>