From d5e877b4da777fe61c4229fdbb3c0f3d1225dca6 Mon Sep 17 00:00:00 2001
From: Terry Geng <gengyanda@gmail.com>
Date: Sun, 8 Mar 2020 15:16:37 +0800
Subject: [PATCH] fix: wrong place to place the lock

---
 interface.py         |  6 +++---
 templates/index.html | 16 ++++++++++------
 2 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/interface.py b/interface.py
index 28957f7..a8607b9 100644
--- a/interface.py
+++ b/interface.py
@@ -133,12 +133,12 @@ def recur_dir(dirobj):
 @web.route("/", methods=['GET'])
 @requires_auth
 def index():
-    tags_color_lookup = build_tags_color_lookup()
-    path_tags_lookup = build_path_tags_lookup()
-
     while var.library.dir_lock.locked():
         time.sleep(0.1)
 
+    tags_color_lookup = build_tags_color_lookup()
+    path_tags_lookup = build_path_tags_lookup()
+
     return render_template('index.html',
                            all_files=var.library.files,
                            tags_lookup=path_tags_lookup,
diff --git a/templates/index.html b/templates/index.html
index 4042697..a4ee0b5 100644
--- a/templates/index.html
+++ b/templates/index.html
@@ -2,8 +2,8 @@
 <ul class="list-group">
     {% if dir and dir.get_subdirs().items() %}
     {% for subdirname, subdirobj in dir.get_subdirs().items() %}
-    {%- set subdirpath = os.path.relpath(subdirobj.fullpath, music_library.fullpath) %}
-    {%- set subdirid = subdirpath.replace("/","-") %}
+    {% set subdirpath = os.path.relpath(subdirobj.fullpath, music_library.fullpath) %}
+    {% set subdirid = subdirpath.replace("/","-") %}
     <li class="directory list-group-item list-group-item-primary">
         <div class="btn-group" role="group">
             <div class="btn-group" role="group">
@@ -45,12 +45,12 @@
         </div>
     </li>
     <div class="collapse multi-collapse" id="multiCollapse-{{ subdirid }}">
-        {{- dirlisting(subdirobj, subdirpath) -}}
+        {{ dirlisting(subdirobj, subdirpath) -}}
     </div>
     {% endfor %}
     {% endif %}
-    {%- set files = dir.get_files() %}
-    {%- if files %}
+    {% set files = dir.get_files() %}
+    {% if files %}
     {% for file in files %}
     {% set filepath = os.path.relpath(os.path.join(dir.fullpath, file), music_library.fullpath) %}
     <li class="file list-group-item">
@@ -80,9 +80,13 @@
             <div class="btn-space"><i class="fa fa-music" aria-hidden="true"></i></div>
             {{ filepath }}
         </div>
+            {% if tags_lookup[filepath] %}
             {% for tag in tags_lookup[filepath] %}
                 <span class="badge badge-{{ tags_color_lookup[tag] }}">{{ tag }}</span>
             {% endfor %}
+            {% else %}
+                Error: {{ filepath }}
+            {% endif %}
 
         <div class="btn-group" style="float: right;">
             <form action="./download" method="get" class="file file_download">
@@ -96,7 +100,7 @@
         </div>
     </li>
     {% endfor %}
-    {%- endif %}
+    {% endif %}
 </ul>
 {%- endmacro %}