effective_events 0.10.1 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cfe22059403401435f15bb3de7a04e64d5f6d66d67b5fa538e552f743bc80657
-  data.tar.gz: 9dde412444e3898074f38c5b4c8cb845bb7b22afce81b2af11d46ec9fe9f22db
+  metadata.gz: 2c3cd1bb2313ea9a4be719350a54cb9f8bae0012040c26cd3661355a7590aba1
+  data.tar.gz: 82fa202806b65f47708ac748df70a44a3e2d5c253860049baad26925edf6e668
 SHA512:
-  metadata.gz: 8a9b73d87d24d66c537427daf7a5f23d4be354eaaf24f34f35646bebd5527727d6236e2a7d8cd8728c9e00d96a061783ba7d772c5c4ab97662ab7bc8a96afdd2
-  data.tar.gz: 6f84878e674939ad1843be2a35ee6d4946a1dc1a4a2dd05a6c477a9453d1ac04ada9903fe31e5c65e7c60cfd1d447f30c1ad3c4f5c0b2bc6ccd83b55c19c01e4
+  metadata.gz: af8087ebab4902bbce1ecb33310333f42943029f612477a2490d68d57c085ccaecded40fb10bcdbbd3cfbc28fb6506aeb1d146dafb22863560d35f2a46e6aa55
+  data.tar.gz: 90f1f61faed9c5991bdfc7289ca21071005f2b238cceea535e4648167102881f07b461021a9a6fc75bd117ab280bf3991d72b293485c5d090ad8503c1c38b389

data/app/controllers/effective/event_registrations_controller.rb

@@ -1,29 +1,28 @@
 module Effective
   class EventRegistrationsController < ApplicationController
-
     if defined?(Devise)
       before_action :authenticate_user!, unless: -> { action_name == 'new' || (action_name == 'show' && params[:id] == 'start') }
     end
 
     include Effective::WizardController
 
+    before_action :redirect_unless_registerable, only: [:new, :show]
+
     resource_scope -> {
       event = Effective::Event.find(params[:event_id])
       EffectiveEvents.EventRegistration.deep.where(owner: current_user, event: event)
     }
 
-    # Override ready_checkout to also check for any currently sold out tickets or addons
-    # Prevents people from coming back and purchasing them
-    def ready_checkout
-      return unless step == :checkout
-      return unless resource.class.try(:acts_as_purchasable_wizard?)
-
-      if resource.sold_out_event_registrants.present? || resource.sold_out_event_addons.present?
-        flash[:danger] = "Your selected event tickets are sold out. This event registration is no longer available."
-        return redirect_to('/dashboard')
-      end
+    # If the event is no longer registerable, do not let them continue
+    def redirect_unless_registerable
+      return if resource.blank?
+      return if resource.submitted?
+      return if resource.event.blank?
+      return if resource.event.registerable?
+      return if resource.submit_order&.deferred?
 
-      resource.ready!
+      flash[:danger] = "Your selected event is no longer available for registration. This event registration is no longer available."
+      return redirect_to('/dashboard')
     end
 
   end

data/app/datatables/admin/effective_event_addons_datatable.rb

@@ -23,8 +23,8 @@ module Admin
         col :event_product, search: :string
       end
 
-      col :purchased_order
-      col :owner
+      col :purchased_order, visible: false
+      col :owner, visible: false
 
       col :first_name
       col :last_name

data/app/datatables/admin/effective_event_products_datatable.rb

@@ -24,13 +24,21 @@ module Admin
         end
       end
 
-      col :purchased_event_addons_count, label: 'Purchased'
+      col :registered_event_addons_count, label: 'Registered'
+      col :purchased_event_addons_count, label: 'Purchased', visible: false
+
       col :capacity, visible: false
       col :capacity_available, visible: false
 
-      col :purchased_event_addons, label: 'Name' do |product|
-        product.purchased_event_addons.reject(&:archived?).sort_by(&:to_s).map do |purchase|
-          content_tag(:div, purchase.owner.to_s, class: 'col-resource_item')
+      col :registered_event_addons, label: 'Registered Names' do |product|
+        product.registered_event_addons.reject(&:archived?).sort_by(&:to_s).map do |addon|
+          content_tag(:div, addon.owner.to_s, class: 'col-resource_item')
+        end.join.html_safe
+      end
+
+      col :purchased_event_addons, label: 'Purchased Names', visible: false do |product|
+        product.purchased_event_addons.reject(&:archived?).sort_by(&:to_s).map do |addon|
+          content_tag(:div, addon.owner.to_s, class: 'col-resource_item')
         end.join.html_safe
       end
 

