effective_memberships 0.9.5 → 0.9.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bea643081128f664a7c004dbc49a3e8e06f940781267f85f63c7c6d4e69fe6d1
-  data.tar.gz: a769c9c7216ccdc090fc8a3e6107d654e49eeca2b20a171c48769b46d27a0cb0
+  metadata.gz: c9916218e0d606b59b230debdc5c7a70e1d3754333f786298a4b9f3cf21222ce
+  data.tar.gz: 7ee9f5e5f4f64ea6764c206b053a81dc0d253a596de57d2e20324e3c64b86aa4
 SHA512:
-  metadata.gz: 4553d6b65132b346f36882658baac0ae47ec8945f2f592c19fac705b2e5414d1f4597e5473d2209cdb6a6c0528768a4aa65fe75b59b9712f14ffccdc6c5203ca
-  data.tar.gz: f6c02642c63397a2484fb0d9cd4b3fd471767b9c1b8449528e720ac2f73946a86520094ba39bd0b7056df0544e230a0f1a3bd60ddeb64ba689431664d9c3938b
+  metadata.gz: 6e6f8b7f49df86cd4461dba072f3106e3a095076d44f298ced308857e7ee86b3f91455d6823716470d6b4dccdb75ed42e8e05cc2be9bc0208e314aaef011e759
+  data.tar.gz: 936b9c462cede66151ae4a9d186a3b3d6aee60989f284f6157373fb043f73498e486678e09d39af0f75f648a5ace5826f66f0ec553427fbeb77375a1b5822fa9

data/app/controllers/admin/registrar_actions_controller.rb

@@ -14,6 +14,10 @@ module Admin
       success: -> { "#{resource.owner} has been reclassified to #{resource.owner.membership.category}" },
       redirect: -> { admin_owners_path(resource) }
 
+    submit :reinstate, 'Reinstate',
+      success: -> { "#{resource.owner} has been reinstated to #{resource.owner.membership.category}" },
+      redirect: -> { admin_owners_path(resource) }
+
     submit :status_assign, 'Assign Status',
       success: -> { "#{resource.owner} has been assigned #{resource.owner.membership.statuses_sentence}" },
       redirect: -> { admin_owners_path(resource) }

data/app/models/concerns/effective_memberships_applicant.rb

@@ -21,6 +21,10 @@ module EffectiveMembershipsApplicant
       [:start, :select, :summary, :billing, :checkout, :submitted]
     end
 
+    # Apply to Join - Anyone can do this
+    # Apply to Reclassify - Must have an existing membership and category
+    # Apply for Reinstatement - Must have a removed membership. Membership history.
+
     def categories
       ['Apply to Join', 'Apply to Reclassify', 'Apply for Reinstatement']
     end
@@ -101,7 +105,6 @@ module EffectiveMembershipsApplicant
     # Required for Apply to Reclassify
     belongs_to :from_category, polymorphic: true, optional: true
 
-    # Required for Apply for Reinstatement
     belongs_to :from_status, polymorphic: true, optional: true
 
     has_many :applicant_reviews, -> { order(:id) }, as: :applicant, inverse_of: :applicant, dependent: :destroy
@@ -202,8 +205,8 @@ module EffectiveMembershipsApplicant
         self.from_category = owner.membership.categories.first
       end
 
-      if owner.membership.present? && reinstatement?
-        self.from_status = owner.membership.statuses.find { |status| status.reinstatable? }
+      if reinstatement?
+        self.from_category = owner.reinstatement_membership_category
       end
     end
 
@@ -215,7 +218,6 @@ module EffectiveMembershipsApplicant
     validates :user, presence: true
 
     validates :from_category, presence: true, if: -> { reclassification? }
-    validates :from_status, presence: true, if: -> { reinstatement? }
 
     validate(if: -> { reclassification? && category_id.present? }) do
       errors.add(:category_id, "can't reclassify to existing category") if category_id == from_category_id
@@ -603,10 +605,12 @@ module EffectiveMembershipsApplicant
 
   # Used by the select step
   def can_apply_applicant_types_collection
-    if owner.blank? || owner.membership.blank? || owner.membership.categories.blank?
+    if owner.blank?
       ['Apply to Join']
