update

src/main/java/au/com/royalpay/payment/manage/merchants/core/impls/ClientManagerImpl.java

@@ -77,7 +77,6 @@ import au.com.royalpay.payment.manage.notice.core.MailService;
 import au.com.royalpay.payment.manage.signin.beans.TodoNotice;
 import au.com.royalpay.payment.manage.signin.core.ManagerTodoNoticeProvider;
 import au.com.royalpay.payment.manage.signin.core.SignInAccountService;
-import au.com.royalpay.payment.manage.support.sms.SmsSender;
 import au.com.royalpay.payment.manage.system.core.ClientContractService;
 import au.com.royalpay.payment.manage.system.core.MailGunService;
 import au.com.royalpay.payment.manage.tradelog.beans.TradeLogQuery;
@@ -116,6 +115,7 @@ import com.alibaba.fastjson.JSONObject;
 import com.github.miemiedev.mybatis.paginator.domain.Order;
 import com.github.miemiedev.mybatis.paginator.domain.PageBounds;
 import com.github.miemiedev.mybatis.paginator.domain.PageList;
+import com.github.qcloudsms.SmsSingleSender;
 
 import org.apache.commons.io.IOUtils;
 import org.apache.commons.lang3.ArrayUtils;
@@ -308,14 +308,21 @@ public class ClientManagerImpl implements ClientManager, ManagerTodoNoticeProvid
     @Value("${app.agreetemplate.aggregate.path}")
     private String aggregateAgreetemplatePdfPath;
 
+    @Value("${royalpay.sms.appid:1400094878}")
+    private static int appId;
+    @Value("${royalpay.sms.appkey:43390d81e20c5191c278fbf4cd275be2}")
+    private  static String appKey;
     @Resource
     private MpWechatApiProvider mpWechatApiProvider;
 
     private final int REFUSE_CLIENT_TEMPLID = 166108;
+    private SmsSingleSender sender = null;
+
 
     @PostConstruct
     public void init() {
         tags.add("account");
+        sender = new SmsSingleSender(appId, appKey);
     }
 
     @Override
@@ -3052,14 +3059,14 @@ public class ClientManagerImpl implements ClientManager, ManagerTodoNoticeProvid
         clientModifySupport.processClientModify(auditModify);
 
         try {
-            sendMessageToBD(client, refuse_remark);
+//            sendMessageToBD(client, refuse_remark);
             if (client.getIntValue("source") == 4) {
                 String contact_phone = client.getString("contact_phone");
                 if (contact_phone.startsWith("+61")) {
                     ArrayList<String> param = new ArrayList<>();
                     param.add(refuse_remark);
                     try {
-                        SmsSender.getSender().sendWithParam("+61", contact_phone.replace("+61", ""), REFUSE_CLIENT_TEMPLID, param, "RoyalPay", "", "");
+                        sender.sendWithParam("+61", contact_phone.replace("+61", ""), REFUSE_CLIENT_TEMPLID, param, "RoyalPay", "", "");
                     } catch (Exception ignore) {
                         throw new ServerErrorException("Phone number is wrong :" + contact_phone);
                     }
@@ -3746,4 +3753,5 @@ public class ClientManagerImpl implements ClientManager, ManagerTodoNoticeProvid
         MpWechatApi api = mpWechatApiProvider.getNewPaymentApi();
         return api.registerShortUrl(longUrl);
     }
+
 }

src/main/java/au/com/royalpay/payment/manage/support/sms/SmsSender.java

@@ -1,26 +0,0 @@
-package au.com.royalpay.payment.manage.support.sms;
-
-import com.github.qcloudsms.SmsSingleSender;
-
-import org.springframework.beans.factory.annotation.Value;
-import org.springframework.stereotype.Service;
-
-/**
- * @author kira
- * @date 2018/8/1
- */
-@Service
-public class SmsSender {
-    @Value("${royalpay.sms.appid:1400094878}")
-    private static int appId;
-    @Value("${royalpay.sms.appkey:43390d81e20c5191c278fbf4cd275be2}")
-    private  static String appKey;
-
-    private static class SmsSenderIner{
-        private static SmsSingleSender sender = new SmsSingleSender(appId, appKey);
-    }
-
-    public static SmsSingleSender getSender(){
-        return SmsSenderIner.sender;
-    }
-}