Revert of Fix param_name for Properties.

recipe_engine/loader.py

Index: recipe_engine/loader.py
diff --git a/recipe_engine/loader.py b/recipe_engine/loader.py
index b39fc165b5fa12b705fd216b28d1faf1f279dd71..e0c99a72cf32fcee6062a669e8e6f5015416872d 100644
--- a/recipe_engine/loader.py
+++ b/recipe_engine/loader.py
@@ -481,7 +481,6 @@
     self._instances[mod] = self._instantiator(mod, deps_dict)
     return self._instances[mod]
 
-
 def _invoke_with_properties(callable_obj, all_props, prop_defs, arg_names,
                             **additional_args):
   """Internal version of invoke_with_properties.
@@ -496,33 +495,21 @@
           "You tried to invoke {} with an unbound Property {} named {}".format(
               callable, prop, name))
 
-  # Maps parameter names to property names
-  param_name_mapping = {
-              prop.param_name: name for name, prop in prop_defs.iteritems()}
-
   props = []
-
-  for param_name in arg_names:
-    if param_name in additional_args:
-      props.append(additional_args.pop(param_name))
+  for arg in arg_names:
+    if arg in additional_args:
+      props.append(additional_args.pop(arg))
       continue
 
-    if param_name not in param_name_mapping:
+    if arg not in prop_defs:
       raise UndefinedPropertyException(
-        "Missing property definition for parameter '{}'.".format(param_name))
-
-    prop_name = param_name_mapping[param_name]
-
-    if prop_name not in prop_defs:
-      raise UndefinedPropertyException(
-        "Missing property value for '{}'.".format(prop_name))
-
-    prop = prop_defs[prop_name]
+        "Missing property definition for '{}'.".format(arg))
+
+    prop = prop_defs[arg]
     props.append(prop.interpret(all_props.get(
-      prop_name, PROPERTY_SENTINEL)))
+      prop.param_name, PROPERTY_SENTINEL)))
 
   return callable_obj(*props, **additional_args)
-
 
 def invoke_with_properties(callable_obj, all_props, prop_defs,
                            **additional_args):
@@ -533,10 +520,9 @@
     callable_obj: The function to call, or class to instantiate.
                   This supports passing in either RunSteps, or a recipe module,
                   which is a class.
-    all_props: A dictionary containing all the properties (strings) currently
-               defined in the system.
-    prop_defs: A dictionary of property name to property definitions
-               (BoundProperty) for this callable.
+    all_props: A dictionary containing all the properties (instances of BoundProperty)
+               currently defined in the system.
+    prop_defs: A dictionary of name to property definitions for this callable.
     additional_args: kwargs to pass through to the callable.
                      Note that the names of the arguments can correspond to
                      positional arguments as well.