remove unneccessary return statements

this is how ruby style works... apparently

app/models/custom_value/date_strategy.rb

@@ -36,8 +36,8 @@ class CustomValue::DateStrategy < CustomValue::FormatStrategy
 
   def validate_type_of_value
     Date.iso8601(value)
-    return nil
+    nil
   rescue
-    return :not_a_date
+    :not_a_date
   end
 end

app/models/custom_value/float_strategy.rb

@@ -36,8 +36,8 @@ class CustomValue::FloatStrategy < CustomValue::FormatStrategy
 
   def validate_type_of_value
     Kernel.Float(value)
-    return nil
+    nil
   rescue
-    return :not_a_number
+    :not_a_number
   end
 end

app/models/custom_value/int_strategy.rb

@@ -36,8 +36,8 @@ class CustomValue::IntStrategy < CustomValue::FormatStrategy
 
   def validate_type_of_value
     Kernel.Integer(value)
-    return nil
+    nil
   rescue
-    return :not_an_integer
+    :not_an_integer
   end
 end

app/models/custom_value/list_strategy.rb

@@ -36,7 +36,7 @@ class CustomValue::ListStrategy < CustomValue::FormatStrategy
 
   def validate_type_of_value
     unless custom_field.possible_values.include?(value)
-      return :inclusion
+      :inclusion
     end
   end
 end

app/models/custom_value/string_strategy.rb

@@ -29,7 +29,7 @@
 
 class CustomValue::StringStrategy < CustomValue::FormatStrategy
   def typed_value
-    return value
+    value
   end
 
   def validate_type_of_value

app/models/custom_value/user_strategy.rb

@@ -36,7 +36,7 @@ class CustomValue::UserStrategy < CustomValue::FormatStrategy
 
   def validate_type_of_value
     unless custom_field.possible_values(custom_value.customized).include?(value)
-      return :inclusion
+      :inclusion
     end
   end
 end

app/models/custom_value/version_strategy.rb

@@ -36,7 +36,7 @@ class CustomValue::VersionStrategy < CustomValue::FormatStrategy
 
   def validate_type_of_value
     unless custom_field.possible_values(custom_value.customized).include?(value)
-      return :inclusion
+      :inclusion
     end
   end
 end