There is an assert in `CWalletTx::RelayWalletTransaction` that gets hit.  All other code paths check `fBroadcastTransactions` before getting to `CWallet::ResendWalletTransactionsBefore` except the rpc call `resendwalletransactions`.

It's an easy fix, but we just need to decide whether to throw an error or return an empty array of rebroadcasted transaction ids.

Should be fixed for 0.15.
{
    LOCK(cs_wallet);
    auto mi = mapWallet.find(originalHash);
    
    assert(mi != mapWallet.end());
    CWalletTx& wtx = (*mi).second;
    
    assert(wtx.mapValue.count("replaced_by_txid") == 0);
    wtx.mapValue["replaced_by_txid"] = newHash.ToString();
    CWalletDB walletdb(*dbw, "r+");
    bool success = true;
    if (!walletdb.WriteTx(wtx)) {
        LogPrintf("%s: Updating walletdb tx %s failed", __func__, wtx.GetHash().ToString());
        success = false;
    }
    NotifyTransactionChanged(this, originalHash, CT_UPDATED);
    return success;
}
bool CWallet::AddToWallet(const CWalletTx& wtxIn, bool fFlushOnClose)
{
    LOCK(cs_wallet);
    CWalletDB walletdb(*dbw, "r+", fFlushOnClose);
    uint256 hash = wtxIn.GetHash();