Ravi Nori has uploaded a new change for review.

Change subject: cli: incorrect property to python built-in symbol encapsulation 
(#1015975)
......................................................................

cli: incorrect property to python built-in symbol encapsulation (#1015975)

If a object has a property with the same
name as an internal property. We do not
append '_' to the property name.

Change-Id: Ie6efa65f785b33ccd46fc23bf4e223fa8e76dfad
Bug-Url: https://bugzilla.redhat.com/1015975
Signed-off-by: Ravi Nori <rn...@redhat.com>
---
M src/ovirtcli/command/command.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine-cli refs/changes/40/19940/1

diff --git a/src/ovirtcli/command/command.py b/src/ovirtcli/command/command.py
index eeb269b..9b9b6dc 100644
--- a/src/ovirtcli/command/command.py
+++ b/src/ovirtcli/command/command.py
@@ -135,7 +135,8 @@
                                 spplited_param_data = param_data[0].split('.')
                                 for param_period in spplited_param_data:
                                     if spplited_param_data[-1] == param_period:
-                                        param_period = 
self.fixParamNameIfParamIsKeyword(param_period)
+                                        if not hasattr(obj_params_set_cand, 
param_period):
+                                            param_period = 
self.fixParamNameIfParamIsKeyword(param_period)
                                         if hasattr(obj_params_set_cand, 
param_period):
                                             if getattr(obj_params_set_cand, 
param_period) != None:
                                                 getattr(obj, 
props[i]).append(obj_params_set_cand)


-- 
To view, visit http://gerrit.ovirt.org/19940
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie6efa65f785b33ccd46fc23bf4e223fa8e76dfad
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to