make the request when the value is set

lib/fog/vcloudng/models/compute/tag.rb

@@ -10,37 +10,20 @@ module Fog
         identity  :id, :aliases => :key
         attribute :value
 
-        
+        def value=(new_value)
-        def save
+          has_changed = ( value != new_value )
-          if value_changed?
+          not_first_set = !value.nil?
-            puts "Debug: change the value from #{attributes[:old_value]} to #{attributes[:value]}"
+          attributes[:value] = new_value
-            set_value(value)
+          if not_first_set && has_changed
-            attributes[:value_task].wait_for { :ready? }
+            response = service.put_vm_metadata_value(attributes[:vm_id], id, value)
+            service.process_task(response.body)
           end
         end
         
-        def value=(new_value)
-          attributes[:old_value] ||= attributes[:value]
-          attributes[:value] = new_value
-        end
-        
-        def value_changed?
-          return false unless attributes[:old_value]
-          attributes[:value] != attributes[:old_value]
-        end
-        
-        def set_value(new_value)
-          response = service.put_vm_metadata_value(attributes[:vm_id], id, value)
-          task = response.body
-          task[:id] = task[:href].split('/').last
-          attributes[:value_task] = service.tasks.new(task)
-        end
         
         def destroy
           response = service.delete_vm_metadata(attributes[:vm_id], id)
-          task = response.body
+          service.process_task(response.body)
-          task[:id] = task[:href].split('/').last
-          attributes[:destroy_tag_task] = service.tasks.new(task)
         end
 
       end

lib/fog/vcloudng/models/compute/tags.rb

@@ -11,7 +11,7 @@ module Fog
         attribute :vm_id
         
         def index
-          tags.keys.map{ |key| new({key: key, value: tags[key] }.merge(vm_id: vm_id))}
+          tags.keys.map{ |key| new({key: key, value: tags[key] }.merge(vm_id: vm_id)) }
         end 
         
         def all
@@ -19,9 +19,8 @@ module Fog
         end
 
         def get(tag_id)
-          tag = tags.detect{ |tag| tag.keys.first == tag_id }
+          return nil unless tags[tag_id]
-          return nil unless tag
+          new({key: tag_id, value: tags[tag_id] }.merge(vm_id: vm_id))
-          new(tag.merge(vm_id: vm_id))
         end
         
         def create(key,value)
@@ -29,9 +28,7 @@ module Fog
           data = Fog::Generators::Compute::Vcloudng::Metadata.new(@tags)
           data.add_item(key,value)
           response = service.post_vm_metadata(vm_id, data.attrs)
-          task = response.body
+          service.process_task(response.body)
-          task[:id] = task[:href].split('/').last
-          attributes[:crate_tag_task] = service.tasks.new(task)
         end
         
 #        private