Size: a a a

2020 March 30

AM

Alexander Makarov in Yii Framework 3
Ну даже если выкинуть этот пункт...
источник

AM

Alexander Makarov in Yii Framework 3
Пункт 2 и у тебя и у @xepozz одинаково вышел. Круто, но что с заголовками-то делать?
источник

А

Алексей R in Yii Framework 3
Alexander Makarov
Пункт 2 и у тебя и у @xepozz одинаково вышел. Круто, но что с заголовками-то делать?
я пока пришёл к решению, что лучше, чтобы конвертер предоставлял список желаемых заголовков
источник

А

Алексей R in Yii Framework 3
а мидлварь сама их вешала
источник

А

Алексей R in Yii Framework 3
но сделал это отдельным методом конвертера
источник

А

Алексей R in Yii Framework 3
> ➖ Router configuration got more complicated.
из-за ActionCaller'а?
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
коммит уже видел. вот это ты упустил 😐
Не упустил, я ответил там, читай внимательнее. Все элементарно делается без непосредственной передачи инстанса.
$route->addMiddleware(function (ServerRequestInterface $request, RequestHandlerInterface $handler) {
      $key = 'sdf6&-$<@#asf';
      return (new Middlewares\ImageManipulation($key))->process($request, $handler);
  });
источник

А

Алексей R in Yii Framework 3
Dmitriy S
Не упустил, я ответил там, читай внимательнее. Все элементарно делается без непосредственной передачи инстанса.
$route->addMiddleware(function (ServerRequestInterface $request, RequestHandlerInterface $handler) {
      $key = 'sdf6&-$<@#asf';
      return (new Middlewares\ImageManipulation($key))->process($request, $handler);
  });
и что теперь везде кложуры втыкать?
источник

А

Алексей R in Yii Framework 3
чем они лучше объекта MiddlewareInterface?
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
я пока пришёл к решению, что лучше, чтобы конвертер предоставлял список желаемых заголовков
Конвертер стрима не должен ничего знать о заголовках, это не его скоуп.
источник

AM

Alexander Makarov in Yii Framework 3
Алексей R
я пока пришёл к решению, что лучше, чтобы конвертер предоставлял список желаемых заголовков
Я как-бы про свои заголовки. Вот надо мне e-tag добавить, например. Как я это сделаю, если у меня инстанса response?
источник

А

Алексей R in Yii Framework 3
Dmitriy S
Конвертер стрима не должен ничего знать о заголовках, это не его скоуп.
о типе предоставляемых данных знать он должен )
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
и что теперь везде кложуры втыкать?
Только если нужно собирать объект не через контейнер. Это не так уж часто бывает - это раз. Два - с кложей он собирается лениво, только если нужен. Если у тебя манипулятор картинок не в текущем роуте, то он не будет собран, так как он нафиг не нужен.
источник

А

Алексей R in Yii Framework 3
Alexander Makarov
Я как-бы про свои заголовки. Вот надо мне e-tag добавить, например. Как я это сделаю, если у меня инстанса response?
если можно взять фабрику для стрима/респонса, что мешает сделать респонс из данных?
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
о типе предоставляемых данных знать он должен )
Ок, будет у тебя в конвертере тайп джсон записан. Где ты его в заголовки будешь мапить?
источник

А

Алексей R in Yii Framework 3
Dmitriy S
Ок, будет у тебя в конвертере тайп джсон записан. Где ты его в заголовки будешь мапить?
у меня это в мидлваре сейчас делается 😐
источник

А

Алексей R in Yii Framework 3
самое место там и делать )
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
самое место там и делать )
У тебя сейчас конвертер стрима выставляет заголовки
https://github.com/yiisoft/yii-demo/pull/75/files#diff-fb57afcd45ad253e10121e720eb734aaR41
источник

AM

Alexander Makarov in Yii Framework 3
Алексей R
если можно взять фабрику для стрима/респонса, что мешает сделать респонс из данных?
public function myAction()
{
   // how to add e-tag to response here?
   return ['user' => 'Alexey'];
}
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
public function myAction()
{
   // how to add e-tag to response here?
   return ['user' => 'Alexey'];
}
Тоже норм пример. Заголовки вообще нельзя выставить в экшине в этом решении. А оно вполне может понадобиться.
источник