Commit dcdac7f2 authored by Sven Strickroth's avatar Sven Strickroth

Do not store remote URL if not used in push/pull/fetch dialogs

Thanks to XhmikosR for spotting this.
Signed-off-by: Sven Strickroth's avatarSven Strickroth <[email protected]>
parent 844dd96b
......@@ -301,11 +301,13 @@ void CPullFetchDlg::OnBnClickedOk()
{
m_Other.GetWindowTextW(m_RemoteURL);
m_RemoteBranchName=m_RemoteBranch.GetString();
// only store URL in history if it's value was used
m_Other.SaveHistory();
}
m_RemoteReg = m_Remote.GetString();
m_Other.SaveHistory();
m_RemoteBranch.SaveHistory();
this->m_regRebase=this->m_bRebase;
......
......@@ -427,7 +427,8 @@ void CPushDlg::OnBnClickedOk()
else
{
// do not store branch names on removal
this->m_RemoteURL.SaveHistory();
if (m_RemoteURL.IsWindowEnabled())
m_RemoteURL.SaveHistory(); // do not store Remote URLs if not used
this->m_BranchRemote.SaveHistory();
m_RemoteReg = m_Remote.GetString();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment