Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Переопределение модуля фреймворка. #2

Open
prokopov-nikolai opened this issue Feb 2, 2018 · 5 comments

Comments

@prokopov-nikolai
Copy link

Всем привет!
Подскажите пожалуйста как переопределить модуль фреймворка?

  1. Создаю модуль
    /application/classes/modules/notify/Notify.class.php
    Как правильно написать?
class ModuleNotify extends Inherit_ModuleNotify
{
}

Или так нельзя?

@mzhelskiy
Copy link
Contributor

Привет. Только через плагин как обычно.

@prokopov-nikolai
Copy link
Author

А можно дополнить функционал фреймворка такой возможностью?

@mzhelskiy
Copy link
Contributor

Смысла особого нет дублировать, можно же сделать некий системный плагин и вынести в него подобные переопределения.

@prokopov-nikolai
Copy link
Author

Когда пишешь сайт с уникальным функционалом, а задача этой ветки (livestreet-boilerplate) именно такая, то выносить базовые вещи в плагины мне кажется не особо логично. Сейчас в модуле Notify не хватает возможности прикреплять файлы к письмам. Такой функционал я бы отнес к базовому, а не к дополнительному.
Свою проблему я решил создав класс Notifier и унаследовав его от Notify. Но такой код выглядит немного коряво.

@mzhelskiy
Copy link
Contributor

Ну доработка фреймворка это же не базовые вещи, поэтому он отдельно от кода приложения. Плагин дорабатывающий фреймворк тоже может быть отдельно от приложения.
Можешь просто запулить расширение класса Notify в ветку с фреймворком.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants