Merge branch 'main' into admin-billing

.discourse-compatibility

@@ -1,2 +0,0 @@
-3.1.999: c4f8220a5572ef9a42a9a71500e001d0df206c43
-2.9.0.beta9: 39b7253c082270e33839f4b0821fbf75abfdd826

.github/workflows/discourse-plugin.yml

@@ -1,11 +0,0 @@
-name: Discourse Plugin
-
-on:
-  push:
-    branches:
-      - main
-  pull_request:
-
-jobs:
-  ci:
-    uses: discourse/.github/.github/workflows/discourse-plugin.yml@v1

.github/workflows/plugin-linting.yml

@@ -0,0 +1,57 @@
+name: Linting
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+concurrency:
+  group: plugin-linting-${{ format('{0}-{1}', github.head_ref || github.run_number, github.job) }}
+  cancel-in-progress: true
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16
+          cache: yarn
+
+      - name: Yarn install
+        run: yarn install
+
+      - name: Set up ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 2.7
+          bundler-cache: true
+
+      - name: ESLint
+        if: ${{ always() }}
+        run: yarn eslint --ext .js,.js.es6 --no-error-on-unmatched-pattern {test,assets}/javascripts
+
+      - name: Prettier
+        if: ${{ always() }}
+        shell: bash
+        run: |
+          yarn prettier -v
+          if [ 0 -lt $(find assets -type f \( -name "*.scss" -or -name "*.js" -or -name "*.es6" \) 2> /dev/null | wc -l) ]; then
+            yarn prettier --list-different "assets/**/*.{scss,js,es6}"
+          fi
+          if [ 0 -lt $(find test -type f \( -name "*.js" -or -name "*.es6" \) 2> /dev/null | wc -l) ]; then
+            yarn prettier --list-different "test/**/*.{js,es6}"
+          fi
+
+      - name: Ember template lint
+        if: ${{ always() }}
+        run: yarn ember-template-lint --no-error-on-unmatched-pattern assets/javascripts
+
+      - name: Rubocop
+        if: ${{ always() }}
+        run: bundle exec rubocop .

.github/workflows/plugin-tests.yml

