2018-03-20 Попутно Ничего Нельзя Менять

Нарушено правило: никогда ничего нельзя править попутно при решении задачи.
Постановка задачи: добавить текст сообщения «Демо-версия».
Ошибка: разработчик попутно переместил global $DEMO_DAYS_MAX в начало функции, как требуют правила хорошего тона php.
Так делать КАТЕГОРИЧЕСКИ нельзя, код бы отлажен и оттестирован именно в варианте, когда объявление было в теле if, и если выше этой переменной нет и или если она не объявилась в if, то весь алгоритм уже отлажен и работает с учетом этой ошибки.
Если исправить эту техническую ошибку, с огромной вероятностью возникнет логическая ошибка, которую не факт, что отловят автотесты.

function do_act($params)
{
+    global $DEMO_DAYS_MAX;
     debug('DO_ACT');
     $potential = potential_by_reg($params['reg_num']);
@@ -181,7 +182,6 @@ function do_act($params)
     // Если текущая стадия не проданная и не внедрение, то это демо
     if (!in_array($potential['sales_stage'], $won_sales_stages)) {
-        global $DEMO_DAYS_MAX;
         debug('DO_ACT DEMO  sales_stage:' . $potential['sales_stage']);
         $interval_days = (strtotime($potential['cf_regdemoend']) - strtotime(date('d.m.Y', time()))) / (3600 * 24);
         $result['demo_days'] = $interval_days;
+        $result['demo_msg'] = 'Демо-версия';

~~OWNERAPPROVE~~