Commit 5d650d54b10d6248be826034129687d81295a9fc
1 parent
27bc1909
Exists in
master
and in
5 other branches
Update class name for edit user #7
Showing
2 changed files
with
6 additions
and
6 deletions
Show diff stats
users/urls.py
@@ -11,6 +11,6 @@ urlpatterns = [ | @@ -11,6 +11,6 @@ urlpatterns = [ | ||
11 | url(r'^profile/$', views.Profile.as_view(), name='profile'), | 11 | url(r'^profile/$', views.Profile.as_view(), name='profile'), |
12 | url(r'^profile/editar/(?P<username>[\w_-]+)/$', views.EditProfile.as_view(), name='edit_profile'), | 12 | url(r'^profile/editar/(?P<username>[\w_-]+)/$', views.EditProfile.as_view(), name='edit_profile'), |
13 | # | 13 | # |
14 | - url(r'^profile/update/$', views.UpdateUser.as_view(), name='update_profile'), | 14 | + url(r'^profile/update/$', views.UpdateProfile.as_view(), name='update_profile'), |
15 | url(r'^profile/delete/$', views.DeleteUser.as_view(), name='delete_profile'), | 15 | url(r'^profile/delete/$', views.DeleteUser.as_view(), name='delete_profile'), |
16 | ] | 16 | ] |
users/views.py
@@ -8,7 +8,7 @@ from django.core.urlresolvers import reverse_lazy | @@ -8,7 +8,7 @@ from django.core.urlresolvers import reverse_lazy | ||
8 | from django.utils.translation import ugettext_lazy as _ | 8 | from django.utils.translation import ugettext_lazy as _ |
9 | from rolepermissions.shortcuts import assign_role | 9 | from rolepermissions.shortcuts import assign_role |
10 | from .models import User | 10 | from .models import User |
11 | -from .forms import UserForm, ProfileForm, UpdateUserForm | 11 | +from .forms import UserForm, UpdateProfileForm, UpdateUserForm |
12 | 12 | ||
13 | # ================ ADMIN ======================= | 13 | # ================ ADMIN ======================= |
14 | class UsersListView(HasRoleMixin, LoginRequiredMixin, generic.ListView): | 14 | class UsersListView(HasRoleMixin, LoginRequiredMixin, generic.ListView): |
@@ -103,13 +103,13 @@ def delete(request,username): | @@ -103,13 +103,13 @@ def delete(request,username): | ||
103 | 103 | ||
104 | 104 | ||
105 | 105 | ||
106 | -class UpdateUser(LoginRequiredMixin, generic.edit.UpdateView): | 106 | +class UpdateProfile(LoginRequiredMixin, generic.edit.UpdateView): |
107 | 107 | ||
108 | allowed_roles = ['student'] | 108 | allowed_roles = ['student'] |
109 | login_url = reverse_lazy("core:home") | 109 | login_url = reverse_lazy("core:home") |
110 | template_name = 'users/edit_profile.html' | 110 | template_name = 'users/edit_profile.html' |
111 | - form_class = UpdateUserForm | ||
112 | - success_url = reverse_lazy('users:update_profile') | 111 | + form_class = UpdateProfileForm |
112 | + success_url = reverse_lazy('users:profile') | ||
113 | 113 | ||
114 | def get_object(self): | 114 | def get_object(self): |
115 | user = get_object_or_404(User, username = self.request.user.username) | 115 | user = get_object_or_404(User, username = self.request.user.username) |
@@ -119,7 +119,7 @@ class UpdateUser(LoginRequiredMixin, generic.edit.UpdateView): | @@ -119,7 +119,7 @@ class UpdateUser(LoginRequiredMixin, generic.edit.UpdateView): | ||
119 | form.save() | 119 | form.save() |
120 | messages.success(self.request, _('Profile edited successfully!')) | 120 | messages.success(self.request, _('Profile edited successfully!')) |
121 | 121 | ||
122 | - return super(UpdateUser, self).form_valid(form) | 122 | + return super(UpdateProfile, self).form_valid(form) |
123 | 123 | ||
124 | class DeleteUser(LoginRequiredMixin, generic.edit.DeleteView): | 124 | class DeleteUser(LoginRequiredMixin, generic.edit.DeleteView): |
125 | allowed_roles = ['student'] | 125 | allowed_roles = ['student'] |