| # |
Jul 20th 2017, 19:24 |
 jeremyharris |
or if it’s a string you want to set, I’ve got to look it up |
| # |
Jul 20th 2017, 19:24 |
alysson-azevedo |
that works, thanks |
| # |
Jul 20th 2017, 19:23 |
jeremyharris |
is this post body data that you want to change then? you can use withParsedBody |
| # |
Jul 20th 2017, 19:22 |
alysson-azevedo |
yes. |
| # |
Jul 20th 2017, 19:21 |
jeremyharris |
sorry misunderstood your code. looks like it’s middleware that you want to modify the body? |
| # |
Jul 20th 2017, 19:21 |
jeremyharris |
it’s a response method |
| # |
Jul 20th 2017, 19:21 |
alysson-azevedo |
i'll try find what method returns that object |
| # |
Jul 20th 2017, 19:20 |
alysson-azevedo |
withStringBody does not exist |
| # |
Jul 20th 2017, 19:19 |
jeremyharris |
try ->withStringBody if it’s a string |
| # |
Jul 20th 2017, 19:19 |
alysson-azevedo |
how can i make that streaminterface? |
| # |
Jul 20th 2017, 19:18 |
alysson-azevedo |
how can i create a new request element with same headers but diferent body? ``` $data = somefunc( $request->input() ); $request = $request->withBody($data); return $next($request, $response); ``` returns the error `Argument 1 passed to Cake\\Http\\ServerRequest::withBody() must implement interface Psr\\Http\\Message\\StreamInterface, string given` |
| # |
Jul 20th 2017, 19:09 |
jeremyharris |
I’ve had no problems last few years |
| # |
Jul 20th 2017, 19:09 |
mikesmoniker |
Good to know. |
| # |
Jul 20th 2017, 19:08 |
jeremyharris |
yea they had a rough start, but they’ve been solid for a few years straight now after fixing their mesh issues |
| # |
Jul 20th 2017, 19:08 |
mikesmoniker |
Er. They’ve gotten better recently? A few years ago they were kind of rough. |
| # |
Jul 20th 2017, 19:08 |
mikesmoniker |
They' |
| # |
Jul 20th 2017, 19:07 |
jeremyharris |
Pagoda Box |
| # |
Jul 20th 2017, 19:07 |
mikesmoniker |
Which PaaS, if you don’t mind? |
| # |
Jul 20th 2017, 19:07 |
jeremyharris |
that’s a good question, and one I can’t really answer well because I don’t manage my infrastructure - I use a PaaS. You bring up a good point about gracefully shutting them down, and it’s something I’m going to have to ask my provider about so I have a better understanding of how it works for me personally |
| # |
Jul 20th 2017, 19:02 |
mikesmoniker |
Worth noting I have supervisord to spin up new workers. I’m just focusing on attractive ways of bringing down stale ones. |
| # |
Jul 20th 2017, 19:01 |
mikesmoniker |
The latter is a bit of extra work for me because this app doesn’t have a redis/memcached/etc. backed cache right now. |
| # |
Jul 20th 2017, 19:00 |
mikesmoniker |
I’m curious what strategies its users have employed for cycling out the workers (the shell) during/after app deployments. Right now I’m not setting a max # of jobs/worker, but doing so would be one way. Also considered some kind of global cache kill switch in the worker loop (I think Laravel does this). I obviously don’t want to just `kill` them, but of course need a way to update those processes when I deploy new code. |
| # |
Jul 20th 2017, 18:58 |
jeremyharris |
I use it in my app, I might be able to help if he’s not around |
| # |
Jul 20th 2017, 18:58 |
mikesmoniker |
@savant If you have a minute sometime I’d love to ask you a couple questions about queuesadilla. |
| # |
Jul 20th 2017, 16:56 |
inoas |
admad jeremyharris here is the issue https://github.com/cakephp/cakephp/issues/10930 |
| # |
Jul 20th 2017, 16:47 |
inoas |
but prefix with/set to modify |
| # |
Jul 20th 2017, 16:47 |
inoas |
there could be readers or immutable objects too where we could omit get |
| # |
Jul 20th 2017, 16:47 |
inoas |
not sure how that can happen but for builders, aka objects where we set a lot of properties again and again, omitting set may be good |
| # |
Jul 20th 2017, 16:44 |
admad |
All the get/set prefixes make the code too verbose for my taste, wouldn't mind doing away with them were possible |
| # |
Jul 20th 2017, 16:38 |
inoas |
and get prefix for getting ... and for read/write objects use set + get prefix |
| # |
Jul 20th 2017, 16:38 |
inoas |
admad when splitting the magic setters/getters wouldn't it make sense for builder like objects to have the pure method name (without set prefix) for the setting? |
| # |
Jul 20th 2017, 16:37 |
inoas |
I take admad by his words - he doesn't put them into plush |
| # |
Jul 20th 2017, 16:33 |
hmic |
this is a no @inoas :p or do you read it differently? :d |
| # |
Jul 20th 2017, 16:32 |
admad |
I haven't been following the discussion |
| # |
Jul 20th 2017, 16:30 |
inoas |
(always a good barometer!) |
| # |
Jul 20th 2017, 16:30 |
inoas |
admad would you think it makes sense to open a ticket or will it just noise-annoy you? |
| # |
Jul 20th 2017, 16:29 |
inoas |
the builders would not "need" to offer getters, but they "could" |
| # |
Jul 20th 2017, 16:28 |
jeremyharris |
I don’t know. That might be appropriate in a separate discussion, with some pseudo code examples. I would like that for the ORM specifically, so we keep the pretty ->contain()->select() and if you want to get, use getContain, etc. |
| # |
Jul 20th 2017, 16:26 |
inoas |
what's the core teams stance on splitting stateful objects into builders (orm for instance) and read/write objects? https://github.com/cakephp/cakephp/pull/10887#issuecomment-316465007 |
| # |
Jul 20th 2017, 16:26 |
inoas |
but its more a proof of concept |
| # |
Jul 20th 2017, 16:25 |
inoas |
yes avoiding duplicate code is good also the validator already got unit tests |