From 2db61d00612cf97d1c733277bc8828b23446969c Mon Sep 17 00:00:00 2001 From: Elena Mamontova Date: Wed, 27 Jul 2016 17:52:37 +0300 Subject: [PATCH 1/1] Ticket 37. The switch case construction for changePolicy checking added in the ApplyChanges() function. --- .../configuration/sgconfig/parser_users.cpp | 54 +++++++++++++++++-- 1 file changed, 50 insertions(+), 4 deletions(-) diff --git a/projects/stargazer/plugins/configuration/sgconfig/parser_users.cpp b/projects/stargazer/plugins/configuration/sgconfig/parser_users.cpp index dbd82b0b..ce07b094 100644 --- a/projects/stargazer/plugins/configuration/sgconfig/parser_users.cpp +++ b/projects/stargazer/plugins/configuration/sgconfig/parser_users.cpp @@ -596,11 +596,57 @@ int CHG_USER::ApplyChanges() { const TARIFF * tariff = u->GetTariff(); const TARIFF * newTariff = m_tariffs.FindByName(m_ucr.tariffName.const_data()); - if (m_tariffs.FindByName(m_ucr.tariffName.const_data())) + if (newTariff) { - if (!u->GetProperty().tariffName.Set(m_ucr.tariffName.const_data(), &m_currAdmin, m_login, &m_store)) - return -1; - u->ResetNextTariff(); + switch (tariff->GetChangePolicy()) + { + case TARIFF::ALLOW: + { + if (!u->GetProperty().tariffName.Set(m_ucr.tariffName.const_data(), &m_currAdmin, m_login, &m_store)) + return -1; + u->ResetNextTariff(); + break; + } + case TARIFF::TO_CHEAP: + { + if (newTariff->GetFee() < tariff->GetFee()) + { + if (!u->GetProperty().tariffName.Set(m_ucr.tariffName.const_data(), &m_currAdmin, m_login, &m_store)) + return -1; + u->ResetNextTariff(); + } + else + GetStgLogger()("Tariff change is prohibited for user %s due to the policy %s. Current tariff %s is more cheap than new tariff %s.", + u->GetLogin().c_str(), + TARIFF::ChangePolicyToString(tariff->GetChangePolicy()).c_str(), + u->GetTariff()->GetName().c_str(), + newTariff->GetName().c_str()); + break; + } + case TARIFF::TO_EXPENSIVE: + { + if (newTariff->GetFee() > tariff->GetFee()) + { + if (!u->GetProperty().tariffName.Set(m_ucr.tariffName.const_data(), &m_currAdmin, m_login, &m_store)) + return -1; + u->ResetNextTariff(); + } + else + GetStgLogger()("Tariff change is prohibited for user %s due to the policy %s. Current tariff %s is more expensive than new tariff %s.", + u->GetLogin().c_str(), + TARIFF::ChangePolicyToString(tariff->GetChangePolicy()).c_str(), + u->GetTariff()->GetName().c_str(), + newTariff->GetName().c_str()); + break; + } + case TARIFF::DENY: + { + GetStgLogger()("Tariff change is prohibited for user %s. Tariff %s.", + u->GetLogin().c_str(), + u->GetTariff()->GetName().c_str()); + break; + } + } } else { -- 2.44.2