slightly simpler logic

morioc/pvinfo.py

@@ -62,17 +62,18 @@ def infer_type(value, parse_string=True): #TODO arrays? strings?
 def make_pvdb(pvinfos):
     pvdb = {}
     for pvi in pvinfos:
-        name, dtype = pvi.name, pvi.dtype
+        name = pvi.name
 
-        if not is_managed(name):
-            print("will create", name)
-            pvdb[name] = pvi.to_dict()
-        else:
+        if is_managed(name):
+            dtype = pvi.dtype
             managed_dtype = get_managed_dtype(name)
-            if managed_dtype != dtype:
-                print(f"type for {name} changed:", managed_dtype, "->", dtype) #TODO what should be done if this happens?
-#                self.deletePV(name)
-#                pvdb[name] = {"type": dtype}
+            if dtype != managed_dtype:
+                print(f"dtype for {name} changed:", managed_dtype, "->", dtype) #TODO what should be done if this happens?
+#                server.deletePV(name) #?
+            continue # no need to re-create
+
+        print("will create", name)
+        pvdb[name] = pvi.to_dict()
 
     return pvdb