Commit 7eed1c076936a8d9bcb7e4d409eafd6f0d800975
1 parent
ec6c233a
Exists in
master
and in
2 other branches
Now when the user is deleted, his profile picture is also deleted
Showing
1 changed file
with
6 additions
and
3 deletions
Show diff stats
users/views.py
@@ -30,7 +30,7 @@ from django.core.mail import EmailMessage | @@ -30,7 +30,7 @@ from django.core.mail import EmailMessage | ||
30 | from django.core.mail.backends.smtp import EmailBackend | 30 | from django.core.mail.backends.smtp import EmailBackend |
31 | 31 | ||
32 | from mailsender.models import MailSender | 32 | from mailsender.models import MailSender |
33 | - | 33 | +import os |
34 | #API IMPORTS | 34 | #API IMPORTS |
35 | from rest_framework import viewsets | 35 | from rest_framework import viewsets |
36 | from .serializers import UserSerializer | 36 | from .serializers import UserSerializer |
@@ -209,6 +209,8 @@ class DeleteView(braces_mixins.LoginRequiredMixin, LogMixin, generic.DeleteView) | @@ -209,6 +209,8 @@ class DeleteView(braces_mixins.LoginRequiredMixin, LogMixin, generic.DeleteView) | ||
209 | email = self.kwargs.get('email', None) | 209 | email = self.kwargs.get('email', None) |
210 | user = self.get_object() | 210 | user = self.get_object() |
211 | 211 | ||
212 | + | ||
213 | + | ||
212 | if email is None: | 214 | if email is None: |
213 | self.log_action = 'remove_account' | 215 | self.log_action = 'remove_account' |
214 | 216 | ||
@@ -233,11 +235,12 @@ class DeleteView(braces_mixins.LoginRequiredMixin, LogMixin, generic.DeleteView) | @@ -233,11 +235,12 @@ class DeleteView(braces_mixins.LoginRequiredMixin, LogMixin, generic.DeleteView) | ||
233 | redirect_url = redirect(error_url) | 235 | redirect_url = redirect(error_url) |
234 | else: | 236 | else: |
235 | self.log_context['dependencies'] = False | 237 | self.log_context['dependencies'] = False |
236 | - | 238 | + image_path_to_delete = user.image.path |
237 | user.delete() | 239 | user.delete() |
238 | 240 | ||
239 | messages.success(self.request, success_msg) | 241 | messages.success(self.request, success_msg) |
240 | - | 242 | + #deleting the user image |
243 | + os.remove(image_path_to_delete) | ||
241 | redirect_url = redirect(success_url) | 244 | redirect_url = redirect(success_url) |
242 | 245 | ||
243 | super(DeleteView, self).createLog(self.request.user, self.log_component, self.log_action, self.log_resource, self.log_context) | 246 | super(DeleteView, self).createLog(self.request.user, self.log_component, self.log_action, self.log_resource, self.log_context) |