data/app/datatables/admin/effective_event_tickets_datatable.rb

@@ -25,7 +25,11 @@ module Admin
         end
       end
 
-      col :purchased_event_registrants_count, label: 'Registered' do |event|
+      col :registered_event_registrants_count, label: 'Registered' do |event|
+        event.event_registrants.registered.unarchived.count
+      end
+
+      col :purchased_event_registrants_count, label: 'Purchased', visible: false do |event|
         event.event_registrants.purchased.unarchived.count
       end
 

data/app/helpers/effective_events_helper.rb

@@ -20,7 +20,7 @@ module EffectiveEventsHelper
       remaining = (ticket.capacity.present? ? "#{ticket.capacity_available} remaining" : nil)
 
       label = [title, price, remaining].compact.join(' - ')
-      disabled = { disabled: :disabled } unless (authorized || ticket.available?)
+      disabled = { disabled: :disabled } unless (authorized || event.event_ticket_available?(ticket, quantity: 1))
 
       [label, ticket.to_param, disabled].compact
     end
@@ -39,7 +39,7 @@ module EffectiveEventsHelper
       remaining = (product.capacity.present? ? "#{product.capacity_available} remaining" : nil)
 
       label = [title, price, remaining].compact.join(' - ')
-      disabled = { disabled: :disabled } unless (authorized || product.available?)
+      disabled = { disabled: :disabled } unless (authorized || event.event_product_available?(product, quantity: 1))
 
       [label, product.to_param, disabled].compact
     end

data/app/models/concerns/effective_events_event_registration.rb

@@ -77,7 +77,13 @@ module EffectiveEventsEventRegistration
       timestamps
     end
 
-    scope :deep, -> { includes(:owner, :event_registrants, :event_addons, event: [:event_products, :event_tickets]) }
+    scope :deep, -> { 
+      includes(:owner)
+      .includes(event: [:rich_texts, event_products: :purchased_event_addons, event_tickets: :purchased_event_registrants])
+      .includes(event_registrants: [event_ticket: :purchased_event_registrants])
+      .includes(event_addons: [event_product: :purchased_event_addons])
+    }
+
     scope :sorted, -> { order(:id) }
 
     scope :in_progress, -> { where.not(status: [:submitted]) }
@@ -99,16 +105,17 @@ module EffectiveEventsEventRegistration
       self.errors.add(:event_registrants, "can't be blank") unless present_event_registrants.present?
     end
 
-    # Validate all items are available
-    validate(unless: -> { current_step == :checkout || done? }) do
-      sold_out_event_registrants.each do |item|
-        errors.add(:base, "The #{item.event_ticket} ticket is sold out and no longer available for purchase")
-        item.errors.add(:event_ticket_id, "#{item.event_ticket} is unavailable for purchase")
+    # Validate all tickets are available for registration
+    validate(if: -> { current_step == :tickets }) do
+      unavailable_event_tickets.each do |event_ticket|
+        errors.add(:base, "The requested number of #{event_ticket} tickets are not available")
       end
+    end
 
-      sold_out_event_addons.each do |item|
-        errors.add(:base, "The #{item.event_product} product is sold out and no longer available for purchase")
-        item.errors.add(:event_product_id, "#{item.event_product} is unavailable for purchase")
+    # Validate all products are available for registration
+    validate(if: -> { current_step == :addons }) do
+      unavailable_event_products.each do |event_product|
+        errors.add(:base, "The requested number of #{event_product} add-ons are not available")
       end
     end
 
@@ -132,15 +139,6 @@ module EffectiveEventsEventRegistration
       notifications = event.event_notifications.select(&:registrant_purchased?)
       notifications.each { |notification| notification.notify!(event_registrants: event_registrants) }
     end
-
-    def sold_out_event_registrants
-      event_registrants.reject { |er| er.purchased? || er.event_ticket&.available? }
-    end
-
-    def sold_out_event_addons
-      event_addons.reject { |ep| ep.purchased? || ep.event_product&.available? }
-    end
-
   end
 
   # Instance Methods
@@ -200,10 +198,34 @@ module EffectiveEventsEventRegistration
     event_addons
   end
 
