« first day (1614 days earlier)      last day (3348 days later) » 

9:00 PM
ah, I do not understand it.. but I think it is very cool
 
with 32 bit it'd be another hash
Sure, it randomly just happens with a probability with 1:2^32 (on 32 bit builds)… but it's nonetheless an issue…
 
@bwoebi Yeah, I noticed that bogus assert as well
 
@bwoebi erm.....that's a pretty bad thing. It could be a real DOS attack. Maybe here isnt the best place to talk about it.
 
Should either drop it or mask the hash to be non-zero
@Danack it's only for php 7 ;)
 
@Danack it's a PHP 7 only issue.
 
9:05 PM
ah - k. Anyone running that in production now deserves it then.
 
sort of
 
@Danack and also only for debug builds, of course
 
I don't know if there are any other assumptions on the hash being not zero except for the check whether the hash already was calculated… @NikiC ?
 
@bwoebi I think it's only relevant for the hashval caching, but not 100% sure
 
So… just remove the assert for now?
It's not an issue if the hashval is recalculated each time for one of 2^32 or 2^64 strings…
 
9:09 PM
@bwoebi that's what I'd do
remove the assert and add a test :)
 
Okay… let me just quickly calculate a hash which is zero on 32 bit...
 
@bwoebi last I tried doing something stupid like function x() { x(); } x(); it would segfault.. now it runs out of memory.. but then keeps doing it on every command after that
php > function x() { x(); } x();

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 262144 bytes) in php shell code on line 1
php > echo 42;

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 262144 bytes) in Unknown on line 0
 
yeah, it's not doing a full shutdown here...
 
It is expected punishment or a bug?
 
so, @NikiC I've run into a weird situation
 
9:13 PM
@RonniSkansing no idea. I'd guess bug.
 
exceptions don't append the "defined in %s on line %d" bit, since the thrown information (where it was thrown) has it. For a userland function, the $ex->line and $ex->file are the declaration. For an internal function, they are the call site
so what I did was modified zend_exception_error to append the and defined bit before raising the error for type exceptions. This makes good uncaught fatals, but it also puts wrong info for the internal functions
 
# using mod 2^32
php > var_dump($hash("\x8e\x1a\x63\x0f\x61"));
string(1) "0"
 
eep, posting on twitter as an app on a users behalf is killing me. I couldn't do it in python, so I moved on to trying in node with no dice, so now I've finally fallen back to php. I have hi hopes for it though. Seems to be one of those things that is hard to find good examples about because so many people created the same tutorial for something simple 1000 times to look smart.
 
@ircmaxell not sure I understand what you mean
 
any who, I just needed to vent somewhere
that is all
 
9:17 PM
why would that happen in zend_exception_error?
Isn't the "and defined" directly part of the error message?
 
no
it used to be all the error messages ended with "defined"
 
so zend_error would append " in %s on line %d"
 
@ircmaxell yeah, I'd just leave that ^^
 
uhhh
^^ that test is in master
Argument 1 passed to blah() must be an instance of Foo, instance of stdClass given, called in %scatchable_error_002.php on line 18 and defined
 
9:19 PM
@rdlowrey is there a reason why $request->getStringFormField('fieldname') works, but $request->get('fieldname') causes this error for the same request? The fieldname is set otherwise the first would not work, but the 2nd errors.
 
that's $e->getMessage()
 
@PeeHaa thats what I'm talking about. All I see there are app only authentication examples
everything out there is app only
 
@ircmaxell yeah sure it's a bit ugly, but I don't see how you can avoid it without adding even uglier special cases
 
I want to post to a users timeline as the user after they grant my app permissions
not post to my own timeline
that would be eas
thats why I'm so frustrated
 
@Rooster You need an app for that either way
 
9:21 PM
:/
 
@crypticツ isnt the get for query data and the FormField a fast way to get from form data? (sorry didnt look up code, just throwing a guesss)
 
well, that's what I was asking about. Is there a reasonably clean way to forward that information to zend_exception_error to make the determination there if it was an internal function or not
 
