Refactor logic for readability

samvera · Jul 13, 2015 · 5ec5bbd · 5ec5bbd
1 parent f5a5f4e
commit 5ec5bbd
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/lib/hydra/works/models/concerns/generic_file/mime_types.rb b/lib/hydra/works/models/concerns/generic_file/mime_types.rb
@@ -29,10 +29,15 @@ def collection?
     end
 
     def file_format
-      return nil if mime_type.blank? and format_label.blank?
-      return mime_type.split('/')[1]+ " ("+format_label.join(", ")+")" unless mime_type.blank? or format_label.blank?
-      return mime_type.split('/')[1] unless mime_type.blank?
-      return format_label
+      if mime_type.present? && format_label.present?
+        "#{mime_type.split('/').last}  (#{format_label.join(", ")})"
+      elsif mime_type.present?
+        mime_type.split('/').last
+      elsif format_label.present?
+        format_label
+      else
+        nil
+      end
     end
 
     module ClassMethods