use with_for_update for User class too

oml/item/icons.py

@@ -134,7 +134,6 @@ def get_icons_db_path():
     icons_db_path = os.path.join(metadata, 'icons.db')
     old_icons_db_path = os.path.join(settings.data_path, 'icons.db')
     if not os.path.exists(icons_db_path) and os.path.exists(old_icons_db_path):
-        print(icons_db_path, icons_db_path)
         shutil.move(old_icons_db_path, icons_db_path)
     return icons_db_path
 

oml/user/models.py

@@ -42,8 +42,11 @@ class User(db.Model):
         return self.id
 
     @classmethod
-    def get(cls, id):
+    def get(cls, id, for_update=False):
-        user = cls.query.filter_by(id=id).first()
+        qs = cls.query.filter_by(id=id)
+        if for_update:
+            qs = qs.with_for_update()
+        user = qs.first()
         if user and not user.info:
             user.info = {}
         return user
@@ -596,13 +599,13 @@ def export_list(data):
 
 def update_user_peering(user_id, peered, username=None):
     with db.session():
-        u = User.get(user_id)
+        u = User.get(user_id, for_update=True)
         if u:
             u.update_peering(peered, username)
 
 def remove_local_info(id):
     with db.session():
-        u = User.get(id)
+        u = User.get(id, for_update=True)
         if u and 'local' in u.info:
             del u.info['local']
             u.save()
@@ -610,7 +613,7 @@ def remove_local_info(id):
 
 def add_local_info(data):
     with db.session():
-        u = User.get(data['id'])
+        u = User.get(data['id'], for_update=True)
         if u:
             if u.info['username'] != data['username']:
                 u.info['username'] = data['username']

static/js/folders.js

@@ -353,7 +353,7 @@ oml.ui.folders = function() {
         oml.$ui.folder[index].options({title: Ox.encodeHTMLEntities(name)});
         oml.getLists(function(lists) {
             var items = lists.filter(function(list) {
-                    return list.user == name && list.type != 'library';
+                    return list.user == name && list.type != 'library' && list.name != 'Public';
                 }),
                 library = lists.filter(function(list) {
                     return list.user == name && list.type == 'library';