If you go to your Facebook profile, go to About, and then Places You've Lived, for some people there is a Residences subsection. How do you add one of those, or join an existing one?
 
I mean, I could add a member to the class, but that's not ideal
 
@PeeHaa yeah, but that link doesnt help me do what I want to do is what I'm saying
i guess step 1 would be useful
if you didn't know you needed to register an app
 
9:22 PM
@ircmaxell whatever you do will be non-ideal in any case, because any other "things" handling it aren't unlikely to use the same message structure as PHP does
 
@Patrick you there?
 
@NikiC what do you mean?
 
Well isn't it just a matter if getting the user token and using it? @Rooster
 
what other "things"?
 
yeah
no examples of that around though
well, I actually may have finally found one
 
9:23 PM
@Patrick followed your tutorial... trying to unit test a controller now, It doesn't find the class, how would I go on about writing a bootstrap loading the autoloader et cetera, or can I just include src/Bootstrap.php?
 
But isn't that what jimbo's code does?
 
@MarcelBurkhard yeah. just finishing my taxes
 
but meh, I'm just bitchy cause this is my 3rd day working on it and it really seems like it should be a trivial thing
 
@crypticツ on closer inspection the Form methods are for POST
 
@Rooster :P
 
9:24 PM
basically
 
@Patrick haven't done those yet aswell ^^
 
hoping doing it in php will help me understand the process enough to be able to translate it to python
 
@NikiC also: isn't this a regression? Since now the information about where classes are defined goes away if you don't catch the exception?
 
I really just think Im spread out over too many concepts at the moment
 
@Rooster I would recommend reading the oauth spec first
 
9:25 PM
@RonniSkansing so the form methods are only accessible for POST? So I assume get() is only for GET?
 
not enough processing power
 
@ircmaxell huh?
 
The documentation is pretty okish
Especially twitter oauth pages are useful
 
./vendor/bin/phpunit --bootstrap vendor/autoload.php test
last part being your test folder. that lets you use the composer autoloader for your tests
 
why would that info get lost?
 
9:25 PM
I found it to be one of the best resources to understand it
 
Hiya Niki! Goodnight, room.
 
@NikiC meaning that 3v4l.org/ad8dT looses the "defined in" part, making the error message less useful
 
all the redirection is mostly whats confusing me
which didn't use too
 
@crypticツ use getQueryParameter($field) for GET =]
 
@Patrick nice that works
 
9:26 PM
@ircmaxell why would it loose that part?
I think I still don't get the issue
 
@NikiC it loses it on master right now
 
been spoiled for a couple years and haven't had to deal with that sort of thing
 
@NikiC github.com/php/php-src/blob/master/tests/lang/… <-- that test explicitly ends with "defined" not including the "in ... on line ..."
 
nikic@saturn:~/php-src$ sapi/cli/php t023.php

Fatal error: Argument 1 passed to foo() must be an instance of stdClass, integer given, called in /home/nikic/php-src/t023.php on line 5 and defined in /home/nikic/php-src/t023.php on line 3
the "and defined in" is there for me
@ircmaxell that's just the bare exception message
 
mostly just amazed theres not a straightforward example of how to do what I want somewhere easy to find
for a service as big as twitter
 
9:28 PM
@NikiC that's my point
 
The file/line info is in $ex->getFile() and $ex->getLine()
 
The examples contain the entire flow
 
@crypticツ I think the get() are for headers and ASGI map (or what they are called), try calling all() for everything available to get()
 
@RonniSkansing then what is the purpose of get($field)?
 
@NikiC which for userland functions is the declaration or the return line respectively. For an internal function call, it's the call site
hence the inconsistency
 
9:30 PM
Think it retrives $_SERVER[$field] or ASGI mapped
 
@ircmaxell Yes, the error message reflects the different usages though
 
@NikiC nevermind
/me will figure something out
 
@ircmaxell unrelated to scalar type hints in any case ;)
 
it is related
because I need to make sensable error messages
 
and fyi this issue has been around since forever
 
9:34 PM
@crypticツ ronnieskansing.github.io/arya .. If you ever need a API ref ... just noticed it is missing the Request... =/
 
