diff --git a/circle/dashboard/models.py b/circle/dashboard/models.py index 5104c31..9615f16 100644 --- a/circle/dashboard/models.py +++ b/circle/dashboard/models.py @@ -54,7 +54,9 @@ from .validators import connect_command_template_validator logger = getLogger(__name__) -pwgen = User.objects.make_random_password + +def pwgen(): + return User.objects.make_random_password() class Favourite(Model): diff --git a/circle/dashboard/templates/dashboard/operate.html b/circle/dashboard/templates/dashboard/operate.html index 1d5b240..173d8ef 100644 --- a/circle/dashboard/templates/dashboard/operate.html +++ b/circle/dashboard/templates/dashboard/operate.html @@ -20,7 +20,7 @@ Do you want to perform the following operation on <a class="btn btn-default" href="{{object.get_absolute_url}}" data-dismiss="modal">{% trans "Cancel" %}</a> <button class="btn btn-{{ opview.effect }} btn-op-form-send" type="submit" id="op-form-send"> - {% if opview.icon %}<i class="fa fa-fw fa-{{opview.icon}}"></i> {% endif %}{{ op|capfirst }} + {% if opview.icon %}<i class="fa fa-fw fa-{{opview.icon}}"></i> {% endif %}{{ op.name|capfirst }} </button> </div> </form> diff --git a/circle/dashboard/views/graph.py b/circle/dashboard/views/graph.py index 472ea1e..a590762 100644 --- a/circle/dashboard/views/graph.py +++ b/circle/dashboard/views/graph.py @@ -62,7 +62,7 @@ class GraphViewBase(LoginRequiredMixin, View): metric = self.create_class(metric)(instance) return HttpResponse(metric.get_graph(graphite_url, time), - mimetype="image/png") + content_type="image/png") def get_object(self, request, pk): instance = self.model.objects.get(id=pk) diff --git a/circle/dashboard/views/user.py b/circle/dashboard/views/user.py index 496c0e0..d9f4f72 100644 --- a/circle/dashboard/views/user.py +++ b/circle/dashboard/views/user.py @@ -267,6 +267,9 @@ class UserCreationView(LoginRequiredMixin, PermissionRequiredMixin, template_name = 'dashboard/user-create.html' permission_required = "auth.add_user" + def get_success_url(self): + reverse('dashboard.views.group-detail', args=[self.group.pk]) + def get_group(self, group_pk): self.group = get_object_or_404(Group, pk=group_pk) if not self.group.profile.has_level(self.request.user, 'owner'): diff --git a/circle/dashboard/views/vm.py b/circle/dashboard/views/vm.py index 2bce624..26bb322 100644 --- a/circle/dashboard/views/vm.py +++ b/circle/dashboard/views/vm.py @@ -1135,7 +1135,7 @@ def get_vm_screenshot(request, pk): # TODO handle this better raise Http404() - return HttpResponse(image, mimetype="image/png") + return HttpResponse(image, content_type="image/png") class InstanceActivityDetail(CheckedDetailView): diff --git a/circle/network/views.py b/circle/network/views.py index 22c4191..d4b648d 100644 --- a/circle/network/views.py +++ b/circle/network/views.py @@ -46,11 +46,10 @@ from itertools import chain from dashboard.views import AclUpdateView from dashboard.forms import AclUserOrGroupAddForm -from django.utils import simplejson - try: from django.http import JsonResponse except ImportError: + from django.utils import simplejson class JsonResponse(HttpResponse): """JSON response for Django < 1.7 https://gist.github.com/philippeowagner/3179eb475fe1795d6515 diff --git a/circle/vm/models/instance.py b/circle/vm/models/instance.py index 4b69464..e0fe497 100644 --- a/circle/vm/models/instance.py +++ b/circle/vm/models/instance.py @@ -110,9 +110,8 @@ class VirtualMachineDescModel(BaseResourceConfigModel): "for hosting the VM."), verbose_name=_("required traits")) system = TextField(verbose_name=_('operating system'), - help_text=(_('Name of operating system in ' - 'format like "%s".') % - 'Ubuntu 12.04 LTS Desktop amd64')) + help_text=(_('Name of operating system in format like ' + '"Ubuntu 12.04 LTS Desktop amd64".'))) tags = TaggableManager(blank=True, verbose_name=_("tags")) has_agent = BooleanField(verbose_name=_('has agent'), default=True, help_text=_(