effective_memberships 0.7.3 → 0.7.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e5630148361e363550dea41e32b75f80ce1534bd93a4c18873d724c7b0b9cd61
-  data.tar.gz: 46e7aa62a886f604e4ebbf9b80e76f44f3fe5ee933f9e5f468513d412007fd62
+  metadata.gz: 0100c4901a84ee663094b0c2875ea528d7345f68f99659e09f5d6daf973ce54c
+  data.tar.gz: 17df43487cab37459c5ea999396724f556b74f05e7ed853eaf55bfb3f0e3b279
 SHA512:
-  metadata.gz: 91fb28c6ba34f3ac0146219b14f613fb4d19b883963e268d1117bcd56d9f7fcdfb4c5bd807230c53d14d2aefc49eb525867cc23c370948562d881ad79a29029a
-  data.tar.gz: eb61306191817a7bde172a548d2cf373f3210b235857c1a35d6923522a7ffddafbeeff6124ce5ef2e8d2bb9161c32d0de606465738ee88aeb74adc2e15efc92e
+  metadata.gz: 3e06ad2ca2dc09ec15d83a7d413d6cd07663c198f16b6ea8716c0e669dd1671d6141396cf2ce0d903cf47b0c22a2bccf710764981595ee55d43a624bbc91be54
+  data.tar.gz: c0c489c0dfc6deda6be5fdb9ebb70f8647c7afb41a10d8905d59e78b7dcf1ad03d3bc46733169b1089d28d0a767f4c46d8bbc6a26f2459602d88466c93a53426

data/app/models/concerns/effective_memberships_owner.rb

@@ -264,7 +264,7 @@ module EffectiveMembershipsOwner
 
     # Only continue if there is a late renewal fee for the same period
     renewal_fee = fees.find { |fee| fee.fee_type == 'Renewal' && fee.period == period && fee.category_id == category.id && fee.category_type == category.class.name }
-    return unless fee.present? || renewal_fee&.late?
+    return if fee.blank? && (renewal_fee.blank? || renewal_fee.late? == false)
 
     # Build the late fee
     fee ||= fees.build()
@@ -292,7 +292,7 @@ module EffectiveMembershipsOwner
     # Assign in bad standing
     if membership.bad_standing_admin?
       # Nothing to do
-    elsif bad_standing_fees.present?
+    elsif bad_standing_fees.present? && membership.categories.any?(&:create_bad_standing?)
       membership.bad_standing = true
       membership.bad_standing_reason = 'Unpaid Fees'
     else

data/app/models/concerns/effective_memberships_registrar.rb

@@ -401,7 +401,7 @@ module EffectiveMembershipsRegistrar
     GC.start
 
     # Update Membership Status - Assign In Bad Standing
-    memberships.find_each { |membership| update_membership_status!(membership, period: period) }
+    memberships.find_each { |membership| create_bad_standing!(membership, period: period) }
     GC.start
 
     true
@@ -416,7 +416,7 @@ module EffectiveMembershipsRegistrar
       raise("expected build_renewal_fee to return a fee for period #{period}") unless fee.kind_of?(Effective::Fee)
       next if fee.purchased?
 
-      puts("Created renewal fee for #{membership.owner}") if fee.new_record? && !Rails.env.test?
+      puts("Created renewal fee for #{membership.owner}") if fee.new_record? && debug?
 
       fee.save!
     end
@@ -426,10 +426,18 @@ module EffectiveMembershipsRegistrar
     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?
+
+      puts("Created late fee for #{membership.owner}") if fee.new_record? && debug?
+
       fee.save!
     end
   end
 
+  def create_bad_standing!(membership, period:)
+    return unless membership.categories.any?(&:create_bad_standing?)
+    update_membership_status!(membership, period: period)
+  end
+
   def update_membership_status!(membership, period: nil)
     membership.owner.update_membership_status!
   end
@@ -442,6 +450,10 @@ module EffectiveMembershipsRegistrar
 
   protected
 
+  def debug?
+    false
+  end
+
   def add_member_role(owner)
     owner.add_role(:member)
 

data/lib/effective_memberships/version.rb

@@ -1,3 +1,3 @@
 module EffectiveMemberships
-  VERSION = '0.7.3'
+  VERSION = '0.7.4'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: effective_memberships
 version: !ruby/object:Gem::Version
-  version: 0.7.3
+  version: 0.7.4
 platform: ruby
 authors:
 - Code and Effect