Merge pull request #1087 from richardboehme/rb/update-current-event

Use next event as the "current" event

Author: salzig

app/models/event.rb

@@ -22,10 +22,10 @@ class Event < ApplicationRecord
   default_scope       -> { where(label: Whitelabel[:label_id]) }
 
   scope :with_topics, -> { joins(:topics).distinct }
-  scope :current,     -> { where(date: Date.today.to_time..(Time.now + 9.weeks)).limit(1).order('date ASC') }
-  scope :latest,      -> { where('date < ?', Date.today.to_time).order('date DESC') }
+  scope :current,     -> { where(date: Date.today.to_time..).limit(1).order(date: :asc) }
+  scope :latest,      -> { where('date < ?', Date.today.to_time).order(date: :desc) }
   scope :unpublished, -> { where('published IS NULL') }
-  scope :ordered,     -> { order('date DESC') }
+  scope :ordered,     -> { order(date: :desc) }
 
   def end_date
     date + 2.hours

app/models/highlight.rb

@@ -5,7 +5,7 @@ class Highlight < ApplicationRecord
 
   default_scope -> { where(label: Whitelabel[:label_id]) }
 
-  scope :active, -> { where('end_at > ?', Time.now).order('start_at').limit(1) }
+  scope :active, -> { where('end_at > ?', Time.now).order(start_at: :asc).limit(1) }
 
   def disabled?
     end_at <= Time.now

spec/models/event_spec.rb

@@ -24,6 +24,8 @@
     it 'finds a current event' do
       event_next = create(:event, date: 2.days.from_now)
       expect(Event.current.first).to eql(event_next)
+      event_next.update(date: 5.months.from_now)
+      expect(Event.current.first).to eql(event_next)
     end
   end