This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/848/

cli/lmi/shell/LMIMethod.py (Diff revision 1)
def __handle_synchro_method_call_polling(self, job_inst):
349
            kwargs -- CIMMethod keyword arguments
353
            kwargs -- CIMMethod keyword arguments (deprecated)
Is it necessary to mark it as *deprecated*? I still prefer this way more than passing a dictionary.

- Michal Minar


On September 10th, 2013, 3:35 p.m. UTC, Peter Hatina wrote:

Review request for OpenLMI Developers.
By Peter Hatina.

Updated Sept. 10, 2013, 3:35 p.m.

Repository: openlmi-tools

Description

This patch unifies the means of passing arguments to method call and instance filtering.

Example:
> instance.MethodCall({"Param1" : val1, "Param2" : val2, ...})

compared to:

> instance.MethodCall(Param1=val1, Param2=val2, ...)

NOTE: The previous way, how to pass parameters to the method call still works.

Diffs

  • cli/lmi/shell/LMIMethod.py (8168ca406df98ba4da49ff31ba7b4188195593b7)

View Diff