effective_memberships 0.12.2 → 0.12.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eee6457653137fc81d4b2bbb9523c2158da073a2a3742678f17e305047854b45
-  data.tar.gz: 5ad84fb485691eda2cef81d3b08b7de3c41306e07bd2adf62cef22903b42cdb6
+  metadata.gz: d7b99c254dcb327566bd4de3e1893a6b94874e055f92b03f79a6f5ee156e48c6
+  data.tar.gz: 732a011869b5756269d5869b2465aec166b1cdaa8cafeb6f359114d8b544d382
 SHA512:
-  metadata.gz: da419613c6a9c13734575d280cbfbbc9d25c7cda500db77f3fd062d181b3c1a03de55a9915aedbbd2943e79d7f206be285945b723fb123fbeb8ba39f4d31bd90
-  data.tar.gz: 1f58b027945b279185489021ef228ecd21cf6df2cefd9a473d67ed50002fa1b8a2bc847efd1cf1cd6ad70a6e06357db62439613ab8e0d5e81ce208fafb32c3ea
+  metadata.gz: 0f1ebd7606208e5603b2039368623427b87ca92888f635f7593efb1b822d40e2821c6a4894df4d935a4cdc5ac9269ce80269e17fc99e772dd00bf1123ef04648
+  data.tar.gz: 299039b2e0ec8fce83317e9033e62cb6cc6688156ee78fbd6b296c15ccfb0e6ea75a36de48a32d782b539eb06188fb5d2929e547de44bc827deb026839fde6c8

data/app/models/concerns/effective_memberships_registrar.rb

@@ -19,17 +19,17 @@ module EffectiveMembershipsRegistrar
   end
 
   def renewal_fee_date(date:)
-    Date.new(date.year, 12, 1) # Fees roll over every December 1st
+    Time.zone.local(date.year, 12, 1) # Fees roll over every December 1st
     raise('to be implemented by app registrar')
   end
 
   def late_fee_date(period:)
-    Date.new(period.year, 2, 1) # Fees are late after February 1st
+    Time.zone.local(period.year, 2, 1) # Fees are late after February 1st
     raise('to be implemented by app registrar')
   end
 
   def not_in_good_standing_date(period:)
-    Date.new(period.year, 3, 1) # Membership in bad standing after March 1st
+    Time.zone.local(period.year, 3, 1) # Membership in bad standing after March 1st
     raise('to be implemented by app registrar')
   end
 
@@ -54,7 +54,7 @@ module EffectiveMembershipsRegistrar
     EffectiveMemberships.Status.where(title: 'Not In Good Standing').first!
   end
 
-  # Should two could be overridden if we do non 1-year periods
+  # These two could be overridden if we do non 1-year periods
   def advance_period(period:, number:)
     period.advance(years: number).beginning_of_year
   end
@@ -147,8 +147,8 @@ module EffectiveMembershipsRegistrar
 
     # Or, Build Fees
     unless skip_fees
-      fee = owner.build_prorated_fee(date: date)
-      raise('already has purchased prorated fee') if fee.purchased?
+      fees = build_register_fees(owner: owner, membership: membership, date: date)
+      raise('unexpected purchased fee') if Array(fees).compact.any?(&:purchased?)
     end
 
     # Assign member role
@@ -159,6 +159,10 @@ module EffectiveMembershipsRegistrar
     owner.save!
   end
 
