From: Matthijs Kooijman <matthijs@stdin.nl>
Date: Sat, 7 Aug 2010 19:37:38 +0000 (+0200)
Subject: handlers: Make create_action pass any kwargs to the constructor.
X-Git-Url: https://git.stderr.nl/gitweb?p=matthijs%2Fprojects%2Fbackupninja.git;a=commitdiff_plain;h=0c1dd1831642f274eaef605fb2b75ac536791612;ds=sidebyside

handlers: Make create_action pass any kwargs to the constructor.
---

diff --git a/src/lib/backupninja/handlers/__init__.py b/src/lib/backupninja/handlers/__init__.py
index 4846771..50abb41 100644
--- a/src/lib/backupninja/handlers/__init__.py
+++ b/src/lib/backupninja/handlers/__init__.py
@@ -107,10 +107,11 @@ class Action(object):
         except ConfigParser.NoOptionError:
             raise config.ConfigError("Option '%s' in section '%s' is mandatory, please configure it" % (option, section))
 
-def create_action(ty):
+def create_action(ty, **kwargs):
     """
     Create a new (subclass of) Action object for an action with the
-    given type.
+    given type. Any extra keyword arguments are passed to the
+    constructor.
 
     If the handler class for this type cannot be loaded, an exception is
     thrown.
@@ -137,7 +138,7 @@ def create_action(ty):
                           % (module.__file__))
 
     # Call the "handler" function to create the actual action
-    action = module.handler()
+    action = module.handler(**kwargs)
    
     # Check if the handler returned is really a subclass of Action
     if not isinstance(action, Action):