Change in ovirt-engine-sdk-java[master]: Fix tag names of action parameters

omachace at redhat.com omachace at redhat.com
Mon Sep 5 15:24:56 UTC 2016


Ondra Machacek has submitted this change and it was merged.

Change subject: Fix tag names of action parameters
......................................................................


Fix tag names of action parameters

Currently when the SDK generates the XML document to invoke an action
method it assumes that the names of the elements inside the 'action'
match the names of the types of the parameters. This isn't correct,
they should match the names of the parameters, not the names of the
types. For example, the 'Host.iscsi_discover' action method has a parameter
of type 'IscsiDetails' and name 'iscsi', so the tag should be 'iscsi',
not 'iscsi_details'. To solve this issue this patch chages the SDK so
that the implementation of action methods create an instance of the
'Action' type, and render it uusing the 'ActionWriter' class, which
knows how to write it correctly.

Change-Id: I25494bc638e3dc1098b1086bad94901ca03a72c9
Signed-off-by: Ondra Machacek <omachace at redhat.com>
---
M generator/src/main/java/org/ovirt/sdk/java/ServicesImplGenerator.java
1 file changed, 9 insertions(+), 34 deletions(-)

Approvals:
  Ondra Machacek: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/63320
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I25494bc638e3dc1098b1086bad94901ca03a72c9
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine-sdk-java
Gerrit-Branch: master
Gerrit-Owner: Ondra Machacek <omachace at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondra Machacek <omachace at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list