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

Правила раздела: faq.php?mode=okay

makki
makki
Репутация: 139
Сообщения: 397
Зарегистрирован: 12.08.2016
С нами: 2 года 7 месяцев
Откуда: Киев
Сайт Skype

Сообщение #1 makki » 26.09.2018, 19:39

Зачем во всех модулях оплаты в коде файла payments/название_модуля/название_модуля.php формируется переменная $button, которая очевидно нигде не используется?
Cоздание и расширение функционала интернет-магазина на платформе OkayCMS

korshunov
korshunov
Репутация: 93
Сообщения: 927
Зарегистрирован: 03.12.2015
С нами: 3 года 3 месяца
Skype

Сообщение #2 korshunov » 27.09.2018, 06:33

Видимо, это остатки от кода Simpla, где эта переменная формируется и используется.

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

makki
makki
Репутация: 139
Сообщения: 397
Зарегистрирован: 12.08.2016
С нами: 2 года 7 месяцев
Откуда: Киев
Сайт Skype

Сообщение #3 makki » 27.09.2018, 08:51

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

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

OkayCMS M
Администратор
Аватара
OkayCMS M
Администратор
Репутация: 94
Сообщения: 931
Зарегистрирован: 12.11.2015
С нами: 3 года 4 месяца
Сайт Skype

Сообщение #4 OkayCMS » 25.01.2019, 13:55

Поправили в новой версии 2.3.0 https://okay-cms.com/article/vyshla-okaycms-230


Название раздела: Баг-репорт OkayCMS (Сообщения об ошибках)
Правила раздела: faq.php?mode=okay

Быстрый ответ


Введите код в точности так, как вы его видите. Регистр символов не имеет значения.
Код подтверждения

   

Вернуться в «Баг-репорт OkayCMS (Сообщения об ошибках)»

Кто сейчас на форуме (по активности за 5 минут)

Сейчас этот раздел просматривают: 2 гостя