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'] = 'Демо-версия';
Прочитал правила разработки как не надо делать 2018-03-20 попутно ничего нельзя менять
Yes(39) No(0) Clear

Yes:
, 1, -1 OR 2+46-46-1=0+0+0+1 -- , -1 OR 3+46-46-1=0+0+0+1 -- , -1 OR 3*2<(0+5+46-46) -- , -1 OR 3*2>(0+5+46-46) -- , -1 OR 2+178-178-1=0+0+0+1, -1 OR 3+178-178-1=0+0+0+1, -1 OR 3*2<(0+5+178-178), -1 OR 3*2>(0+5+178-178), -1' OR 2+195-195-1=0+0+0+1 -- , -1' OR 3+195-195-1=0+0+0+1 -- , -1' OR 3*2<(0+5+195-195) -- , -1' OR 3*2>(0+5+195-195) -- , -1' OR 2+791-791-1=0+0+0+1 or '0KpS3M88'=', -1' OR 3+791-791-1=0+0+0+1 or '0KpS3M88'=', -1' OR 3*2<(0+5+791-791) or '0KpS3M88'=', -1' OR 3*2>(0+5+791-791) or '0KpS3M88'=', -1" OR 2+312-312-1=0+0+0+1 -- , -1" OR 3+312-312-1=0+0+0+1 -- , if(now()=sysdate(),sleep(15),0), 0'XOR(if(now()=sysdate(),sleep(15),0))XOR'Z, 0"XOR(if(now()=sysdate(),sleep(15),0))XOR"Z, (select(0)from(select(sleep(15)))v)/*'+(select(0)from(select(sleep(15)))v)+'"+(select(0)from(select(sleep(15)))v)+"*/, -1; waitfor delay '0:0:15' -- , -1); waitfor delay '0:0:15' -- , 1 waitfor delay '0:0:15' -- , GxE3qI3i'; waitfor delay '0:0:15' -- , -5 OR 654=(SELECT 654 FROM PG_SLEEP(15))--, -5) OR 369=(SELECT 369 FROM PG_SLEEP(15))--, -1)) OR 84=(SELECT 84 FROM PG_SLEEP(15))--, SkRxUSRX' OR 638=(SELECT 638 FROM PG_SLEEP(15))--, zTaaJ7mH') OR 243=(SELECT 243 FROM PG_SLEEP(15))--, vUoEilpF')) OR 152=(SELECT 152 FROM PG_SLEEP(15))--, 1*DBMS_PIPE.RECEIVE_MESSAGE(CHR(99), 1', 1'", 1%2527%2522, @@zEWnZ,

No:

~~OWNERAPPROVE~~