Don't always return action, raise an Exception if we cannot check what the action is

rhodecode/lib/hooks.py

@@ -90,7 +90,6 def log_pull_action(ui, repo, **kwargs):
     :param ui:
     :param repo:
     """
-
     extras = dict(repo.ui.configitems('rhodecode_extras'))
     username = extras['username']
     repository = extras['repository']

rhodecode/lib/middleware/simplehg.py

@@ -226,6 +226,9 class SimpleHg(BaseVCSController):
 
         return 'pull'
 
+        raise Exception('Unable to detect pull/push action !!'
+                        'Are you using non standard command or client ?')
+
     def __inject_extras(self, repo_path, baseui, extras={}):
         """
         Injects some extra params into baseui instance