В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

Yii Framework 3

439 membersпожаловаться на группу
2020 August 20

AM

Alexander Makarov in Yii Framework 3
Sergei Iamskoi
👍 А в чем сомнения? Я пока еще не разбрилася с системой логгирования в yii3, но сейчас глянул: кидаем sentry/sdk в зависимости SentryTarget, реализум export() в котором из getMessage() закидываем сообщения в SentryClient ( или как он там сейчас называется). Вроде так?
Сомнения в том, то наш Error Handler ловит абсолютно всё. И что в логгер передаётся всё необходимое чтобы в Sentry заслать.
источник
13:42пожаловаться#1

AM

Alexander Makarov in Yii Framework 3
Но да, в идеале это log target.
источник
13:43пожаловаться#2

AM

Alexander Makarov in Yii Framework 3
Переименуем пакет, если что...
источник
13:43пожаловаться#3

Д

Дмитрий in Yii Framework 3
log-target это про интеграцию с logger.
не все будут использовать yiisoft/logger в качестве логгера
источник
13:49пожаловаться#4

AM

Alexander Makarov in Yii Framework 3
Дмитрий
log-target это про интеграцию с logger.
не все будут использовать yiisoft/logger в качестве логгера
Sentry под Monolog имеется.
источник
14:01пожаловаться#5

Д

Дмитрий in Yii Framework 3
почему бы и не под yiisoft/logger?
источник
14:02пожаловаться#6

AM

Alexander Makarov in Yii Framework 3
Дмитрий
почему бы и не под yiisoft/logger?
Да, я всецело за, если это будет так и всё уложится в target логгера.
источник
14:08пожаловаться#7

Д

Дмитрий in Yii Framework 3
Alexander Makarov
Sentry под Monolog имеется.
такое уже есть, остаётся только законфижить
источник
14:09пожаловаться#8

AM

Alexander Makarov in Yii Framework 3
Ну, под наш логгер нет пока.
источник
14:11пожаловаться#9

СП

Сергей Предводителев... in Yii Framework 3
https://github.com/yiisoft/yii-demo/pull/122#issuecomment-676627974

м?
GitHub
Add parameters injections to ViewRenderer + minor refactoring by vjik · Pull Request #122 · yiisoft/yii-demo
Q
A




Is bugfix?
❌


New feature?
✔️


Breaks BC?
❌


Fixed issues
-




Move adding of "csrf" and "user" parameters to external classes CsrfInjection and User...
источник
14:43пожаловаться#10

O

OSW in Yii Framework 3
Welcome, Nikolay Kolev!

Это официальная группа "Yii 3 PHP Framework Community"!

🤝 Присоединяйтесь к обсуждениям, помогайте новичкам и коллегам, участвуйте в разработке фреймворка!

▪️ Документация: https://github.com/yiisoft/docs

▪️ Текущие задачи: https://trello.com/b/GiAnIAir/yii-3

▪️ Поддержать Yii материально: https://www.yiiframework.com/news/204/preparing-yii-for-the-long-run

Чат Yii 2:  @yii2ru
Чат Yii 1.1: @yii1ru
источник
14:48пожаловаться#11

СП

Сергей Предводителев... in Yii Framework 3
https://github.com/yiisoft/yii-demo/pull/122#issuecomment-677590151

Что если вынести в отдельный пакет yii-csrf два класса:
CsrfViewInjection и мидлварю Csrf из yii-web ?
GitHub
Add parameters injections to ViewRenderer + minor refactoring by vjik · Pull Request #122 · yiisoft/yii-demo
Q
A




Is bugfix?
❌


New feature?
✔️


Breaks BC?
❌


Fixed issues
-




Move adding of "csrf" and "user" parameters to external classes CsrfInjection and User...
источник
14:55пожаловаться#12

СП

Сергей Предводителев... in Yii Framework 3
https://github.com/yiisoft/yii-view/pull/2
Добавьте на ревью...
GitHub
Add implementation of CsrfViewInjectionInterface. by vjik · Pull Request #2 · yiisoft/yii-view
Q
A




Is bugfix?
❌


New feature?
✔️


Breaks BC?
❌


Fixed issues
-
источник
16:52пожаловаться#13

AM

Alexander Makarov in Yii Framework 3
Сергей Предводителев
https://github.com/yiisoft/yii-view/pull/2
Добавьте на ревью...
GitHub
Add implementation of CsrfViewInjectionInterface. by vjik · Pull Request #2 · yiisoft/yii-view
Q
A




Is bugfix?
❌


New feature?
✔️


Breaks BC?
❌


Fixed issues
-
Добавил.
источник
16:55пожаловаться#14

СП

Сергей Предводителев... in Yii Framework 3
Спасбио
источник
16:55пожаловаться#15

АН

Александр Некрасов... in Yii Framework 3
https://github.com/yiisoft/yii-view/blob/master/src/ViewRenderer.php#L35 а зачем этот параметр, если уже есть  https://github.com/yiisoft/yii-view/blob/master/src/ViewRenderer.php#L38 ?
GitHub
yiisoft/yii-view
Yii binding for view rendering. Contribute to yiisoft/yii-view development by creating an account on GitHub.
источник
17:01пожаловаться#16

АН

Александр Некрасов... in Yii Framework 3
как-то слишком много инжекторов (интерфейсов), по мне проще сделать 1 интерфейс Injection и там константами типы, и getOptions(), оставить 1 метод в ViewRender injection который обработает все это.
источник
17:06пожаловаться#17

СП

Сергей Предводителев... in Yii Framework 3
Александр Некрасов
https://github.com/yiisoft/yii-view/blob/master/src/ViewRenderer.php#L35 а зачем этот параметр, если уже есть  https://github.com/yiisoft/yii-view/blob/master/src/ViewRenderer.php#L38 ?
GitHub
yiisoft/yii-view
Yii binding for view rendering. Contribute to yiisoft/yii-view development by creating an account on GitHub.
Это для withCsrf
источник
17:21пожаловаться#18

СП

Сергей Предводителев... in Yii Framework 3
Александр Некрасов
как-то слишком много инжекторов (интерфейсов), по мне проще сделать 1 интерфейс Injection и там константами типы, и getOptions(), оставить 1 метод в ViewRender injection который обработает все это.
Нельзя один метод. Там часть нужно обрабатывать для контента, а часть для лейаута. Ну и в предыдущих обсуждениях пришли как раз к разделению на интерфейсы и методы.
источник
17:22пожаловаться#19

Д

Дмитрий in Yii Framework 3
Почему бы не инжектить и в контент и в лэйаут сразу всё?
источник
17:28пожаловаться#20