-    elsif owner.membership.statuses.any?(&:reinstatable?)
+    elsif owner.membership_removed?
       ['Apply for Reinstatement', 'Apply to Join']
+    elsif owner.membership.blank? || owner.membership.categories.blank?
+      ['Apply to Join']
     else
       ['Apply to Reclassify']
     end
@@ -848,6 +852,8 @@ module EffectiveMembershipsApplicant
 
     approved!
 
+    EffectiveMemberships.Registrar.delete_fees!(owner)
+
     if apply_to_join?
       EffectiveMemberships.Registrar.register!(
         owner,
@@ -859,7 +865,7 @@ module EffectiveMembershipsApplicant
     elsif reclassification?
       EffectiveMemberships.Registrar.reclassify!(owner, to: to_category, status: to_status)
     elsif reinstatement?
-      EffectiveMemberships.Registrar.reinstate!(owner, from: from_status)
+      EffectiveMemberships.Registrar.reinstate!(owner, to: to_category)
     else
       raise('unsupported approval applicant_type')
     end

data/app/models/concerns/effective_memberships_owner.rb

@@ -99,6 +99,15 @@ module EffectiveMembershipsOwner
     fees.select { |fee| fee.fee_payment_fee? && !fee.purchased? }
   end
 
+  def outstanding_fee_payments
+    fee_payments = EffectiveMemberships.FeePayment.all
+
+    fee_payments = fee_payments.where(user: self) if self.class.respond_to?(:effective_memberships_user?)
+    fee_payments = fee_payments.where(organization: self) if self.class.respond_to?(:effective_memberships_organization?)
+
+    fee_payments.select { |fee_payment| fee_payment.in_progress? && fee_payment.orders.none?(&:purchased?) }
+  end
+
   def outstanding_fee_payment_orders
     orders.select { |order| order.parent_type.to_s.include?('FeePayment') && !order.purchased? }
   end
@@ -121,8 +130,25 @@ module EffectiveMembershipsOwner
   end
 
   def membership_removed_on
-    return nil unless membership_removed?
-    membership_histories.find { |history| history.removed? }.start_on
+    removed_membership_history.start_on if membership_removed?
+  end
+
+  # The membership history that is removed. Has exit statuses that may affect reinstatement fees.
+  def removed_membership_history
+    membership_histories.reverse.find { |history| history.removed? } if membership_removed?
+  end
+
+  # The membership history we would reinstate to. Should be the one just before removed.
+  def reinstatement_membership_history
+    membership_histories.reverse.find { |history| !history.removed? } if membership_removed?
+  end
+
+  def reinstatement_membership_category
+    reinstatement_membership_history.membership_categories.first if membership_removed?
+  end
+
+  def reinstatement_membership_statuses
+    reinstatement_membership_history.membership_statuses if membership_removed?
   end
 
   def registrar_action_categories(action)
@@ -130,7 +156,7 @@ module EffectiveMembershipsOwner
   end
 
   def registrar_action_statuses(action)
-    EffectiveMemberships.Status.sorted.all.where.not(id: EffectiveMemberships.Registrar.not_in_good_standing_status)
+    EffectiveMemberships.Status.sorted.all
   end
 
   def after_build_fee(fee)
@@ -302,9 +328,7 @@ module EffectiveMembershipsOwner
       EffectiveMemberships.Registrar.in_good_standing!(self)
     end
 
-    if membership_histories.blank?
-      build_membership_history()
-    end
+    build_membership_history() if membership_histories.blank?
 
     save!
   end
@@ -352,9 +376,10 @@ module EffectiveMembershipsOwner
     errors = []
     history = membership_histories.first
     last_history = membership_histories.last
+    was_removed = membership_histories.any? { |history| history.removed? }
 
     # Check membership joined on date matches first history start date
-    if membership.joined_on != history.start_on
+    if membership.joined_on != history.start_on && !was_removed
       errors << "The joined date #{membership.joined_on.strftime('%F')} does not match the first history start date of #{history.start_on.strftime('%F')}. Please change the first history start date to #{membership.joined_on.strftime('%F')} or update the joined date above."
     end
 

data/app/models/concerns/effective_memberships_registrar.rb

@@ -131,9 +131,12 @@ module EffectiveMembershipsRegistrar
     membership.registration_on = date  # Always new registration_on
 
     # Assign Number (this could be nil)
-    number = next_membership_number(owner, to: to) if number.blank?
-    membership.number = number
-    membership.number_as_integer = (Integer(number) rescue nil)
+    number = next_membership_number(owner, to: to) if number.nil?
+
+    if number.present?
+      membership.number = number
+      membership.number_as_integer = (Integer(number) rescue nil)
+    end
 
     # Assign Category
     membership.build_membership_category(category: to)
@@ -173,11 +176,13 @@ module EffectiveMembershipsRegistrar
 
     date ||= Time.zone.now
 
+    # Existing Membership
     membership = owner.membership
 
-    # Assign Category
+    # Last Changed Date
     membership.registration_on = date
 
+    # Assign Category
     membership.build_membership_category(category: to)
     membership.membership_category(category: from).mark_for_destruction
 
@@ -195,24 +200,60 @@ module EffectiveMembershipsRegistrar
     save!(owner, date: date)
   end
 
-  def reinstate!(owner, from:, date: nil, skip_fees: false)
+  def reinstate!(owner, to: nil, date: nil, skip_fees: false)
     raise('expecting a memberships owner') unless owner.class.respond_to?(:effective_memberships_owner?)
-    raise('owner must have an existing membership. use register! instead') if owner.membership.blank?
+    raise('owner must have a removed membership.') unless owner.membership_removed?
 
-    raise('expecting a from') if from.blank?
-    raise('expected a from status or category') unless from.class.respond_to?(:effective_memberships_status?) || from.class.respond_to?(:effective_memberships_category?)
+    history = owner.reinstatement_membership_history
+    raise('owner must have a reinstatement membership history.') if history.blank?
 
+    to ||= owner.reinstatement_membership_category
+    raise('expecting a to memberships category') unless to.class.respond_to?(:effective_memberships_category?)
+
+    statuses = owner.reinstatement_membership_statuses
+
+    # Default Date
     date ||= Time.zone.now
+    period = period(date: date)
+    period_end_on = period_end_on(date: date)
 
-    membership = owner.membership
-    membership.membership_status(status: from).mark_for_destruction if from.class.respond_to?(:effective_memberships_status?)
-    membership.membership_category(category: from).mark_for_destruction if from.class.respond_to?(:effective_memberships_category?)
+    # Build a membership
+    membership = owner.build_membership
 
+    # Assign Dates
+    membership.joined_on = date
+    membership.registration_on = date
+
+    # Assign Number (this could be nil)
+    if history.number.present?
+      membership.number = history.number
+      membership.number_as_integer = (Integer(history.number) rescue nil)
+    end
+
+    # Assign Category
+    membership.build_membership_category(category: to)
+
+    # Assign Statuses
+    statuses.each do |status|
+      membership.build_membership_status(status: status)
+    end
+
+    # Assign fees paid through period
+    if skip_fees
+      membership.fees_paid_period = period
+      membership.fees_paid_through_period = period_end_on
+    end
+
+    # Or, Build Fees
     unless skip_fees
       fee = owner.build_prorated_fee(date: date)
       # This might already be present and purchased if joined and reinstated in the same period
     end
 
+    # Assign member role
+    add_member_role(owner)
+
+    # Save owner
     save!(owner, date: date)
   end
 
@@ -267,16 +308,17 @@ module EffectiveMembershipsRegistrar
     true
   end
 
-  def remove!(owner, date: nil)
+  # When you remove, you can set a status(es) which may affect reinstatement applications
+  def remove!(owner, statuses: nil, date: nil, notes: nil)
+    statuses = Array(statuses)
+
     raise('expecting a memberships owner') unless owner.class.respond_to?(:effective_memberships_owner?)
     raise('expected a member with a membership') unless owner.membership.present?
+    raise('expecting an membership status (optional)') if statuses.present? && statuses.any? { |status| !status.class.respond_to?(:effective_memberships_status?) }
 
     # Date
     date ||= Time.zone.now
 
-    # Remove Membership
-    owner.membership.mark_for_destruction
-
     # Delete unpurchased fees and orders
     owner.outstanding_fee_payment_fees.each { |fee| fee.mark_for_destruction }
     owner.outstanding_fee_payment_orders.each { |order| order.mark_for_destruction }
@@ -284,7 +326,17 @@ module EffectiveMembershipsRegistrar
     # Remove member role
     remove_member_role(owner)
 
-    save!(owner, date: date)
+    # Remove Membership
+    owner.membership.mark_for_destruction
+
+    # We do the save a bit differently here because we want to set the removed statuses
+    history = owner.build_membership_history(start_on: date, notes: notes)
+
+    if statuses.present?
+      history.assign_attributes(statuses: statuses.map(&:to_s), status_ids: statuses.map(&:id))
+    end
+
+    owner.save!
   end
 
   def not_in_good_standing!(owner, date: nil, notes: nil)
@@ -380,24 +432,51 @@ module EffectiveMembershipsRegistrar
     periods(from: from, to: to).reverse.map { |period| [period_end_on(date: period), period] }
   end
 
+  # Called by applicant.approve!
+  def delete_fees!(resource)
+    owner = (resource.class.respond_to?(:effective_memberships_owner?) ? resource : resource.try(:owner))
+    raise('expecting a memberships owner') unless owner.class.respond_to?(:effective_memberships_owner?)
+
+    # Delete unpurchased fees and orders
+    owner.outstanding_fee_payment_fees.each { |fee| fee.mark_for_destruction }
+    owner.outstanding_fee_payment_orders.each { |order| order.mark_for_destruction }
+    owner.save!
+
+    owner.outstanding_fee_payments.each { |fee_payment| fee_payment.destroy! }
+
+    true
+  end
+
   # This is intended to be run once per day in a rake task
   # rake effective_memberships:create_fees
   # Create Renewal and Late fees
-  def create_fees!(period: nil)
+  # Assigns NIGS
+  def create_all_fees!(period: nil)
     period ||= current_period
     memberships = Effective::Membership.deep.with_unpaid_fees_through(period)
 
+    memberships.find_each do |membership|
+      create_fees!(membership, period: period)
+      GC.start
+    end
+
+    true
+  end
+
+  def create_fees!(resource, period: nil)
+    period ||= current_period
+
+    membership = (resource.class.respond_to?(:effective_memberships_owner?) ? resource.membership : resource)
+    raise('expected a membership') unless membership.present? && membership.kind_of?(Effective::Membership)
+
     # Create Renewal Fees
-    memberships.find_each { |membership| create_renewal_fees!(membership, period: period) }
-    GC.start
+    create_renewal_fees!(membership, period: period)
 
     # Create Late Fees
-    memberships.find_each { |membership| create_late_fees!(membership, period: period) }
-    GC.start
+    create_late_fees!(membership, period: period)
 
     # Update Membership Status - Assign Not In Good Standing
-    memberships.find_each { |membership| create_not_in_good_standing!(membership, period: period) }
-    GC.start
+    create_not_in_good_standing!(membership, period: period)
 
     true
   end

data/app/models/effective/membership.rb

@@ -252,9 +252,8 @@ module Effective
       return true if paid_fees_through?(period)
 
       # Otherwise build fees right now
-      EffectiveMemberships.Registrar.create_renewal_fees!(self, period: period)
-      EffectiveMemberships.Registrar.create_late_fees!(self, period: period)
-      EffectiveMemberships.Registrar.create_not_in_good_standing!(self, period: period)
+      EffectiveMemberships.Registrar.delete_fees!(self)
+      EffectiveMemberships.Registrar.create_fees!(self, period: period)
 
       true
     end

data/app/models/effective/membership_history.rb

@@ -25,13 +25,18 @@ module Effective
     serialize :status_ids, Array
 
     scope :deep, -> { includes(:owner) }
-    scope :sorted, -> { order(:start_on) }
+    scope :sorted, -> { order(:start_on).order(:id) }
 
     scope :removed, -> { where(removed: true) }
 
     validates :owner, presence: true
     validates :start_on, presence: true
 
+    with_options(if: -> { removed? }) do
+      validates :number, absence: { message: 'must be blank when removed' }
+      validates :category_ids, absence: { message: 'must be blank when removed' }
+    end
+
     def to_s
       'membership history'
     end

data/app/models/effective/registrar_action.rb

@@ -17,6 +17,9 @@ module Effective
     attr_accessor :membership_number
     attr_accessor :skip_fees
 
+    # Reinstate
+    attr_accessor :skip_fees
+
     # Status Change
     attr_accessor :status_ids
     attr_accessor :status_id
@@ -28,6 +31,7 @@ module Effective
     # Mark Fees Paid - Order Attributes
     attr_accessor :payment_provider
     attr_accessor :payment_card
+    attr_accessor :purchased_at
     attr_accessor :note_to_buyer
     attr_accessor :note_internal
 
@@ -63,6 +67,11 @@ module Effective
       EffectiveMemberships.Registrar.reclassify!(owner, to: category, skip_fees: skip_fees?)
     end
 
+    def reinstate!
+      update!(current_action: :reinstate)
+      EffectiveMemberships.Registrar.reinstate!(owner, skip_fees: skip_fees?)
+    end
+
     def status_assign!
       update!(current_action: :status_assign)
       EffectiveMemberships.Registrar.status_assign!(owner, status: statuses)
@@ -85,7 +94,7 @@ module Effective
 
     def remove!
       update!(current_action: :remove)
-      EffectiveMemberships.Registrar.remove!(owner)
+      EffectiveMemberships.Registrar.remove!(owner, statuses: statuses)
     end
 
     def assign_attributes(atts)
@@ -136,6 +145,7 @@ module Effective
       {
         payment_provider: @payment_provider.presence,
         payment_card: @payment_card.presence,
+        purchased_at: @purchased_at.presence,
         note_to_buyer: @note_to_buyer.presence,
         note_internal: @note_internal.presence
       }.compact

data/app/views/admin/applicants/_form_approve.html.haml

@@ -4,9 +4,13 @@
   - categories = EffectiveMemberships.Category.all
   = f.select :category_id, categories, label: 'Approve to'
 
+  - reinstatement = applicant.owner.reinstatement_membership_history if applicant.reinstatement?
+
   %p
     - if applicant.owner.membership&.number_was.present?
       The member will keep their existing membership number: #{applicant.owner.membership.number}.
+    - elsif applicant.reinstatement? && reinstatement.number.present?
+      The member will keep their previous membership number: #{reinstatement.number}.
     - else
       - number = EffectiveMemberships.Registrar.next_membership_number(applicant.owner, to: applicant.category)
 
@@ -15,8 +19,11 @@
       - else
         No membership number will be assigned.
 
+  - if applicant.reinstatement?
+    %p The member will be reinstated with their previous status: #{badges(reinstatement.membership_statuses) || 'None'}.
+
   %h3 Fees
-  %p The following fee(s) will be created:
+  %p The following fees will be created:
   - month = Time.zone.now.strftime('%B')
 
   %ul
@@ -27,6 +34,19 @@
       %li A #{month} prorated fee to the new category
       %li A #{month} discount fee from their old category
 
+    - if applicant.reinstatement?
+      %li A #{month} prorated fee to their reinstatement category
+
+  %p The following unpurchased fees will be deleted:
+  - outstanding_fee_payment_fees = applicant.owner.outstanding_fee_payment_fees
+
+  %ul
+    - outstanding_fee_payment_fees.each do |fee|
+      %li= fee
+
+    - if outstanding_fee_payment_fees.blank?
+      %li None
+
   %h3 Email to send
   - email_templates = applicant.approve_email_templates
 

data/app/views/admin/memberships/_form.html.haml

@@ -15,3 +15,7 @@
 - if membership.blank? && owner.membership_histories.present?
   = card('History') do
     = render_datatable(Admin::EffectiveMembershipHistoriesDatatable.new(owner: owner), inline: true, simple: true)
+
+- if owner.membership_removed?
+  - registrar_action = Effective::RegistrarAction.new(current_user: current_user, owner: owner)
+  = render 'admin/registrar_actions/form_reinstatement', registrar_action: registrar_action

data/app/views/admin/memberships/_form_membership.html.haml

@@ -23,6 +23,6 @@
 
     = f.select :change_fees_paid_period, collection, label: 'Fees Paid Through', hint: 'Which period this user has fees paid through. Determines which renewal fees should be created. Setting this to a past or blank period may create a renewal fee in the current period.'
 
-    %p.text-muted To update the current membership categories, use the 'Assign' or 'Reclassify' actions below
+    %p.text-muted To update the current membership categories or statuses, use the 'Reclassify' or 'Status Change' actions below
 
     = f.submit 'Update Membership', border: false, center: true, 'data-confirm': "Really update #{f.object.owner}?"

data/app/views/admin/registrar_actions/_form.html.haml

@@ -15,3 +15,6 @@
   .mb-4= render 'admin/registrar_actions/form_not_in_good_standing', registrar_action: registrar_action
   .mb-4= render 'admin/registrar_actions/form_fees_paid', registrar_action: registrar_action
   .mb-4= render 'admin/registrar_actions/form_remove', registrar_action: registrar_action
+
+
+- # The reinstatement form is added by memberships/form and not here

data/app/views/admin/registrar_actions/_form_fees_paid.html.haml

@@ -31,6 +31,7 @@
           %p An order will be created and marked as paid with the following information:
 
           = f.select :payment_provider, EffectiveOrders.admin_payment_providers, required: true
+          = f.datetime_field :purchased_at, label: 'Purchased or completed'
 
           = f.text_field :payment_card,
             label: 'Payment card type, cheque or transaction number',

data/app/views/admin/registrar_actions/_form_reinstatement.html.haml

@@ -0,0 +1,44 @@
+.card
+  .card-body
+    %h5.card-title Reinstate
+
+    = effective_form_with(model: [:admin, registrar_action], url: effective_memberships.admin_registrar_actions_path) do |f|
+      = f.hidden_field :owner_id
+      = f.hidden_field :owner_type
+
+      - removed = f.object.owner.removed_membership_history
+      - reinstatement = f.object.owner.reinstatement_membership_history
+
+      - raise('expected a removed membership history') unless removed.present?
+      - raise('expected a reinstatement membership history') unless reinstatement.present?
+
+      - period = EffectiveMemberships.Registrar.current_period
+
+      %p.text-muted
+        Reinstate to previous membership and optionally create fees.
+
+      %p
+        This member was removed on #{removed.start_on.strftime('%F')} from
+        = reinstatement.membership_categories.first
+        = badges(reinstatement.membership_statuses)
+        - if reinstatement.number.present?
+          with number #{reinstatement.number}.
+
+      = f.check_box :current_action, label: 'Yes, reinstate this member to their previous category, status and number'
+
+      = f.show_if :current_action, true do
+        = f.check_box :skip_fees, label: 'Yes, skip creating fees and just set the category'
+
+        = f.hide_if :skip_fees, true do
+          %p The following fee(s) will be created:
+          - month = Time.zone.now.strftime('%B')
+
+          %ul
+            %li A #{month} prorated fee for the #{reinstatement.membership_categories.first} category
+
+          %p The member will be required to return to the website and make a fee payment
+
+        = f.show_if :skip_fees, true do
+          %p No fees will be created
+
+        = f.submit 'Reinstate', border: false, center: true, 'data-confirm': "Really reinstate #{f.object.owner}?"

data/app/views/admin/registrar_actions/_form_remove.html.haml

@@ -13,5 +13,10 @@
       = f.check_box :current_action, label: 'Yes, remove this member'
 
       = f.show_if :current_action, true do
+        - statuses = f.object.owner.registrar_action_statuses(:remove)
+
+        - if statuses.present?
+          = f.select :status_ids, statuses, label: 'Remove with status', multiple: true, hint: 'Will be assigned to the membership history and may affect fees generated for reinstatement applicants'
+
         = f.submit 'Remove', border: false, center: true,
           'data-confirm': "Really remove #{f.object.owner}?"

data/app/views/effective/applicants/select/_apply_for_reinstatement.html.haml

@@ -1,3 +1 @@
-- membership = f.object.owner.membership
-
-%p Apply for reinstatement from #{badges(f.object.from_status || f.object.from_category)}.
+%p Apply for reinstatement to #{resource.owner.reinstatement_membership_category}.

data/app/views/effective/applicants/select.html.haml

@@ -3,7 +3,8 @@
 
   - categories = resource.can_apply_categories_collection()
   - applicant_types = resource.can_apply_applicant_types_collection()
-  - existing_category = resource.owner&.membership&.category
+  - existing_category = resource.owner.membership&.category
+  - reinstatement_category = resource.owner.reinstatement_membership_category
 
   = card do
     = effective_form_with(model: resource, url: wizard_path(step), method: :put) do |f|
@@ -20,11 +21,6 @@
         = render('effective/applicants/select/apply_to_join', f: f)
         = render('effective/applicants/select/categories', f: f, categories: categories)
 
-      - elsif applicant_types == ['Apply for Reinstatement']
-        = f.hidden_field :applicant_type, value: applicant_types.first
-        = f.hidden_field :category_id, value: existing_category.id
-        = render('effective/applicants/select/apply_for_reinstatement', f: f)
-
       - elsif applicant_types == ['Apply to Reclassify']
         = render('effective/applicants/select/apply_to_reclassify', f: f)
         = render('effective/applicants/select/categories', f: f, categories: categories - [existing_category])
@@ -32,6 +28,11 @@
       - else
         = f.select :applicant_type, applicant_types, label: 'Apply to...'
 
+        - if applicant_types.include?('Apply for Reinstatement')
+          = f.show_if :applicant_type, 'Apply for Reinstatement' do
+            = f.hidden_field :category_id, value: reinstatement_category.id
+            = render('effective/applicants/select/apply_for_reinstatement', f: f)
+
         - if applicant_types.include?('Apply to Join')
           = f.show_if :applicant_type, 'Apply to Join' do
             = render('effective/applicants/select/apply_to_join', f: f)
@@ -42,9 +43,4 @@
             = render('effective/applicants/select/apply_to_reclassify', f: f)
             = render('effective/applicants/select/categories', f: f, categories: categories - [existing_category])
 
-        - if applicant_types.include?('Apply for Reinstatement')
-          = f.show_if :applicant_type, 'Apply for Reinstatement' do
-            = f.hidden_field :category_id, value: existing_category.id
-            = render('effective/applicants/select/apply_for_reinstatement', f: f)
-
       = f.save 'Save and Continue'

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

@@ -35,6 +35,7 @@
 
 - if current_user.membership_removed?
   %p Your membership was removed on #{current_user.membership_removed_on.strftime('%F')}.
+  %p You may Apply for Reinstatement to #{current_user.reinstatement_membership_category} by clicking the Apply to Join button below and then complete the wizard.
 
 - if membership_organizations.present?
   %p You are a representative for #{pluralize(membership_organizations.length, 'member organization')}.

data/lib/effective_memberships/version.rb

@@ -1,3 +1,3 @@
 module EffectiveMemberships
-  VERSION = '0.9.5'
+  VERSION = '0.9.7'
 end

data/lib/tasks/effective_memberships_tasks.rake

@@ -11,7 +11,7 @@ namespace :effective_memberships do
     begin
       if ActiveRecord::Base.connection.table_exists?(:memberships)
         EffectiveLogger.info "Running effective_memberships:create_fees scheduled task" if defined?(EffectiveLogger)
-        EffectiveMemberships.Registrar.create_fees!
+        EffectiveMemberships.Registrar.create_all_fees!
       end
     rescue => e
       ExceptionNotifier.notify_exception(e) if defined?(ExceptionNotifier)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_memberships
 version: !ruby/object:Gem::Version
-  version: 0.9.5
+  version: 0.9.7
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-11-24 00:00:00.000000000 Z
+date: 2022-12-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -383,6 +383,7 @@ files:
 - app/views/admin/registrar_actions/_form_not_in_good_standing.html.haml
 - app/views/admin/registrar_actions/_form_reclassify.html.haml
 - app/views/admin/registrar_actions/_form_register.html.haml
+- app/views/admin/registrar_actions/_form_reinstatement.html.haml
 - app/views/admin/registrar_actions/_form_remove.html.haml
 - app/views/admin/registrar_actions/_form_status_change.html.haml
 - app/views/admin/representatives/_form.html.haml