Fix how tags are saved in the DB

pwman/data/nodes.py

@@ -122,7 +122,8 @@ class Node(object):
 
     @tags.setter
     def tags(self, value):
-        self._tags = [tag for tag in value]
+        enc = CryptoEngine.get()
+        self._tags = [enc.encrypt(tag).strip() for tag in value]
 
     @property
     def url(self):

pwman/ui/baseui.py

@@ -394,8 +394,7 @@ class BaseCommands(HelpUIMixin, AliasesMixin):
                 menu.add(CliMenuItem("Url", node.url))
                 menunotes = CliMenuItem("Notes", node.notes)
                 menu.add(menunotes)
-                tgetter = lambda: ', '.join(t for t in node.tags)
-                menu.add(CliMenuItem("Tags", tgetter()))
+                menu.add(CliMenuItem("Tags", ','.join(node.tags)))
             menu.run(node)
             self._db.editnode(i, **node.to_encdict())
             # when done with node erase it

pwman/ui/tools.py

@@ -306,15 +306,13 @@ def set_selection(new_node, items, selection, reader):  # pragma: no cover
         new_node.url = getinput("Url:")
         items[2].getter = new_node.url
     elif selection == 3:  # for notes
-        # new_node.notes = getinput("Notes:")
         new_node.notes = reader("Notes:")
         items[3].getter = new_node.notes
     elif selection == 4:
         taglist = getinput("Tags:")
-        tagstrings = taglist.split()
-        tags = [tn for tn in tagstrings]
+        tags = taglist.split()
         new_node.tags = tags
-        items[4].getter = new_node.tags
+        items[4].getter = ','.join(new_node.tags)
 
 
 class CMDLoop(object):