dashboard, vm: move is_console_available logic to model

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

@@ -113,8 +113,8 @@
               <i class="icon-tasks icon-2x"></i><br>
               {% trans "Resources" %}</a>
           </li>
-          <li {% if instance.state != "RUNNING" %}class="disabled"{% endif %}>
-            <a href="#{% if instance.state == "RUNNING" %}console" data-toggle="pill{% endif %}" data-target="#_console" class="text-center">
+          <li {% if not instance.is_console_available %}class="disabled"{% endif %}>
+            <a href="#console" data-toggle="pill" data-target="#_console" class="text-center">
               <i class="icon-desktop icon-2x"></i><br>
               {% trans "Console" %}</a></li>
           <li>

circle/vm/models/instance.py

@@ -270,6 +270,9 @@ class Instance(AclBase, VirtualMachineDescModel, TimeStampedModel):
             act = None
         return 'NOSTATE' if act is None else act.resultant_state
 
+    def is_console_available(self):
+        return self.state in ('RUNNING', )
+
     def manual_state_change(self, new_state, reason=None, user=None):
         # TODO cancel concurrent activity (if exists)
         act = InstanceActivity.create(code_suffix='manual_state_change',