@@ -0,0 +1,156 @@
+name: Plugin Tests
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+concurrency:
+  group: plugin-tests-${{ format('{0}-{1}', github.head_ref || github.run_number, github.job) }}
+  cancel-in-progress: true
+
+jobs:
+  build:
+    name: ${{ matrix.build_type }}
+    runs-on: ubuntu-latest
+    container: discourse/discourse_test:slim${{ startsWith(matrix.build_type, 'frontend') && '-browsers' || '' }}
+    timeout-minutes: 30
+
+    env:
+      DISCOURSE_HOSTNAME: www.example.com
+      RUBY_GLOBAL_METHOD_CACHE_SIZE: 131072
+      RAILS_ENV: test
+      PGUSER: discourse
+      PGPASSWORD: discourse
+
+    strategy:
+      fail-fast: false
+
+      matrix:
+        build_type: ["backend", "frontend-legacy", "frontend"]
+
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: discourse/discourse
+          fetch-depth: 1
+
+      - name: Install plugin
+        uses: actions/checkout@v3
+        with:
+          path: plugins/${{ github.event.repository.name }}
+          fetch-depth: 1
+
+      - name: Setup Git
+        run: |
+          git config --global user.email "[email protected]"
+          git config --global user.name "Discourse CI"
+
+      - name: Start redis
+        run: |
+          redis-server /etc/redis/redis.conf &
+
+      - name: Start Postgres
+        run: |
+          chown -R postgres /var/run/postgresql
+          sudo -E -u postgres script/start_test_db.rb
+          sudo -u postgres psql -c "CREATE ROLE $PGUSER LOGIN SUPERUSER PASSWORD '$PGPASSWORD';"
+
+      - name: Bundler cache
+        uses: actions/cache@v3
+        with:
+          path: vendor/bundle
+          key: ${{ runner.os }}-gem-${{ hashFiles('**/Gemfile.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-gem-
+
+      - name: Setup gems
+        run: |
+          gem install bundler --conservative -v $(awk '/BUNDLED WITH/ { getline; gsub(/ /,""); print $0 }' Gemfile.lock)
+          bundle config --local path vendor/bundle
+          bundle config --local deployment true
+          bundle config --local without development
+          bundle install --jobs 4
+          bundle clean
+
+      - name: Lint English locale
+        if: matrix.build_type == 'backend'
+        run: bundle exec ruby script/i18n_lint.rb "plugins/${{ github.event.repository.name }}/locales/{client,server}.en.yml"
+
+      - name: Get yarn cache directory
+        id: yarn-cache-dir
+        run: echo "::set-output name=dir::$(yarn cache dir)"
+
+      - name: Yarn cache
+        uses: actions/cache@v3
+        id: yarn-cache
+        with:
+          path: ${{ steps.yarn-cache-dir.outputs.dir }}
+          key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-yarn-
+
+      - name: Yarn install
+        run: yarn install
+
+      - name: Fetch app state cache
+        uses: actions/cache@v3
+        id: app-cache
+        with:
+          path: tmp/app-cache
+          key: >-
+            ${{ hashFiles('.github/workflows/tests.yml') }}-
+            ${{ hashFiles('db/**/*', 'plugins/**/db/**/*') }}-
+
+      - name: Restore database from cache
+        if: steps.app-cache.outputs.cache-hit == 'true'
+        run: psql -f tmp/app-cache/cache.sql postgres
+
+      - name: Restore uploads from cache
+        if: steps.app-cache.outputs.cache-hit == 'true'
+        run: rm -rf public/uploads && cp -r tmp/app-cache/uploads public/uploads
+
+      - name: Create and migrate database
+        if: steps.app-cache.outputs.cache-hit != 'true'
+        run: |
+          bin/rake db:create
+          bin/rake db:migrate
+
+      - name: Dump database for cache
+        if: steps.app-cache.outputs.cache-hit != 'true'
+        run: mkdir -p tmp/app-cache && pg_dumpall > tmp/app-cache/cache.sql
+
+      - name: Dump uploads for cache
+        if: steps.app-cache.outputs.cache-hit != 'true'
+        run: rm -rf tmp/app-cache/uploads && cp -r public/uploads tmp/app-cache/uploads
+
+      - name: Check spec existence
+        id: check_spec
+        shell: bash
+        run: |
+          if [ 0 -lt $(find plugins/${{ github.event.repository.name }}/spec -type f -name "*.rb" 2> /dev/null | wc -l) ]; then
+            echo "::set-output name=files_exist::true"
+          fi
+
+      - name: Plugin RSpec
+        if: matrix.build_type == 'backend' && steps.check_spec.outputs.files_exist == 'true'
+        run: bin/rake plugin:spec[${{ github.event.repository.name }}]
+
+      - name: Check qunit existence
+        id: check_qunit
+        shell: bash
+        run: |
+          if [ 0 -lt $(find plugins/${{ github.event.repository.name }}/test/javascripts -type f \( -name "*.js" -or -name "*.es6" \) 2> /dev/null | wc -l) ]; then
+            echo "::set-output name=files_exist::true"
+          fi
+
+      - name: Plugin QUnit
+        if: matrix.build_type == 'frontend-legacy' && steps.check_qunit.outputs.files_exist == 'true'
+        run: QUNIT_EMBER_CLI=0 bundle exec rake plugin:qunit['${{ github.event.repository.name }}','1200000']
+        timeout-minutes: 10
+
+      - name: Plugin QUnit (Ember CLI)
+        if: matrix.build_type == 'frontend' && steps.check_qunit.outputs.files_exist == 'true'
+        run: QUNIT_EMBER_CLI=1 bundle exec rake plugin:qunit['${{ github.event.repository.name }}','1200000']
+        timeout-minutes: 10

.rubocop.yml

@@ -1,5 +1,5 @@
 inherit_gem:
-  rubocop-discourse: stree-compat.yml
+  rubocop-discourse: default.yml
 AllCops:
   Exclude:
     - "gems/**/*"

.streerc

@@ -1,2 +0,0 @@
---print-width=100
---plugins=plugin/trailing_comma

Gemfile

@@ -1,9 +1,8 @@
 # frozen_string_literal: true
 
-source "https://rubygems.org"
+source 'https://rubygems.org'
 
 group :development do
-  gem "translations-manager", git: "https://github.com/discourse/translations-manager.git"
+  gem 'translations-manager', git: 'https://github.com/discourse/translations-manager.git'
-  gem "rubocop-discourse"
+  gem 'rubocop-discourse'
-  gem "syntax_tree"
 end

Gemfile.lock

@@ -8,43 +8,37 @@ GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.2)
-    json (2.6.2)
     parallel (1.22.1)
-    parser (3.1.2.1)
+    parser (3.1.2.0)
       ast (~> 2.4.1)
-    prettier_print (1.2.0)
     rainbow (3.1.1)
-    regexp_parser (2.6.0)
+    regexp_parser (2.5.0)
     rexml (3.2.5)
-    rubocop (1.36.0)
+    rubocop (1.30.1)
-      json (~> 2.3)
       parallel (~> 1.10)
-      parser (>= 3.1.2.1)
+      parser (>= 3.1.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.20.1, < 2.0)
+      rubocop-ast (>= 1.18.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.21.0)
+    rubocop-ast (1.18.0)
       parser (>= 3.1.1.0)
-    rubocop-discourse (3.0)
+    rubocop-discourse (2.5.0)
       rubocop (>= 1.1.0)
       rubocop-rspec (>= 2.0.0)
-    rubocop-rspec (2.13.2)
+    rubocop-rspec (2.11.1)
-      rubocop (~> 1.33)
+      rubocop (~> 1.19)
     ruby-progressbar (1.11.0)
-    syntax_tree (5.1.0)
+    unicode-display_width (2.1.0)
-      prettier_print (>= 1.2.0)
-    unicode-display_width (2.3.0)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
   rubocop-discourse
-  syntax_tree
   translations-manager!
 
 BUNDLED WITH
-   2.3.4
+   2.2.19

app/controllers/concerns/stripe.rb

@@ -4,15 +4,12 @@ module DiscourseSubscriptions
   module Stripe
     extend ActiveSupport::Concern
 
-    AUTH_PROBLEM_IDENTIFIER = "#{DiscourseSubscriptions::PLUGIN_NAME}.stripe_auth_error"
-
     def set_api_key
       ::Stripe.api_key = SiteSetting.discourse_subscriptions_secret_key
     end
 
     def is_stripe_configured?
-      SiteSetting.discourse_subscriptions_public_key.present? &&
+      SiteSetting.discourse_subscriptions_public_key.present? && SiteSetting.discourse_subscriptions_secret_key.present?
-        SiteSetting.discourse_subscriptions_secret_key.present?
     end
   end
 end

app/controllers/discourse_subscriptions/admin/coupons_controller.rb

@@ -18,24 +18,23 @@ module DiscourseSubscriptions
       end
 
       def create
-        params.require(%i[promo discount_type discount active])
+        params.require([:promo, :discount_type, :discount, :active])
         begin
-          coupon_params = { duration: "forever" }
+          coupon_params = {
+            duration: 'forever',
+          }
 
           case params[:discount_type]
-          when "amount"
+          when 'amount'
             coupon_params[:amount_off] = params[:discount].to_i * 100
             coupon_params[:currency] = SiteSetting.discourse_subscriptions_currency
-          when "percent"
+          when 'percent'
             coupon_params[:percent_off] = params[:discount]
           end
 
           coupon = ::Stripe::Coupon.create(coupon_params)
 
-          promo_code =
+          promo_code = ::Stripe::PromotionCode.create({ coupon: coupon[:id], code: params[:promo] }) if coupon.present?
-            ::Stripe::PromotionCode.create(
-              { coupon: coupon[:id], code: params[:promo] },
-            ) if coupon.present?
 
           render_json_dump promo_code
         rescue ::Stripe::InvalidRequestError => e
@@ -44,9 +43,14 @@ module DiscourseSubscriptions
       end
 
       def update
-        params.require(%i[id active])
+        params.require([:id, :active])
         begin
-          promo_code = ::Stripe::PromotionCode.update(params[:id], { active: params[:active] })
+          promo_code = ::Stripe::PromotionCode.update(
+            params[:id],
+            {
+              active: params[:active]
+            }
+          )
 
           render_json_dump promo_code
         rescue ::Stripe::InvalidRequestError => e

app/controllers/discourse_subscriptions/admin/plans_controller.rb

@@ -27,11 +27,15 @@ module DiscourseSubscriptions
             active: params[:active],
             metadata: {
               group_name: params[:metadata][:group_name],
-              trial_period_days: params[:trial_period_days],
+              trial_period_days: params[:trial_period_days]
-            },
+            }
           }
 
-          price_object[:recurring] = { interval: params[:interval] } if params[:type] == "recurring"
+          if params[:type] == 'recurring'
+            price_object[:recurring] = {
+              interval: params[:interval]
+            }
+          end
 
           plan = ::Stripe::Price.create(price_object)
 
@@ -52,16 +56,14 @@ module DiscourseSubscriptions
           end
 
           interval = nil
