effective_memberships 0.9.4 → 0.9.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 611d01eb6651211b44bbc681c6060aa4a74541a08ec9ac59d3b0d1972ceacbfc
-  data.tar.gz: 78796ebd87001ae6e9448f48892d61b034077637ceea0437084a259746824e57
+  metadata.gz: bea643081128f664a7c004dbc49a3e8e06f940781267f85f63c7c6d4e69fe6d1
+  data.tar.gz: a769c9c7216ccdc090fc8a3e6107d654e49eeca2b20a171c48769b46d27a0cb0
 SHA512:
-  metadata.gz: 60bd9e7864f520588a47b4ef69a9460c5cbadbc7fbd0718086566787a16e785348666e87c16c41a6e95826c6b4cd5b93a5c9c67a239b14a5ed82699681e6c279
-  data.tar.gz: 5032cf5a31bda06e919a7abc9009c00ae2d3fcaf25c2889453184839b625cdaa8987c88c95ae7c462c0fb501d6303126b1f2a0b8f14778bbb7d945d49574c9ff
+  metadata.gz: 4553d6b65132b346f36882658baac0ae47ec8945f2f592c19fac705b2e5414d1f4597e5473d2209cdb6a6c0528768a4aa65fe75b59b9712f14ffccdc6c5203ca
+  data.tar.gz: f6c02642c63397a2484fb0d9cd4b3fd471767b9c1b8449528e720ac2f73946a86520094ba39bd0b7056df0544e230a0f1a3bd60ddeb64ba689431664d9c3938b

data/app/datatables/admin/effective_fees_datatable.rb

@@ -21,6 +21,13 @@ module Admin
 
       col :fee_type, search: EffectiveMemberships.fee_types
 
+      col :period, search: { collection: EffectiveMemberships.Registrar.periods_collection } do |fee|
+        EffectiveMemberships.Registrar.period_end_on(date: fee.period)
+      end
+
+      col :category, search: { collection: EffectiveMemberships.Category.all, polymorphic: false }
+      col :title
+
       col :price, as: :price
       col :qb_item_name, label: 'Quickbooks Item Name'
       col :tax_exempt, visible: false
@@ -31,9 +38,8 @@ module Admin
         col :parent, search: :string, visible: false
       end
 
-      col :purchased_order, visible: false
+      col :purchased_order
 
-      col :category, search: { collection: EffectiveMemberships.Category.all, polymorphic: false }
       col :checkout_type, search: Effective::Fee::CHECKOUT_TYPES, visible: false
 
       unless attributes[:total] == false

data/app/models/concerns/effective_memberships_applicant.rb

@@ -623,6 +623,7 @@ module EffectiveMembershipsApplicant
     category_ids = user.membership.category_ids
 
     categories.select do |cat|
+      cat.can_apply_new? ||
       cat.can_apply_existing? ||
       (cat.can_apply_restricted? && (category_ids & cat.can_apply_restricted_ids).present?)
     end

data/app/models/concerns/effective_memberships_registrar.rb

@@ -375,6 +375,11 @@ module EffectiveMembershipsRegistrar
     retval
   end
 
+  def periods_collection(from: nil, to: nil)
+    from ||= Time.zone.now - 10.years
+    periods(from: from, to: to).reverse.map { |period| [period_end_on(date: period), period] }
+  end
+
   # This is intended to be run once per day in a rake task
   # rake effective_memberships:create_fees
   # Create Renewal and Late fees

data/app/models/effective/fee.rb

@@ -44,9 +44,13 @@ module Effective
       self.with_status ||= owner.membership.statuses.first if owner.membership.statuses.length == 1
     end
 
+    before_validation(if: -> { changes['title'].blank? && (changes.keys & ['period', 'category_id', 'fee_type']).present? }) do
+      self.title = default_title()
+    end
+
     before_validation do
       self.period ||= EffectiveMemberships.Registrar.current_period
-      self.title ||= default_title()
+      self.title = default_title() if self.title.blank?
     end
 
     before_validation(if: -> { fee_type == 'Renewal' && category.present? && period.present? }) do

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

@@ -6,45 +6,38 @@
     - collection = { 'Users' => current_user.class.sorted, 'Organizations' => EffectiveMemberships.Organization.sorted }
     = f.select :owner, collection, polymorphic: true
 