@NikiC nobody's tried refactoring them to use the same error system before
 
@rdlowrey Which reminds me; how's Lusitanian's mom doing?
@ircmaxell Or at least they didn't succeed :)
I guess that should be a frowning face though :(
 
@ircmaxell I'm sure I the fault is on my side (as I had to modfiy zend_API.c to let the build suceed, but function foo1(\Foo\String $baz) { var_dump($baz); } segfaults for me with your branch.
 
@Rangad I'm working on the branch pretty aggressively, when did you build it?
also: you had to modify zend_API.c??? what did you do?
 
f040538448
the latest on github
 
9:39 PM
what modification?
 
@Danack By the way… the usual vector here are hash collisions… which you always could easily calculate…
 
@DaveRandom I'm done with hex \o/ github.com/Ralt/agni/commit/…
 
@bwoebi I was assuming that a failing assert would crash an FPM worker, which would allow someone to take a server offline.
 
move the is_method declaration up to the other declarations in zend_parse_method_parameters
 
oh windows ^^
 
9:42 PM
@Danack yeah… would have been bad if code really relied on this hash being non-zero…
 
@NikiC I hear you're going to be at IPC and that we'll both be talking about PHP7...
 
@bwoebi btw second line in git commit message should always be empty
 
@NikiC ah? why?
 
@bwoebi Because the first line is the "Subject" portion.
 
yeah, and from second line on, like body portion…
 
9:44 PM
git shows the messages nicely formatted if the commits are done according to a few rules
 
@Sara But I hope not the same topic?
 
@Sara not sure why second line should be empty…
 
@NikiC No, same track in fact.
 
first line <72 chars (but <53 recommended), 2nd line empty, then paragraphs
 
@NikiC Anyway, we should coordinate what we'll be talking about specifically at some point.
 
9:45 PM
@Sara I mean as in, will you be talking about userland changes?
 
@NikiC That was my plan, yeah
 
It would be pretty weird to have two talks on PHP 7 internals
 
Yeah, I heard you were going to be talking about internals... ast, memory, hash tables, etc...
 
Anyway, awesome that you're coming :)
 
@PeeHaa besides the fact that this tutorial doesnt tell or provide any of the file prefixed with Epi, it does what I want (finally!) 9lessons.info/2014/04/…
 
9:46 PM
@NikiC Yeah, looking forward to meeting you in person finally
 
now I get to reverse engineer it into nodey pythony things
horray
 
me too :)
 
@Sara don't hurt him too much
 
@Rooster SQL injection galore \o/
 
@FlorianMargaine Does a job offer at Facebook count as harm?
 
9:48 PM
definitely
 
@Sara If that means he will work more on hack/hhvm instead of actual php then yes
 
see https://wiki.openstack.org/wiki/GitCommitMessages
http://stopwritingramblingcommitmessages.com
http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
http://stackoverflow.com/questions/2290016/git-commit-messages-50-72-formatting
https://github.com/erlang/otp/wiki/Writing-good-commit-messages
http://sethrobertson.github.io/GitBestPractices/#usemsg
http://chris.beams.io/posts/git-commit/
 
nah it'd be fine, such a relaxation to work on php-src's C after working all day on hhvm's C++
 
> Bad, because the community has the shape of a pyramid. The bottom of that pyramid are those who know PHP but don't know "how" to be good developers/programmers. They never heard of Laravel, Composer and unit testing.
The top are very good developers like Fabien Potencier and Taylor Otwell (and many others btw, these are just 2 names I can think of right now) who make the PHP community better.
 
9:49 PM
lul
 
Uhm. I don't think they are the top, with all due respect.
 
lolwtf
 
Maybe middle of the pyramid.
 
@LeviMorrison who would you consider as such? :-)
 
@PeeHaa wellllllll, I'd like to think that since its a proof of concept they were just leaving that out for simplicities sake
 
9:51 PM
@FlorianMargaine You want specific names?
 
@FlorianMargaine No way man. C99 versus C++11? No contest. C++11 all the way.
 