-          interval = plan[:recurring][:interval] if plan[:recurring] && plan[:recurring][:interval]
+          if plan[:recurring] && plan[:recurring][:interval]
+            interval = plan[:recurring][:interval]
+          end
 
-          serialized =
+          serialized = plan.to_h.merge(trial_period_days: trial_days, currency: plan[:currency].upcase, interval: interval)
-            plan.to_h.merge(
-              trial_period_days: trial_days,
-              currency: plan[:currency].upcase,
-              interval: interval,
-            )
 
           render_json_dump serialized
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
@@ -69,16 +71,15 @@ module DiscourseSubscriptions
 
       def update
         begin
-          plan =
+          plan = ::Stripe::Price.update(
-            ::Stripe::Price.update(
+            params[:id],
-              params[:id],
+            nickname: params[:nickname],
-              nickname: params[:nickname],
+            active: params[:active],
-              active: params[:active],
+            metadata: {
-              metadata: {
+              group_name: params[:metadata][:group_name],
-                group_name: params[:metadata][:group_name],
+              trial_period_days: params[:trial_period_days]
-                trial_period_days: params[:trial_period_days],
+            }
-              },
+          )
-            )
 
           render_json_dump plan
         rescue ::Stripe::InvalidRequestError => e

app/controllers/discourse_subscriptions/admin/products_controller.rb

@@ -27,15 +27,20 @@ module DiscourseSubscriptions
 
       def create
         begin
-          create_params = product_params.merge!(type: "service")
+          create_params = product_params.merge!(type: 'service')
 
-          create_params.except!(:statement_descriptor) if params[:statement_descriptor].blank?
+          if params[:statement_descriptor].blank?
+            create_params.except!(:statement_descriptor)
+          end
 
           product = ::Stripe::Product.create(create_params)
 
-          Product.create(external_id: product[:id])
+          Product.create(
+            external_id: product[:id]
+          )
 
           render_json_dump product
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
@@ -46,6 +51,7 @@ module DiscourseSubscriptions
           product = ::Stripe::Product.retrieve(params[:id])
 
           render_json_dump product
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
@@ -53,9 +59,13 @@ module DiscourseSubscriptions
 
       def update
         begin
-          product = ::Stripe::Product.update(params[:id], product_params)
+          product = ::Stripe::Product.update(
+            params[:id],
+            product_params
+          )
 
           render_json_dump product
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
@@ -68,6 +78,7 @@ module DiscourseSubscriptions
           Product.delete_by(external_id: params[:id])
 
           render_json_dump product
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
@@ -84,8 +95,8 @@ module DiscourseSubscriptions
           statement_descriptor: params[:statement_descriptor],
           metadata: {
             description: params.dig(:metadata, :description),
-            repurchaseable: params.dig(:metadata, :repurchaseable),
+            repurchaseable: params.dig(:metadata, :repurchaseable)
-          },
+          }
         }
       end
     end

app/controllers/discourse_subscriptions/admin/subscriptions_controller.rb

@@ -16,23 +16,20 @@ module DiscourseSubscriptions
             has_more: false,
             data: [],
             length: 0,
-            last_record: params[:last_record],
+            last_record: params[:last_record]
           }
 
           if subscription_ids.present? && is_stripe_configured?
             while subscriptions[:length] < PAGE_LIMIT
               current_set = get_subscriptions(subscriptions[:last_record])
 
-              until valid_subscriptions =
+              until valid_subscriptions = find_valid_subscriptions(current_set[:data], subscription_ids) do
-                      find_valid_subscriptions(current_set[:data], subscription_ids)
                 current_set = get_subscriptions(current_set[:data].last)
                 break if current_set[:has_more] == false
               end
 
               subscriptions[:data] = subscriptions[:data].concat(valid_subscriptions.to_a)
-              subscriptions[:last_record] = current_set[:data].last[:id] if current_set[
+              subscriptions[:last_record] = current_set[:data].last[:id] if current_set[:data].present?
-                :data
-              ].present?
               subscriptions[:length] = subscriptions[:data].length
               subscriptions[:has_more] = current_set[:has_more]
               break if subscriptions[:has_more] == false
@@ -53,11 +50,10 @@ module DiscourseSubscriptions
           refund_subscription(params[:id]) if params[:refund]
           subscription = ::Stripe::Subscription.delete(params[:id])
 
-          customer =
+          customer = Customer.find_by(
-            Customer.find_by(
+            product_id: subscription[:plan][:product],
-              product_id: subscription[:plan][:product],
+            customer_id: subscription[:customer]
-              customer_id: subscription[:customer],
+          )
-            )
 
           Subscription.delete_by(external_id: params[:id])
 
@@ -69,6 +65,7 @@ module DiscourseSubscriptions
           end
 
           render_json_dump subscription
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
@@ -77,11 +74,7 @@ module DiscourseSubscriptions
       private
 
       def get_subscriptions(start)
-        ::Stripe::Subscription.list(
+        ::Stripe::Subscription.list(expand: ['data.plan.product'], limit: PAGE_LIMIT, starting_after: start)
-          expand: ["data.plan.product"],
-          limit: PAGE_LIMIT,
-          starting_after: start,
-        )
       end
 
       def find_valid_subscriptions(data, ids)
@@ -92,11 +85,11 @@ module DiscourseSubscriptions
       # this will only refund the most recent subscription payment
       def refund_subscription(subscription_id)
         subscription = ::Stripe::Subscription.retrieve(subscription_id)
-        invoice = ::Stripe::Invoice.retrieve(subscription[:latest_invoice]) if subscription[
+        invoice = ::Stripe::Invoice.retrieve(subscription[:latest_invoice]) if subscription[:latest_invoice]
-          :latest_invoice
-        ]
         payment_intent = invoice[:payment_intent] if invoice[:payment_intent]
-        refund = ::Stripe::Refund.create({ payment_intent: payment_intent })
+        refund = ::Stripe::Refund.create({
+          payment_intent: payment_intent,
+        })
       end
     end
   end

app/controllers/discourse_subscriptions/hooks_controller.rb

@@ -4,9 +4,7 @@ module DiscourseSubscriptions
   class HooksController < ::ApplicationController
     include DiscourseSubscriptions::Group
     include DiscourseSubscriptions::Stripe
-
     layout false
-
     skip_before_action :check_xhr
     skip_before_action :redirect_to_login_if_required
     skip_before_action :verify_authenticity_token, only: [:create]
@@ -14,59 +12,71 @@ module DiscourseSubscriptions
     def create
       begin
         payload = request.body.read
-        sig_header = request.env["HTTP_STRIPE_SIGNATURE"]
+        sig_header = request.env['HTTP_STRIPE_SIGNATURE']
         webhook_secret = SiteSetting.discourse_subscriptions_webhook_secret
 
         event = ::Stripe::Webhook.construct_event(payload, sig_header, webhook_secret)
       rescue JSON::ParserError => e
