Allow max_attendees to be nil

app/models/event.rb

@@ -35,7 +35,6 @@ class Event < ActiveRecord::Base
   validates :program, presence: true
   validates :max_attendees, numericality: { only_integer: true, greater_than_or_equal_to: 1, allow_nil: true }
 
-  validate :max_attendees_and_require_registration
   validate :max_attendees_no_more_than_room_size
 
   scope :confirmed, -> { where(state: 'confirmed') }
@@ -78,7 +77,8 @@ class Event < ActiveRecord::Base
   end
 
   def registration_possible?
-    return false unless max_attendees
+    return false unless require_registration && state == 'confirmed'
+    return true if max_attendees.nil?
     registrations.count < max_attendees
   end
 
@@ -225,14 +225,6 @@ class Event < ActiveRecord::Base
 
   private
 
-  ##
-  # If max_attendees variable is set (higher than 0)
-  # variable require_registration must also be set
-  def max_attendees_and_require_registration
-    errors.add(:require_registration, 'must be enabled, when you set max_attendees') if max_attendees && !require_registration
-    errors.add(:max_attendees, 'must be enabled, when you set require_registration') if require_registration && max_attendees.nil?
-  end
-
   ##
   # Do not allow, for the event, more attendees than the size of the room
   def max_attendees_no_more_than_room_size

app/models/program.rb

@@ -13,8 +13,7 @@ class Program < ActiveRecord::Base
     end
 
     def with_registration_open
-      where(require_registration: true, state: :confirmed).
-      map { |e| e if e.max_attendees > e.registrations.count }.compact
+      select { |e| e if e.registration_possible? }
     end
 
     # All confirmed events of the conference with attribute require_registration

app/views/admin/events/registrations.html.haml

@@ -2,7 +2,11 @@
   .col-md-12
     .page-header
       %h1
-        Registrations (#{@event_registrations.length}/#{@event.max_attendees})
+        Registrations
+        - if @event.max_attendees
+          (#{@event.events_registrations.length}/#{@event.max_attendees})
+        - else
+          (#{@event.events_registrations.length})
       .text-muted
         for
         = @event.title

app/views/proposal/registrations.html.haml

@@ -3,7 +3,11 @@
     .col-md-10.col-md-offset-1
       .page-header
         %h1
-          Registrations (#{@event.events_registrations.length}/#{@event.max_attendees})
+          Registrations
+          - if @event.max_attendees
+            (#{@event.events_registrations.length}/#{@event.max_attendees})
+          - else
+            (#{@event.events_registrations.length})
         .text-muted
           for
           = @event.title