« first day (2448 days earlier)      last day (2505 days later) » 

4:08 PM
 
Does PHP suck?
 
yes
 
Anonymous
 
https://t.co/h2h3RFCjAn
 
@JayIsTooCommon Wat… #NotMyPresident
 
new avatar? @PeeHaa
 
Where?
 
the peehaa one :B
 
Anonymous
4:29 PM
@WesStark shouldn't that be my avatar?
 
now i'm just gonna pretend i didn't waste an afternoon
 
@WesStark Can I? Last time you wanted me to pay you for your services :P
 
but for a good cause.
please someone do the tweet script :B
 
@WesStark thanks :D
 
@tereško also facepalm was inspired by you i.imgur.com/zyUuena.png
@PeeHaa i'm not like @JayIsTooCommon's mum
 
4:53 PM
you guys just don't have enough stereotypes
ok i ran out of ideas
 
why does the top one have an tentacle coming out of the end of it's arm?
 
it's a slice of pizza.
 
ohhh
goes away again
 
:B
 
@WesStark aw... I almost thought that you were into tentacle porn
 
5:03 PM
that's you :B
 
the real semantics of semantic versioning: major = a breaking change minor = a minor breaking change patch = a little-bitty breaking change
 
i wish :x
 
Hahah
Evening room
 
@WesStark this may be useful
no centering/justifying though
 
5:16 PM
cc @DaveRandom ^
 
@WesStark awesome :) you are very creative
 
thanks :B
 
:)
 
5:40 PM
 
@WesStark how did you match the figure so closely?
😉
 
:P
 
:B
 
how do update price and quantity by asin code in mws api
 
Anonymous
 
Loading PHP extension with already registered function name leads to a crash – #74832
 
6:32 PM
@Jeeves As it should. Closed as "Won't Fix/Not A Bug"
 
@Dereleased Good. That was how I interpreted it.
@Dereleased You sound like my friend.
 
@Jeeves I am your friend!
 
@Dereleased No you're not, I don't know you.
 
@Jeeves My mistake. I just sound like someone who's your friend. Anyone I know?
 
@Dereleased If you have friends that tell you nobody loves you then you need new friends...
 
6:34 PM
@Jeeves I agree, but that hardly answers my question... unless this is a cry for help! @Jeeves, who hurt you!?
 
@Dereleased Just making small chat.
 
Noite all.
 
Anonymous
!!wotd
 
6:40 PM
ophidian: a snake.
 
@WesStark you need one with a square wheel
 
7:29 PM
ugh. I'm kinda depressed. how do I regain productivity, at work?
 
7:43 PM
@FélixGagnon-Grenier Take a nap then look at the things you need to do and do one of them.
 
right. the things I need to do. I'll start by doing a list.
 
@WesStark that kind of stuff is why I think I want to ship PHP code with the extension. Doing that in PHP is trivial. Doing it in C, not so trivial.
Fyi I am in Amsterdam and am incredibly likely to walk in front if a cyclist or car before the weekend is over.
 
8:00 PM
@Danack wanna go for beer on sat and make it even more likely?
 
8:21 PM
@PeeHaa Still having issues with Message?
 
Hello
Can I get the domain that is sending a $_POST request to me?
The reason I ask this, is that I want to make an "email relay" script for identified sources
to allow only specific domains to use this "email relay"
so I can host a simple .html script on any environment that is capable of sending emails just by posting the info to this script
 
I think the only issue in front of me now is Loop exceptionally stopped without resolving the promise @kelunik
 
@Ekin Promise or watcher callback throwing an exception?
 
I'm not sure about what's actually going on yet
 
The Error instance thrown there should have a previous exception.
s/Promise/onResolve-callback
 
8:39 PM
... So weird. I have no changes on any file but now I get "Error: Cannot use a promise as success value" for github.com/ekinhbayar/gitamp/blob/v2_rewrite/src/Client/… instead of above
 
@Trowski That should throw, no?
 
It should.
 
@Ekin Do you use Amp\PHPUnit\LoopReset?
 
I don't think so, no
 
@Trowski No, I don't think there's one for a loop that stops early.
 
8:41 PM
@Ekin $this->resultFactory->build($response) returns a promise. Just change that line to return yield $this->resultFactory->build($response);
@kelunik Not stopping early, "exceptionally stopped"
 
Both appear on errors and warnings
 
@Trowski Oh, then yes.
 
`1) ekinhbayar\GitAmpTests\Client\GitAmpTest::testListenReturnsResults
Error: Loop stopped without resolving the promise`
now got that ^ after the yield and I see Loop exceptionally stopped on warnings
 
@Ekin Why on warnings?
 
8:45 PM
@Ekin Is this line ever reached? If so, does the yielded promise resolve?
 
> Failed asserting that exception of type "Error" matches expected exception "ekinhbayar\GitAmp\Client\RequestFailedException"
 
What are the previous exceptions?
@Ekin Just change that to return $this->resultFactory->build($response);, no need for yield.
 
@Trowski Yes. I get the body string
 
