Страница 1 из 1

Модули оплаты

Добавлено: 26.09.2018, 19:39
makki
Зачем во всех модулях оплаты в коде файла payments/название_модуля/название_модуля.php формируется переменная $button, которая очевидно нигде не используется?

Добавлено: 27.09.2018, 06:33
korshunov
Видимо, это остатки от кода Simpla, где эта переменная формируется и используется.

А вообще по переделкам этой части возникают разные вопросы, например,
1. Зачем усложнены модули оплаты и в них включено формирование по шаблонам? Шаблон предназначен для тех случаев, когда надо дать возможность владельцу магазина изменить отображение. В данном же случае владельцу никак не рекомендуется лезть в этот код, он в нем обычно ничего не понимает и редактировать ему там особо нечего - разве что может испортить функционал.
2. Все формы для платежных систем собраны в одном файле. Это совсем нерационально, ибо в каждом конкретном случае требуется ОДНА. Если уж разработчикам так захотелось экономить на числе файлов, тогда уж логичнее было бы собирать все payments/название_модуля/название_модуля.php тоже в один файл...
3. Изначально платежные средства сделаны так, что все относящееся к конкретному средству, собрано в одной папке. А теперь эта четкая структура размывается - часть в папке, часть - в шаблоне. Вряд ли это хорошо.

Добавлено: 27.09.2018, 08:51
makki
korshunov писал(а):Видимо, это остатки от кода Simpla, где эта переменная формируется и используется.
Спасибо, понятно. Тоже самое и с переменной $button_text, которая еще и является аргументом функции checkout_form.
Такое впечатление, что сделано на скорую руку, лишь бы работало.

С нижесказанным полностью согласен
korshunov писал(а):А вообще по переделкам этой части возникают разные вопросы, например,
1. Зачем усложнены модули оплаты и в них включено формирование по шаблонам? Шаблон предназначен для тех случаев, когда надо дать возможность владельцу магазина изменить отображение. В данном же случае владельцу никак не рекомендуется лезть в этот код, он в нем обычно ничего не понимает и редактировать ему там особо нечего - разве что может испортить функционал.
2. Все формы для платежных систем собраны в одном файле. Это совсем нерационально, ибо в каждом конкретном случае требуется ОДНА. Если уж разработчикам так захотелось экономить на числе файлов, тогда уж логичнее было бы собирать все payments/название_модуля/название_модуля.php тоже в один файл...
3. Изначально платежные средства сделаны так, что все относящееся к конкретному средству, собрано в одной папке. А теперь эта четкая структура размывается - часть в папке, часть - в шаблоне. Вряд ли это хорошо.

Добавлено: 25.01.2019, 13:55
OkayCMS
Поправили в новой версии 2.3.0 https://okay-cms.com/article/vyshla-okaycms-230