Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
...........
var name = '';
var price = 0.0;
var priceFormated = '';
var priceBase = 0.0;
var priceBaseFormat = '';
var priceType = '';
var currency = '';
var priceDiscount = 0.0;
var quantity = 0;
var summaFormated = '';
var weight = 0;
var vatRate = 0.0;
var module = '';
var valutaFormat = '';
var catalogXmlID = '';
var productXmlID = '';
var url = '';
var urlImg = '';
var urlEdit = '';
var balance = '';
var priceTotalFormated = '';
var discountPercent = '';
var callback = '';
var orderCallback = '';
var cancelCallback = '';
var payCallback = '';
var productProviderClass = '';
var arSkuProps = [];
var barcodeMulti = '';
var arStores = [];
var productType = '';
var setParentId = '';
if (!!arParams.id)
product_id = arParams.id;
if (!!arParams.name)
name = arParams.name;
if (!!arParams.price)
price = arParams.price;
if (!!arParams.priceFormated)
priceFormated = arParams.priceFormated;
if (!!arParams.priceBase)
priceBase = arParams.priceBase;
if (!!arParams.priceBaseFormat)
priceBaseFormat = arParams.priceBaseFormat;
if (!!arParams.priceType)
priceType = arParams.priceType;
if (!!arParams.currency)
currency = arParams.currency;
if (!!arParams.priceDiscount)
priceDiscount = arParams.priceDiscount;
if (!!arParams.quantity)
quantity = arParams.quantity;
if (!!arParams.summaFormated)
summaFormated = arParams.summaFormated;
if (!!arParams.weight)
weight = arParams.weight;
if (!!arParams.vatRate)
vatRate = arParams.vatRate;
if (!!arParams.module)
module = arParams.module;
if (!!arParams.valutaFormat)
valutaFormat = arParams.valutaFormat;
if (!!arParams.catalogXmlID)
catalogXmlID = arParams.catalogXmlID;
if (!!arParams.productXmlID)
productXmlID = arParams.productXmlID;
if (!!arParams.url)
url = arParams.url;
if (!!arParams.urlImg)
urlImg = arParams.urlImg;
if (!!arParams.urlEdit)
urlEdit = arParams.urlEdit;
if (!!arParams.balance)
balance = arParams.balance;
if (!!arParams.priceTotalFormated)
priceTotalFormated = arParams.priceTotalFormated;
if (!!arParams.discountPercent)
discountPercent = arParams.discountPercent;
if (!!arParams.callback)
callback = arParams.callback;
if (!!arParams.orderCallback)
orderCallback = arParams.orderCallback;
if (!!arParams.cancelCallback)
cancelCallback = arParams.cancelCallback;
if (!!arParams.payCallback)
payCallback = arParams.payCallback;
if (!!arParams.productProviderClass)
productProviderClass = arParams.productProviderClass;
if (!!arParams.skuProps)
{
arSkuProps = (BX.type.isString(arParams.skuProps) ? eval('('+arParams.skuProps+')') : arParams.skuProps);
}
if (!!arParams.barcodeMulti)
barcodeMulti = arParams.barcodeMulti;
if (!!arParams.productType)
productType = arParams.productType;
if (!!arParams.setParentId)
.......... и так далее еще очень долго)
Это вообще очень неприятный кусок из /bitrix/admin/sale_order_new.php
Адский файл в несколько тысяч строк лапши.
Вообще мне нравится битрикс, но от таких файлов мне хочется бесконечно тыкать отверткой в бочину автору сего файла.
Нет! Нет!
Этот код кричит "убейте меня"
Адский файл в несколько тысяч строк лапши.
Вообще мне нравится битрикс, но от таких файлов мне хочется бесконечно тыкать отверткой в бочину автору сего файла.