From ccf87f07a71d64decb047bd0239c6da070d28a01 Mon Sep 17 00:00:00 2001
From: Kirill Solomko <ksolomko@sipwise.com>
Date: Tue, 9 Apr 2024 12:56:02 +0200
Subject: [PATCH] MT#56301 move lock_provisioning_voip_subscriber before
 underrun_lock

* POST /api/subscribers/ invoke lock_provisioning_voip_subscriber()
  before underrun_lock(), so that if the subscriber is about
  to be locked due to customer balance costs, the
  lock_provisioning_lock_subscriber() invocation will not unlock it
  back straight after

Change-Id: I55020f844c9aa76df2e2f057a88b2ae7c9ebbfcc
(cherry picked from commit c6aaaddd87067fb573df7a8d127541689d2d5b28)
---
 lib/NGCP/Panel/Controller/API/Subscribers.pm | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/lib/NGCP/Panel/Controller/API/Subscribers.pm b/lib/NGCP/Panel/Controller/API/Subscribers.pm
index cf14b837f7..b645511df0 100644
--- a/lib/NGCP/Panel/Controller/API/Subscribers.pm
+++ b/lib/NGCP/Panel/Controller/API/Subscribers.pm
@@ -417,13 +417,12 @@ sub POST :Allow {
                     level => $resource->{lock} || 4,
                 );
             } else {
-                NGCP::Panel::Utils::ProfilePackages::underrun_lock_subscriber(c => $c, subscriber => $subscriber);
-
                 NGCP::Panel::Utils::Subscriber::lock_provisoning_voip_subscriber(
                     c => $c,
                     prov_subscriber => $subscriber->provisioning_voip_subscriber,
                     level => $resource->{lock} || 0,
                 ) if exists $resource->{lock};
+                NGCP::Panel::Utils::ProfilePackages::underrun_lock_subscriber(c => $c, subscriber => $subscriber);
             }
             NGCP::Panel::Utils::Subscriber::update_subscriber_numbers(
                 c              => $c,