ruby-rails-extensions 1.1.1 → 2.0.0

data/lib/ruby-rails-extensions/extensions/in_utc.rb

@@ -3,19 +3,15 @@
 require 'date'
 
 DateTime.class_eval do
-  unless method_defined?(:in_utc)
-    # @return [DateTime]
-    def in_utc
-      utc? ? self : in_time_zone('UTC').to_datetime
-    end
+  # @return [DateTime]
+  def in_utc
+    utc? ? self : in_time_zone('UTC').to_datetime
   end
 end
 
 Time.class_eval do
-  unless method_defined?(:in_utc)
-    # @return [Time]
-    def in_utc
-      utc? ? self : in_time_zone('UTC')
-    end
+  # @return [Time]
+  def in_utc
+    utc? ? self : in_time_zone('UTC')
   end
 end

data/lib/ruby-rails-extensions/extensions/input.rb

@@ -3,19 +3,15 @@
 require 'date'
 
 DateTime.class_eval do
-  unless method_defined?(:input)
-    # @return [String] - yyyy-mm-dd[Thh:mm::ss]
-    def input(all = false)
-      strftime(all ? '%F %T' : '%F')
-    end
+  # @return [String] - yyyy-mm-dd[Thh:mm::ss]
+  def input(all = false)
+    strftime(all ? '%F %T' : '%F')
   end
 end
 
 Date.class_eval do
-  unless method_defined?(:input)
-    # @return [String] - yyyy-mm-dd
-    def input(_all = false)
-      strftime('%F')
-    end
+  # @return [String] - yyyy-mm-dd
+  def input(_all = false)
+    strftime('%F')
   end
 end

data/lib/ruby-rails-extensions/extensions/month_and_year.rb

@@ -3,10 +3,8 @@
 require 'date'
 
 Date.class_eval do
-  unless method_defined?(:month_and_year)
-    # @return [String] - mm/yy
-    def month_and_year
-      strftime('%m/%y')
-    end
+  # @return [String] - mm/yy
+  def month_and_year
+    strftime('%m/%y')
   end
 end

data/lib/ruby-rails-extensions/extensions/month_year.rb

@@ -3,10 +3,8 @@
 require 'date'
 
 Date.class_eval do
-  unless method_defined?(:month_year)
-    # @return [String] - yyyy-mm
-    def month_year
-      input[0, 7]
-    end
+  # @return [String] - yyyy-mm
+  def month_year
+    input[0, 7]
   end
 end

data/lib/ruby-rails-extensions/extensions/no_keys.rb

@@ -1,10 +1,8 @@
 # frozen_string_literal: true
 
 Hash.class_eval do
-  unless method_defined?(:no_keys?)
-    # @see Enumerable#none?
-    def no_keys?(...)
-      keys.none?(...)
-    end
+  # @see Enumerable#none?
+  def no_keys?(...)
+    keys.none?(...)
   end
 end

data/lib/ruby-rails-extensions/extensions/no_values.rb

@@ -1,10 +1,8 @@
 # frozen_string_literal: true
 
 Hash.class_eval do
-  unless method_defined?(:no_values?)
-    # @see Enumerable#none?
-    def no_values?(...)
-      values.none?(...)
-    end
+  # @see Enumerable#none?
+  def no_values?(...)
+    values.none?(...)
   end
 end

data/lib/ruby-rails-extensions/extensions/only_some.rb

@@ -1,28 +1,26 @@
 # frozen_string_literal: true
 
 Array.class_eval do
-  unless method_defined?(:only_some?)
-    # Same as `my_array.any?(&block) && !my_array.all?(&block)`
-    #
-    # @return [Boolean]
-    #
-    def only_some?(*meth_args)
-      # Check arg size since `any?` arg take presence over block
-      return any?(*meth_args) && !all?(*meth_args) if meth_args.size.positive?
+  # Same as `my_array.any?(&block) && !my_array.all?(&block)`
+  #
+  # @return [Boolean]
+  #
+  def only_some?(*meth_args)
+    # Check arg size since `any?` arg take presence over block
+    return any?(*meth_args) && !all?(*meth_args) if meth_args.size.positive?
 
-      any = false
-      all = true
+    any = false
+    all = true
 
-      each do |*args|
-        result = yield(*args)
+    each do |*args|
+      result = yield(*args)
 
-        any ||= result
-        all &&= result
+      any ||= result
+      all &&= result
 
-        break if any && !all
-      end
-
-      return any && !all
+      break if any && !all
     end
+
+    return any && !all
   end
 end

data/lib/ruby-rails-extensions/extensions/overlaps.rb

@@ -4,18 +4,14 @@ require 'forwardable'
 require 'set'
 
 Array.class_eval do
