From: Elena Mamontova <helenh463@gmail.com>
Date: Wed, 27 Jul 2016 14:52:37 +0000 (+0300)
Subject: Ticket 37. The switch case construction for changePolicy
X-Git-Tag: 2.409~138
X-Git-Url: https://git.stg.codes/stg.git/commitdiff_plain/2db61d00612cf97d1c733277bc8828b23446969c?ds=sidebyside;hp=c949afc4f9940c71c63ce6dda771eb32e6fd8c65

Ticket 37. The switch case construction for changePolicy
checking added in the ApplyChanges() function.
---

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
         {