Fix #1491 as a shop owner I can bulk stop all subscriptions in my shop

Author: chrisjsimpson

subscribie/blueprints/admin/__init__.py

@@ -384,6 +384,61 @@ def stripe_create_charge():
     return jsonify(paymentIntent.status)
 
 
+@background_task
+def do_delete_stripe_subscription(
+    subscription_id, app=None
+):
+    """Stop (delete) stripe_subscription via its
+    Stripe subscription_id
+    """
+    with app.app_context():
+        stripe.api_key = get_stripe_secret_key()
+        connect_account_id = get_stripe_connect_account_id()
+        subscription = Subscription.query.filter_by(
+            stripe_subscription_id=subscription_id
+        ).first()
+
+        log.info(f"Deleting (stopping) Strip subscription {subscription_id}")
+        if subscription_id is None:
+            log.error("subscription_id cannot be None")
+            return False
+        try:
+            try:
+                stripe_subscription = stripe.Subscription.retrieve(
+                    subscription_id, stripe_account=connect_account_id
+                )
+
+                stripe.Subscription.delete(
+                    subscription_id, stripe_account=connect_account_id
+                )
+                log.debug(f"Subscription deleted ({subscription_id})")
+
+                # instead of inserting cancel directly into the db,
+                # it refresh status after cancellation
+                update_stripe_subscription_statuses()
+                flash("Subscription cancelled")
+
+            except stripe._error.InvalidRequestError as e:
+                if e.code == "resource_missing":
+                    msg = (
+                        "stripe subscription id: "
+                        f"{subscription_id} does not exist "
+                        f"for account {connect_account_id}. "
+                        "perhaps show owner has switched from test mode "
+                        "to live mode and has a new stripe connect account"
+                    )
+                    log.warning(msg)
+                    subscription.stripe_status = "resource_missing"
+                    database.session.commit()
+                msg = f"Error could not retrieve subscription ({subscription_id}) to stop (is it deleted already?)"  # noqa: E501
+                log.error(f"{msg}. {e}")
+                return False
+        except Exception as e:
+            msg = f"Error deleting subscription ({subscription_id})"
+            log.error(f"{msg}. {e}")
+            raise
+
+
 @background_task
 def do_pause_stripe_subscription_payment_collection(
     subscription_id, pause_collection_behavior="keep_as_draft", app=None
@@ -481,6 +536,24 @@ def do_pause_all_stripe_subscriptions(app=None):
                     f"Error trying to pause subscription {subscription.uuid}. Error: {e}"  # noqa: E501
                 )
 
+@background_task
+def do_stop_all_stripe_subscriptions(app=None):
+    # For each Subscription object, get it's Stripe subscription
+    # object and stop it using stop_stripe_subscription.
+    with app.app_context():
+        subscriptions = Subscription.query.all()
+        for subscription in subscriptions:
+            log.debug(f"Attempting to stop subscription {subscription.uuid}")
+            stripe_subscription_id = subscription.stripe_subscription_id
+            try:
+                do_delete_stripe_subscription(
+                    stripe_subscription_id, app=current_app
+                )
+            except Exception as e:
+                log.error(
+                    f"Error trying to stop subscription {subscription.uuid}. Error: {e}"  # noqa: E501
+                )
+
 
 @admin.route("/stripe/subscriptions/<subscription_id>/actions/pause")
 @login_required
@@ -521,6 +594,14 @@ def pause_all_subscribers_subscriptions():
     return """All payment collections are being paused in the background. You can move away from this page."""  # noqa: E501
 
 
+@admin.route("/stripe/subscriptions/all/actions/stop")
+@login_required
+def stop_all_subscribers_subscriptions():
+    """Bulk action to stop all subscriptions in the shop"""
+    do_stop_all_stripe_subscriptions()  # Background task
+    return """All subscriptions are being stopped in the background. You can move away from this page."""  # noqa: E501
+
+
 @admin.route("/stripe/subscriptions/<subscription_id>/actions/resume")
 @login_required
 def resume_stripe_subscription(subscription_id):

subscribie/blueprints/admin/templates/admin/dashboard.html

@@ -117,11 +117,11 @@ <h3 class="card-title justify-content-between d-flex">Stats</h3>
               <div class="px-3 py-3 my-3">
                 <p class="card-subtitle mb-3 text-muted">
                     Perform bulk actions across all subscribers such as
-                    pause all Subscribers payment collections.
+                    pause all Subscribers payment collections, or permanently stop all subscriptions.
                 </p>
                 <a class="btn btn-success btn-block"
                     href="{{ url_for('admin.subscribers_bulk_operations_index') }}">
-                    Pause all Subscribers payment collections
+                    Pause all Subscribers payment collections, or stop all collections permanently.
                 </a>
             </div>
         </div>

subscribie/blueprints/admin/templates/admin/subscribers_bulk_operations_index.html

@@ -17,7 +17,7 @@ <h2 class="text-center text-dark mb-3">Subscribers Bulk Operations</h2>
 <main>
   <div class="section">
     <div class="container">
-      {% if 'confirm' not in request.args %}
+      {% if 'confirm' not in request.args and 'confirm_stop_all_subscriptions' not in request.args %}
       <div class="col-md-12">
         <p>These bulk operations effect <em>all</em> Subscribers in your shop. This can save you time, for example,
           if you want to pause payment collection from every subscriber in your shop.</p>
@@ -34,6 +34,37 @@ <h3>Pause payment collection for every Subscription</h3>
           </a>
         </div>
       </div>
+      <hr />
+      <div class="col-md-12">
+        <h3>Stop all Subscriptions</h3>
+        <p>Stop <em>permanently</em> all current Subscriptions in this shop. Stopped Subscriptions cannot be re-started. Invoices are no longer raised
+          for stopped subscriptions. </p>
+        <p>If you intend to resume payment collection for <a href=""></a> Subscription at a later date, then you should pause
+          the subscription rather than stop them.
+        </p>
+        <div class="">
+          <a href="{{ url_for('admin.subscribers_bulk_operations_index', confirm_stop_all_subscriptions=0) }}" class="btn btn-danger">
+            Stop permanently every Subscription in this Shop
+          </a>
+        </div>
+      </div>
+
+      {% elif 'confirm_stop_all_subscriptions' in request.args %}
+      <div class="col-md-12">
+        <h3>Are you sure you want to cancel <em>all</em> Subscriptions for every Subscriber?</h3>
+        <p>Once cancelled, no payment collections will take place, and no future invoices will be generated for those
+          Subscriptions.
+        </p>
+        <p>This shop currently has {{ num_active_subscribers }} active <a href="{{ url_for('admin.subscribers', action='show_active') }}">subscriptions</a>.</p>
+        <div class="">
+          <a href="{{ url_for('admin.stop_all_subscribers_subscriptions', confirm=1) }}" class="btn btn-danger">
+            <span class="cancel-yes">Yes</span>
+          </a>
+          <a href="{{ url_for('admin.subscribers_bulk_operations_index') }}" class="btn btn-success">
+            <span class="cancel-no">No</span>
+          </a>
+        </div>
+      </div>
       {% else %}
       <div class="col-md-12">
         <h3>Are you sure you want to pause payment collection for <em>all</em> Subscriptions for every Subscriber?</h3>