@LeviMorrison yeah, to have an idea of which level you mean
@Sara too much increment
 
who's that guy that hacked the gibson?
 
@Rooster I'd like to think that the person who wrote it should stop spreading vulnerable applications to innocent readers
 
@Sara same track meaning at the same time?
@Sara definitely, no question there ^^
 
9:52 PM
@LeviMorrison you can do so in a private room if you prefer
 
@NikiC no
 
same room typically (means you're definitely not up against each other
 
Hey @Sara, do you know if Andrei Alexandrescu has written PHP code at Facebook?
 
isn't he working on D exclusively?
 
9:53 PM
@ircmaxell ah okay
 
^^ As @ircmaxell said
 
		retval_ptr = GET_OP1_ZVAL_PTR(BP_VAR_R);

		if (EXPECTED((opline->extended_value & 2) == 0)) {
			/* Does not return by reference */
			/* TODO: separate zval if it's a reference to prevent the cast from harming anything */
			if (UNEXPECTED(Z_ISREF_P(retval_ptr))) {
				SEPARATE_ZVAL(retval_ptr);
			}
		} else {
			ZVAL_DEREF(retval_ptr);
		}

		zend_verify_return_type(EX(func), retval_ptr, opline->extended_value & 1);
 
I had misunderstood your original question, that you asked "At the same time?" not "About the same thing?"
 
^^ does that look sane for handling the ref return issue @NikiC?
the todo isn't needed, that's done
 
@FlorianMargaine Anyway, guys like Andrei Alexandrescu. Even if he hasn't written PHP I know there are caliber programmers like him in PHP.
 
9:54 PM
opline->extended_value & 2 would mean it returns by reference
 
@Levi He was on the HHVM team for awhile, so he probably wrote PHP tests at the very least.
 
@Sara Thanks.
 
@LeviMorrison What am I? Chopped liver?
 
@Sara :) Definitely not.
I was trying to pick people who don't frequent this room.
 
@LeviMorrison I don't know much about the guy except he made D tbh.
 
9:55 PM
Sec... I have a photo...
 
 
@ircmaxell not entirely ... you need to separate in any case, even if the ref one (where you separate the derefed value)
 
^^ That's Andrei
 
@NikiC so just a SEPARATE_ZVAL ?
 
9:57 PM
ah, no
 
no check if the function returns by reference?
 
Well, that's a blackmailing picture
 
Regarding C vs C++: C++ generally has a lot more complexity when you define things; the benefit is that you generally have less complexity at the call site. In C the complexity is generally a bit more spread throughout.
 
		if (EXPECTED((opline->extended_value & 2) == 0)) {
			/* Does not return by reference */
			SEPARATE_ZVAL(retval_ptr);
		} else {
			ZVAL_DEREF(retval_ptr);
		}
^^ that?
 
That's one reason I like C++: you define things fewer times than you use them (it's at least a 1:1 mapping, unless you keep C++ code that isn't used anywhere) so the net complexity is hidden from you.
 
9:58 PM
@Gordon yea thanks for that collection of links :-D
 
@ircmaxell and SEPARATE_ZVAL_NO_REF after the DEREF
I think ^^
 
I'm checking both cases in a test now, standby
 
@bwoebi I collected them when I introduced the git message format to my team
 
I'm really unsure about how the VERIFY_RETURN_TYPE interfacts with RETURN_REF
 
@Gordon hehe … That somehow was funny… all these links …
 
10:01 PM
except for the last one. I found that only yesterday
 
@FlorianMargaine Here's another for ya
 
If we had C with templates and other compile-time features like constexpr I probably wouldn't use C++ very much.
(Inheritance is used way too often in my opinion)
 
@Sara woa, even in gif!
 
@Levi If I had a $1 for every time I saw virtual get abused
 
@Sara The good thing about C++ is that virtual is explicit
As opposed to certain other popular languages that shall remain unnamed
 
10:02 PM
@NikiC Yeah, I'm just talking about the times when it's not particularly a good design choice
:D
 
		if (EXPECTED((opline->extended_value & 2) == 0)) {
			/* Does not return by reference */
			SEPARATE_ZVAL(retval_ptr);
		} else {
			ZVAL_DEREF(retval_ptr);
			SEPARATE_ZVAL_NOREF(retval_ptr);
		}
 