You guys ever implement login with facebook or twitter on your PHP application?
 
@kelunik right, although it does the same thing
 
8:51 PM
I still like yielding the promise and returning the resolved value.
That's mostly a personal preference. Only matters if using yield from.
@Ekin Then it's not immediately obvious to me why the promise isn't being resolved.
 
@Ekin You simply don't add a body to your mock?
 
@Ekin This Response is going to need a Body.
$response->method('getBody')->willReturn(new Message(new InMemoryStream("mock data")));
The loop exceptionally stopped probably because null was being returned instead of a Message instance.
 
I see
How about mocking the Client github.com/ekinhbayar/gitamp/blob/v2_rewrite/tests/Client/… which should be just BasicClient but I can't mock that, right?
 
@Ekin Why should it be BasicClient?
 
I think it's what it expects at the moment
 
9:05 PM
Change the type declarations to Client.
 
Well, that takes me back to stopping loop issue
 
exceptionally or without resolving?
 
Again the error is without resolving
 
Yes it does
 
9:10 PM
@JoeWatkins Do you mind if I update some of the content on the pthreads.org website (such as the broken examples)? Or would you rather I left that to you?
 
@Ekin github.com/ekinhbayar/gitamp/blob/v2_rewrite/src/Client/… This is calling a method that can throw.
That should not be in an onResolve callback, but checked after yielding the promise.
 
Oh right, it does
 
@Ekin Do you set a loop error handler somewhere that hides the exceptions?
 
I have Amp\Loop::setErrorHandler on server.php
 
Do you include that somewhere in the tests?
I'd recommend using github.com/amphp/phpunit-util to run each test automatically with a new event loop.
 
9:13 PM
@kelunik I don't think we do
But I'm not sure, /cc @PeeHaa
It goes into github.com/ekinhbayar/gitamp/blob/v2_rewrite/src/Client/… listen() though. Confused whether the flow here is okay now
 
@Ekin It should do that anyway, yes, but does the line later get reached?
 
Yes
 
9:29 PM
Till the yield I think
 
Then what does it react in appendResponse :P
 
I'm trying to see that :P
 
Just add a few fputs(STDERR, "...");
 
@Ekin Nope. Not in tests afaik @kelunik
 
9:50 PM
Streaming video to HTML5 video transcoding on the fly with ffmpeg...anyone think/know whether or not it would work?
 
10:02 PM
@Ekin If you're providing an empty body, then json_try_decode probably fails. Though you'd think you'd see an exception from the test then.
 
nothing throws in there
afaics
Stepping through the code now
 
Random crashes, seemingly related to DateTimeImmutable and mod_php – #74833
 
10:24 PM
@Trowski Can you pull something and run it / open teamviewer to me?
 
@PeeHaa Can definitely pull something and try it.
 
kk 1 sec. Let me push something specially for you :P
From the trowski branch run
phpdbg -qrr ./vendor/phpunit/phpunit/phpunit --filter testListenReturnsResults
It will reach github.com/ekinhbayar/gitamp/blob/trowski/src/Client/… just fine, but doesn't reach into the build method after that github.com/ekinhbayar/gitamp/blob/trowski/src/Response/…
FWIW @Trowski @kelunik I am already proxying room11 github pages so if the amp pages also needs to be proxied just let me know
 
10:50 PM
@PeeHaa The response factory wasn't configured, so the build method doesn't return anything.
 
@poo
oooooooh
Oooooooooooh
I see. It's because of the change of the yielding of the body that it needs to return something now
 
Yeah… though all that method is really testing then is your ability to create mocks.
 
Yeah was just a simple test added to cover a specific path
<3
@Ekin ^^^ he did it \o/
 
Oh well, great
\o/
 
tnx
 
10:56 PM
If you want to actually test something, feed in some valid JSON for the request body and give it a real factory.
 
Going to pass in json into the mock factory
...
 
$this->factory->method('build')->willReturn(new Success($this->createMock(Results::class))); :-D
n/m, that fails because then the request object is never used.
 
That's... exactly what I have now :P
 
@PeeHaa Oh, that's because the code that called getStatus was removed.
 
yeah. tried to minimize it for you
What you shared works for us
 
11:09 PM
@PeeHaa @Ekin If it matters, this is what I did with the GitAmp class: gist.github.com/trowski/1d12934757cb3969a680ae08a1e8bf49
 
That's actually useful
 
Can probably catch a more specific exception on line 51.
And log the error. You get the idea. :-D
 
yeah ta
 
@pmmaga yes, sounds good. I'm flying out sunday evening.
 
Hi guys. I am new to wordpress development. Why the the php code in wordpress themes look different from standard php syntax? Endwhile? Endif? no curly braces? Is it some sort of templeting language?
 
11:22 PM
!!docs alternative.syntax
 
2 messages moved to Trash
 
Ok thanks. Wasn't aware of this php alternative syntax.
 
np
 
11:47 PM
@Ekin ping
 

« first day (2448 days earlier)      last day (2505 days later) »