-        return render_json_error e.message
+        render_json_error e.message
+        return
       rescue ::Stripe::SignatureVerificationError => e
-        return render_json_error e.message
+        render_json_error e.message
+        return
       end
 
       case event[:type]
-      when "customer.subscription.created"
+      when 'customer.subscription.updated'
-      when "customer.subscription.updated"
+        customer = Customer.find_by(
-        customer =
+          customer_id: event[:data][:object][:customer],
-          Customer.find_by(
+          product_id: event[:data][:object][:plan][:product]
-            customer_id: event[:data][:object][:customer],
+        )
-            product_id: event[:data][:object][:plan][:product],
-          )
 
-        return render_json_error "customer not found" if !customer
+        if customer && subscription_completion?(event)
-        return head 200 if event[:data][:object][:status] != "complete"
+          user = ::User.find(customer.user_id)
-
+          group = plan_group(event[:data][:object][:plan])
-        user = ::User.find_by(id: customer.user_id)
+          group.add(user) if group
-        return render_json_error "user not found" if !user
-
-        if group = plan_group(event[:data][:object][:plan])
-          group.add(user)
-        end
-      when "customer.subscription.deleted"
-        customer =
-          Customer.find_by(
-            customer_id: event[:data][:object][:customer],
-            product_id: event[:data][:object][:plan][:product],
-          )
-
-        return render_json_error "customer not found" if !customer
-
-        Subscription.find_by(
-          customer_id: customer.id,
-          external_id: event[:data][:object][:id],
-        )&.destroy!
-
-        user = ::User.find(customer.user_id)
-        return render_json_error "user not found" if !user
-
-        if group = plan_group(event[:data][:object][:plan])
-          group.remove(user)
         end
 
-        customer.destroy!
+      when 'customer.subscription.deleted'
+        delete_subscription(event)
       end
 
       head 200
     end
+
+    private
+
+    def subscription_completion?(event)
+      subscription_complete?(event) && previously_incomplete?(event)
+    end
+
+    def subscription_complete?(event)
+      event && event[:data] && event[:data][:object] && event[:data][:object][:status] && event[:data][:object][:status] == 'complete'
+    end
+
+    def previously_incomplete?(event)
+      event && event[:data] && event[:data][:previous_attributes] && event[:data][:previous_attributes][:status] && event[:data][:previous_attributes][:status] == 'incomplete'
+    end
+
+    def delete_subscription(event)
+      customer = Customer.find_by(
+        customer_id: event[:data][:object][:customer],
+        product_id: event[:data][:object][:plan][:product]
+      )
+
+      if customer
+        sub_model = Subscription.find_by(
+          customer_id: customer.id,
+          external_id: [:id]
+        )
+
+        sub_model.delete if sub_model
+
+        user = ::User.find(customer.user_id)
+        customer.delete
+        group = plan_group(event[:data][:object][:plan])
+        group.remove(user) if group
+      end
+    end
   end
 end

app/controllers/discourse_subscriptions/subscribe_controller.rb

@@ -5,7 +5,7 @@ module DiscourseSubscriptions
     include DiscourseSubscriptions::Stripe
     include DiscourseSubscriptions::Group
     before_action :set_api_key
-    requires_login except: %i[index contributors show]
+    requires_login except: [:index, :contributors, :show]
 
     def index
       begin
@@ -13,12 +13,19 @@ module DiscourseSubscriptions
         products = []
 
         if product_ids.present? && is_stripe_configured?
-          response = ::Stripe::Product.list({ ids: product_ids, active: true })
+          response = ::Stripe::Product.list({
+            ids: product_ids,
+            active: true
+          })
+
+          products = response[:data].map do |p|
+            serialize_product(p)
+          end
 
-          products = response[:data].map { |p| serialize_product(p) }
         end
 
         render_json_dump products
+
       rescue ::Stripe::InvalidRequestError => e
         render_json_error e.message
       end
@@ -29,11 +36,7 @@ module DiscourseSubscriptions
       contributor_ids = Set.new
 
       campaign_product = SiteSetting.discourse_subscriptions_campaign_product
-      if campaign_product.present?
+      campaign_product.present? ? contributor_ids.merge(Customer.where(product_id: campaign_product).last(5).pluck(:user_id)) : contributor_ids.merge(Customer.last(5).pluck(:user_id))
-        contributor_ids.merge(Customer.where(product_id: campaign_product).last(5).pluck(:user_id))
-      else
-        contributor_ids.merge(Customer.last(5).pluck(:user_id))
-      end
 
       contributors = ::User.where(id: contributor_ids)
 
@@ -46,7 +49,10 @@ module DiscourseSubscriptions
         product = ::Stripe::Product.retrieve(params[:id])
         plans = ::Stripe::Price.list(active: true, product: params[:id])
 
-        response = { product: serialize_product(product), plans: serialize_plans(plans) }
+        response = {
+          product: serialize_product(product),
+          plans: serialize_plans(plans)
+        }
 
         render_json_dump response
       rescue ::Stripe::InvalidRequestError => e
@@ -55,60 +61,47 @@ module DiscourseSubscriptions
     end
 
     def create
-      params.require(%i[source plan])
+      params.require([:source, :plan])
       begin
-        customer =
+        customer = find_or_create_customer(params[:source])
-          find_or_create_customer(
-            params[:source],
-            params[:cardholder_name],
-            params[:cardholder_address],
-          )
         plan = ::Stripe::Price.retrieve(params[:plan])
 
         if params[:promo].present?
           promo_code = ::Stripe::PromotionCode.list({ code: params[:promo] })
           promo_code = promo_code[:data][0] # we assume promo codes have a unique name
 
-          if promo_code.blank?
+          return render_json_error I18n.t("js.discourse_subscriptions.subscribe.invalid_coupon") if promo_code.blank?
-            return render_json_error I18n.t("js.discourse_subscriptions.subscribe.invalid_coupon")
-          end
         end
 
-        recurring_plan = plan[:type] == "recurring"
+        recurring_plan = plan[:type] == 'recurring'
 
         if recurring_plan
-          trial_days = plan[:metadata][:trial_period_days] if plan[:metadata] &&
+          trial_days = plan[:metadata][:trial_period_days] if plan[:metadata] && plan[:metadata][:trial_period_days]
-            plan[:metadata][:trial_period_days]
 
           promo_code_id = promo_code[:id] if promo_code
 
