From 8762d6742efd0a3dbaa21de9c6cbd35f8bc54e82 Mon Sep 17 00:00:00 2001 From: Blake Erickson Date: Wed, 22 Jun 2022 18:51:14 -0500 Subject: [PATCH] FIX: Import script confirmation flag logic (#126) The import script has a prompt asking if this is a "procourse" import or not, but the logic for handling the answer is backwards and will incorrectly set the `procourse_import` flag to `true`. This commit also fixes a minor typo. --- lib/tasks/subscriptions_import.rake | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/tasks/subscriptions_import.rake b/lib/tasks/subscriptions_import.rake index 0218388..cc50b64 100644 --- a/lib/tasks/subscriptions_import.rake +++ b/lib/tasks/subscriptions_import.rake @@ -11,7 +11,7 @@ task 'subscriptions:subscriptions_import' => :environment do procourse_import = false procourse_import_response = ask("Were the subscriptions you are importing created in Procourse Memberships?: (y/N)") - if procourse_import_response.downcase != 'y' + if procourse_import_response.downcase == 'y' procourse_import = true end @@ -147,10 +147,10 @@ def import_subscriptions(procourse_import) discourse_user = User.find(user_id) puts "Discourse User: #{discourse_user.username_lower} found for Strip metadata update ..." - updated_subsciption = Stripe::Subscription.update(subscription_id, + updated_subscription = Stripe::Subscription.update(subscription_id, { metadata: { user_id: user_id, username: discourse_user.username_lower } }) - puts "Stripe Subscription: #{updated_subsciption[:id]}, metadata: #{updated_subsciption[:metadata]} UPDATED" + puts "Stripe Subscription: #{updated_subscription[:id]}, metadata: #{updated_subscription[:metadata]} UPDATED" updated_customer = Stripe::Customer.update(customer_id, { email: discourse_user.email }) puts "Stripe Customer: #{updated_customer[:id]}, email: #{updated_customer[:email]} UPDATED"