Сообщение #4 makki » 13.08.2018, 14:27
Еще раз сегодня убедился, что вышеуказанный кусок кода из scripts.tpl - это зло, которое нужно срочно искоренить из движка.
Зачем в каталоге товаров дергать 24 селекта и 24 раза запускать $(document).on('change', '.fn_variant', function(), в которой тысяча операций? А что если выставлено 50 товаров на странице сайта?
И все это ради того чтобы определить вариант товара по-умолчанию у которого наличие больше ноля.
Не лучше ли это сделать в том же ProductsView.php, в котором итак зачем-то назначается вариантом по-умолчанию первый вариант товара.
Предлагаю разработчикам обратить внимание на этот момент и оптимизировать Okay
Еще раз сегодня убедился, что вышеуказанный кусок кода из scripts.tpl - это зло, которое нужно срочно искоренить из движка.
Зачем в каталоге товаров дергать 24 селекта и 24 раза запускать $(document).on('change', '.fn_variant', function(), в которой тысяча операций? А что если выставлено 50 товаров на странице сайта?
И все это ради того чтобы определить вариант товара по-умолчанию у которого наличие больше ноля.
Не лучше ли это сделать в том же ProductsView.php, в котором итак зачем-то назначается вариантом по-умолчанию первый вариант товара.
Предлагаю разработчикам обратить внимание на этот момент и оптимизировать Okay
Cоздание и расширение функционала интернет-магазина на платформе OkayCMS 2 (с 3-й и 4-й версией не работаю)