+  def unavailable_event_tickets
+    unavailable = []
+
+    present_event_registrants.map(&:event_ticket).group_by { |t| t }.each do |event_ticket, event_tickets|
+      unavailable << event_ticket unless event.event_ticket_available?(event_ticket, quantity: event_tickets.length)
+    end
+
+    unavailable
+  end
+
+  def unavailable_event_products
+    unavailable = []
+
+    present_event_addons.map(&:event_product).group_by { |p| p }.each do |event_product, event_products|
+      unavailable << event_product unless event.event_product_available?(event_product, quantity: event_products.length)
+    end
+
+    unavailable
+  end
+
   private
 
   def present_event_registrants
-    event_registrants.reject(&:marked_for_destruction?)
+    event_registrants.reject(&:marked_for_destruction?).reject(&:archived?)
+  end
+
+  def present_event_addons
+    event_addons.reject(&:marked_for_destruction?).reject(&:archived?)
   end
 
 end

data/app/models/effective/event.rb

@@ -24,6 +24,10 @@ module Effective
     has_many :event_notifications, -> { order(:id) }, inverse_of: :event, dependent: :destroy
     accepts_nested_attributes_for :event_notifications, allow_destroy: true
 
+    # Used by the registration_available checks
+    has_many :registered_event_registrants, -> { EventRegistrant.registered }, class_name: 'Effective::EventRegistrant', inverse_of: :event
+    has_many :registered_event_addons, -> { EventAddon.registered }, class_name: 'Effective::EventAddon', inverse_of: :event
+
     # rich_text_body - Used by the select step
     has_many_rich_texts
 
@@ -186,7 +190,8 @@ module Effective
     end
 
     def sold_out?
-      false
+      return false unless event_tickets.present?
+      event_tickets.none? { |event_ticket| event_ticket_available?(event_ticket, quantity: 1) }
     end
 
     def early_bird?
@@ -229,5 +234,35 @@ module Effective
       start_at
     end
 
+    # Can I register/purchase this many new event tickets?
+    def event_ticket_available?(event_ticket, quantity:)
+      raise('expected an EventTicket') unless event_ticket.kind_of?(Effective::EventTicket)
+      raise('expected quantity to be greater than 0') unless quantity.to_i > 0
+
+      return false if event_ticket.archived?
+      return true if event_ticket.capacity.blank?   # No capacity enforced for this ticket
+
+      # Total number already sold
+      registered = registered_event_registrants.count { |r| r.event_ticket_id == event_ticket.id }
+
+      # If there's capacity for this many more
+      (registered + quantity) <= event_ticket.capacity
+    end
+
+    # Can I register/purchase this many new event products?
+    def event_product_available?(event_product, quantity:)
+      raise('expected an EventProduct') unless event_product.kind_of?(Effective::EventProduct)
+      raise('expected quantity to be greater than 0') unless quantity.to_i > 0
+
+      return false if event_product.archived?
+      return true if event_product.capacity.blank?   # No capacity enforced for this product
+
+      # Total number already sold
+      registered = registered_event_addons.count { |r| r.event_product_id == event_product.id }
+
+      # If there's capacity for this many more
+      (registered + quantity) <= event_product.capacity
+    end
+
   end
 end

data/app/models/effective/event_addon.rb

@@ -42,6 +42,7 @@ module Effective
 
     scope :sorted, -> { order(:id) }
     scope :deep, -> { includes(:event, :event_product) }
+    scope :registered, -> { purchased_or_deferred.unarchived }
 
     before_validation(if: -> { event_registration.present? }) do
       self.event ||= event_registration.event

data/app/models/effective/event_product.rb

@@ -9,7 +9,8 @@ module Effective
     belongs_to :event
 
     has_many :event_addons
-    has_many :purchased_event_addons, -> { EventAddon.purchased }, class_name: 'Effective::EventAddon'
+    has_many :purchased_event_addons, -> { EventAddon.purchased.unarchived }, class_name: 'Effective::EventAddon'
+    has_many :registered_event_addons, -> { EventAddon.registered.unarchived }, class_name: 'Effective::EventAddon'
 
     log_changes(to: :event) if respond_to?(:log_changes)
 
@@ -45,23 +46,17 @@ module Effective
       title.presence || 'New Event Product'
     end
 
