runner: simpler len() and if: logic, improved naming

slapos/runner/utils.py

@@ -278,11 +278,11 @@ def config_SR_folder(config):
         continue  # there is a broken config file
       list.append(cfg[1])
   folder_list = os.listdir(config['software_root'])
-  if len(folder_list) < 1:
+  if not folder_list:
     return
   current_project = open(os.path.join(config['etc_dir'], ".project")).read()
   projects = current_project.split('/')
-  name = projects[len(projects) - 2]
+  name = projects[-2]
   for folder in folder_list:
     if folder in list:
       continue  # this folder is already registered

slapos/runner/views.py

@@ -182,17 +182,17 @@ def editInstanceProfile():
 # get status of all computer partitions and process state
 @login_required()
 def inspectInstance():
-  file_content = ''
-  result = ''
   if os.path.exists(app.config['instance_root']):
-    file_content = 'instance_root'
-    result = getSvcStatus(app.config)
-    if len(result) == 0:
-      result = []
+    file_path = 'instance_root'
+    supervisor = getSvcStatus(app.config)
+  else:
+    file_path = ''
+    supervisor = []
   return render_template('instanceInspect.html',
-      file_path=file_content, supervisor=result,
-      slap_status=getSlapStatus(app.config),
-      supervisore=result, partition_amount=app.config['partition_amount'])
+                         file_path=file_path,
+                         supervisor=supervisor,
+                         slap_status=getSlapStatus(app.config),
+                         partition_amount=app.config['partition_amount'])
 
 
 #Reload instance process ans returns new value to ajax