- 1
- 2
$arParams["SET_TITLE"] = $arParams["SET_TITLE"]!="N";
$arParams["SET_LAST_MODIFIED"] = $arParams["SET_LAST_MODIFIED"]==="Y";
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+3
$arParams["SET_TITLE"] = $arParams["SET_TITLE"]!="N";
$arParams["SET_LAST_MODIFIED"] = $arParams["SET_LAST_MODIFIED"]==="Y";
news.detail - cтандартный компонент Битрикса
kropotor 27.07.2016 15:40 # +1
1. Да, не против
2. Нет, не против
3. Секретная опция "N"
inkanus-gray 27.07.2016 15:55 # +2
1. Да, не нужна.
2. Нет, не нужна.
3. Секретная опция "Y".
В общем, параметры нужно разделить на те, у которых молчание означает согласие, и на те, у которых молчание означает «нет».
guest 19.08.2016 17:25 # −61
И буквально следующей строкой еще одна конструкция:
$arParams["SET_BROWSER_TITLE"] = (isset($arParams["SET_BROWSER_TITLE"]) && $arParams["SET_BROWSER_TITLE"] === 'N' ? 'N' : 'Y');
guest 06.08.2016 10:13 # −61
guest 19.08.2016 17:15 # −61
строка 70,71
До меня кажется дошло, они так дефолтные значения задают.