+  def build_register_fees(owner:, membership:, date:)
+    owner.build_prorated_fee(date: date)
+  end
+
   def reclassify!(owner, to:, status: 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?
@@ -187,18 +191,8 @@ module EffectiveMembershipsRegistrar
     membership.build_membership_status(status: status) if status.present?
 
     unless skip_fees
-      existing = owner.membership_period_fee(category: from, period: period)
-
-      if existing
-        fee = owner.build_prorated_fee(date: date)
-        raise('already has purchased prorated fee') if fee.purchased?
-
-        fee = owner.build_discount_fee(date: date, from: from)
-        raise('already has purchased discount fee') if fee.purchased?
-      else
-        fee = owner.build_renewal_fee(category: to, period: period)
-        raise('already has purchased renewal fee') if fee.purchased?
-      end
+      fees = build_reclassify_fees(owner: owner, membership: membership, to: to, from: from, period: period, date: date)
+      raise('unexpected purchased fee') if Array(fees).compact.any?(&:purchased?)
     end
 
     # Save owner
@@ -206,6 +200,19 @@ module EffectiveMembershipsRegistrar
     owner.save!
   end
 
+  def build_reclassify_fees(owner:, membership:, to:, from:, period:, date:)
+    existing = owner.membership_period_fee(category: from, period: period)
+
+    if existing
+      [
+        owner.build_prorated_fee(date: date),
+        owner.build_discount_fee(date: date, from: from)
+      ]
+    else
+      owner.build_renewal_fee(category: to, period: period)
+    end
+  end
+
   def reinstate!(owner, to: nil, date: nil, skip_fees: false)
     raise('expecting a memberships owner') unless owner.class.respond_to?(:effective_memberships_owner?)
 
@@ -251,7 +258,7 @@ module EffectiveMembershipsRegistrar
 
     # Or, Build Fees
     unless skip_fees
-      fee = owner.build_prorated_fee(date: date)
+      build_reinstate_fees(owner: owner, membership: membership, date: date)
       # This might already be present and purchased if joined and reinstated in the same period
     end
 
@@ -263,6 +270,10 @@ module EffectiveMembershipsRegistrar
     owner.save!
   end
 
+  def build_reinstate_fees(owner:, membership:, date:)
+    owner.build_prorated_fee(date: date)
+  end
+
   def status_clear!(owner, date: nil)
     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?
@@ -464,8 +475,7 @@ module EffectiveMembershipsRegistrar
     memberships = Effective::Membership.deep.with_unpaid_fees_through(period)
 
     memberships.find_each do |membership|
-      create_fees!(membership, period: period)
-      GC.start
+      create_fees!(membership, period: period); GC.start
     end
 
     true
@@ -496,9 +506,9 @@ module EffectiveMembershipsRegistrar
 
       fee = membership.owner.build_renewal_fee(category: category, period: period)
       raise("expected build_renewal_fee to return a fee for period #{period}") unless fee.kind_of?(Effective::Fee)
-      next if fee.purchased?
+      next if fee.blank? || fee.persisted?
 
-      puts("Created renewal fee for #{membership.owner}") if fee.new_record? && debug?
+      puts("Created renewal fee for #{membership.owner}") if debug?
 
       fee.save!
     end
@@ -507,16 +517,18 @@ module EffectiveMembershipsRegistrar
   def create_late_fees!(membership, period:)
     membership.categories.select(&:create_late_fees?).map do |category|
       fee = membership.owner.build_late_fee(category: category, period: period)
-      next if fee.blank? || fee.purchased?
+      next if fee.blank? || fee.persisted?
 
-      puts("Created late fee for #{membership.owner}") if fee.new_record? && debug?
+      puts("Created late fee for #{membership.owner}") if debug?
 
       fee.save!
     end
   end
 
   def create_not_in_good_standing!(membership, period:)
+    return unless Time.zone.now.beginning_of_day == not_in_good_standing_date(period: period).beginning_of_day
     return unless membership.categories.any?(&:create_not_in_good_standing?)
+
     membership.owner.update_membership_status!
   end
 

data/app/models/effective/registrar.rb

@@ -4,15 +4,15 @@ module Effective
     include EffectiveMembershipsRegistrar
 
     def renewal_fee_date(date:)
-      Date.new(date.year, 12, 1) # Fees roll over every December 1st
+      Time.zone.local(date.year, 12, 1) # Fees roll over every December 1st
     end
 
     def late_fee_date(period:)
-      Date.new(period.year, 2, 1) # Fees are late after February 1st
+      Time.zone.local(period.year, 2, 1) # Fees are late after February 1st
     end
 
     def not_in_good_standing_date(period:)
-      Date.new(period.year, 3, 1) # Membership are Not In Good Standing after March 1st
+      Time.zone.local(period.year, 3, 1) # Membership are Not In Good Standing after March 1st
     end
 
   end

data/app/views/admin/categories/_fees_additional.html.haml

@@ -0,0 +1 @@
+- # Intentionally blank. Used to extend category fees form.

data/app/views/admin/categories/_fees_applicant.html.haml

@@ -0,0 +1,23 @@
+%h3 Applicant Fees
+.row
+  .col= f.price_field :applicant_fee
+  .col= f.text_field :qb_item_name, label: "Quickbooks Item Name"
+
+%h3 Pro-rated fees based on date of approval
+.row
+  .col-lg-3
+    = f.price_field :prorated_jan, label: "January"
+    = f.price_field :prorated_feb, label: "February"
+    = f.price_field :prorated_mar, label: "March"
+  .col-lg-3
+    = f.price_field :prorated_apr, label: "April"
+    = f.price_field :prorated_may, label: "May"
+    = f.price_field :prorated_jun, label: "June"
+  .col-lg-3
+    = f.price_field :prorated_jul, label: "July"
+    = f.price_field :prorated_aug, label: "August"
+    = f.price_field :prorated_sep, label: "September"
+  .col-lg-3
+    = f.price_field :prorated_oct, label: "October"
+    = f.price_field :prorated_nov, label: "November"
+    = f.price_field :prorated_dec, label: "December"

data/app/views/admin/categories/_fees_late.html.haml

@@ -0,0 +1,10 @@
+%h3 Late Fees
+= f.check_box :create_late_fees, label: 'Yes, late fees should be created'
+
+= f.show_if(:create_late_fees, true) do
+  - period = EffectiveMemberships.Registrar.current_period
+  - date = EffectiveMemberships.Registrar.late_fee_date(period: period)
+
+  %p Charged to members, on #{date.strftime('%B %e')}, with outstanding renewal dues or other fees.
+  .row
+    .col-3= f.price_field :late_fee

data/app/views/admin/categories/_fees_nigs.html.haml

@@ -0,0 +1,8 @@
+%h3 Not In Good Standing
+= f.check_box :create_not_in_good_standing, label: 'Yes, Not in Good Standing status should be applied'
+
+= f.show_if(:create_not_in_good_standing, true) do
+  - period = EffectiveMemberships.Registrar.current_period
+  - date = EffectiveMemberships.Registrar.not_in_good_standing_date(period: period)
+
+  %p Members with outstanding fees, on #{date.strftime('%B %e')}, will automatically be marked #{badge('Not In Good Standing')}.

data/app/views/admin/categories/_fees_renewal.html.haml

@@ -0,0 +1,9 @@
+%h3 Renewal Fees
+= f.check_box :create_renewal_fees, label: 'Yes, annual renewal fees should be created'
+
+= f.show_if(:create_renewal_fees, true) do
+  - date = EffectiveMemberships.Registrar.renewal_fee_date(date: Time.zone.now)
+
+  %p Charged to members, every #{date.strftime('%B %e')}, to renew their dues.
+  .row
+    .col-3= f.price_field :renewal_fee

data/app/views/admin/categories/_form_fees.html.haml

@@ -1,56 +1,7 @@
 = effective_form_with(model: [:admin, category], engine: true) do |f|
-  %h3 Applicant Fees
-  .row
-    .col= f.price_field :applicant_fee
-    .col= f.text_field :qb_item_name, label: "Quickbooks Item Name"
-
-  %h3 Pro-rated fees based on date of approval
-  .row
-    .col-lg-3
-      = f.price_field :prorated_jan, label: "January"
-      = f.price_field :prorated_feb, label: "February"
-      = f.price_field :prorated_mar, label: "March"
-    .col-lg-3
-      = f.price_field :prorated_apr, label: "April"
-      = f.price_field :prorated_may, label: "May"
-      = f.price_field :prorated_jun, label: "June"
-    .col-lg-3
-      = f.price_field :prorated_jul, label: "July"
-      = f.price_field :prorated_aug, label: "August"
-      = f.price_field :prorated_sep, label: "September"
-    .col-lg-3
-      = f.price_field :prorated_oct, label: "October"
-      = f.price_field :prorated_nov, label: "November"
-      = f.price_field :prorated_dec, label: "December"
-
-  %h3 Renewal Fees
-  = f.check_box :create_renewal_fees, label: 'Yes, annual renewal fees should be created'
-
-  = f.show_if(:create_renewal_fees, true) do
-    - date = EffectiveMemberships.Registrar.renewal_fee_date(date: Time.zone.now)
-
-    %p Charged to members, every #{date.strftime('%B %e')}, to renew their dues.
-    .row
-      .col-3= f.price_field :renewal_fee
-
-  %h3 Late Fees
-  = f.check_box :create_late_fees, label: 'Yes, late fees should be created'
-
-  = f.show_if(:create_late_fees, true) do
-    - period = EffectiveMemberships.Registrar.current_period
-    - date = EffectiveMemberships.Registrar.late_fee_date(period: period)
-
-    %p Charged to members, on #{date.strftime('%B %e')}, with outstanding renewal fees
-    .row
-      .col-3= f.price_field :late_fee
-
-  %h3 Bad Standing
-  = f.check_box :create_not_in_good_standing, label: 'Yes, Not in Good Standing status should be applied'
-
-  = f.show_if(:create_not_in_good_standing, true) do
-    - period = EffectiveMemberships.Registrar.current_period
-    - date = EffectiveMemberships.Registrar.not_in_good_standing_date(period: period)
-
-    %p Members with outstanding fees, on #{date.strftime('%B %e')}, will automatically be marked #{badge('Not In Good Standing')}.
-
+  = render('admin/categories/fees_applicant', f: f)
+  = render('admin/categories/fees_renewal', f: f)
+  = render('admin/categories/fees_late', f: f)
+  = render('admin/categories/fees_additional', f: f)
+  = render('admin/categories/fees_nigs', f: f)
   = f.submit

data/lib/effective_memberships/version.rb

@@ -1,3 +1,3 @@
 module EffectiveMemberships
-  VERSION = '0.12.2'
+  VERSION = '0.12.3'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_memberships
 version: !ruby/object:Gem::Version
-  version: 0.12.2
+  version: 0.12.3
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-10 00:00:00.000000000 Z
+date: 2023-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -363,6 +363,11 @@ files:
 - app/views/admin/applicants/_form_process.html.haml
 - app/views/admin/applicants/_form_transcripts.html.haml
 - app/views/admin/applicants/_status.html.haml
+- app/views/admin/categories/_fees_additional.html.haml
+- app/views/admin/categories/_fees_applicant.html.haml
+- app/views/admin/categories/_fees_late.html.haml
+- app/views/admin/categories/_fees_nigs.html.haml
+- app/views/admin/categories/_fees_renewal.html.haml
 - app/views/admin/categories/_form.html.haml
 - app/views/admin/categories/_form_applicant_content.html.haml
 - app/views/admin/categories/_form_applicant_steps.html.haml