Но с JS вы "здорово" придумали.
А как бы вы реализовали бы данные вещи? Без JS. 
Но с JS вы "здорово" придумали.
А как бы вы реализовали бы данные вещи? Без JS.
Где стрелка - там показывается комплектация цвет, а называется фото.
Всё дело в том, что там должно быть фото. В многомерках на вкладке комплектации есть радиокнопка, показа как фото комплектации. Например, если у Вас есть цвет, то цвет можно выводит маленькой фотографией. Но... Если у Вас для комплектации не отмечено ни одно фото, которое соответствует например цвету, то оно будет выводится как текст(у каждой комплектации можно его отметить). Поэтому всё там правильно, на наш взгляд.
Кстати, а чем вас prev() не устраивает? По-моему, хороший метод. Сам не проверял, но по логике, должен работать в разы быстрее чем closest() так как объекты на prev и next у каждого объекта jquery при инициализации прописываются.
Дело в том, что prev и next это неточные обращения. Особенно если, кто-то захочет поменять вёрстку, то надо будет и javascript Ваш менять, и несколько раз допустим прописывать ещё prev prev prev. А если обращаться по идентификатору, как указано выше или через closest, то можно имея класс или идентификатор сразу обратиться к нужному элементу, не особо следя за вёрсткой.
Скажите, а предусмотрена ли (или может в планах) такая вещь, как скрытие пустых комплектаций (с 0 остатком)? Чтобы при смене, например, цвета, менялся и список остальных свойств, в зависимости от наличия на складе.
Да, в этом году поставлено задачей.
И еще вопрос - у вас в демках и в моей теме тоже остатки отображаются стиками, индикатором, возможно ли как то быстро отказаться от этой системы, и выводить остатки обычным числом?
Можно. Но нужно немного допилить js и вёрстку. Остатки по складам, присутствуют у каждой комплектации. Догрузить из можно с помощью двух методов у объекта товара
$full_stock=$product->getWarehouseFullStock() - остатки общие для комплектациям по складам
и
$offers_stock=$product->getWarehouseStock() - остатки по складам и комплектациямПлюс у комплектаций есть поле num с общим остатком по складам.
Т.е. можно при смене комплектации например брать num и подставлять в каждый нужный класс отображения.
Товар также обладает полем num - общий остаток по всем складам.
]]>{* Многомерные комплектации *}
<span class="pname">{$product.offer_caption|default:'Комплектация'}</span>
{* Подгрузим у многомерных комплектаций фото к их вариантам *}
{$product->fillMultiOffersPhotos()}
{* Переберём доступные многомерные комплектации *}
<div class="multiOffers">
{foreach $product.multioffers.levels as $level}
{if !empty($level.values)}
<div class="title">{if $level.title}{$level.title}{else}{$level.prop_title}{/if}</div>
====> {if !$level.is_photo && !isset($level.values_photos)} {* Если отображать не как фото (выпадающим списком)*}
<select name="multioffers[{$level.prop_id}]" data-prop-title="{if $level.title}{$level.title}{else}{$level.prop_title}{/if}">
{foreach $level.values as $value}
<option value="{$value.val_str}">{$value.val_str}</option>
{/foreach}
</select>
{else} {* Как фото *}
<div class="multiOfferValues">
<input type="hidden" name="multioffers[{$level.prop_id}]" data-prop-title="{if $level.title}{$level.title}{else}{$level.prop_title}{/if}"/>
{foreach $level.values as $value}
{if isset($level.values_photos[$value.val_str])}
<a class="multiOfferValueBlock {if $value@first}sel{/if}" data-value="{$value.val_str}" data-image="{$level.values_photos[$value.val_str]->getUrl(310,310,'axy')}" data-is-dialog="1" title="{$value.val_str}"><img src="{$level.values_photos[$value.val_str]->getUrl(30,30,'axy')}"/></a>
{else}
<a class="multiOfferValueBlock likeString {if $value@first}sel{/if}" data-value="{$value.val_str}" data-is-dialog="1" title="{$value.val_str}">{$value.val_str}</a>
{/if}
{/foreach}
</div>
{/if}
{/if}
{/foreach}
</div>
{if $product->isOffersUse()}
{foreach from=$product.offers.items key=key item=offer name=offers}
<input value="{$key}" type="hidden" name="hidden_offers" class="hidden_offers" {if $smarty.foreach.offers.first}checked{/if} id="offer_{$key}" data-info='{$offer->getPropertiesJson()}' {if $check_quantity}data-num="{$offer.num}"{/if} data-change-cost='{ ".offerBarcode": "{$offer.barcode|default:$product.barcode}", ".myCost": "{$product->getCost(null, $key)}", ".lastPrice": "{$product->getOldCost($key)}"}' data-images='{$offer->getPhotosJson()}' data-sticks='{$offer->getStickJson()}'/>
{/foreach}
<input type="hidden" name="offer" value="0"/>
{/if}Где стрелка - там показывается комплектация цвет, а называется фото. Хотя у вас в документации http://readyscript.ru/dev-manual/dev_co … v_code_var написано:
Имя переменной должно отражать суть её содержимого.
Я не был в курсе, что photo - это цвет, а наш макет не подразумевает фотографий в месте смены комплектаций (у нас просто квадраты), и поэтому, в первый раз, я сделал вот так:
<div class="multiOffers">
{foreach $product.multioffers.levels as $level}
{if !empty($level.values) && $level.title != 'Цвет'}
<div class="multiofferTitle">{if $level.title}{$level.title}{else}{$level.prop_title}{/if}</div>
{if !$level.is_photo && !isset($level.values_photos)} {* Если отображать не как фото (выпадающим списком)*}
<select name="multioffers[{$level.prop_id}]" data-prop-title="{if $level.title}{$level.title}{else}{$level.prop_title}{/if}">
{foreach $level.values as $value}
<option value="{$value.val_str}">{$value.val_str}</option>
{/foreach}
</select>
<div class="multiofferBlock multiofferBlock{$level.prop_id}">
{foreach $level.values as $value key=i}
<div class="moItem{if $i == 1} active{/if}"><div class="moItemInner">{$value.val_str}</div></div>
{/foreach}
</div>
<div class="clearfix"></div>
{else}
<div class="multiOfferValues">
<input type="hidden" name="multioffers[{$level.prop_id}]" data-prop-title="{if $level.title}{$level.title}{else}{$level.prop_title}{/if}"/>
{foreach $level.values as $value}
{if isset($level.values_photos[$value.val_str])}
<a class="multiOfferValueBlock {if $value@first}sel{/if}" data-value="{$value.val_str}" title="{$value.val_str}"><img src="{$level.values_photos[$value.val_str]->getUrl(40,40,'axy')}"/></a>
{else}
<a class="multiOfferValueBlock likeString {if $value@first}sel{/if}" data-value="{$value.val_str}" title="{$value.val_str}">{$value.val_str}</a>
{/if}
{/foreach}
</div>
{/if}
{else if !empty($level.values) && $level.title == 'Цвет'}
<div class="multiofferTitle">{if $level.title}{$level.title}{else}{$level.prop_title}{/if}</div>
<select name="multioffers[{$level.prop_id}]" data-prop-title="{if $level.title}{$level.title}{else}{$level.prop_title}{/if}">
{foreach $level.values as $value}
<option value="{$value.val_str}">{$value.val_str}</option>
{/foreach}
</select>
<div class="multiofferBlock multiofferBlock{$level.prop_id}">
{foreach $level.values as $value key=i}
<div class="moItem{if $i == 1} active{/if}"><div class="moItemInner" style="background-color:#fff;background-image: linear-gradient( -45deg, {$colors.colors[$value.val_str].color2} 0%, {$colors.colors[$value.val_str].color2} 50%, {$colors.colors[$value.val_str].color1} 50%, {$colors.colors[$value.val_str].color1} 50%);" title="{$value.val_str}">{$value.val_str}</div></div>
{/foreach}
</div>
<div class="clearfix"></div>
{/if}
{/foreach}
</div>Сейчас все нормально, спасибо за подсказки.
Но с JS вы "здорово" придумали. Посмотрите на код - у меня там дублируется цикл и таким образом генерирует значение, так что мне его не нужно получать, оно априори есть в $(this). В итоге - это было последнее, исправление. Поменял обратно на prev(), тк как closest() начистую отказался работать.
Кстати, а чем вас prev() не устраивает? По-моему, хороший метод. Сам не проверял, но по логике, должен работать в разы быстрее чем closest() так как объекты на prev и next у каждого объекта jquery при инициализации прописываются.
Скажите, а предусмотрена ли (или может в планах) такая вещь, как скрытие пустых комплектаций (с 0 остатком)? Чтобы при смене, например, цвета, менялся и список остальных свойств, в зависимости от наличия на складе.
И еще вопрос - у вас в демках и в моей теме тоже остатки отображаются стиками, индикатором, возможно ли как то быстро отказаться от этой системы, и выводить остатки обычным числом?
]]>$(data.options.hiddenOffersName,context).each(function(i){Попробуйте его подебажить, может у Вас что-то не то выбирается.
И на всякий случай Возьмите ещё код jquery.changeoffer.js из дефолтной темы.
Что касается Вашего кода. Ваших ссылках записать во первых в data id(идентификатор) выпадающего списка, который реализует отображение вариантов с выбором, тогда можно считать точный селектор того выпадающего списка. И в другой data записать значение, которое нужно выбрать из этого списка. Т.е. при нажатии на вашу ссылку, вы сразу знаете какой выпадающий список и его значение нужно применить. Применяете это значение к селекту и уже у него вызываете событие change. Тогда код по идее точно отработает. Возможно, что вы просто не тому селекту присваиваете значения.
Попробуйте покосолеложить выбранные селекты.
Вам нужен файл jquery.changeoffer.js
Для проверки отключил свой JS, оставил только ваш, и все равно, смена комплектаций не работает в модальном окне.
В итоге, после 2,5 часов дебага я выяснил, что ошибка действительно, в jquery.changeoffer.js, а именно в этой функции во вложенных циклах, я пометил:
changeMultiOffer = function (){
var context = $this.closest(data.options.context);
var selected = []; //Массив, что выбрано
//Соберём информацию, что изменилось
$(data.options.multiOfferName,context).each(function(i){
selected[i] = {};
selected[i]['title'] = $(this).data('propTitle');
selected[i]['value'] = $(this).val();
});
//Найдём инпут с комплектацией
var input_info = data.options.multiOffersInfo;
var offer = false; //Cпрятанная комплектация, которую мы выбрали
for(var j=0;j<input_info.length;j++){
var info = input_info[j]['info']; //Группа с информацией
// console.log("input_info", input_info[j]);
var found = 0; //Флаг, что найдены все совпадения
for(var m=0;m<info.length;m++){
for(var i=0;i<selected.length;i++){
if ((selected[i]['title']==info[m][0])&&(selected[i]['value']==info[m][1])){
found++;
// console.log("found", found);
// console.log("selected_match", selected[i]);
}
}
if (found==selected.length){ //Если удалось найди совпадение, то выходим
offer = input_info[j]['id'] // ВОТ ЗДЕСЬ ДАННЫЕ НЕКОРРЕКТНЫЕ ВЫНИМАЮТСЯ
// console.log("compl", selected[i]);
break;
}
}
}
//Отметим выбранную комплектацию
var offer_val = 0;
if (offer){ // Если комплектация выбранная присутствует
offer_val = $(offer).val();
$(data.options.theOffer,context).val(offer_val);
}else{ // Если комплектации такой не нашлось, выберем нулевую компл.
$(data.options.theOffer,context).val(offer_val);
}
//Добавим параметр комплектаций к ссылке купить в 1 клик, если купить в 1 клик присутствует
if ($(data.options.buyOneClick,context).length>0){
var clickHref = $(data.options.buyOneClick,context).data('href').split('?'); //Получим урл
//Соберём информацию что выбрано из многомерных
var multi_selected = [];
$(selected).each(function(i){
multi_selected.push('multioffers[]='+encodeURIComponent(selected[i]['title'])+": "+encodeURIComponent(selected[i]['value']));
});
multi_selected = multi_selected.join('&');
//Запишем урл
$(data.options.buyOneClick,context).data('href',clickHref[0]+"?"+multi_selected);
}
$(data.options.offerProperty).addClass(data.options.hiddenClass);
$(data.options.offerProperty+'[data-offer="'+offer_val+'"]').removeClass('hidden');
//Поменяем цену за комплектацию
var dataCost = $(offer).data('changeCost');
for(var i in dataCost){
$(i,context).html(dataCost[i]);
}
//Сменим единицу измерения, если нужно
if ((typeof($(offer).data('unit'))!='undefined') && ($(offer).data('unit')!="")){
$(data.options.unitBlock,context).show();
$(data.options.unitBlock+" .unit",context).text($(offer).data('unit'));
}else{
$(data.options.unitBlock,context).hide();
}
//Показываем фото комплектаций
if ($(data.options.hiddenOffersName+"[value='"+offer_val+"']",context).length>0){
$selected = $(data.options.hiddenOffersName+"[value='"+offer_val+"']",context);
showOfferPhotos($selected);
}
//Покажем наличие товара после выбора комплектации
showAvailability(offer_val);
},Еще проблема, что мой имеющийся рабочий (на странице товара) JS код, который инициализирует смену селекта (и комплектации соотв.) по клику на квадратик не работает в модальном окне. Это как раз тот код, по которому вы делали мне замечания.
$('.moItem').on('click', function() {
var value = $(this).text();
$(this).addClass('active').siblings('.moItem').removeClass('active');
$(this).parent().closest('[name^="multioffers["]').val(value).trigger('change');
});theme.js у Вас срабатывает. У Вас в шаблоне используются многомерные комплектации. Поэтому Вам надо посмотреть метод changeMultiOffer.
такого метода нет( 
или я не там ищу? или у меня theme.js старый
Вам признак multiOffers: true здесь не нужен. Скрипт всё должен и так подхватить. Можно просто оставить
Я просто попробовал.
Странно, т.к. он приведён в Вашем коде выше с модальным окном в самом низу.
Код там появился именно потому, что не работает из theme.js
Там и пара кусков закомментированных есть, я не стал чистить для полноты картины.
PS Ну и проблема же фотку добавить у вас, может drop-область привязать прямо слева от textarea?
]]>$(this).parent().prev()
этого кода вообще там не должно быть, он находится здесь http://22.groupvm.z8.ru/templates/autof … s/theme.js
он там и есть, только не работаетсейчас сделаю как вы советуете.
Странно, т.к. он приведён в Вашем коде выше с модальным окном в самом низу.
]]>$('[name="offer"]').changeOffer({
multiOffers: true
});Вам признак multiOffers: true здесь не нужен. Скрипт всё должен и так подхватить. Можно просто оставить
$('[name="offer"]').changeOffer();$(this).parent().prev()
этого кода вообще там не должно быть, он находится здесь http://22.groupvm.z8.ru/templates/autof … s/theme.js
он там и есть, только не работает
сейчас сделаю как вы советуете.
]]>jquery.changeoffer.js у Вас должен подключаться в list_product.tpl - шаблон со списком товаров общий в модуле catalog.
так он подключен, просто в colorbox не работает почему-то
зайдите проверьте http://22.groupvm.z8.ru/catalog/dlya-muzhchin/
даже мой код из theme.js не работает
]]>