Нарушено правило: никогда ничего нельзя править попутно при решении задачи.
Постановка задачи: добавить текст сообщения «Демо-версия».
Ошибка: разработчик попутно переместил 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: No: |
~~OWNERAPPROVE~~