Add favorite mark, display on detail page

circle/dashboard/migrations/0008_auto_20180420_1054.py

+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.6 on 2018-04-20 08:54
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('dashboard', '0007_profile_network_limit'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='favourite',
+            name='instance',
+            field=models.CharField(max_length=50),
+        ),
+        migrations.AlterField(
+            model_name='favourite',
+            name='user',
+            field=models.CharField(max_length=50),
+        ),
+    ]

circle/dashboard/models.py

@@ -82,8 +82,8 @@ class Message(TimeStampedModel, TimeFramedModel):
 
 
 class Favourite(Model):
-    instance = ForeignKey("vm.Instance")
-    user = ForeignKey(User)
+    instance = CharField(max_length=50, blank=False)
+    user = CharField(max_length=50, blank=False)
 
 
 class Notification(TimeStampedModel):

circle/dashboard/templates/dashboard/vm-detail.html

@@ -73,7 +73,7 @@
         {{ instance.name }}
       </div>
       <small>{{ instance.primary_host.get_fqdn }}</small>
-      <small class="dashboard-vm-favourite" style="line-height: 39.6px;" data-vm="{{ instance.pk }}">
+      <small class="dashboard-vm-favourite" style="line-height: 39.6px;" data-vm="{{ instance.id }}">
         {% if fav %}
           <i class="fa fa-star text-primary title-favourite" title="{% trans "Unfavourite" %}"></i>
         {% else %}

circle/dashboard/views/vm.py

@@ -137,7 +137,7 @@ class VmDetailView(LoginRequiredMixin, GraphMixin, DetailView):
             'op': {i.op: i for i in ops},
             # 'connect_commands': user.profile.get_connect_commands(instance),
             'hide_tutorial': hide_tutorial,
-            'fav': [], #instance.favourite_set.filter(user=user).exists(),
+            'fav': Favourite.objects.filter(user=user.id, instance=instance.id).exists(),
             'instance': self.object
         })
 
@@ -1392,15 +1392,13 @@ def vm_activity(request, pk):
 class FavouriteView(TemplateView):
 
     def post(self, *args, **kwargs):
-        user = self.request.user
-        vm = Instance.objects.get(pk=self.request.POST.get("vm"))
-        if not vm.has_level(user, 'user'):
-            raise PermissionDenied()
+        user_id = self.request.user.id
+        vm_id = self.request.POST.get("vm")
         try:
-            Favourite.objects.get(instance=vm, user=user).delete()
+            Favourite.objects.get(instance=vm_id, user=user_id).delete()
             return HttpResponse("Deleted.")
         except Favourite.DoesNotExist:
-            Favourite(instance=vm, user=user).save()
+            Favourite(instance=vm_id, user=user_id).save()
             return HttpResponse("Added.")
 #
 #

circle/request/migrations/0006_auto_20180420_1054.py

+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.6 on 2018-04-20 08:54
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('request', '0005_auto_20180403_1342'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='request',
+            options={},
+        ),
+    ]

circle/vm/migrations/0011_auto_20180420_1054.py

+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.6 on 2018-04-20 08:54
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('vm', '0010_auto_20180313_1628'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='node',
+            name='host',
+        ),
+        migrations.RemoveField(
+            model_name='node',
+            name='tags',
+        ),
+        migrations.RemoveField(
+            model_name='node',
+            name='traits',
+        ),
+        migrations.RemoveField(
+            model_name='nodeactivity',
+            name='node',
+        ),
+        migrations.RemoveField(
+            model_name='nodeactivity',
+            name='parent',
+        ),
+        migrations.RemoveField(
+            model_name='nodeactivity',
+            name='user',
+        ),
+        migrations.RemoveField(
+            model_name='instance',
+            name='node',
+        ),
+        migrations.DeleteModel(
+            name='Node',
+        ),
+        migrations.DeleteModel(
+            name='NodeActivity',
+        ),
+    ]