Merge remote-tracking branch 'upstream/pull/4954'

[rails.git] / app / helpers / user_blocks_helper.rb

diff --git a/app/helpers/user_blocks_helper.rb b/app/helpers/user_blocks_helper.rb
index 0528140f4cd8bbad5a4627f5eb2fa1c886e014e5..3a55e1cdedfcc12d9f3d02ca1135ed7f8f339feb 100644 (file)
--- a/app/helpers/user_blocks_helper.rb
+++ b/app/helpers/user_blocks_helper.rb
@@ -8,7 +8,7 @@ module UserBlocksHelper
     if block.active?
       # if the block hasn't expired yet show the date, if the user just needs to login show that
       if block.needs_view?
     if block.active?
       # if the block hasn't expired yet show the date, if the user just needs to login show that
       if block.needs_view?

-        if block.ends_at > Time.now.getutc
+        if block.ends_at > Time.now.utc

           t("user_blocks.helper.time_future_and_until_login_html", :time => friendly_date(block.ends_at))
         else
           t("user_blocks.helper.until_login")
           t("user_blocks.helper.time_future_and_until_login_html", :time => friendly_date(block.ends_at))
         else
           t("user_blocks.helper.until_login")


@@ -26,6 +26,8 @@ module UserBlocksHelper
   end
 
   def block_duration_in_words(duration)
   end
 
   def block_duration_in_words(duration)

+    # Ensure the requested duration isn't negative, even by a millisecond
+    duration = 0 if duration.negative?

     parts = ActiveSupport::Duration.build(duration).parts
     if duration < 1.day
       t("user_blocks.helper.block_duration.hours", :count => parts.fetch(:hours, 0))
     parts = ActiveSupport::Duration.build(duration).parts
     if duration < 1.day
       t("user_blocks.helper.block_duration.hours", :count => parts.fetch(:hours, 0))