-    # Available for purchase
-    def available?
-      return false if archived?
-      capacity_available?
-    end
-
-    def capacity_available?
-      capacity.blank? || (capacity_available > 0)
-    end
-
     def capacity_available
       return nil if capacity.blank?
-      [(capacity - purchased_event_addons_count), 0].max
+      [(capacity - registered_event_addons_count), 0].max
+    end
+
+    def registered_event_addons_count
+      registered_event_addons.length
     end
 
     def purchased_event_addons_count
-      purchased_event_addons.reject(&:archived?).length
+      purchased_event_addons.length
     end
 
   end

data/app/models/effective/event_registrant.rb

@@ -41,6 +41,7 @@ module Effective
 
     scope :sorted, -> { order(:last_name) }
     scope :deep, -> { includes(:event, :event_ticket) }
+    scope :registered, -> { purchased_or_deferred.unarchived }
 
     validates :first_name, presence: true
     validates :last_name, presence: true

data/app/models/effective/event_ticket.rb

@@ -9,7 +9,8 @@ module Effective
     belongs_to :event
 
     has_many :event_registrants
-    has_many :purchased_event_registrants, -> { EventRegistrant.purchased }, class_name: 'Effective::EventRegistrant'
+    has_many :purchased_event_registrants, -> { EventRegistrant.purchased.unarchived }, class_name: 'Effective::EventRegistrant'
+    has_many :registered_event_registrants, -> { EventRegistrant.registered.unarchived }, class_name: 'Effective::EventRegistrant'
 
     log_changes(to: :event) if respond_to?(:log_changes)
 
@@ -51,23 +52,17 @@ module Effective
       event.early_bird? ? early_bird_price : regular_price
     end
 
-    # Available for purchase
-    def available?
-      return false if archived?
-      capacity_available?
-    end
-
-    def capacity_available?
-      capacity.blank? || (capacity_available > 0)
-    end
-
     def capacity_available
       return nil if capacity.blank?
-      [(capacity - purchased_event_registrants_count), 0].max
+      [(capacity - registered_event_registrants_count), 0].max
+    end
+
+    def registered_event_registrants_count
+      registered_event_registrants.length
     end
 
     def purchased_event_registrants_count
-      purchased_event_registrants.reject(&:archived?).length
+      purchased_event_registrants.length
     end
 
   end

data/app/views/effective/event_registrations/_dashboard.html.haml

@@ -1,5 +1,5 @@
 -# In progress registration
-- registration = EffectiveEvents.EventRegistration.in_progress.for(current_user).first
+- registration = EffectiveEvents.EventRegistration.in_progress.includes(event: :event_products).for(current_user).first
 
 - datatable = EffectiveResources.best('EffectiveEventRegistrationsDatatable').new(self, namespace: :effective)
 

data/app/views/effective/event_registrations/checkout.html.haml

@@ -1,6 +1,18 @@
 = render 'layout' do
   = render 'effective/event_registrations/content', resource: resource
 
-  .card
-    .card-body
-      = render_checkout_step2(resource.submit_order, purchased_url: wizard_path(:submitted), deferred_url: wizard_path(:checkout), declined_url: wizard_path(:checkout))
+  - if resource.submit_order.deferred?
+    .card
+      .card-body
+        = render_checkout_step2(resource.submit_order, purchased_url: wizard_path(:submitted), deferred_url: wizard_path(:checkout), declined_url: wizard_path(:checkout))
+  - elsif resource.event.registerable? == false
+    .alert.alert-danger Your selected event is no longer available for registration.
+  - elsif resource.unavailable_event_tickets.present?
+    .alert.alert-danger Your selected number of event tickets are no longer available.
+  - elsif resource.unavailable_event_products.present?
+    .alert.alert-danger Your selected number of event add-ons are no longer available.
+  - else
+    .card
+      .card-body
+        = render_checkout_step2(resource.submit_order, purchased_url: wizard_path(:submitted), deferred_url: wizard_path(:checkout), declined_url: wizard_path(:checkout))
+

data/lib/effective_events/version.rb

@@ -1,3 +1,3 @@
 module EffectiveEvents
-  VERSION = '0.10.1'.freeze
+  VERSION = '0.11.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_events
 version: !ruby/object:Gem::Version
-  version: 0.10.1
+  version: 0.11.0
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-12-01 00:00:00.000000000 Z
+date: 2024-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails