« first day (4500 days earlier)      last day (448 days later) » 

10:02 AM
Morning
 
10:58 AM
@Derick So it seems that dokuwiki should work on PHP 8.2, albeit with occasional warnings. But if we are using any plugins, they may not work on PHP 8.
I will try to test it out tonight if I get the chance
but I don't know if we are using any plugins
 
we are using one I think, for the voting
 
And probably something for the php.net authentication.
 
 
3 hours later…
2:23 PM
Can someone reproduce the CI memory leak of github.com/php/php-src/pull/10520? I cannot do it locally
 
2:50 PM
@Girgias I can.
==726739== 32 bytes in 1 blocks are definitely lost in loss record 9 of 21
==726739==    at 0x483B7F3: malloc (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==726739==    by 0x7519D3: __zend_malloc (zend_alloc.c:3081)
==726739==    by 0x75034E: _malloc_custom (zend_alloc.c:2452)
==726739==    by 0x75050F: _emalloc (zend_alloc.c:2571)
==726739==    by 0x717865: zend_string_alloc (zend_string.h:150)
==726739==    by 0x7178D8: zend_string_init (zend_string.h:172)
==726739==    by 0x71B5E4: zend_scan_escape_string (zend_language_scanner.l:922)
 
What command did you use? I tried ASAN, MSAN, Valgrind and just letting ZMM tell me but none showed up
Also that leak kinda looks weird
 
sapi/cli/php ext/spl/tests/gh8271.phpt gave the Zend MM output. After disabling it, valgrind reported the above.
I compiled on Ubuntu 20.04.
 
Okay yeah running it directly does work, why does my test runner not pick it up...
oh ffs, opcache was hiding the issue
 
Is there anything from the US or New Mexico or Texas I should bring over, that won't get me on trouble with smuggling laws?
 
BBQ?
(that's a joke)
 
2:58 PM
Was considering getting some kind of green chile salsa, but it'd have to be a small enough jar
 
3:18 PM
@Tiffany Are there any local candies that aren't probably already available in the UK?
 
3:41 PM
@Danack very probably, I'll do some searching today
I have an interview so I'll have to do it around that
There's the NM Tea Co that makes loose leaf teas (they sponsored a Star Trek podcast at least once)
Piñon candy appears to be popular in NM
Well, native to NM, but not as popular as Skittles, apparently
 
4:20 PM
green chile fudge :D
though, I'm more of a fan of red chiles
I think I'll go there today
I'll bring some green chile fudge, red chile fudge and pinon brittle
 
4:34 PM
@PeeHaa So... my supervisor resigned... I'm now Interim VP as of yesterday.
 
4:50 PM
Hey, I was wondering whether somebody here wants to give their opinion about github.com/php/php-src/pull/10476 ? Thanks :)
 
5:07 PM
@nielsdos Is the addition of the new parameter a breaking change for anyone deriving from SplFileObject? Then I suspect this requires an RFC.
 
@TimWolla yes, they would need to update the method signature
I can look into doing an rfc
I did already post the pr link with some explanation on the mailing list to see how people would react but nobody replied unfortunately
So I'm not sure if an RFC would be useful as nobody seems interested
 
@StatikStasis congrats but also.....you sure you're ready for it?
@nielsdos It would be useful, if only to avoid potential drama.
 
@nielsdos Experience says folks are interested once the vote starts, not earlier.
 
The mailing list seems to be going through a period of 'teh dumb', which does happen from time-to-time. Submitting it as an RFC, even if no-one comments on it, means that no-one can accuse people of favouritism. But as Tim said, as it's a BC break it probably needs one.
 
5:24 PM
@Danack Just to be clear: I did not mean to propose to merge the PR without RFC :) When I said "not useful" I meant not doing an RFC and dropping the PR. I do not want to cause drama ofc :p
@TimWolla alright, I can try to look into the rfc process, thanks
@Danack I understand, thanks :)
 
@nielsdos This is also silly but.....probably best not to put copyrighted lyrics in source control.
oh, is it already there?
 
Copyrighted lyrics?
 
> 17 years ago
 
I'm confused :p
 
yeah.....Swamped by Lacuna Coil. added 17 years ago...
 
5:30 PM
Which commit?
 
oh well. Let's pretend I didn't notice that.
 
@Danack We'll see... I've been here for 17+ years and training for this role for the last 2.5 years... I just didn't think it would be this soon.
 
Nvm found, yeah I didnt add that :p, I could replace that file in my PR as I need to make changes close to that file anyway
 
5:46 PM
@nielsdos There's quite a few tests that would be affected....so I think don't bother.
 
 
1 hour later…
7:02 PM
I am able to print a variable which returned from $_GET super global variable after submitting a post on a same page
 $x = $_GET['doctornum'];
 echo $x;
But I get error when I use $x variable in a query
 $stmt =  $pdo->query('SELECT * FROM  patient_reserv_view  where doctor_id = $x');
 
single quotes don't substitute $variables
Also don't do your query that way, that's vulnerable to sql injections
 
Fatal error: Uncaught PDOException: SQLSTATE[42S22]: Column not found: 1054 Unknown column '$x' in 'where clause
 
use prepared s tatements instead
 
^
 
29 secs ago, by nielsdos
use prepared s tatements instead
but also: "$x" vs '$x'
 
7:06 PM
Mar 28, 2021 at 17:32, by Tiffany
@PHPFan We've told you this several times, perhaps... I dunno... listen?
 
@Tiffany Yeah you have told me
 
I mean, fair that this is different... but still, if prepared statements were used at the start, you wouldn't have encountered the problem. :)
 
ok
 
I apologize for dogpiling... felt like I had a moment of deja vu
 
@Tiffany no no, this is appropriate moderation. At some point the refusal of someone to take advice, which results in them asking for the same help repeatedly gets to be a....problem.
 
7:09 PM
Especially looking at the contents of the query, it looks like the application is going to handle health information. There are laws in both the EU and the US that require special protection for this type of information.
 
@Tiffany No problem
 
Even with prepared statements to protect against the most obvious vulnerabilities, you likely want to have an experienced security engineer on the team.
 
@Danack still worth treating people like people with their own problems
there is context we don't know
 
@TimWolla Why you said that
 
I still agree with what Past Me said, but I should have taken a step away from my desk so that I was less annoyed
 
7:14 PM
@PHPFan Because mishandling patient data is a big no-no both under the EU's GDPR and the US' HIPAA.
 
Sure, I didn't say not to treat them as people. "there is context we don't know" - we've had this conversation before, and I suspect we can do it again over refreshing beverages soon....we don't owe random people who have received help here before a level courtesy. When they refuse, over a period of many years, to take advice and so keep taking up our time with the same tedious questions, it is appropriate to start being annoyed at them.
 
I've been trying to become less annoyed... but I acknowledge that this is my own personal journey :P
like, I understand it's a human emotion, but it builds on top of other emotions that overall make me a less happy person, or a person less willing to help others...which is not what I want to become
 
I disagree, but definitely a conversation for over refreshing beverages.
 
yup :)
I'm not saying that I expect others to behave the same, we have different moderation styles... diversity... this is a self-imposed thing
sorry, I realize now where this started, my phrasing of treating people like people
 
7:32 PM
looks like semi decent weather next week
A shout out to anyone interested, in the next few months I'm going to be looking to hire between 2 and 4 PHP developers, juniors, mid and senior working to work on a large CMS + event management platform, joining an existing 14 person team. Ping me if you're interested in knowing more.
4
 
Ok, so I am testing Dokuwiki with PHP 8.2 and I see no issues, but... I have got no access to the PHP plugins. They are no on the marketplace and I cannot test them to see if they are compatible with PHP 8.2. They're probably not. If I had access I could modernize the code.
@Girgias Do you have access to them?
 
@Dharman I have access to nothing, so nope :/
 
same here.
Does anyone employed by PHP foundation have access and time to check this out?
 
Well other than Derick don't think so
 
I don't want to reach an impass. PHP should lead by example and using PHP 7.0 is really bad
but I can only do so much with limited access
It would probably be a good idea to do a mock upgrade on a local machine
Copy everything from wiki.php.net and upgrade locally. Test then upgrade on the site.
 
JRL
8:08 PM
Hmmm. That reminds me of the person a year ago or so where I told them they need to hire an expert, and they basically demanded that I tell them how to do it instead.
To which I replied something like "No, because even though I know more about this than you do, I'm still not confident about my ability to do this without creating liability for your company."
I can't remember what the application was though.
 
8:21 PM
@StatikStasis Congratz!
Is the interim a way to get you to VP position or are you guys looking for somebody else?
 
8:39 PM
It's like getting a 6 month lease with an option to buy.
 
Sweet
 
Wes
microsoft: doubts
 
 
1 hour later…
9:49 PM
@Tiffany for clarity, what day and time are we all meeting next week, and any idea where?
 
10:41 PM
Tuesday, figured we would start at Rack and Tenter with Derick and whoever else at 15:00
Jan 26 at 10:16, by Derick
@Tiffany We usually meet at the Rack & Tenter the day before the conference/day of workshops: https://www.openstreetmap.org/node/4518611096
though, I will be at the workshop that day, and I intend to stay as long as possible, but I will duck out to hang
 
cool.
 
I'm guessing I will probably leave the workshop about 3:00, so I will be at Rack and Tenter hopefully somewhere around 3:15.. I need to figure out directions
oh, it may not take me 15 minutes, it's a fast walk
6 hours ago, by Tiffany
I'll bring some green chile fudge, red chile fudge and pinon brittle
this was acquired
 

« first day (4500 days earlier)      last day (448 days later) »