- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
/* Здесь мы имеем сраный унаследованный пиздец: количество товара передается в параметрах POST-запроса,
имеющих имена вида productX_count, где X - ИД товара. Как предыдущие разработчики разобирали это счастье?
Правильно, перебирали все (блядь, ВСЕ, блядь, 20000 штук!) товары и проверяли, есть ли соответствующее
значение в запросе.
*/
$items5 = GetIBlockElementList(5);
while($arItem = $items5->GetNext())
if(isset($_POST['product'.$arItem['ID']."_id"]) && is_numeric($_POST['product'.$arItem['ID']."_id"]))
{
$_PRODUCTS[$arItem['ID']]['count'] = $_POST['product'.$arItem['ID']."_count"];
}
В этом коде меня утешает только боль и ненависть комментария.
Vindicar 20.01.2014 08:06 # +2
Кстати, удивил закос под суперглобальный массив на 11й строке.
kegdan 20.01.2014 08:46 # 0
bormand 20.01.2014 09:29 # +1
com1 20.01.2014 14:06 # +2
закос под суперглобальность приколен, но приведенный код может являться частью функции... конечно приставка "супер" в этом случае отпадает, но вот "глобальность" может и появиться. )))
Vasiliy 20.01.2014 14:25 # +1
com1 20.01.2014 15:31 # 0
Lure Of Chaos 21.01.2014 02:20 # 0
а скорей всего в $_POST
Vindicar 20.01.2014 17:12 # 0
Я-то предлагал в рамках "меньшего говна и малых правок".
com1 20.01.2014 23:17 # 0
а разгребать увы... ))) (((
Vindicar 23.01.2014 09:08 # 0
А так - вполне себе подход, получше моего.
psycho-coder 20.01.2014 10:02 # 0
Homess 22.01.2014 22:52 # 0