effective_memberships 0.9.15 → 0.9.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e299db8cf3241529169d0f61a9524263dce1b1752fb04cbec395951ae7d1d029
-  data.tar.gz: e04637274efa4c3bffae73bbd27b62588ec3f6c74379050f51c58609f513bbc0
+  metadata.gz: 90078c67d46aca1f293b49ffacac17288bb24592a4007079e6843847845a4e48
+  data.tar.gz: 0063d4f68e0c2ff073c06eb62812006d13f050b50eecbc10163aa80f0a862a04
 SHA512:
-  metadata.gz: 9d01059e8b8e98c6c5ed0838a15a4b126a6d031cbaea7f76adb19146985f687514f5d518ca05bebbf6b9b3735739efd26c63a5d3a970097572d836cadb478c01
-  data.tar.gz: 00da75ee33edb063d76280cfb6bd1c771903ff24bdab9db0bf5c0ca06cc76e0998bd72313705ae0bf2ef90acf0c37dc4703b9fc5b3d15b34808f13606ccf38bc
+  metadata.gz: 01b87e4c9a1aa5664e5bb0fb14ca95a3fac57ea8f2ca0542e379e7e11a1d0450c7aef6fcd4b8d29ab3312e6db33e4a3729b60301b54870e0d7abe20f144df93b
+  data.tar.gz: e71ea11d0e5d126f4e37474726a234440f6e2105131a57fd1a6427c9c2ab897884f3502df0f2150a89318748b494d156a1e11d3fd81334aab8d9c19d9c2ef533

data/app/datatables/admin/effective_fees_datatable.rb

@@ -42,6 +42,9 @@ module Admin
 
       col :checkout_type, search: Effective::Fee::CHECKOUT_TYPES, visible: false
 
+      col :late_on, visible: false
+      col :not_in_good_standing_on, visible: false
+
       unless attributes[:total] == false
         aggregate :total
       end

data/app/models/concerns/effective_memberships_owner.rb

@@ -333,6 +333,25 @@ module EffectiveMembershipsOwner
     save!
   end
 
+  # Delete the last membership history
+  def rollback_membership_history!
+    raise('expected membership to be present') unless membership.present?
+
+    (history, last) = membership_histories.last(2)
+    raise('expected 2 or more membership histories') unless history.present? && last.present?
+
+    last.mark_for_destruction
+    history.assign_attributes(end_on: nil)
+
+    membership.assign_attributes(
+      number: history.number,
+      categories: history.membership_categories,
+      statuses: history.membership_statuses
+    )
+
+    save!
+  end
+
   def build_membership_history(start_on: nil, notes: nil)
     raise('expected membership to be present') unless membership.present?
 

data/app/models/concerns/effective_memberships_registrar.rb

@@ -76,27 +76,8 @@ module EffectiveMembershipsRegistrar
     membership.number ||= number
     membership.number_as_integer ||= (Integer(number) rescue nil)
 
-    # Delete any removed statuses
-    membership.membership_statuses.each do |membership_status|
-      next if statuses.include?(membership_status.status)
-      membership_status.mark_for_destruction
-    end
-
-    # Build any additional statuses
-    statuses.each do |status|
-      membership.build_membership_status(status: status)
-    end
-
-    # Delete any removed categories
-    membership.membership_categories.each do |membership_category|
-      next if categories.include?(membership_category.category)
-      membership_category.mark_for_destruction
-    end
-
-    # Build any additional categories
-    categories.each do |category|
-      membership.build_membership_category(category: category)
-    end
+    # Builds the membership_categories and membership_statuses. Marks for destruction ones that don't exist.
+    membership.assign_attributes(statuses: statuses, categories: categories)
 
     changed = membership.membership_categories.any? { |mc| mc.new_record? || mc.marked_for_destruction? }
     changed ||= membership.membership_statuses.any? { |ms| ms.new_record? || ms.marked_for_destruction? }

data/app/models/effective/membership.rb

@@ -156,6 +156,21 @@ module Effective
       membership_categories.reject(&:marked_for_destruction?).map(&:category_id)
     end
 
+    def categories=(categories)
+      categories = Array(categories)
+      raise('expecting a membership category') if categories.any? { |cat| !cat.class.respond_to?(:effective_memberships_category?) }
+
+      # Delete any removed categories
+      membership_categories.each do |membership_category|
+        membership_category.mark_for_destruction unless categories.include?(membership_category.category)
+      end
+
+      # Build any additional categories
+      categories.each { |category| build_membership_category(category: category) }
+
+      nil
+    end
+
     # We might want to use singular memberships.
     def category
       raise('expected singular usage but there are more than one membership category') if categories.length > 1
@@ -192,6 +207,21 @@ module Effective
       membership_statuses.reject(&:marked_for_destruction?).map(&:status_id)
     end
 
+    def statuses=(statuses)
+      statuses = Array(statuses)
+      raise('expecting a membership category') if statuses.any? { |status| !status.class.respond_to?(:effective_memberships_status?) }
+
+      # Delete any removed statuses
+      membership_statuses.each do |membership_status|
+        membership_status.mark_for_destruction unless statuses.include?(membership_status.status)
+      end
+
+      # Build any additional statuses
+      statuses.each { |status| build_membership_status(status: status) }
+
+      nil
+    end
+
     def statuses_sentence
       statuses.map(&:to_s).to_sentence.presence || 'None'
     end

data/lib/effective_memberships/version.rb

@@ -1,3 +1,3 @@
 module EffectiveMemberships
-  VERSION = '0.9.15'
+  VERSION = '0.9.16'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_memberships
 version: !ruby/object:Gem::Version
-  version: 0.9.15
+  version: 0.9.16
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-01-13 00:00:00.000000000 Z
+date: 2023-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails