Raey

Потому что ты проверяешь каждый элемент массива, если добавить else в цикле, то он вернет -1 при первом несовпадении. (те i = 0, a[0] = 1, ты ищешь 24 или хоть что-то кроме 1, сразу вернет -1 и все, цикл закончен)

Разобрался!
var close = document.querySelector('.close-cross') находил крестик из соседней формы, заменил на var close = document.querySelector('.popup-cart .close-cross') и все заработало
Извините, ошибка на внимательность)