Lander

При обработке запроса сохраняйте utm-метки в сессию, если они есть. Если их нет, берите из сессии.

if - это оператор, а не функция, соответственно он не возвращает значение, а значит и не может конкатенироваться.

После include "bitrix.php"; да. Ужасен. Не глядя.

Разбирать ваш код, я, конечно же разбирать не стал. У меня только вопрос:

Собственно при шифровании вызываю code и передаю "1", при расшифровании code и передаю "2".

Зачем?
Для RC4 P = C(C(P));
Соответственно для расшифрования можно использовать просто повторное шифрование с тем же ключом. К тому же при разных параметрах функции code у вас пароль берётся из разных полей. Вы уверены что они совпадают?

if ( in_array(date("W", $hours), [1, 2, 3, 4, 5]) ) {
    //Рабочий день, выполняем нужные действия
}

Если собственник платит вам деньги и у вас нет полномочий продукт-менеджера, а следовательно вы не несёте никакой ответственности за это, то ставьте.

Ну концептуально это два разных лейаута, в которые выводится контент. Поэтому делайте две разные вьюхи и используйте их в разных контроллерах.

Давно читал статью (сейчас к сожалению не найду никак), так там решили проверить что лучше работает. И оказалось, что хоть люди и понимают, что 2.99 это 3, но конверсия у 2.99 оказалась в два раза выше! Кроме того, люди вне любят круглые суммы (тоже из той статьи). То есть за 5240 они возьмут охотнее чем за 5000. Это создаёт иллюзию что цена товара была взята не с потолка, а вычислена по какой то модели ценообразования. Но как совершенно верно заметил Владимир Проскурин нужно тестировать эти гипотезы в вашем конкретном случае!

Вы явно что то делаете не так... В принципе ситуация, когда из одного контроллера нужно дёргать другой - это нарушение архитектуры. А данные получаются через модели. Если модель используется только во frontend или backend, то она должна лежать в соответствующем каталоге. Если это общая модель, то используйте common.

Используйте транзакции базы данных. В этом случае следующий запрос не получит баланс, пока он не обновится предыдущей транзакцией.

Использование $this вне контекста объекта. unregister точно является методом класса?