Commit 1b4f8e9ce8dd948574f49c163754b4e78c70db67
1 parent
5d731a29
Exists in
master
and in
3 other branches
Adjusting category deletion
Showing
5 changed files
with
22 additions
and
10 deletions
Show diff stats
amadeus/static/css/base/amadeus.css
categories/templates/categories/list.html
@@ -16,7 +16,11 @@ | @@ -16,7 +16,11 @@ | ||
16 | {% if messages %} | 16 | {% if messages %} |
17 | {% for message in messages %} | 17 | {% for message in messages %} |
18 | <script type="text/javascript"> | 18 | <script type="text/javascript"> |
19 | - alertify.success('{{message}}'); | 19 | + {% if message.tags == "danger" %} |
20 | + alertify.error('{{message}}'); | ||
21 | + {% else %} | ||
22 | + alertify.success('{{message}}'); | ||
23 | + {% endif %} | ||
20 | </script> | 24 | </script> |
21 | {% endfor %} | 25 | {% endfor %} |
22 | {% endif %} | 26 | {% endif %} |
categories/views.py
1 | -from django.shortcuts import render, get_object_or_404 | 1 | +from django.shortcuts import render, get_object_or_404, redirect |
2 | from django.views.generic import ListView, CreateView, DeleteView, UpdateView | 2 | from django.views.generic import ListView, CreateView, DeleteView, UpdateView |
3 | from .models import Category | 3 | from .models import Category |
4 | from django.core.urlresolvers import reverse_lazy | 4 | from django.core.urlresolvers import reverse_lazy |
@@ -139,17 +139,20 @@ class DeleteCategory(LogMixin, DeleteView): | @@ -139,17 +139,20 @@ class DeleteCategory(LogMixin, DeleteView): | ||
139 | 139 | ||
140 | login_url = reverse_lazy("users:login") | 140 | login_url = reverse_lazy("users:login") |
141 | redirect_field_name = 'next' | 141 | redirect_field_name = 'next' |
142 | + | ||
142 | model = Category | 143 | model = Category |
143 | template_name = 'categories/delete.html' | 144 | template_name = 'categories/delete.html' |
144 | 145 | ||
145 | def delete(self, request, *args, **kwargs): | 146 | def delete(self, request, *args, **kwargs): |
146 | category = get_object_or_404(Category, slug = self.kwargs.get('slug')) | 147 | category = get_object_or_404(Category, slug = self.kwargs.get('slug')) |
147 | subjects = Subject.objects.filter(category = category) | 148 | subjects = Subject.objects.filter(category = category) |
149 | + | ||
150 | + print(self.request.META.get('HTTP_REFERER')) | ||
148 | 151 | ||
149 | if subjects.count() > 0: | 152 | if subjects.count() > 0: |
150 | - #objeto = category | ||
151 | - #messages.success(self.request, _('cannot delete Category "%s" ')%(objeto)) | ||
152 | - return reverse_lazy('categories:index') | 153 | + messages.error(self.request, _('The category cannot be removed, it contains one or more virtual enviroments attach.')) |
154 | + | ||
155 | + return redirect(self.request.META.get('HTTP_REFERER')) | ||
153 | 156 | ||
154 | return super(DeleteCategory, self).delete(self, request, *args, **kwargs) | 157 | return super(DeleteCategory, self).delete(self, request, *args, **kwargs) |
155 | 158 | ||
@@ -161,8 +164,8 @@ class DeleteCategory(LogMixin, DeleteView): | @@ -161,8 +164,8 @@ class DeleteCategory(LogMixin, DeleteView): | ||
161 | super(DeleteCategory, self).createLog(self.request.user, self.log_component, self.log_action, self.log_resource, self.log_context) | 164 | super(DeleteCategory, self).createLog(self.request.user, self.log_component, self.log_action, self.log_resource, self.log_context) |
162 | 165 | ||
163 | messages.success(self.request, _('Category removed successfully!')) | 166 | messages.success(self.request, _('Category removed successfully!')) |
164 | - return reverse_lazy('categories:index') | ||
165 | 167 | ||
168 | + return self.request.META.get('HTTP_REFERER') | ||
166 | 169 | ||
167 | class UpdateCategory(LogMixin, UpdateView): | 170 | class UpdateCategory(LogMixin, UpdateView): |
168 | log_component = 'category' | 171 | log_component = 'category' |
subjects/templates/subjects/list.html
@@ -20,10 +20,15 @@ | @@ -20,10 +20,15 @@ | ||
20 | {% if messages %} | 20 | {% if messages %} |
21 | {% for message in messages %} | 21 | {% for message in messages %} |
22 | <script type="text/javascript"> | 22 | <script type="text/javascript"> |
23 | - alertify.success('{{message}}'); | 23 | + {% if message.tags == "danger" %} |
24 | + alertify.error('{{message}}'); | ||
25 | + {% else %} | ||
26 | + alertify.success('{{message}}'); | ||
27 | + {% endif %} | ||
24 | </script> | 28 | </script> |
25 | {% endfor %} | 29 | {% endfor %} |
26 | - {% endif %} | 30 | + {% endif %} |
31 | + | ||
27 | <div id="core-subjects-options-div"> | 32 | <div id="core-subjects-options-div"> |
28 | <ul class="core-subjects-options"> | 33 | <ul class="core-subjects-options"> |
29 | {% if all %} | 34 | {% if all %} |
users/views.py
1 | -from django.shortcuts import get_object_or_404,redirect, render | 1 | +from django.shortcuts import get_object_or_404, redirect, render |
2 | from django.views import generic | 2 | from django.views import generic |
3 | from django.contrib import messages | 3 | from django.contrib import messages |
4 | from django.contrib.auth import authenticate, login as login_user | 4 | from django.contrib.auth import authenticate, login as login_user |