Почему считает избранное начиная со второго товара?

Question

[Павел Сидоров]

Вот страница. на ней стразу товары со сердечками.
НЕ могу понять где ошибка , счетчик избранного начинает считать начиная со второго товара , добавленного в избранное. Если 1 товар добавить, то счетчик все равно будет равен 0

вот код

п.с. первый товар видел в отладке в куках, просто счетчик сверху не крутит с.каа

$(document).ready(function () {

    $(document).on('click', '.to_favorites', function() { // работа с закладками
        var name = 'FOREVER';
        var cookie_zac = BX.getCookie(name);

        if($(this).hasClass('active')){
            $(this).removeClass('active');
            var znach = $(this).attr('data-cookieid')+"|";
            var new_zac = cookie_zac.replace(znach,"");

            $('.goods_icon-counter').html(
                +($('.goods_icon-counter').html())-1
            );
            if($(this).data('remove')) {
                $(this).closest('.favour-item').remove();
            }
            BX.setCookie(name, new_zac, {expires: 86400,path:'/'});
        }else{
            $(this).addClass('active');

            if(cookie_zac == undefined){
                var znach = "|"+$(this).attr('data-cookieid')+"|";
                BX.setCookie(name, znach, {expires: 86400,path:'/'});
            }else{
                var znach = cookie_zac+$(this).attr('data-cookieid')+"|";
                BX.setCookie(name, znach, {expires: 86400,path:'/'});
            }
        }
        activZacladca();
        return false;
    });

    activZacladca();

});

function activZacladca() { // проверка есть ли товар в закладках
    var name = 'FOREVER';

    var cookie_zac = BX.getCookie(name);
    if (cookie_zac !== undefined) {
        var mas = cookie_zac.split('|');

        mas.forEach(function (item, i, mas) {
            if (item != "" ) {

                $('#favour_in .goods_icon-counter').html(
                    i
                );
                $(".to_favorites[data-cookieid='"+item+"']").addClass('active');
            }
        });
    }

}

Answer 1

[v3shin]

В activZacladca у вас проход по mas ведется с 0. Просто сделайте после forEach'а:

$('#favour_in .goods_icon-counter').html(mas.length-1);

Comments

[Павел Сидоров]

сделал

$(document).ready(function () {

    $(document).on('click', '.to_favorites', function() { // работа с закладками
        var name = 'FOREVER';
        var cookie_zac = BX.getCookie(name);

        if($(this).hasClass('active')){
            $(this).removeClass('active');
            var znach = $(this).attr('data-cookieid')+"|";
            var new_zac = cookie_zac.replace(znach,"");

            $('.goods_icon-counter').html(
                +($('.goods_icon-counter').html())-1
            );
            if($(this).data('remove')) {
                $(this).closest('.favour-item').remove();
            }
            BX.setCookie(name, new_zac, {expires: 86400,path:'/'});
        }else{
            $(this).addClass('active');

            if(cookie_zac == undefined){
                var znach = "|"+$(this).attr('data-cookieid')+"|";
                BX.setCookie(name, znach, {expires: 86400,path:'/'});
            }else{
                var znach = cookie_zac+$(this).attr('data-cookieid')+"|";
                BX.setCookie(name, znach, {expires: 86400,path:'/'});
            }
        }
        activZacladca();
        return false;
    });

    activZacladca();

});

function activZacladca() { // проверка есть ли товар в закладках
    var name = 'FOREVER';

    var cookie_zac = BX.getCookie(name);
    if (cookie_zac !== undefined) {
        var mas = cookie_zac.split('|');

        mas.forEach(function (item, i, mas) {
            if (item != "" ) {

                $('#favour_in .goods_icon-counter').html(
                    i
                );
                $(".to_favorites[data-cookieid='"+item+"']").addClass('active');
            }
        });
		$('#favour_in .goods_icon-counter').html(mas.length-1);
    }

}

теперь начинает с 2 ))

[Павел Сидоров]

а не, вроде все ок

[Павел Сидоров]

слушай, чет все таки не так)
начинают с 2 считаться.