+  - registrar = EffectiveMemberships.Registrar
   - fee_types = EffectiveMemberships.fee_types
   - membership_period_fee_types = fee_types.select { |fee_type| Effective::Fee.new(fee_type: fee_type).membership_period_fee? }
 
-  - if f.object.new_record?
-    = f.select :fee_type, fee_types
-  - else
-    = f.static_field :fee_type
-
-  = f.show_if_any(:fee_type, membership_period_fee_types) do
-    = card do
-      %p This is a membership period fee. It will advance the membership fees paid through period when purchased.
-      %p Only one membership period fee -- Prorated or Renewal -- is expected to exist per membership period.
-
-      - registrar = EffectiveMemberships.Registrar
-      - periods = registrar.periods(from: Time.zone.now - 10.years)
-      - collection = periods.reverse.map { |period| [registrar.period_end_on(date: period), period] }
+  - f.object.period ||= registrar.current_period
+  - f.object.checkout_type ||= 'Default'
 
-      - f.object.period ||= registrar.current_period
-      = f.select :period, collection, label: 'Membership period', hint: 'When purchased, set the membership fees paid through period to this period'
+  - f.object.category_id ||= f.object.owner&.membership&.categories&.first&.id
+  - f.object.category ||= f.object.owner&.membership&.categories&.first
 
-      - # Renewal Fees Only
-      - if f.object.persisted? && f.object.renewal_fee?
-        - if f.object.category.create_late_fees?
-          = f.date_field :late_on, hint: 'A late fee will be applied on this date unless purchased'
+  = f.select :period, registrar.periods_collection, label: 'Membership period'
+  = f.select :fee_type, fee_types
+  = f.select :category_id, EffectiveMemberships.Category.all
 
-        - if f.object.category.create_not_in_good_standing?
-          = f.date_field :not_in_good_standing_on, hint: 'The membership will be marked NIGS on this date unless purchased'
+  = f.show_if_any(:fee_type, membership_period_fee_types) do
+    %p.text-muted
+      This is a membership period fee. It will advance the membership fees paid through period when purchased.
+      %br
+      Only one membership period fee -- Prorated or Renewal -- is expected to exist per membership period.
 
-  - f.object.category_id = f.object.owner&.membership&.categories&.first&.id
-  = f.select :category_id, EffectiveMemberships.Category.all
+  - if f.object.category.present?
+    = f.show_if(:fee_type, 'Renewal') do
+      - if f.object.category.create_late_fees?
+        = f.date_field :late_on, hint: 'A late fee will be applied on this date unless purchased. Leave blank to assign the default.'
 
-  - if f.object.parent.blank?
-    - f.object.checkout_type ||= 'Default'
+      - if f.object.category.create_not_in_good_standing?
+        = f.date_field :not_in_good_standing_on, hint: 'The membership will be marked NIGS on this date unless purchased. Leave blank to assign the default.'
 
-    = f.radios :checkout_type, Effective::Fee::CHECKOUT_TYPES, inline: true, label: 'Checkout wizard',
-      hint: 'When default, Applicant and Reinstatement fees will be charged from the applicant wizard, and all other fee types from the fee payment wizard'
-  - else
-    = f.static_field :parent
+  = f.radios :checkout_type, Effective::Fee::CHECKOUT_TYPES, inline: true, label: 'Checkout wizard',
+    hint: 'When default, Applicant and Reinstatement fees will be charged from the applicant wizard, and all other fee types from the fee payment wizard'
 
+  = f.text_field :title, hint: 'Leave blank to assign the default', required: false
   = f.price_field :price
   = f.text_field :qb_item_name, label: 'Quickbooks Item Name'
   = f.check_box :tax_exempt, label: 'Yes, this fee is tax exempt'

data/config/routes.rb

@@ -52,7 +52,7 @@ EffectiveMemberships::Engine.routes.draw do
 
     resources :applicant_reviews, only: [:index, :show]
 
-    resources :fees
+    resources :fees, except: [:show]
     resources :categories, only: [:index, :edit, :update]
 
     resources :applicant_course_areas, except: [:show]

data/lib/effective_memberships/version.rb

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

metadata

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