rbvmomi 1.2.0 → 1.2.1

data/VERSION

@@ -1 +1 @@
-1.2.0
+1.2.1

data/lib/rbvmomi/connection.rb

@@ -26,9 +26,10 @@ class Connection < TrivialSoap
     xml.tag! method, :xmlns => @ns do
       obj2xml xml, '_this', 'ManagedObject', false, this
       descs.each do |d|
-        k = d['name'].to_sym
-        if params.member? k or params.member? k.to_s
-          v = params.member?(k) ? params[k] : params[k.to_s]
+        k = d['name']
+        k = k.to_sym if !params.member?(k) && params.member?(k.to_sym)
+        v = params[k]
+        if not v == nil
           obj2xml xml, d['name'], d['wsdl_type'], d['is-array'], v
         else
           fail "missing required parameter #{d['name']}" unless d['is-optional']

data/test/test_emit_request.rb

@@ -86,6 +86,23 @@ class EmitRequestTest < Test::Unit::TestCase
     EOS
   end
 
+  def test_nil_optional_param
+    desc = [
+      {
+        'name' => 'blah',
+        'is-array' => false,
+        'is-optional' => true,
+        'wsdl_type' => 'xsd:string',
+      }
+    ]
+
+    check desc, <<-EOS, MO, :blah => nil
+<root xmlns="urn:vim25">
+  <_this type="VirtualMachine">foo</_this>
+</root>
+    EOS
+  end
+
   def test_string_key
     desc = [
       {

metadata

@@ -1,12 +1,8 @@
 --- !ruby/object:Gem::Specification 
 name: rbvmomi
 version: !ruby/object:Gem::Version 
-  prerelease: false
-  segments: 
-  - 1
-  - 2
-  - 0
-  version: 1.2.0
+  prerelease: 
+  version: 1.2.1
 platform: ruby
 authors: 
 - Rich Lane
@@ -25,10 +21,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        segments: 
-        - 1
-        - 4
-        - 1
         version: 1.4.1
   type: :runtime
   version_requirements: *id001
@@ -40,8 +32,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        segments: 
-        - 0
         version: "0"
   type: :runtime
   version_requirements: *id002
@@ -53,8 +43,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        segments: 
-        - 0
         version: "0"
   type: :runtime
   version_requirements: *id003
@@ -66,7 +54,6 @@ extensions: []
 
 extra_rdoc_files: 
 - LICENSE
-- README.html
 - README.rdoc
 files: 
 - .yardopts
@@ -116,7 +103,6 @@ files:
 - test/test_parse_response.rb
 - test/test_serialization.rb
 - vmodl.db
-- README.html
 has_rdoc: true
 homepage: https://github.com/rlane/rbvmomi
 licenses: []
@@ -131,23 +117,17 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      segments: 
-      - 1
-      - 8
-      - 7
       version: 1.8.7
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      segments: 
-      - 0
       version: "0"
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.7
+rubygems_version: 1.5.3
 signing_key: 
 specification_version: 3
 summary: Ruby interface to the VMware vSphere API

data/README.html

@@ -1,76 +0,0 @@
-<h1>RbVmomi</h1>
-
-<h2>Introduction</h2>
-
-<p>RbVmomi is a Ruby interface to the vSphere API. Like the Perl and Java SDKs,
-you can use it to manage ESX and VirtualCenter servers. The current release
-supports the vSphere 4.1 API.</p>
-
-<h2>Usage</h2>
-
-<p>A simple example of turning on a VM:</p>
-
-<pre><code>require 'rbvmomi'
-conn = RbVmomi.connect host: 'foo', user: 'bar', password: 'baz'
-dc = conn.serviceInstance.find_datacenter("mydatacenter") or fail "datacenter not found"
-vm = dc.find_vm("myvm") or fail "VM not found"
-vm.PowerOn_Task.wait_for_completion
-</code></pre>
-
-<p>This code uses several RbVmomi extensions to the VI API for concision. The
-expanded snippet below uses only standard API calls and should be familiar to
-users of the Java SDK:</p>
-
-<pre><code>require 'rbvmomi'
-conn = RbVmomi.connect host: 'foo', user: 'bar', password: 'baz'
-rootFolder = conn.serviceInstance.content.rootFolder
-dc = rootFolder.childEntity.grep(RbVmomi::VIM::Datacenter).find { |x| x.name == "mydatacenter" } or fail "datacenter not found"
-vm = dc.vmFolder.childEntity.grep(RbVmomi::VIM::VirtualMachine).find { |x| x.name == "myvm" } or fail "VM not found"
-task = vm.PowerOn_Task
-filter = conn.propertyCollector.CreateFilter(
-  spec: {
-    propSet: [{ type =&gt; 'Task', all: false, pathSet: ['info.state']}],
-    objectSet: [{ obj: task }]
-  },
-  partialUpdates: false
-)
-ver = ''
-while true
-  result = conn.propertyCollector.WaitForUpdates(version: ver)
-  ver = result.version
-  break if ['success', ['error'].member? task.info.state
-end
-filter.DestroyPropertyFilter
-raise task.info.error if task.info.state == 'error'
-</code></pre>
-
-<p>As you can see, the extensions RbVmomi adds can dramatically decrease the code
-needed to perform simple tasks while still letting you use the full power of
-the API when necessary. RbVmomi extensions are often more efficient than a
-naive implementation; for example, the find_vm method on VIM::Datacenter used
-in the first example uses the SearchIndex for fast lookups.</p>
-
-<p>A few important points:</p>
-
-<ul>
-<li>Ruby 1.9 is required.</li>
-<li>Properties are exposed as methods: vm.summary</li>
-<li>All class, method, parameter, and property names match the <a href="http://www.vmware.com/support/developer/vc-sdk/visdk41pubs/ApiReference/index.html">official documentation</a>.</li>
-<li>Data object types can usually be inferred from context, so you may simply use a hash instead.</li>
-<li>Enumeration values are simply strings.</li>
-<li>Example code is included in the examples/ directory.</li>
-<li>A set of helper methods for Trollop is included to speed up development of
-command line apps. See the included examples for usage.</li>
-<li>This is a side project of a VMware developer and is entirely unsupported by VMware.</li>
-</ul>
-
-<p>Built-in extensions are in lib/rbvmomi/extensions.rb. You are encouraged to
-reopen VIM classes in your applications and add extensions of your own. If you
-write something generally useful please send it to me and I'll add it in. One
-important point about extensions is that since VIM classes are lazily loaded,
-you need to trigger this loading before you can reopen the class. Putting the
-class name on a line by itself before reopening is enough.</p>
-
-<h2>Development</h2>
-
-<p>Send patches to rlane@vmware.com.</p>