А
из-за использования ссылок получается
dirname('D:\\yii-dev-tool\\dev\\composer-config-plugin\\src', 4), что в результате даёт корень диска )Size: a a a
2020 March 23
AM
поправишь?
AM
@xepozz protected function render(...$data): ResponseInterface - а зачем тут ...?
AM
$this->render($data, $user, $file, $bla-bla) - как-то не очень, по-моему, такое разрешать...
AM
ну и имя render немного путает, намекает на view
А
поправишь?
фикс готов. Также можно переместить $csrf мидлварь под роутер, даже под сессию. А то после роутера какой смысл?
Д
@xepozz protected function render(...$data): ResponseInterface - а зачем тут ...?
ну да, как-то не айс
AM
фикс готов. Также можно переместить $csrf мидлварь под роутер, даже под сессию. А то после роутера какой смысл?
+
Д
ну и имя render немного путает, намекает на view
я потом придумаю, чтобы можно было return $object, а дальше автоматом конвертить
AM
Route::get('/api/post', new FormatConverter($callable, 'json'))
AM
так?
Д
не, чтобы без этого всего
AM
> TODO Need to implement passing the Paginator to this class
А что мешает сделать?
А что мешает сделать?
Д
не пришло время :)
Д
я не знаю, у него вообще есть getLinks
AM
Ещё чуть до мёржа:
1. Учесть в AbstractResponseFactory и JsonResponseFactory выставление заголовков.
2. AbstractController::render() переименовать в createResponse()
1. Учесть в AbstractResponseFactory и JsonResponseFactory выставление заголовков.
2. AbstractController::render() переименовать в createResponse()
AM
3. Убрать TODO should be moved to yii-web
AM
фикс готов. Также можно переместить $csrf мидлварь под роутер, даже под сессию. А то после роутера какой смысл?
pull request?
А
зачем этот код?
А
это создание базовой группы роутов?