-  unless method_defined?(:overlaps?)
-    # @return [Boolean]
-    def overlaps?(other)
-      !(self & Array(other)).empty?
-    end
+  # @return [Boolean]
+  def overlaps?(other)
+    !(self & Array(other)).empty?
   end
 end
 
 Set.class_eval do
-  unless method_defined?(:overlaps?)
-    extend Forwardable
+  extend Forwardable
 
-    def_delegators :to_a, :overlaps?
-  end
+  def_delegators :to_a, :overlaps?
 end

data/lib/ruby-rails-extensions/extensions/pipe.rb

@@ -4,19 +4,15 @@ require 'forwardable'
 require 'set'
 
 Array.class_eval do
-  unless method_defined?(:pipe)
-    # Replace my_array.map(&:method1).map(&:method2) with my_array.pipe(:method1, :method2)
-    # @return [Array]
-    def pipe(*fns)
-      map { |a| fns.reduce(a, &:send) }
-    end
+  # Replace my_array.map(&:method1).map(&:method2) with my_array.pipe(:method1, :method2)
+  # @return [Array]
+  def pipe(*fns)
+    map { |a| fns.reduce(a, &:send) }
   end
 end
 
 Set.class_eval do
-  unless method_defined?(:pipe)
-    extend Forwardable
+  extend Forwardable
 
-    def_delegators :to_a, :pipe
-  end
+  def_delegators :to_a, :pipe
 end

data/lib/ruby-rails-extensions/extensions/presence_bang.rb

@@ -1,17 +1,15 @@
 # frozen_string_literal: true
 
 Object.class_eval do
-  unless method_defined?(:presence!)
-    # Returns self if `present?`
-    #
-    # @raise [BlankObject] if `blank?`
-    #
-    # @return [*]
-    #
-    def presence!
-      raise(RubyRailsExtensions::Errors::BlankObjectError, 'Blank object') if blank?
+  # Returns self if `present?`
+  #
+  # @raise [BlankObject] if `blank?`
+  #
+  # @return [*]
+  #
+  def presence!
+    raise(RubyRailsExtensions::Errors::BlankObjectError, 'Blank object') if blank?
 
-      self
-    end
+    self
   end
 end

data/lib/ruby-rails-extensions/extensions/quarter_dates.rb

@@ -1,47 +1,41 @@
 # frozen_string_literal: true
 
 String.class_eval do
-  if !method_defined?(:quarter_start) && Date.instance_methods.include?(:beginning_of_quarter)
-    # Return the quarter start date.
-    #
-    # @return [Date]
-    #
-    def quarter_start
-      quarter_date.beginning_of_quarter
-    end
+  # Return the quarter start date.
+  #
+  # @return [Date]
+  #
+  def quarter_start
+    quarter_date.beginning_of_quarter
   end
 
-  if !method_defined?(:quarter_end) && Date.instance_methods.include?(:end_of_quarter)
-    # Return the quarter end date.
-    #
-    # @return [Date]
-    #
-    def quarter_end
-      quarter_date.end_of_quarter
-    end
+  # Return the quarter end date.
+  #
+  # @return [Date]
+  #
+  def quarter_end
+    quarter_date.end_of_quarter
   end
 
   private
 
-    unless private_method_defined?(:quarter_date)
-      # Parses a string for a quarter date. Expects the string to follow: 'qQyy' where
-      # q = 1/2/3/4 and yy to be the last two digits of the year.
-      #
-      # @raise [RubyRailsExtensions::Errors::InvalidFormatError] if `self` is not qQyy
-      #
-      # @return [Date]
-      #
-      def quarter_date
-        unless match?(/\A[1-4]q\d\d\z/i)
-          raise(RubyRailsExtensions::Errors::InvalidFormatError, "Unexpected format (given #{self}, expected qQyy)")
-        end
+    # Parses a string for a quarter date. Expects the string to follow: 'qQyy' where
+    # q = 1/2/3/4 and yy to be the last two digits of the year.
+    #
+    # @raise [RubyRailsExtensions::Errors::InvalidFormatError] if `self` is not qQyy
+    #
+    # @return [Date]
+    #
+    def quarter_date
+      unless match?(/\A[1-4]q\d\d\z/i)
+        raise(RubyRailsExtensions::Errors::InvalidFormatError, "Unexpected format (given #{self}, expected qQyy)")
+      end
 
-        values = scan(/\d/)
-        quarter = values.shift.to_i
-        year = "20#{values.join('')}".to_i
-        month = quarter * 3
+      values = scan(/\d/)
+      quarter = values.shift.to_i
+      year = "20#{values.join('')}".to_i
+      month = quarter * 3
 
-        Date.new(year, month, 1)
-      end
+      Date.new(year, month, 1)
     end
 end

data/lib/ruby-rails-extensions/extensions/range_add.rb

@@ -1,47 +1,45 @@
 # frozen_string_literal: true
 
 Range.class_eval do
