Merge branch 'revert-52ce2907' into 'master'

Revert APPM subscription improvement

See merge request entgra/carbon-device-mgt!786

components/application-mgt/org.wso2.carbon.device.application.mgt.core/src/main/java/org/wso2/carbon/device/application/mgt/core/impl/SubscriptionManagerImpl.java

@@ -848,16 +848,16 @@ public class SubscriptionManagerImpl implements SubscriptionManager {
                     }
                 }
 
+                subscriptionDAO.addDeviceSubscription(username, subInsertingDeviceIds, subType,
+                        Operation.Status.PENDING.toString(), applicationReleaseId, tenantId);
                 if (!subUpdatingDeviceIds.isEmpty()) {
                     subscriptionDAO.updateDeviceSubscription(username, subUpdatingDeviceIds, action, subType,
                             Operation.Status.PENDING.toString(), applicationReleaseId, tenantId);
                 }
-
+                subUpdatingDeviceIds.addAll(subInsertingDeviceIds);
-                if (!subInsertingDeviceIds.isEmpty()) {
+                if (!subUpdatingDeviceIds.isEmpty()) {
-                    subscriptionDAO.addDeviceSubscription(username, subInsertingDeviceIds, subType,
-                            Operation.Status.PENDING.toString(), applicationReleaseId, tenantId);
                     List<Integer> deviceSubIds = new ArrayList<>(
-                            subscriptionDAO.getDeviceSubIds(subInsertingDeviceIds, applicationReleaseId, tenantId));
+                            subscriptionDAO.getDeviceSubIds(subUpdatingDeviceIds, applicationReleaseId, tenantId));
                     subscriptionDAO.addOperationMapping(operationId, deviceSubIds, tenantId);
                 }
             }