| # |
Dec 25th 2019, 16:36 |
 ndm |
no problem |
| # |
Dec 25th 2019, 16:36 |
admad |
@ndm thanks for the doc update |
| # |
Dec 25th 2019, 16:32 |
challgren |
But the exception makes more sense now to figure out how to semver a release back to prerelease and add the beta tag |
| # |
Dec 25th 2019, 16:30 |
challgren |
No try { in https://book.cakephp.org/4/en/core-libraries/email.html |
| # |
Dec 25th 2019, 16:30 |
ndm |
I thought I've seen an example in docs that wrapped the `Email::send()` call in `try ... catch`, but I can find it, so maybe that's just my imagination |
| # |
Dec 25th 2019, 16:28 |
challgren |
Ok so I got some work on the mailgun plugin to fix things up |
| # |
Dec 25th 2019, 16:26 |
ndm |
Failures in transports should be indicated by throwing exceptions |
| # |
Dec 25th 2019, 16:25 |
challgren |
Im not sure honestly, but my question is since it returns an array and it looks like it always has headers and messages how does one detect a failure |
| # |
Dec 25th 2019, 16:25 |
ndm |
Wasn't it always `array`, or least for quite some time? I don't remember |
| # |
Dec 25th 2019, 16:22 |
challgren |
Yeah, before it returned mixed but now its array |
| # |
Dec 25th 2019, 16:22 |
ndm |
Not sure what exactly you mean by "_it_"? The transport? |
| # |
Dec 25th 2019, 16:18 |
challgren |
But it no longer returns false on failure :( |
| # |
Dec 25th 2019, 16:18 |
challgren |
Ok @ndm you got something with replacing the transport |
| # |
Dec 25th 2019, 15:56 |
ndm |
gn8 |
| # |
Dec 25th 2019, 15:55 |
ndm |
Just sprinkle some more swapping magic on top :) |
| # |
Dec 25th 2019, 15:54 |
challgren |
I'll try later though, it's bedtime |
| # |
Dec 25th 2019, 15:53 |
challgren |
It's used during testing, it's magically swapped out in testing |
| # |
Dec 25th 2019, 15:52 |
ndm |
How so? Where do you use that? |
| # |
Dec 25th 2019, 15:50 |
challgren |
@ndm using a different transport would kill DebugTransport |
| # |
Dec 25th 2019, 15:48 |
challgren |
I give up for today |
| # |
Dec 25th 2019, 15:43 |
ndm |
Not anymore now though... until the next build that is |
| # |
Dec 25th 2019, 15:41 |
ndm |
Sure, the change is totally fine, it's just that the docs still say `_csrfToken` ;) |
| # |
Dec 25th 2019, 15:39 |
admad |
@ndm The reason for the change is getAttribute() is a method of the psr interface while getParam() is cake specific, so static analyzers get sad if you use the latter in middlewares methods as they have the argument typehinted to ServerRequestInterface |
| # |
Dec 25th 2019, 15:38 |
challgren |
Debug kit is being a POS hmmm may have to try that |
| # |
Dec 25th 2019, 15:35 |
ndm |
How about using a dummy/mock transport instead? |
| # |
Dec 25th 2019, 15:31 |
challgren |
Grrrr Im starting to not like Cake 4 |
| # |
Dec 25th 2019, 15:20 |
ndm |
OK then, first of all, .NET MVVM Framworks... their documentations are so absolutely terrible, I could even forgive @admad about the `_csrfToken` mistake :) |
| # |
Dec 25th 2019, 15:13 |
challgren |
_waits at the end of the line_ |
| # |
Dec 25th 2019, 15:12 |
ndm |
@challgren You'll have to line up in the back, I've got a lot of problems with all sorts of people! |
| # |
Dec 25th 2019, 15:09 |
slackebot |
<challgren> |
| # |
Dec 25th 2019, 15:08 |
challgren |
To simulate a failure |
| # |
Dec 25th 2019, 15:08 |
challgren |
@ndm I have a bunch of grievances! |
| # |
Dec 25th 2019, 15:08 |
admad |
Why do you even need to mock a mailer? |
| # |
Dec 25th 2019, 15:08 |
challgren |
Wanted to discuss that issue from a baker perspective, but we can chat later |
| # |
Dec 25th 2019, 15:07 |
admad |
@challgren am now, on mobile |
| # |
Dec 25th 2019, 15:07 |
ndm |
Sounds like I need to get my Festivus pole |
| # |
Dec 25th 2019, 15:06 |
admad |
Merry Christmas bakers :christmas_tree: |
| # |
Dec 25th 2019, 15:06 |
admad |
@ndm in 3.x it was even a request param not request attribute :) |
| # |
Dec 25th 2019, 15:06 |
challgren |
Hmmm how do I mock a mailer in cake4. Before it was as simple as `$mock = $this->getMockBuilder('Cake\Mailer\Mailer')->getMock();` now that throws a warning `Static method "getConfig" cannot be invoked on mock object` |
| # |
Dec 25th 2019, 14:53 |
ashesh.social |
Great working @ndm Thanks ! |
| # |
Dec 25th 2019, 14:48 |
ndm |
*now |