-  unless method_defined?(:+)
-    # Allows adding a to a range. This will create a new range since ranges are immutable.
-    #
-    # @note Make sure to assign this to a new var or inline replacement `+=`.
-    #
-    # @raise [RubyRailsExtensions::Errors::NonNumericError]
-    #
-    # @param other [Numeric, Range<Numeric>]
-    #
-    # @return [Range]
-    #
-    def +(other)
-      unless self.begin.is_a?(Numeric)
-        raise(
-          RubyRailsExtensions::Errors::NonNumericError,
-          "Wrong range type (given #{self.begin.class}, expected Numeric)"
-        )
-      end
-
-      new_begin, new_end =
-        if other.is_a?(Range)
-          unless other.begin.is_a?(Numeric)
-            raise(
-              RubyRailsExtensions::Errors::NonNumericError,
-              "Wrong range type (given #{other.begin.class}, expected Numeric)"
-            )
-          end
+  # Allows adding a to a range. This will create a new range since ranges are immutable.
+  #
+  # @note Make sure to assign this to a new var or inline replacement `+=`.
+  #
+  # @raise [RubyRailsExtensions::Errors::NonNumericError]
+  #
+  # @param other [Numeric, Range<Numeric>]
+  #
+  # @return [Range]
+  #
+  def +(other)
+    unless self.begin.is_a?(Numeric)
+      raise(
+        RubyRailsExtensions::Errors::NonNumericError,
+        "Wrong range type (given #{self.begin.class}, expected Numeric)"
+      )
+    end
 
-          [self.begin + other.begin, self.end + other.end]
-        else
-          unless other.is_a?(Numeric)
-            raise(
-              RubyRailsExtensions::Errors::NonNumericError,
-              "Wrong type (given #{other.class}, expected Numeric)"
-            )
-          end
+    new_begin, new_end =
+      if other.is_a?(Range)
+        unless other.begin.is_a?(Numeric)
+          raise(
+            RubyRailsExtensions::Errors::NonNumericError,
+            "Wrong range type (given #{other.begin.class}, expected Numeric)"
+          )
+        end
 
-          [self.begin + other, self.end + other]
+        [self.begin + other.begin, self.end + other.end]
+      else
+        unless other.is_a?(Numeric)
+          raise(
+            RubyRailsExtensions::Errors::NonNumericError,
+            "Wrong type (given #{other.class}, expected Numeric)"
+          )
         end
 
-      Range.new(new_begin, new_end, exclude_end?)
-    end
+        [self.begin + other, self.end + other]
+      end
+
+    Range.new(new_begin, new_end, exclude_end?)
   end
 end

data/lib/ruby-rails-extensions/extensions/range_multiply.rb

@@ -1,47 +1,45 @@
 # frozen_string_literal: true
 
 Range.class_eval do
-  unless method_defined?(:*)
-    # Allows multiplying a range. This will create a new range due to range objects being immutable.
-    #
-    # @note Make sure to assign this to a new var or do inline replacement `*=`.
-    #
-    # @raise [RubyRailsExtensions::Errors::NonNumericError]
-    #
-    # @param other [Numeric, Range<Numeric>]
-    #
-    # @return [Range]
-    #
-    def *(other)
-      unless self.begin.is_a?(Numeric)
-        raise(
-          RubyRailsExtensions::Errors::NonNumericError,
-          "Wrong range type (given #{self.begin.class}, expected Numeric)"
-        )
-      end
-
-      new_begin, new_end =
-        if other.is_a?(Range)
-          unless other.begin.is_a?(Numeric)
-            raise(
-              RubyRailsExtensions::Errors::NonNumericError,
-              "Wrong range type (given #{other.begin.class}, expected Numeric)"
-            )
-          end
+  # Allows multiplying a range. This will create a new range due to range objects being immutable.
+  #
+  # @note Make sure to assign this to a new var or do inline replacement `*=`.
+  #
+  # @raise [RubyRailsExtensions::Errors::NonNumericError]
+  #
+  # @param other [Numeric, Range<Numeric>]
+  #
+  # @return [Range]
+  #
+  def *(other)
+    unless self.begin.is_a?(Numeric)
+      raise(
+        RubyRailsExtensions::Errors::NonNumericError,
+        "Wrong range type (given #{self.begin.class}, expected Numeric)"
+      )
+    end
 
-          [self.begin * other.begin, self.end * other.end]
-        else
-          unless other.is_a?(Numeric)
-            raise(
-              RubyRailsExtensions::Errors::NonNumericError,
-              "Wrong type (given #{other.class}, expected Numeric)"
-            )
-          end
+    new_begin, new_end =
+      if other.is_a?(Range)
+        unless other.begin.is_a?(Numeric)
+          raise(
+            RubyRailsExtensions::Errors::NonNumericError,
+            "Wrong range type (given #{other.begin.class}, expected Numeric)"
+          )
+        end
 
