Вот так работает:
<a data-id="632" class="buy-now" data-price="XXX"
onclick="add(this)">Купить</a>
а так нет
<a data-id="632" class="buy-now" data-price="XXX"
onclick="return cart.add(this)">Купить</a>
А в чем разница?
И если data-price="XXX" лучше, как это реализовать?