-          transaction =
+          transaction = ::Stripe::Subscription.create(
-            ::Stripe::Subscription.create(
+            customer: customer[:id],
-              customer: customer[:id],
+            items: [{ price: params[:plan] }],
-              items: [{ price: params[:plan] }],
+            metadata: metadata_user,
-              metadata: metadata_user,
+            trial_period_days: trial_days,
-              trial_period_days: trial_days,
+            promotion_code: promo_code_id
-              promotion_code: promo_code_id,
+          )
-            )
 
-          payment_intent = retrieve_payment_intent(transaction[:latest_invoice]) if transaction[
+          payment_intent = retrieve_payment_intent(transaction[:latest_invoice]) if transaction[:status] == 'incomplete'
-            :status
-          ] == "incomplete"
         else
-          coupon_id = promo_code[:coupon][:id] if promo_code && promo_code[:coupon] &&
+          coupon_id = promo_code[:coupon][:id] if promo_code && promo_code[:coupon] && promo_code[:coupon][:id]
-            promo_code[:coupon][:id]
+          invoice_item = ::Stripe::InvoiceItem.create(
-          invoice_item =
+            customer: customer[:id],
-            ::Stripe::InvoiceItem.create(
+            price: params[:plan],
-              customer: customer[:id],
+            discounts: [{ coupon: coupon_id }]
-              price: params[:plan],
+          )
-              discounts: [{ coupon: coupon_id }],
+          invoice = ::Stripe::Invoice.create(
-            )
+            customer: customer[:id]
-          invoice = ::Stripe::Invoice.create(customer: customer[:id])
+          )
           transaction = ::Stripe::Invoice.finalize_invoice(invoice[:id])
-          payment_intent = retrieve_payment_intent(transaction[:id]) if transaction[:status] ==
+          payment_intent = retrieve_payment_intent(transaction[:id]) if transaction[:status] == 'open'
-            "open"
+          transaction = ::Stripe::Invoice.pay(invoice[:id]) if payment_intent[:status] == 'successful'
-          transaction = ::Stripe::Invoice.pay(invoice[:id]) if payment_intent[:status] ==
-            "successful"
         end
 
         finalize_transaction(transaction, plan) if transaction_ok(transaction)
@@ -122,7 +115,7 @@ module DiscourseSubscriptions
     end
 
     def finalize
-      params.require(%i[plan transaction])
+      params.require([:plan, :transaction])
       begin
         price = ::Stripe::Price.retrieve(params[:plan])
         transaction = retrieve_transaction(params[:transaction])
@@ -139,15 +132,17 @@ module DiscourseSubscriptions
 
       group.add(current_user) if group
 
-      customer =
+      customer = Customer.create(
-        Customer.create(
+        user_id: current_user.id,
-          user_id: current_user.id,
+        customer_id: transaction[:customer],
-          customer_id: transaction[:customer],
+        product_id: plan[:product]
-          product_id: plan[:product],
+      )
-        )
 
-      if transaction[:object] == "subscription"
+      if transaction[:object] == 'subscription'
-        Subscription.create(customer_id: customer.id, external_id: transaction[:id])
+        Subscription.create(
+          customer_id: customer.id,
+          external_id: transaction[:id]
+        )
       end
     end
 
@@ -159,47 +154,34 @@ module DiscourseSubscriptions
         name: product[:name],
         description: PrettyText.cook(product[:metadata][:description]),
         subscribed: current_user_products.include?(product[:id]),
-        repurchaseable: product[:metadata][:repurchaseable],
+        repurchaseable: product[:metadata][:repurchaseable]
       }
     end
 
     def current_user_products
       return [] if current_user.nil?
 
-      Customer.select(:product_id).where(user_id: current_user.id).map { |c| c.product_id }.compact
+      Customer
+        .select(:product_id)
+        .where(user_id: current_user.id)
+        .map { |c| c.product_id }.compact
     end
 
     def serialize_plans(plans)
-      plans[:data]
+      plans[:data].map do |plan|
-        .map { |plan| plan.to_h.slice(:id, :unit_amount, :currency, :type, :recurring) }
+        plan.to_h.slice(:id, :unit_amount, :currency, :type, :recurring)
-        .sort_by { |plan| plan[:amount] }
+      end.sort_by { |plan| plan[:amount] }
     end
 
-    def find_or_create_customer(source, cardholder_name = nil, cardholder_address = nil)
+    def find_or_create_customer(source)
       customer = Customer.find_by_user_id(current_user.id)
-      cardholder_address =
-        (
-          if cardholder_address.present?
-            {
-              line1: cardholder_address[:line1],
-              city: cardholder_address[:city],
-              state: cardholder_address[:state],
-              country: cardholder_address[:country],
-              postal_code: cardholder_address[:postalCode],
-            }
-          else
-            nil
-          end
-        )
 
       if customer.present?
         ::Stripe::Customer.retrieve(customer.customer_id)
       else
         ::Stripe::Customer.create(
           email: current_user.email,
-          source: source,
+          source: source
-          name: cardholder_name,
-          address: cardholder_address,
         )
       end
     end

app/controllers/discourse_subscriptions/user/payments_controller.rb

@@ -22,8 +22,7 @@ module DiscourseSubscriptions
               invoices_with_products = parse_invoices(all_invoices, product_ids)
               invoice_ids = invoices_with_products.map { |invoice| invoice[:id] }
               payments = ::Stripe::PaymentIntent.list(customer: customer_id)
-              payments_from_invoices =
+              payments_from_invoices = payments[:data].select { |payment| invoice_ids.include?(payment[:invoice]) }
-                payments[:data].select { |payment| invoice_ids.include?(payment[:invoice]) }
               data = data | payments_from_invoices
             end
           end
@@ -31,6 +30,7 @@ module DiscourseSubscriptions
           data = data.sort_by { |pmt| pmt[:created] }.reverse
 
           render_json_dump data
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
@@ -39,19 +39,16 @@ module DiscourseSubscriptions
       private
 
       def parse_invoices(all_invoices, product_ids)
-        invoices_with_products =
+        invoices_with_products = all_invoices[:data].select do |invoice|
-          all_invoices[:data].select do |invoice|
+          invoice_lines = invoice[:lines][:data][0] if invoice[:lines] && invoice[:lines][:data]
-            invoice_lines = invoice[:lines][:data][0] if invoice[:lines] && invoice[:lines][:data]
+          invoice_product_id = parse_invoice_lines(invoice_lines)
-            invoice_product_id = parse_invoice_lines(invoice_lines)
+          product_ids.include?(invoice_product_id)
-            product_ids.include?(invoice_product_id)
+        end
-          end
       end
 
       def parse_invoice_lines(invoice_lines)
