sync facets saving annotations that are used as filters. i.e. keywords fixes #378

pandora/annotation/models.py

@@ -160,6 +160,10 @@ class Annotation(models.Model):
         if layer.get('type') == 'event' or layer.get('hasEvents'):
             update_matching_events(self.id)
 
+        #update facets if needed
+        if filter(lambda f: f['id'] == self.layer, settings.CONFIG['filters']):
+            self.update_layer_facet(self.layer)
+
     def cleanup_undefined_relations(self):
         layer = self.get_layer()
         if layer.get('type') == 'place':

pandora/item/models.py

@@ -779,11 +779,7 @@ class Item(models.Model):
         #update cached values in clips
         self.clips.all().update(director=s.director, title=s.title)
 
-    def update_layer_facets(self):
-        filters = [f['id'] for f in settings.CONFIG['filters']]
-        for layer in settings.CONFIG['layers']:
-            if layer['id'] in filters:
-                key = layer['id']
+    def update_layer_facet(self, key):
         current_values = [a['value']
             for a in self.annotations.filter(layer=key).distinct().values('value')]
         saved_values = [i.value for i in Facet.objects.filter(item=self, key=key)]
@@ -795,6 +791,12 @@ class Item(models.Model):
                 sortvalue = value
                 Facet.objects.get_or_create(item=self, key=key, value=value, sortvalue=sortvalue)
 
+    def update_layer_facets(self):
+        filters = [f['id'] for f in settings.CONFIG['filters']]
+        for layer in settings.CONFIG['layers']:
+            if layer['id'] in filters:
+                self.update_layer_facet(layer['id'])
+
     def update_facets(self):
         for key in self.facet_keys + ['title']:
             current_values = self.get(key, [])