PriceUU - зачеркнутая старая цена 1.0.1 Alpha

[Enot]

Enot добавил(а) новый ресурс:

BergStudio-PriceUU Alpha - Зачеркнутая старая цена в платных повышениях

Посмотреть вложение 146423

Небольшой аддон, который позволяет на странице с платными повышениями /admin.php?user-upgrades/ указывать старую цену, которая будет отображаться зачеркнутой. Так-же реализована возможность в автоматическом режиме при применении купона отображать цену со скидкой.

Разработка: BergStudio
Заказ...

Узнать больше об этом ресурсе...

 

[inz]

BergStudio,

Зачем полностью переопределять метод? Достаточно просто унаследовать от родителя.


Я думаю не стоит говорить что использование HTML в коде это плохая практика. Вопрос зачем? Всё это можно оформить во фразах, ведь они его поддерживают.

Неизвестно зачем ещё раз был переопределен метод, причем с таким же кодом как у родителя. ???
Если аддон тянет в зависимости другой аддон, то нужно его указывать в addon.json (

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

)
Заместо кучу вложенных ифов можно вложить всё в одну проверку. Подробнее можно ознакомится тут:

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

.
Там есть ещё пару моментов, но уже банально лень расписывать дальше, я думаю и так понятно всё.

 

[BergStudio]

BergStudio,
Заместо кучу вложенных ифов можно вложить всё в одну проверку. Подробнее можно ознакомится тут:

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

.

1. Это практически родной код - XF\Mvc\Entity\Entity\UserUpgrade - getCostPhrase()

BergStudio,

Зачем полностью переопределять метод? Достаточно просто унаследовать от родителя.

Неизвестно зачем ещё раз был переопределен метод, причем с таким же кодом как у родителя. ???

2. Такой код как у родителя? :)
А как передать в formAction - old_cost_amount? подскажи если знаешь!

BergStudio,
Если аддон тянет в зависимости другой аддон, то нужно его указывать в addon.json (

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

)

3. А от чего(кого) зависимость?
Там идет проверка стоит (включен) ли Аддон DBTech/UserUpgradeCoupon

BergStudio,
Я думаю не стоит говорить что использование HTML в коде это плохая практика. Вопрос зачем? Всё это можно оформить во фразах, ведь они его поддерживают.

4. С этим полностью согласен!
Спешил, позаимствовал большую часть кода у - DBTech/UserUpgradeCoupon
на DBTech чего не нападаете?

 

[25517]

inz, я в принципе не понимаю, на кой этот плагин.
Если всё можно решить куда проще.

 

[BergStudio]

inz, я в принципе не понимаю, на кой этот плагин.
Если всё можно решить куда проще.

Посмотреть вложение 146439

Так цены разные, дабы стиль каждый раз не править!

 

[25517]

BergStudio, и ?
Комментарий в коде для кого ?

С помощью модификации шаблонов, добавить уникальный класс.
Обернуть весь код выше в функцию и просто расписать всё это в extra.less

 

[BergStudio]

inz, я в принципе не понимаю, на кой этот плагин.
Если всё можно решить куда проще.

Посмотреть вложение 146439

Так цены разные

BergStudio, и ?
Комментарий в коде для кого ?

С помощью модификации шаблонов, добавить уникальный класс.
Обернуть весь код выше в функцию и просто расписать всё это в extra.less

Так попросил ТС - ПОЛЕ для ввода старой цены!
А цены как менять? Каждый раз в .less или в модификации шаблона?

Не все в этом разбираются!

 

[25517]

Каждый раз в .less или в модификации шаблона?

А какая разница, куда залезать ? В платные повышения, или в экстру ?
Комментарии для удобства никто не запрещал.

А как передать в formAction - old_cost_amount? подскажи если знаешь!

Вот пример, как это делается.

 

[Matew]

1. Это практически родной код XF\Mvc\Entity\Entity\UserUpgrade - getCostPhrase()

Нет, это не практически родной код. В родном коде используется фраза, название которой задания динамически, но не напрямую html код. Так делать не надо

2. Такой код как у родителя? :)
А как передать в formAction - old_cost_amount? подскажи если знаешь!

3. А от чего(кого) зависимость?
Там идет проверка стоит (включен) ли Аддон DBTech/UserUpgradeCoupon

Этот плагин лежит в основе, поэтому от него и вешать зависимость

 

[BergStudio]

Нет, это не практически родной код. В родном коде используется фраза, название которой задания динамически, но не напрямую html код. Так делать не надо


Посмотреть вложение 146442


Этот плагин лежит в основе, поэтому от него и вешать зависимость

Попробуй отключить или удалить - DBTech/UserUpgradeCoupon!!!

А какая разница, куда залезать ? В платные повышения, или в экстру ?
Комментарии для удобства никто не запрещал.


Вот пример, как это делается.

Посмотреть вложение 146440

Спасибо за подсказку! Все сразу не постичь :)

А как правильнее?

Или так

Нет, это не практически родной код. В родном коде используется фраза, название которой задания динамически, но не напрямую html код. Так делать не надо


Посмотреть вложение 146442


Этот плагин лежит в основе, поэтому от него и вешать зависимость

С этим абсолютно согласен!
Родной код - DBTech\UserUpgradeCoupon\XF\Entity\UserUpgrade

 

[Matew]

А как правильнее?

Или так

Вариант 25517, с моей точки зрения, более правильный

 

[BergStudio]

Спасибо за внимание, подсказки!
Замечания (предложения) принял!

Сделаем рефакторинг кода!

 

[TwiX]

А как купон создаётся?
В настройках или как?

 

[BergStudio]

А как купон создаётся?
В настройках или как?

За купоны отвечает - DBTech/UserUpgradeCoupon

После его установки, в Админке добавляется ниже платных повышений прав юзера!

Этот Аддон всего лишь зачеркивает(выделяет) старую цену!

 

[inz]

4. С этим полностью согласен!
Спешил, позаимствовал большую часть кода у - DBTech/UserUpgradeCoupon
на DBTech чего не нападаете?

Кто сказал? Просто автора тут нету, так как он зарубежом, там достаточно заглянуть в eCommerce и уже плохо станет.

 

[BergStudio]

Кто сказал? Просто автора тут нету, так как он зарубежом, там достаточно заглянуть в eCommerce и уже плохо станет.

А как же допустили к публикации?

inz уточните пожалуйста, где, что не так с if else?

 

[Matew]

А как же допустили к публикации?

А почему не должны были? Вы думаете кто-то проверяет код?

 

[BergStudio]

А почему не должны были? Вы думаете кто-то проверяет код?

Простите меня, может много любопытсвую...

Да, думал на официальном форум проверяют, как и на XFI

 

[Matew]

Простите меня, может много любопытсвую...

Да, думал на официальном форум проверяют, как и на XFI

Не, разработчикам и своих дел хватает, себе всего. Если кто-то кинет репорт на явное нарушение - посмотрят. А так...а так всем без разницы что там

 

[BergStudio]

Не, разработчикам и своих дел хватает, себе всего. Если кто-то кинет репорт на явное нарушение - посмотрят. А так...а так всем без разницы что там

OFFTOP

А если Трояна кто внедрит?

Вариант 25517, с моей точки зрения, более правильный

Подскажи пожалуйста ещё разок

В этой функции(ях) идёт запрос (обращение) к БД?

Конкретнее - basicEntitySave ?

p.s. Не вникал!