-        invoice_product_id = invoice_lines[:price][:product] if invoice_lines[:price] &&
+        invoice_product_id = invoice_lines[:price][:product] if invoice_lines[:price] && invoice_lines[:price][:product]
-          invoice_lines[:price][:product]
+        invoice_product_id = invoice_lines[:plan][:product] if invoice_lines[:plan] && invoice_lines[:plan][:product]
-        invoice_product_id = invoice_lines[:plan][:product] if invoice_lines[:plan] &&
-          invoice_lines[:plan][:product]
         invoice_product_id
       end
     end

app/controllers/discourse_subscriptions/user/subscriptions_controller.rb

@@ -12,21 +12,24 @@ module DiscourseSubscriptions
         begin
           customer = Customer.where(user_id: current_user.id)
           customer_ids = customer.map { |c| c.id } if customer
-          subscription_ids =
+          subscription_ids = Subscription.where("customer_id in (?)", customer_ids).pluck(:external_id) if customer_ids
-            Subscription.where("customer_id in (?)", customer_ids).pluck(
-              :external_id,
-            ) if customer_ids
 
           subscriptions = []
 
           if subscription_ids
-            plans = ::Stripe::Price.list(expand: ["data.product"], limit: 100)
+            plans = ::Stripe::Price.list(
+              expand: ['data.product'],
+              limit: 100
+            )
 
-            customers =
+            customers = ::Stripe::Customer.list(
-              ::Stripe::Customer.list(email: current_user.email, expand: ["data.subscriptions"])
+              email: current_user.email,
+              expand: ['data.subscriptions']
+            )
 
-            subscriptions =
+            subscriptions = customers[:data].map do |sub_customer|
-              customers[:data].map { |sub_customer| sub_customer[:subscriptions][:data] }.flatten(1)
+              sub_customer[:subscriptions][:data]
+            end.flatten(1)
 
             subscriptions = subscriptions.select { |sub| subscription_ids.include?(sub[:id]) }
 
@@ -38,6 +41,7 @@ module DiscourseSubscriptions
           end
 
           render_json_dump subscriptions
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
@@ -47,41 +51,18 @@ module DiscourseSubscriptions
         # we cancel but don't remove until the end of the period
         # full removal is done via webhooks
         begin
-          subscription = ::Stripe::Subscription.update(params[:id], { cancel_at_period_end: true })
+          subscription = ::Stripe::Subscription.update(params[:id], { cancel_at_period_end: true, })
 
           if subscription
             render_json_dump subscription
           else
-            render_json_error I18n.t("discourse_subscriptions.customer_not_found")
+            render_json_error I18n.t('discourse_subscriptions.customer_not_found')
           end
+
         rescue ::Stripe::InvalidRequestError => e
           render_json_error e.message
         end
       end
-
-      def update
-        params.require(:payment_method)
-
-        subscription = Subscription.where(external_id: params[:id]).first
-        begin
-          attach_method_to_customer(subscription.customer_id, params[:payment_method])
-          subscription =
-            ::Stripe::Subscription.update(
-              params[:id],
-              { default_payment_method: params[:payment_method] },
-            )
-          render json: success_json
-        rescue ::Stripe::InvalidRequestError
-          render_json_error I18n.t("discourse_subscriptions.card.invalid")
-        end
-      end
-
-      private
-
-      def attach_method_to_customer(customer_id, method)
-        customer = Customer.find(customer_id)
-        ::Stripe::PaymentMethod.attach(method, { customer: customer.customer_id })
-      end
     end
   end
 end

app/jobs/regular/manually_update_campaign_data.rb

@@ -2,6 +2,7 @@
 
 module ::Jobs
   class ManuallyUpdateCampaignData < ::Jobs::Base
+
     def execute(args)
       return unless SiteSetting.discourse_subscriptions_campaign_enabled
       DiscourseSubscriptions::Campaign.new.refresh_data

app/serializers/discourse_subscriptions/payment_serializer.rb

@@ -2,14 +2,14 @@
 
 module DiscourseSubscriptions
   class PaymentSerializer < ApplicationSerializer
-    attributes :payment_intent_id,
+    attributes  :payment_intent_id,
-               :receipt_email,
+                :receipt_email,
-               :url,
+                :url,
-               :created_at_age,
+                :created_at_age,
-               :amount,
+                :amount,
-               :amount_currency,
+                :amount_currency,
-               :username,
+                :username,
-               :user_id
+                :user_id
 
     def created_at_age
       Time.now - object.created_at
@@ -19,7 +19,7 @@ module DiscourseSubscriptions
       ActiveSupport::NumberHelper.number_to_currency(
         object.amount / 100,
         precision: 2,
-        unit: currency_unit,
+        unit: currency_unit
       )
     end
 
@@ -32,7 +32,7 @@ module DiscourseSubscriptions
     def user
       begin
         User.find(object.user_id)
-      rescue StandardError
+      rescue
         nil
       end
     end

app/services/discourse_subscriptions/campaign.rb

@@ -17,39 +17,28 @@ module DiscourseSubscriptions
         product_ids = product_ids.include?(campaign_product) ? [campaign_product] : []
       end
 
-      begin
+      amount = 0.00
-        amount = 0.00
+      subscriptions = get_subscription_data
-        subscriptions = get_subscription_data
+      subscriptions = filter_to_subscriptions_products(subscriptions, product_ids)
-        subscriptions = filter_to_subscriptions_products(subscriptions, product_ids)
 
-        # Fetch product purchases
+      # Fetch product purchases
-        one_time_payments = get_one_time_payments(product_ids)
+      one_time_payments = get_one_time_payments(product_ids)
-        one_time_payments.each { |c| amount += c[:price].to_f / 100.00 }
+      one_time_payments.each do |c|
-
+        amount += c[:price].to_f / 100.00
-        # get number of subscribers
-        SiteSetting.discourse_subscriptions_campaign_subscribers = subscriptions&.length.to_i
-
-        # calculate amount raised
-        subscriptions&.each do |sub|
-          sub_amount = calculate_monthly_amount(sub)
-          amount += sub_amount / 100.00
-        end
-
-        SiteSetting.discourse_subscriptions_campaign_amount_raised = amount.round(2)
-
-        check_goal_status
-      rescue ::Stripe::AuthenticationError
-        message =
-          I18n.t(
-            "discourse_subscriptions.auth.invalid",
-            settingsUrl: "admin/site_settings/category/discourse_subscriptions",
-          )
-        AdminDashboardData.add_found_scheduled_check_problem(
-          AdminDashboardData::Problem.new(message, identifier: AUTH_PROBLEM_IDENTIFIER),
-        )
-      else
-        AdminDashboardData.clear_found_problem(AUTH_PROBLEM_IDENTIFIER)
       end
