Steps to reproduce:

1. put `changetype=bech32` in bitcoin.conf or launch QT with `-changetype=bech32`
2. send money to any legacy address
3. use `decoderawtransaction` to see that the change address is `bech32`, as expected since #12119

4. enable Coin Control in settings menu
5. send money to any legacy address
6. use `decoderawtransaction` to see that the change address is now `p2sh-segwit`
uint256 txhash = out.tx->GetHash();
        COutPoint outpt(txhash, out.i);
        if (model->isSpent(outpt))
        {
            coinControl->UnSelect(outpt);
            continue;
        }
        
        nQuantity++;
        
        nAmount += out.tx->tx->vout[out.i].nValue;
        
        CTxDestination address;
        int witnessversion = 0;
        std::vector<unsigned char> witnessprogram;
        if (out.tx->tx->vout[out.i].scriptPubKey.IsWitnessProgram(witnessversion, witnessprogram))
        {
            nBytesInputs += (32 + 4 + 1 + (107 / WITNESS_SCALE_FACTOR) + 4);
            fWitness = true;
        }
        else if(ExtractDestination(out.tx->tx->vout[out.i].scriptPubKey, address))
        {
            CPubKey pubkey;
            CKeyID *keyid = boost::get<CKeyID>(&address);
            if (keyid && model->getPubKey(*keyid, pubkey))
            {
                nBytesInputs += (pubkey.IsCompressed() ? 148 : 180);
            }