Merge pull request ManageIQ#13944 from gauravaradhye/supports_snapshots

Removed supports_snapshots? methods which were not using supportsFeatureMixin
bzwei · Feb 16, 2017 · 5f79ec5 · 5f79ec5
2 parents f1b93b3 + 13046e4
commit 5f79ec5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/app/models/manageiq/providers/openstack/cloud_manager/vm.rb b/app/models/manageiq/providers/openstack/cloud_manager/vm.rb
@@ -11,6 +11,8 @@ class ManageIQ::Providers::Openstack::CloudManager::Vm < ManageIQ::Providers::Cl
     end
   end
 
+  supports :snapshots
+
   POWER_STATES = {
     "ACTIVE"            => "on",
     "SHUTOFF"           => "off",
@@ -161,8 +163,4 @@ def requires_storage_for_scan?
   def memory_mb_available?
     true
   end
-
-  def supports_snapshots?
-    true
-  end
 end
diff --git a/app/models/vm_or_template.rb b/app/models/vm_or_template.rb
@@ -1801,9 +1801,7 @@ def self.includes_template?(ids)
     MiqTemplate.where(:id => ids).exists?
   end
 
-  def supports_snapshots?
-    false
-  end
+  supports_not :snapshots
 
   def self.batch_operation_supported?(operation, ids)
     VmOrTemplate.where(:id => ids).all? do |vm_or_template|