Fixed a typo that prevented virtual servers from being able to capture images

Fixes #47

Author: SLsthompson

CHANGELOG.textile

@@ -1,5 +1,6 @@
 *2.2*
-* Added the ability to set a timout for network requests.  The timeout is given when a client is created by passing the :timeout hash parameter when creating a client. The value of the parameter is an integer number of seconds.
+* Added the ability to set a timout for network requests. The timeout is given when a client is created by passing the :timeout hash parameter when creating a client. The value of the parameter is an integer number of seconds.
+* Fixed a bug in VirtualServer#capture_image
 
 *2.1.1*
 * Virtual server upgrades no longer raise exceptions

lib/softlayer/VirtualServer.rb

@@ -143,13 +143,15 @@ def upgrade_max_port_speed!(network_speed_in_Mbps)
     #
     # The image_notes should be a string and will be added to the image as notes.
     #
-    def capture_image(image_name, include_attached_storage = false, image_notes = nil)
+    def capture_image(image_name, include_attached_storage = false, image_notes = '')
+      image_notes = '' if !image_notes
+
       disk_filter = lambda { |disk| disk['device'] == '0' }
-      disk_filter = lambda { |disk| disk['device'] == '1' } if include_attached_storage
+      disk_filter = lambda { |disk| disk['device'] != '1' } if include_attached_storage
 
       disks = self.blockDevices.select(&disk_filter)
 
-      self.service.createArchiveTransaction(image_name, disks, notes) if disks && !disks.empty?
+      self.service.createArchiveTransaction(image_name, disks, image_notes) if disks && !disks.empty?
     end
 
     ##