Implement the review fixes
This commit is contained in:
+4
-4
@@ -79,15 +79,15 @@ void AuthenticationManagementService::changePassword(const std::string& newPassw
|
|||||||
{
|
{
|
||||||
DataStoreLockGuard lock(m_dataStore);
|
DataStoreLockGuard lock(m_dataStore);
|
||||||
auto& trackedUsersMap = m_dataStore.getUsers();
|
auto& trackedUsersMap = m_dataStore.getUsers();
|
||||||
|
if (m_authenticatedUser == nullptr)
|
||||||
|
{
|
||||||
|
throw std::runtime_error("There is no user currently logged in!");
|
||||||
|
}
|
||||||
int index = trackedUsersMap.find(m_authenticatedUser->getId());
|
int index = trackedUsersMap.find(m_authenticatedUser->getId());
|
||||||
if (index == -1)
|
if (index == -1)
|
||||||
{
|
{
|
||||||
throw std::runtime_error("User does not exist!\n");
|
throw std::runtime_error("User does not exist!\n");
|
||||||
}
|
}
|
||||||
if (m_authenticatedUser == nullptr)
|
|
||||||
{
|
|
||||||
throw std::runtime_error("There is no user currently logged in!");
|
|
||||||
}
|
|
||||||
m_authenticatedUser->setPassword(newPassword);
|
m_authenticatedUser->setPassword(newPassword);
|
||||||
trackedUsersMap.getValueAt(index).state = RecordState::MODIFIED;
|
trackedUsersMap.getValueAt(index).state = RecordState::MODIFIED;
|
||||||
m_dataStore.saveUsers();
|
m_dataStore.saveUsers();
|
||||||
|
|||||||
Reference in New Issue
Block a user