+
+      # get number of subscribers
+      SiteSetting.discourse_subscriptions_campaign_subscribers = subscriptions&.length.to_i
+
+      # calculate amount raised
+      subscriptions&.each do |sub|
+        sub_amount = calculate_monthly_amount(sub)
+        amount += sub_amount / 100.00
+      end
+
+      SiteSetting.discourse_subscriptions_campaign_amount_raised = amount.round(2)
+
+      check_goal_status
     end
 
     def create_campaign
@@ -68,7 +57,7 @@ module DiscourseSubscriptions
     protected
 
     def goal_met_date_key
-      "subscriptions_goal_met_date"
+      'subscriptions_goal_met_date'
     end
 
     def check_goal_status
@@ -102,9 +91,9 @@ module DiscourseSubscriptions
         SiteSetting.discourse_subscriptions_campaign_group = group[:id]
 
         params = {
-          full_name: I18n.t("js.discourse_subscriptions.campaign.supporters"),
+          full_name: I18n.t('js.discourse_subscriptions.campaign.supporters'),
-          title: I18n.t("js.discourse_subscriptions.campaign.supporter"),
+          title: I18n.t('js.discourse_subscriptions.campaign.supporter'),
-          flair_icon: "donate",
+          flair_icon: "donate"
         }
 
         group.update(params)
@@ -115,11 +104,11 @@ module DiscourseSubscriptions
 
     def create_campaign_product
       product_params = {
-        name: I18n.t("js.discourse_subscriptions.campaign.title"),
+        name: I18n.t('js.discourse_subscriptions.campaign.title'),
         active: true,
         metadata: {
-          description: I18n.t("js.discourse_subscriptions.campaign.body"),
+          description: I18n.t('js.discourse_subscriptions.campaign.body'),
-        },
+        }
       }
 
       product = ::Stripe::Product.create(product_params)
@@ -134,9 +123,13 @@ module DiscourseSubscriptions
       monthly_prices = [3, 5, 10, 25]
       yearly_prices = [50, 100]
 
-      monthly_prices.each { |price| create_price(product[:id], group, price, "month") }
+      monthly_prices.each do |price|
+        create_price(product[:id], group, price, "month")
+      end
 
-      yearly_prices.each { |price| create_price(product[:id], group, price, "year") }
+      yearly_prices.each do |price|
+        create_price(product[:id], group, price, "year")
+      end
     end
 
     def create_price(product_id, group_name, amount, recurrence)
@@ -147,11 +140,11 @@ module DiscourseSubscriptions
         currency: SiteSetting.discourse_subscriptions_currency,
         active: true,
         recurring: {
-          interval: recurrence,
+          interval: recurrence
         },
         metadata: {
-          group_name: group_name,
+          group_name: group_name
-        },
+        }
       }
 
       plan = ::Stripe::Price.create(price_object)
@@ -159,13 +152,18 @@ module DiscourseSubscriptions
 
     def get_one_time_payments(product_ids)
       one_time_payments = []
-      current_set = { has_more: true, last_record: nil }
+      current_set = {
+        has_more: true,
+        last_record: nil
+      }
 
       if product_ids.present?
         # lots of matching because the Stripe API doesn't make it easy to match products => payments except from invoices
         until current_set[:has_more] == false
-          all_invoices =
+          all_invoices = ::Stripe::Invoice.list(
-            ::Stripe::Invoice.list(limit: 100, starting_after: current_set[:last_record])
+            limit: 100,
+            starting_after: current_set[:last_record]
+          )
 
           current_set[:last_record] = all_invoices[:data].last[:id] if all_invoices[:data].present?
           current_set[:has_more] = all_invoices[:has_more]
@@ -175,8 +173,11 @@ module DiscourseSubscriptions
             next if invoice[:paid] != true
             line_item = invoice[:lines][:data][0] if invoice[:lines] && invoice[:lines][:data] # Discourse only makes single-line item charges
             # check if non-subscription and that the plan is active