1 message moved to Orphan GIFs
 
@NikiC You mean Java? :)
 
@LeviMorrison Yes ^^
 
^^ that's what I'm currently working with, and it works in both directions (when it's a ref, and when it's not)
 
10:03 PM
@ircmaxell yes, that looks not entirely unreasonable ^^
 
@LeviMorrison I wouldn't have absolutely no issue with constexpr. Just have fun with marking each function in PHP without side effects as constexpr internally. We wouldn't even need an explicit constexpr hint…
 
now just to create a constant for the &2 vs &1
 
@NikiC I kinda wish override were required as well, tbqh
 
fuck me, I read $1 as a regex backreference
also, mornin
 
@LeviMorrison It's actually possible to pre-evaluate a lot in PHP… we just need strong side-effect analysis.
 
10:08 PM
@NikiC thank you
fixed and pushed
 
10:21 PM
			if ((Z_TYPE_P(arg) == IS_NULL && !cur_arg_info->allow_null)
				|| (Z_TYPE_P(arg) != IS_NULL && !zend_verify_scalar_type_hint(cur_arg_info->type_hint, arg, strict))) {
^^ that's better than the goto?
 
I dunno :)
 
@ircmaxell will be even nicer with the null constant handling ;)
 
it's on my list
for tomorrow
 
maybe stick the conditional in a separate function if you think it's too complicated?
 
10:24 PM
@NikiC that's what I will likely do, because with constant handling it's going to get worse
and it's duplicated 4 times, so perfect chance to extract
so remaining items:
1. Fix null constant issue
2. Change callable E_RECOVERABLE to exception
3. Handle severity for ZPP, somehow
pretty much all the other issues raised by @NikiC so far are "fixed" (yes, quotes intentional)
 
wut
where did Pierre offend Hannes in the email?
 
@FlorianMargaine it's an older issue
Pierre and Hannes ... don't get along well ^^
 
well I can see that...
 
wow, that's an email
 
10:30 PM
especially when you consider the way Hannes treats (some) others, there's a bit of irony in that reply
 
please refrain from talking about that email ever again
 
and here I was worried my remaining popcorn would go to waste
9
 
Nice.
Anyone that gets that offended by being asked to make a process more open and transparent is probably the wrong person to be doing that job.
 
that is something to consider
 
@ircmaxell Hmh… I never had issues with Hannes. So, wondering what's up there :o
 
10:35 PM
no but seriously
 
@ircmaxell And moving github.com/php/php-src/pull/1180/… for msvc compat
 
I don't see where he attacks Hannes
 
I've seen Pierre irritate people before, but over the past few days he's been an amazing force...
 
or "harass"
 
Also… how do you mean the way Hannes treats (some) others?
 
10:35 PM
he's been pretty abrasive to me. Granted, I've deserved it each time, but still.
 
@FlorianMargaine Here:
> However, as of today, you are the blocking point when it comes to
improve the wiki RFCs, registration and voting areas.And this is
really becoming a problem.
 
@NikiC whoops, missed that
 
That accusation is coming out of thin air and seems unrelated to anything
 
Well… Hannes is the maintainer of the box… but the wiki source is under git control… no?
 
@NikiC well, if Hannes is the sole maintainer who has access to wiki.php.net, then yes, Hannes is the blocking point
unless I misunderstood something
 
10:38 PM
fixed
 
According to wiki, there are two people with sudo access to that box.
 
@FlorianMargaine As bob said, source is under git. And even then, unless Hannes is actively declining implemented changes, I don't see how he's a blocking point
On the other hand Pierre is very much a blocking point for all our Windows infrastructure
 
@NikiC the backstory is that Pierre has been trying to get access to the data for like "3 months" to be able to analyze it and see if they can implement restrictions / fixes. And Hannes keeps telling him no and ignoring him
 
@ircmaxell What data exactly does Pierre want?
 
