Conversation
|
Ну сейчас это зависимые методы, нельзя получается сделать updateOnLoad и получить просто обновление при старте без autoLoad, при это если забыть добавить autoLoad то вообще ничего не будет происходить, если мы делаем связанный функционал только лучше через параметр autoLoad($ms, $onLoad = false) но в идеале сделать их независимыми чтобы можно было просто добавить updateOnLoad и получить апдейт фрагмент без полинга |
Да, ок. З.Ы. можно сказать, что у меня раздвоение личности, но так получилось |
ca88b2d to
bd72c8b
Compare
| return $this; | ||
| } | ||
|
|
||
| public function autoUpdate(int $ms, bool $immediate = true): static |
There was a problem hiding this comment.
отсюда можно убрать раз есть отдельный независимый метод
| } | ||
|
|
||
| public function autoUpdate(int $ms): static | ||
| public function updateOnLoad(bool $enabled = true): static |
There was a problem hiding this comment.
давай тут сделаем как принято в проекте через $condition bool|Closure|null только распиши Closure через докблок
There was a problem hiding this comment.
Я чет в примерах не нашел расписанный док блок, что имеешь ввиду
There was a problem hiding this comment.
(Closure(bool): bool) смотри в исходниках, везде есть
bd72c8b to
211e1e2
Compare
| { | ||
| return [ | ||
| 'interval' => $this->interval, | ||
| 'updateOnLoad' => $this->updateOnLoad, |
There was a problem hiding this comment.
ты вот тут делай запускай closure а не сразу в методе, иначе какой от нее толк, в чем отложенность то?
| ); | ||
| } | ||
|
|
||
| public function updateOnLoad(Closure|bool|null $condition = null): static |
There was a problem hiding this comment.
**@PARAM (Closure(static): bool)|bool|null $condition = null **
|
|
||
| public function updateOnLoad(Closure|bool|null $condition = null): static | ||
| { | ||
| $this->updateOnLoad = value($condition, $this) ?? true; |
There was a problem hiding this comment.
да какой смысл сразу колбек вызвать, вызывать надо во viewData, куча же примеров в коде с отложенным вызовом
|
@MKKMXD @korundKrona ping |
211e1e2 to
320a213
Compare
320a213 to
62a2945
Compare
This branch has not been deployed
3 participants
What was changed
Why?
Checklist