Should mark tx as a PS one regardless of change calculations in CreateTransaction (#3193)

Author: UdjinM6

src/wallet/wallet.cpp

@@ -3652,6 +3652,10 @@ bool CWallet::CreateTransaction(const std::vector<CRecipient>& vecSend, CWalletT
 
     wtxNew.fTimeReceivedIsTxTime = true;
     wtxNew.BindWallet(this);
+    if (coin_control.nCoinType == CoinType::ONLY_DENOMINATED) {
+        wtxNew.mapValue["DS"] = "1";
+    }
+
     CMutableTransaction txNew;
 
     // Discourage fee sniping.
@@ -3802,7 +3806,6 @@ bool CWallet::CreateTransaction(const std::vector<CRecipient>& vecSend, CWalletT
                     if (coin_control.nCoinType == CoinType::ONLY_DENOMINATED) {
                         nChangePosInOut = -1;
                         nFeeRet += nChange;
-                        wtxNew.mapValue["DS"] = "1";
                     } else {
                         // Fill a vout to ourself
                         newTxOut = CTxOut(nChange, scriptChange);