-          [self.begin * other, self.end * other]
+        [self.begin * other.begin, self.end * other.end]
+      else
+        unless other.is_a?(Numeric)
+          raise(
+            RubyRailsExtensions::Errors::NonNumericError,
+            "Wrong type (given #{other.class}, expected Numeric)"
+          )
         end
 
-      Range.new(new_begin, new_end, exclude_end?)
-    end
+        [self.begin * other, self.end * other]
+      end
+
+    Range.new(new_begin, new_end, exclude_end?)
   end
 end

data/lib/ruby-rails-extensions/extensions/remove_reserved_keywords.rb

@@ -2,27 +2,25 @@
 
 if defined?(ActiveJob::Base)
   ActiveJob::Base.class_eval do
-    unless method_defined?(:remove_reserved_keywords)
-      # Removes any reserved keywords that can't be passed along.
-      #
-      # @param arguments [...]
-      #
-      # @return [String, Numeric, Hash, Array, nil]
-      #
-      def remove_reserved_keywords(arguments)
-        case arguments
-        when String, Numeric
-          arguments
-        when Hash
-          arguments.each_with_object({}) do |(key, value), obj|
-            next if key.start_with?('_aj')
+    # Removes any reserved keywords that can't be passed along.
+    #
+    # @param arguments [...]
+    #
+    # @return [String, Numeric, Hash, Array, nil]
+    #
+    def remove_reserved_keywords(arguments)
+      case arguments
+      when String, Numeric
+        arguments
+      when Hash
+        arguments.each_with_object({}) do |(key, value), obj|
+          next if key.start_with?('_aj')
 
-            obj[key] = remove_reserved_keywords(value)
-          end
-        when Array
-          arguments.map do |arg|
-            remove_reserved_keywords(arg)
-          end
+          obj[key] = remove_reserved_keywords(value)
+        end
+      when Array
+        arguments.map do |arg|
+          remove_reserved_keywords(arg)
         end
       end
     end

data/lib/ruby-rails-extensions/extensions/safe_parse.rb

@@ -2,13 +2,11 @@
 
 Time.class_eval do
   class << self
-    unless method_defined?(:safe_parse)
-      # Parses the `str` and returns `nil` if invalid
-      def safe_parse(str)
-        parse(str)
-      rescue
-        nil
-      end
+    # Parses the `str` and returns `nil` if invalid
+    def safe_parse(str)
+      parse(str)
+    rescue
+      nil
     end
   end
 end
@@ -16,13 +14,11 @@ end
 if defined?(ActiveSupport::TimeZone)
   ActiveSupport::TimeZone.class_eval do
     class << self
-      unless method_defined?(:safe_parse)
-        # Parses the `str` and returns `nil` if invalid
-        def safe_parse(str)
-          parse(str)
-        rescue
-          nil
-        end
+      # Parses the `str` and returns `nil` if invalid
+      def safe_parse(str)
+        parse(str)
+      rescue
+        nil
       end
     end
   end
@@ -30,13 +26,11 @@ end
 
 Date.class_eval do
   class << self
-    unless method_defined?(:safe_parse)
-      # Parses the `str` and returns `nil` if invalid
-      def safe_parse(str)
-        parse(str)
-      rescue
-        nil
-      end
+    # Parses the `str` and returns `nil` if invalid
+    def safe_parse(str)
+      parse(str)
+    rescue
+      nil
     end
   end
 end

data/lib/ruby-rails-extensions/extensions/select_present.rb

@@ -1,11 +1,7 @@
 # frozen_string_literal: true
 
 Enumerable.class_eval do
-  if method_defined?(:compact_blank) && !method_defined?(:select_present)
-    alias_method :select_present, :compact_blank
-  end
+  alias_method :select_present, :compact_blank
 
-  if method_defined?(:compact_blank!) && !method_defined?(:select_present!)
-    alias_method :select_present!, :compact_blank!
-  end
+  alias_method :select_present!, :compact_blank!
 end

data/lib/ruby-rails-extensions/extensions/select_present_join.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
 Enumerable.class_eval do
-  if method_defined?(:compact_blank_join) && !method_defined?(:select_present_join)
-    alias_method :select_present_join, :compact_blank_join
-  end
+  alias_method :select_present_join, :compact_blank_join
 end

data/lib/ruby-rails-extensions/extensions/set_to_sentence.rb

@@ -3,9 +3,7 @@
 require 'forwardable'
 
 Set.class_eval do
-  unless method_defined?(:to_sentence)
-    extend Forwardable
+  extend Forwardable
 
-    def_delegators :to_a, :to_sentence
-  end
+  def_delegators :to_a, :to_sentence
 end