Do unless instead "if !" to follow the Ruby guideline

modules/exploits/multi/http/struts_code_exec.rb

@@ -133,7 +133,7 @@ class MetasploitModule < Msf::Exploit::Remote
   end
 
   def exploit
-    if !datastore['CMD'].blank?
+    unless datastore['CMD'].blank?
       print_status("Executing user supplied command")
       execute_command(datastore['CMD'])
       return

modules/exploits/multi/http/struts_code_exec_exception_delegator.rb

@@ -186,7 +186,7 @@ class MetasploitModule < Msf::Exploit::Remote
   end
 
   def exploit
-    if !datastore['CMD'].blank?
+    unless datastore['CMD'].blank?
       print_status("Executing user supplied command")
       execute_command(datastore['CMD'])
       return

modules/exploits/windows/antivirus/ams_hndlrsvc.rb

@@ -149,7 +149,7 @@ class MetasploitModule < Msf::Exploit::Remote
 
   def exploit
 
-    if !datastore['CMD'].blank?
+    unless datastore['CMD'].blank?
       print_status("Executing command '#{datastore['CMD']}'")
       execute_command(datastore['CMD'])
       return

modules/exploits/windows/antivirus/ams_xfr.rb

@@ -92,7 +92,7 @@ class MetasploitModule < Msf::Exploit::Remote
   end
 
   def exploit
-    if !datastore['CMD'].blank?
+    unless datastore['CMD'].blank?
       print_status("Executing command '#{datastore['CMD']}'")
       execute_command(datastore['CMD'])
       return

modules/exploits/windows/http/ca_totaldefense_regeneratereports.rb

@@ -110,7 +110,7 @@ class MetasploitModule < Msf::Exploit::Remote
 
   def exploit
 
-    if !datastore['CMD'].blank?
+    unless datastore['CMD'].blank?
       print_status("Executing command '#{datastore['CMD']}'")
       execute_command(datastore['CMD'])
       return

modules/exploits/windows/http/osb_uname_jlist.rb

@@ -93,7 +93,7 @@ class MetasploitModule < Msf::Exploit::Remote
 
   def exploit
 
-    if !datastore['CMD'].blank?
+    unless datastore['CMD'].blank?
       print_status("Executing command '#{datastore['CMD']}'")
       execute_command(datastore['CMD'])
       return