Add various abo fees as mounted attrs (#1590)

Author: njaeggi

app/domain/invoices/sac_memberships/member.rb

@@ -14,7 +14,7 @@ module SacMemberships
     class Member
       attr_reader :person, :context, :sac_membership
 
-      delegate :id, :to_s, :language, :sac_family_main_person?, to: :person
+      delegate :id, :to_s, :language, :sac_family_main_person?, :living_abroad?, to: :person
       delegate :date, :sac_magazine_mailing_list, to: :context
       delegate :zusatzsektion_roles,
         :neuanmeldung_nv_stammsektion_roles, :neuanmeldung_nv_zusatzsektion_roles,
@@ -68,10 +68,6 @@ def sac_ehrenmitglied?
         @sac_ehrenmitglied = sac_membership.sac_ehrenmitglied?
       end
 
-      def living_abroad?
-        !(person.swiss? || person.country.downcase == "li")
-      end
-
       def sac_magazine?
         return @sac_magazine if defined?(@sac_magazine)
 

app/domain/sac_cas.rb

@@ -112,17 +112,6 @@ module SacCas
     .select { |c| c.to_s =~ /MAILING_LIST_.*INTERNAL_KEY/ }
     .map { |c| const_get(c) }
 
-  AboCost = Data.define(:amount, :country)
-  ABO_COSTS = {
-    magazin: [
-      AboCost.new(amount: 60, country: :switzerland),
-      AboCost.new(amount: 76, country: :international)
-    ],
-    tourenportal: [
-      AboCost.new(amount: 45, country: nil)
-    ]
-  }
-
   MEMBERSHIP_OPERATIONS_GROUP_TYPES = [::Group::Sektion.sti_name, ::Group::Ortsgruppe.sti_name].freeze
   MEMBERSHIP_OPERATIONS_EXCLUDED_IDS = [
     2900, 3700, 2249, 2330, 2601, 3030, 3251,

app/mailers/signup/abo_magazin_mailer.rb

@@ -24,12 +24,9 @@ def confirmation(person, group, newsletter_subscribed)
   private
 
   def placeholder_costs
-    cost = if @person.country == "CH"
-      SacCas::ABO_COSTS[:magazin].find { |cost| cost.country == :switzerland }
-    else
-      SacCas::ABO_COSTS[:magazin].find { |cost| cost.country == :international }
-    end
-    formatted_value = number_with_precision(cost.amount,
+    cost = Group.root.abo_alpen_fee
+    cost += Group.root.abo_alpen_postage_abroad if @person.living_abroad?
+    formatted_value = number_with_precision(cost,
       precision: I18n.t("number.currency.format.precision"),
       delimiter: I18n.t("number.currency.format.delimiter"))
 

app/models/group/sac_cas.rb

@@ -26,6 +26,11 @@ class Group::SacCas < Group
   mounted_attr :sac_newsletter_mailing_list_id, :integer
   mounted_attr :sac_magazine_mailing_list_id, :integer
   mounted_attr :sac_fundraising_mailing_list_id, :integer
+  mounted_attr :abo_alpen_fee, :decimal, precision: 10, scale: 2
+  mounted_attr :abo_alpen_postage_abroad, :decimal, precision: 10, scale: 2
+  mounted_attr :abo_touren_portal_fee, :decimal, precision: 10, scale: 2
+  mounted_attr :abo_alpen_fee_article_number, :string
+  mounted_attr :abo_alpen_postage_abroad_article_number, :string
 
   validate :assert_valid_course_admin_email
   validate :assert_mounted_mailing_list_attrs

app/models/sac_cas/person.rb

@@ -107,6 +107,11 @@ def youth?(reference_date: Time.zone.today.end_of_year)
     SacCas::Beitragskategorie::Calculator.new(self, reference_date: reference_date).youth?
   end
 
+  # Liechtenstein is counted as not abroad, extra fees should not apply
+  def living_abroad?
+    !(swiss? || country.downcase == "li")
+  end
+
   def picture_profile_default
     "profile.svg" # default image for profile variant
   end

app/models/wizards/signup/abo_magazin_wizard.rb

@@ -29,16 +29,16 @@ def member_or_applied?
 
     def redirection_message = I18n.t("groups.self_registration.create.already_subscribed_to_abo")
 
-    def costs = SacCas::ABO_COSTS[:magazin]
+    def costs = Group.root.abo_alpen_fee
 
     def requires_policy_acceptance? = false
 
     def calculated_costs
       case step("person_fields").country
-      when "CH"
-        costs.find { |cost| cost.country == :switzerland }.amount
+      when "CH" || "LI"
+        Group.root.abo_alpen_fee
       else
-        costs.find { |cost| cost.country == :international }.amount
+        Group.root.abo_alpen_fee + Group.root.abo_alpen_postage_abroad
       end
     end
 

app/models/wizards/signup/abo_touren_portal_wizard.rb

@@ -25,6 +25,6 @@ def member_or_applied?
 
     def redirection_message = I18n.t("groups.self_registration.create.already_member_of_tourenportal")
 
-    def costs = SacCas::ABO_COSTS[:tourenportal]
+    def costs = Group.root.abo_touren_portal_fee
   end
 end

config/locales/wagon.de.yml

@@ -417,6 +417,11 @@ de:
         sac_magazine_mailing_list_id: Die Alpen Abo ID
         sac_fundraising_mailing_list_id: Spendenaufruf Abo ID
         course_admin_email: E-Mail Kursadministration
+        abo_alpen_fee: Betrag Abonnent Die Alpen
+        abo_alpen_postage_abroad: Betrag Auslandporto Die Alpen
+        abo_touren_portal_fee: Betrag Abonnent Tourenportal
+        abo_alpen_fee_article_number: Artikelnummer Abonnent Die Alpen
+        abo_alpen_postage_abroad_article_number: Artiklenummer Auslandporto Die Alpen
       group/sektion:
         foundation_year: Gründungsjahr
         section_canton: Kanton

spec/mailers/signup/abo_magazin_mailer_spec.rb

@@ -15,7 +15,10 @@
   let(:mail) { described_class.confirmation(person, group, false) }
   let(:custom_content) { CustomContent.get(Signup::AboMagazinMailer::CONFIRMATION) }
 
-  before { person.update(address_care_of: "1A", postbox: "123") }
+  before do
+    person.update(address_care_of: "1A", postbox: "123")
+    Group.root.update!(abo_alpen_fee: 60, abo_alpen_postage_abroad: 16)
+  end
 
   it "sends confirmation email" do
     expect(mail.to).to eq(["e.hillary@hitobito.example.com"])

spec/models/wizards/signup/abo_magazin_wizard_spec.rb

@@ -145,6 +145,10 @@ def build(params = required_attrs)
   end
 
   describe "#calculate_costs" do
+    before do
+      Group.root.update!(abo_alpen_fee: 60, abo_alpen_postage_abroad: 16)
+    end
+
     it "calculates costs for swiss people" do
       expect(wizard.calculated_costs).to eq(60)
     end

spec/support/graphiti/schema.json

@@ -1728,6 +1728,31 @@
           "readable": true,
           "description": null
         },
+        "abo_alpen_fee": {
+          "type": "string",
+          "readable": true,
+          "description": null
+        },
+        "abo_alpen_postage_abroad": {
+          "type": "string",
+          "readable": true,
+          "description": null
+        },
+        "abo_touren_portal_fee": {
+          "type": "string",
+          "readable": true,
+          "description": null
+        },
+        "abo_alpen_fee_article_number": {
+          "type": "string",
+          "readable": true,
+          "description": null
+        },
+        "abo_alpen_postage_abroad_article_number": {
+          "type": "string",
+          "readable": true,
+          "description": null
+        },
         "foundation_year": {
           "type": "string",
           "readable": true,