He's been promising doing certain updates to windows.php.net for like 5 years and nothing ever happens
 
10:40 PM
I have no idea
 
@ircmaxell lol, he should get someone else to ask
 
no do I know why he wants it
 
He'll probably get it right away :D
 
other than
21:47 <Pierre> Hannes is currently preventing anyone to actually implement tech safe guards to avoid bad behaviors, trying to get a dump of the current wiki since literally months
21:47 <Pierre> sad time, but more clarity won't hurt, if done correctly and openly
21:50 <Pierre> my idea is to add code to block edition during votes, ensure a minimum of 2 weeks discussions and ensure a fixed voting period
 
block edition during votes… Well… closing a vote is an edit…
 
10:41 PM
@ircmaxell the last point sounds reasonable
 
not sure why he needs a full dump for that though
 
I'm not sure why he needs "data" if wiki is on github, but whatever
 
Hmph. Need to find that gist. Or whereever I edited that doc...
 
@bwoebi you can detect that and block all others
 
I wouldn't do that tho
 
10:43 PM
@ircmaxell Yay for closing, changing, reopening in one same minute
Also… fixed voting period… no… with that Anthony couldn't have terminated vote on sth
And minimum 2 weeks… there may be reasons why less in exceptional circumstances…
 
well, I think auto-close (you specify end-date and it does the rest) isn't a horrible idea
 
I think automatically closing would be convenient
Especially given how people are asking for UTC close times
 
the discussion minimum is hard to enforce
because what if I resurect one of my RFCs from 3 years ago. Can I just open vote today?
 
auto-close, okay. Blocking edits…no.
 
I don't feel like having to close it myself at some particular time. It's not like I know two weeks in advance if I'll be at home at just that time
 
10:44 PM
Blocking edits is a bad idea, as it'd prevent changes to the document that don't change the meaning, like a FAQ section, or fixing a typo.
 
if you have a typo, why should you open the RFC?
 
@Charles on one hand, I agree. On the other, there's a very fine line in there
 
@FlorianMargaine if you have a typo it's typically because you don't realize you have a typo
 
it's supposed to be in a votable state, I hope typos are gone since long
@PaulCrovella your rfc has supposedly been reviewed by other people
 
@FlorianMargaine that happens from time to time tho
 
10:46 PM
@FlorianMargaine You're not serious are you?
 
@NikiC "supposedly"
there was a discussion period to talk about the rfc
 
@ircmaxell Of course. /s
 
Proposing no editing would get voted against, hard. Proposing a rule that limits the type of edits could pass.
 
