[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[GMG-Devel] [PATCH 40/83] Add try-except block to event listeners.
From: |
Alon Levy |
Subject: |
[GMG-Devel] [PATCH 40/83] Add try-except block to event listeners. |
Date: |
Tue, 25 Feb 2014 21:58:23 +0200 |
From: Praveen Kumar <address@hidden>
---
mediagoblin/plugins/search/__init__.py | 11 ++++++++--
mediagoblin/plugins/search/listeners.py | 39 +++++++++++++++++++++++----------
2 files changed, 36 insertions(+), 14 deletions(-)
diff --git a/mediagoblin/plugins/search/__init__.py
b/mediagoblin/plugins/search/__init__.py
index 8be286d..3701223 100644
--- a/mediagoblin/plugins/search/__init__.py
+++ b/mediagoblin/plugins/search/__init__.py
@@ -71,8 +71,15 @@ def setup_plugin():
]
pluginapi.register_routes(routes)
- register_indices()
- activate_orm_events_listeners()
+ try:
+ register_indices()
+ except Exception as e:
+ _log.debug("Failed to register search indices: %s"%(e))
+
+ try:
+ activate_orm_events_listeners()
+ except Exception as e:
+ _log.debug("Failed to start event listeners: %s"%(e))
#event.listen(MediaEntry, 'after_insert', mediaentry_add_listener)
#event.listen(MediaEntry, 'before_insert', mediaentry_before_add_listener)
#_log.info("Registered listening event")
diff --git a/mediagoblin/plugins/search/listeners.py
b/mediagoblin/plugins/search/listeners.py
index 8212957..1b7a066 100644
--- a/mediagoblin/plugins/search/listeners.py
+++ b/mediagoblin/plugins/search/listeners.py
@@ -14,24 +14,36 @@ class ORMEventsListener(object):
self.listeners = []
def _after_insert_event_listener(self):
- event.listen(self.model, 'after_insert',
+ try:
+ event.listen(self.model, 'after_insert',
self.insert_event_handler)
+ except Exception, e:
+ _log.debug("Listener activation failed: %s"%(e))
_log.info("Listening for after_insert event for
%s",self.model.__name__)
def _after_update_event_listener(self):
- event.listen(self.model, 'after_update',
+ try:
+ event.listen(self.model, 'after_update',
self.update_event_handler)
+ except Exception as e:
+ _log.debug("Listener activation failed: %s"%(e))
_log.info("Listening for after_update event for
%s",self.model.__name__)
def _after_delete_event_listener(self):
- event.listen(self.model, 'after_delete',
+ try:
+ event.listen(self.model, 'after_delete',
self.delete_event_handler)
+ except Exception as e:
+ _log.debug("Listener activation failed: %s"%(e))
_log.info("Listening for after_delete event for
%s",self.model.__name__)
def insert_event_handler(self, mapper, connection, target):
index = self.index_registry.get_index_for_object(target)
- index.add_document_from_model_obj(target)
-
+ try:
+ index.add_document_from_model_obj(target)
+ except Exception as e:
+ _log.debug("Failed to add object to index: %s"%(e))
+
def update_event_handler(self, mapper, connection, target):
index = self.index_registry.get_index_for_object(target)
index.update_document_from_model_obj(target)
@@ -41,10 +53,13 @@ class ORMEventsListener(object):
index.delete_document_from_model_obj(target)
def activate_listeners(self):
- self.listeners = [
- self._after_insert_event_listener,
- self._after_update_event_listener,
- self._after_delete_event_listener,
- ]
- for listener in self.listeners:
- listener()
+ try:
+ self.listeners = [
+ self._after_insert_event_listener,
+ self._after_update_event_listener,
+ self._after_delete_event_listener,
+ ]
+ for listener in self.listeners:
+ listener()
+ except Exception as e:
+ _log.debug("Activating listeners failed: %s"%(e))
--
1.8.5.3
- [GMG-Devel] [PATCH 30/83] fix parameters in search views, (continued)
- [GMG-Devel] [PATCH 30/83] fix parameters in search views, Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 31/83] Refactor SearchIndex and add method to update an index., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 32/83] Add event listener for model update, Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 33/83] Add listener for model delete event., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 34/83] Fix bugs in SearchIndex, Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 35/83] Add indices for MediaEntry and MediaTag., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 36/83] Add schema for MediaTag., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 37/83] Register schemas for MediaEntry and MediaTag., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 38/83] Add gmg command to delete all indices., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 39/83] Clean up and add docstring to search methods., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 40/83] Add try-except block to event listeners.,
Alon Levy <=
- [GMG-Devel] [PATCH 42/83] Store attributes from a model only it its available., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 43/83] Add field for storing search results., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 41/83] Correct method names in event listeners, Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 44/83] Remove the field for storing search results with the index., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 46/83] Add verbose_name and css_id to search results., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 47/83] Adapt results collection to change in result generation., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 48/83] Modify search template to show tabbed results., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 49/83] Show only unique results., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 50/83] Change element ids and some template fixes., Alon Levy, 2014/02/25
- [GMG-Devel] [PATCH 51/83] Change view function for the search page., Alon Levy, 2014/02/25