AM
добавь тест, пофиксим
Size: a a a
2020 April 01
DS
В DataResponse должна передаваться только фабрика
AM
для этого не надо зависимости искажать
AM
он оборачивает Response, а не фабрику
AM
у меня может быть response без фабрики в теории
DS
Попробуй сделать так
$response = $responseFactory->createResponse();
$response = $response->getBody()->write('test1');
$dataResponse = new DataResponse('test2', $response);
$response = $responseFactory->createResponse();
$response = $response->getBody()->write('test1');
$dataResponse = new DataResponse('test2', $response);
AM
ок, попробую
DS
ок, попробую
А еще лучше там массив передай вместо test2 и форматтер примени Json😉
AM
а что ты ожидаешь при этом?
DS
а что ты ожидаешь при этом?
В первом случае склеится, во втором test1 проигнорируется
AM
ну так и есть
AM
но... ты реально хочешь такого поведения?
DS
Хотя не, во втором тоже склеится
DS
Вот это
$response = $responseFactory->createResponse();
$response = $response->getBody()->write('test1');
$dataResponse = new DataResponse(['test' => 'test2'], $response);
$dataResponse->withDataForamatter(JsonDataFormatter);
$dataResponse->getBody()->rewind();
$dataResponse->getBody()->getContents();
Выдаст строку
test1{"test":"test2"}
$response = $responseFactory->createResponse();
$response = $response->getBody()->write('test1');
$dataResponse = new DataResponse(['test' => 'test2'], $response);
$dataResponse->withDataForamatter(JsonDataFormatter);
$dataResponse->getBody()->rewind();
$dataResponse->getBody()->getContents();
Выдаст строку
test1{"test":"test2"}
DS
но... ты реально хочешь такого поведения?
Я хочу как было до твоего исправления
AM
да, makes sense, в общем-то
AM
откатил
DS
Ну и в демке я еще одно улучшение хотел сделать, но то уже я отдельным пр
DS
А в целом с именами норм получилось 👍
DS
Ну и в демке я еще одно улучшение хотел сделать, но то уже я отдельным пр