-            if line_item[:plan] == nil && line_item[:price] &&
+            if line_item[:plan] == nil &&
-                 line_item[:price][:recurring] == nil && line_item[:price][:active] == true
+               line_item[:price] &&
+               line_item[:price][:recurring] == nil &&
+               line_item[:price][:active] == true
+
               product_id = line_item[:price][:product]
               if product_ids.include? product_id
                 line_data = {
@@ -196,15 +197,17 @@ module DiscourseSubscriptions
 
     def get_subscription_data
       subscriptions = []
-      current_set = { has_more: true, last_record: nil }
+      current_set = {
+        has_more: true,
+        last_record: nil
+      }
 
       until current_set[:has_more] == false
-        current_set =
+        current_set = ::Stripe::Subscription.list(
-          ::Stripe::Subscription.list(
+          expand: ['data.plan.product'],
-            expand: ["data.plan.product"],
+          limit: 100,
-            limit: 100,
+          starting_after: current_set[:last_record]
-            starting_after: current_set[:last_record],
+        )
-          )
 
         current_set[:last_record] = current_set[:data].last[:id] if current_set[:data].present?
         subscriptions.concat(current_set[:data].to_a)
@@ -214,14 +217,13 @@ module DiscourseSubscriptions
     end
 
     def filter_to_subscriptions_products(data, ids)
-      valid =
+      valid = data.select do |sub|
-        data.select do |sub|
+        # cannot .dig stripe objects
-          # cannot .dig stripe objects
+        items = sub[:items][:data][0] if sub[:items] && sub[:items][:data]
-          items = sub[:items][:data][0] if sub[:items] && sub[:items][:data]
+        product = items[:price][:product] if items[:price] && items[:price][:product]
-          product = items[:price][:product] if items[:price] && items[:price][:product]
 
-          ids.include?(product)
+        ids.include?(product)
-        end
+      end
       valid.empty? ? nil : valid
     end
 

assets/javascripts/discourse/components/create-coupon-form.hbs

@@ -1,50 +0,0 @@
-<div class="create-coupon-form">
-  <form class="form-horizontal">
-    <p>
-      <label for="promo_code">
-        {{i18n "discourse_subscriptions.admin.coupons.promo_code"}}
-      </label>
-      <Input @type="text" name="promo_code" @value={{promoCode}} />
-    </p>
-
-    <p>
-      <label for="amount">
-        {{i18n "discourse_subscriptions.admin.coupons.discount"}}
-      </label>
-      <ComboBox
-        @content={{discountTypes}}
-        @value={{discountType}}
-        @onChange={{action (mut discountType)}}
-      />
-      <Input
-        class="discount-amount"
-        @type="text"
-        name="amount"
-        @value={{discount}}
-      />
-    </p>
-
-    <p>
-      <label for="active">
-        {{i18n "discourse_subscriptions.admin.coupons.active"}}
-      </label>
-      <Input @type="checkbox" name="active" @checked={{active}} />
-    </p>
-  </form>
-
-  <DButton
-    @action={{action "createNewCoupon"}}
-    @label="discourse_subscriptions.admin.coupons.create"
-    @title="discourse_subscriptions.admin.coupons.create"
-    @icon="plus"
-    class="btn-primary btn btn-icon"
-  />
-
-  <DButton
-    @action={{action "cancelCreate"}}
-    label="cancel"
-    @title="cancel"
-    @icon="times"
-    class="btn btn-icon"
-  />
-</div>

assets/javascripts/discourse/components/ds-button.js

@@ -0,0 +1,12 @@
+import Button from "discourse/components/d-button";
+
+export default Button.extend({
+  seleted: false,
+
+  init() {
+    this._super(...arguments);
+    this.classNameBindings = this.classNameBindings.concat(
+      "selected:btn-primary"
+    );
+  },
+});

assets/javascripts/discourse/components/login-required.hbs

@@ -1,8 +0,0 @@
-<h3>{{i18n "discourse_subscriptions.subscribe.unauthenticated"}}</h3>
-
-<DButton
-  @label="log_in"
-  @action={{route-action "showLogin"}}
-  @icon="user"
-  class="btn btn-primary login-required subscriptions"
-/>

assets/javascripts/discourse/components/login-required.js

@@ -0,0 +1,17 @@
+import Component from "@ember/component";
+import cookie from "discourse/lib/cookie";
+import DiscourseURL from "discourse/lib/url";
+import { default as getURL } from "discourse-common/lib/get-url";
+
+export default Component.extend({
+  classNames: ["login-required", "subscriptions"],
+  actions: {
+    createAccount() {
+      const destinationUrl = window.location.href;
+      const cookiePath = getURL("/");
+
+      cookie("destination_url", destinationUrl, { path: cookiePath });
+      DiscourseURL.redirectTo("/login");
+    },
+  },
+});

assets/javascripts/discourse/components/modal/admin-cancel-subscription.gjs

@@ -1,38 +0,0 @@
-import DButton from "discourse/components/d-button";
-import DModal from "discourse/components/d-modal";
-import Component from "@glimmer/component";
-import { fn, hash } from "@ember/helper";
-import i18n from "discourse-common/helpers/i18n";
-import { Input } from "@ember/component";
-import { tracked } from "@glimmer/tracking";
-
-export default class AdminCancelSubscription extends Component {
-  @tracked refund;
-
-  <template>
-    <DModal
-      @title={{i18n
-        "discourse_subscriptions.user.subscriptions.operations.destroy.confirm"
-      }}
-      @closeModal={{@closeModal}}
-    >
-      <:body>
-        <Input @type="checkbox" @checked={{this.refund}} />
-        {{i18n "discourse_subscriptions.admin.ask_refund"}}
-      </:body>
-      <:footer>
-        <DButton
-          @label="yes_value"
-          @action={{fn
-            @model.cancelSubscription
-            (hash [email protected] refund=this.refund)
-          }}
-          @icon="times"
-          @isLoading={{@model.subscription.loading}}
-          class="btn-danger"
-        />
-        <DButton @label="no_value" @action={{@closeModal}} />
-      </:footer>
-    </DModal>
-  </template>
-}

assets/javascripts/discourse/components/payment-plan.hbs

@@ -1,21 +0,0 @@
-<DButton
-  @action={{action "planClick"}}
-  class={{concat-class "btn-discourse-subscriptions-subscribe" selectedClass}}
->
-  <span class="interval">
-    {{#if this.recurringPlan}}
-      {{i18n
-        (concat
-          "discourse_subscriptions.plans.interval.adverb."
-          this.plan.recurring.interval
-        )
-      }}
-    {{else}}
-      {{i18n "discourse_subscriptions.one_time_payment"}}
-    {{/if}}
-  </span>
-
-  <span class="amount">
-    {{format-currency this.plan.currency this.plan.amountDollars}}
-  </span>
-</DButton>

assets/javascripts/discourse/components/payment-plan.js

@@ -4,11 +4,9 @@ import discourseComputed from "discourse-common/utils/decorators";
 const RECURRING = "recurring";
 
 export default Component.extend({
-  tagName: "",
-
   @discourseComputed("selectedPlan")
-  selectedClass(planId) {
+  selected(planId) {
-    return planId === this.plan.id ? "btn-primary" : "";
+    return planId === this.plan.id;
   },
 
   @discourseComputed("plan.type")

assets/javascripts/discourse/components/subscribe-ca-province-select.js

@@ -1,38 +0,0 @@
-import ComboBoxComponent from "select-kit/components/combo-box";
-import { computed } from "@ember/object";
-import I18n from "I18n";
-
-export default ComboBoxComponent.extend({
-  pluginApiIdentifiers: ["subscribe-ca-province-select"],
-  classNames: ["subscribe-address-state-select"],
-  nameProperty: "name",
-  valueProperty: "value",
-
-  selectKitOptions: {
-    filterable: true,
-    allowAny: false,
-    translatedNone: I18n.t(
-      "discourse_subscriptions.subscribe.cardholder_address.province"
-    ),
-  },
-
-  content: computed(function () {
-    return [
-      ["AB", "Alberta"],
-      ["BC", "British Columbia"],
-      ["MB", "Manitoba"],
-      ["NB", "New Brunswick"],
-      ["NL", "Newfoundland and Labrador"],
-      ["NT", "Northwest Territories"],
-      ["NS", "Nova Scotia"],
-      ["NU", "Nunavut"],
-      ["ON", "Ontario"],
-      ["PE", "Prince Edward Island"],
-      ["QC", "Quebec"],
-      ["SK", "Saskatchewan"],
-      ["YT", "Yukon"],
-    ].map((arr) => {
-      return { value: arr[0], name: arr[1] };
-    });
-  }),
-});

assets/javascripts/discourse/components/subscribe-card.hbs

@@ -1 +0,0 @@
-<div id="card-element"></div>

assets/javascripts/discourse/components/subscribe-card.js

@@ -4,25 +4,6 @@ export default Component.extend({
   didInsertElement() {
     this._super(...arguments);
     this.cardElement.mount("#card-element");
-    this.setCardElementStyles();
-  },
-
-  setCardElementStyles() {
-    const root = document.querySelector(":root");
-    const computedStyle = getComputedStyle(root);
-    const primaryColor = computedStyle.getPropertyValue("--primary");
-    const placeholderColor =
-      computedStyle.getPropertyValue("--secondary-medium");
-    this.cardElement.update({
-      style: {
-        base: {
-          color: primaryColor,
-          "::placeholder": {
-            color: placeholderColor,
-          },
-        },
-      },
-    });
   },
   didDestroyElement() {},
 });