I think a simple but not too unrealistic rule is 1 week from last non-editorial change (fixing typos wouldn't delay voting, for instance).
 
Typos crop up in Nth editions of heavily edited books ... you're crazy if you think having a typo free RFC is a reasonable requirement
 
10:48 PM
Thou Shalt Not Change Thy RFC In Any Way That Makes Voters Regret Their Decisions.
 
@NikiC we're talking about RFCs there... most of them are <1000 words
 
I think I'm more comfortable saying "don't fix a typo during vote" than I am saying "only typo fixes during vote"
 
But if an RFC goes for some period of time without discussion it probably needs to be brought up before voting.
 
unless you heavily define type as "has no meaning change whatsoever to the proposal"
 
Any change to the intended or actual semantic meaning of the proposal during a vote should be a no-go. Fixing a typo, sure. Adding a FAQ, maybe. Fixing wording so that it reads differently, probably not. Changing the proposal itself, no.
You have to let it sit so people can review it... and you need to make sure people review it.
Nobody's gonna catch the typos unless you actually tell them it's about to be a thing.
The care level of php-internals is JIT.
This is a bug, not a feature.
 
10:52 PM
The @doctrineorm team is pleased to announce the immediate availability of doctrine/orm 2.5.0-beta1! http://goo.gl/uKJp7X #doctrine2
for anyone willing to blame me for beta software stability :P
 
@Ocramius betas are annoying… I prefer the dev branches :-D
 
@bwoebi I have plenty of that :D
 
@Charles well, I'd include a clause "if someone raises an issue with a change, it should be reverted" as a safe-guard from someone saying "I don't think it changes it"
@Ocramius congrats!
 
jmp or jump? … but not both :x
 
@ircmaxell Fair enough, though I'll struggle to come up with wording that doesn't sound awkward. I'm gonna see if I can find the old doc I drafted and cut it down to just voting timing and editing rules. The smaller and more narrowly targeted the proposal, the easier it'll be to discuss and pass.
That reminds me, do rule change RFCs require simple majority or 2/3rds? I forget.
 
10:59 PM
@Charles simple majority
until we pass the "all votes require 2/3" proposal
 
Maybe consider doing that one before 7.1 starts? I think all but one or two of the 7.0 RFCs required 2/3rds.
 
yeah, I think start talking about it in May or June, when things die down a bit
 
I'm kinda torn on timing. There's been a few calls for everyone to shut up and fix bugs, which is a very good idea... but ... the voting/editing thing is still fresh in people's minds.
Might just announce it and also note no vote until further along the 7.0 timeline or something. I dunno.
 
let the emotions die a tiny bit
 
That won't be a problem, it'll be a week or two before it'll be ready anyway.
 
11:12 PM
Hannes, dun goofed there. You can't be part of a project and threaten legal action action against other people in the project, for talking to you about the project.
 
@Danack Well, maybe. Depends on the history.
Pierre did address Hannes directly and called him a "blocking point when it comes to
improve the wiki RFC"
 
"Please refrain for talking about me or to me ever again. I will take legal actions if this does not stop." - you can't say that while still being part of a project that the other person is also in.
 
Given the apparent personal history, that might be true only in the context of interacting with Pierre... but, frankly, the way he worded the email that Pierre replied to makes me think he really is a potential block.
 
@Danack I am not so sure.
I'm not sure exactly what Pierre is accusing Hannes of, honestly.
 
refusing to give him a dump of wiki.php.net/rfc
 
11:17 PM
@LeviMorrison hypothetically, what would happen, if someone in a company threatened that against a fellow employee?
 
@Danack HR steps in and tells them both to knock it off.
 
@Charles no. Harassment is a thing.
and usually one party gets fired
but it's harder to fire anyone in this kind of project...
 
It is, but a neutral third party comes in regardless.
 
There seems to be some hostility that happened in private, so we aren't seeing the whole picture.
 
not necessarily
 
11:18 PM
@FlorianMargaine Maybe I've just had good HR departments?
 
or maybe just employees not going straight through the lawyer :)
 
True.
 
1
Q: Too many variables in constructor

user1806244check this out: public function __construct( \Magento\Framework\Model\Context $context, \Magento\Framework\View\DesignInterface $design, \Magento\Framework\Registry $registry, \Magento\Store\Model\App\Emulation $appEmulation, \Magento\Store\Model\StoreMana...

 
@FlorianMargaine they moved just to strict DI. Before they had singletons everywhere
they just made the thing explicit
 
@FlorianMargaine Looks like Magento hasn't discovered the use statement yet...
Also, what's that SRP thing everybody is talking about
 
11:24 PM
hey, one step at a time.
 
@NikiC Suggested Retail Price?
 
and lol @ this:
@SuppressWarnings(PHPMD.ExcessiveParameterList)
 
usually marked up by %20 - you can negotiate it down again.
 
they're even going fully-qualified for things in the same namespace as that class
and actually there's a way better line in the question:
> anyway, I think it has something to do with that dependency injection hipstery thing.
 
Maybe they generated this stuff?
 
11:30 PM
how slow is class_alias? As in: Is it a viable way to use to keep bc impact of renaming String/Int/Float named classes small (on php5)
 
11:44 PM
@Rangad If it's just for these few classes… no issue.
 
I have a ticket from 2007 open here that says that one call to class_alias slowed down one of our applications by ~25%. So it's either "bogus" or no longer true?
(Sorry for that, I'm not near a computer where I could perform meaningful tests)
 
if the application is otherwise just 4 calls to class_alias, seems legit to me
 

« first day (1614 days earlier)      last day (3348 days later) »