Move delete networks out of looping over the clients in CUser::Clone

This was moved in the previous commit when I tried to merge it to the
commit below.

src/User.cpp

@@ -647,8 +647,9 @@ bool CUser::Clone(const CUser& User, CString& sErrorRet, bool bCloneNetworks) {
 				// This line will remove pClient from vClients,
 				// because it's a reference to the internal Network's vector.
 				pClient->SetNetwork(NULL);
-				DeleteNetwork(*it);
 			}
+
+			DeleteNetwork(*it);
 		}
 	}
 	// !Networks