« first day (4035 days earlier)      last day (913 days later) » 

1:32 AM
@Danack I feel like this might be your vibe youtube.com/watch?v=ARgDDNsaItI
 
1:50 AM
@DaveRandom I am apparently in a much more, .....'challenging' music mood: youtube.com/…
 
bit much for 2am ngl :-P
tho I was listening to aphex twin about an hour ago so no room to manoeuvre really :-P
that really is like an auditory K trip though
when you climb down, highly recommend earth pig/adam bork if you fancy a bit of quite technical guitar stuff :-P
 
2:14 AM
@Danack Quotes should have gone around "music"
 
2:26 AM
lol
 
 
6 hours later…
8:31 AM
@Dharman Can you please post a comment on github.com/nikic/test-repo/issues/4? ^^
 
Using PHP 8.1.0RC5 I get a lot of malloc_consolidate(): unaligned fastbin chunk detected messages when running PHPUnit's own test suite when collecting code coverage with PCOV 1.0.9. Is that expected? @JoeWatkins
 
8:53 AM
@Dharman Yay, it works
 
@SebastianBergmann not expected, can you open an issue, I'll take a look ... probably not today though, sorry ...
 
@JoeWatkins Will do, thanks.
 
9:10 AM
@PatrickAllaert It wasn't on my radar - added it to my todo list now.
 
ok Derick, thanks!
 
probably this afternoon
 
 
2 hours later…
user17285887
11:16 AM
Hey guys...
 
user17285887
I'm new to the community, can someone help me with a php project, the source code link is this--> github.com/shubham4538/Image-Slider.git can you help me out to select the circles, because when I select "getElementsByClassName('fa-circle')" as const circles, it returns HTMLCollection and when I try to console log the circles[0], it retuns undefined. I just need each circle as eventListener using for loop.
 
@Shubham748748 This is not the ideal room to ask such questions as your question relates to the browser. You would be better off asking in a room focused on javascript.
 
11:35 AM
@JoeWatkins Migrating to GitHub Issues Anything to add/change?
2
 
@NikiC It misses a "cons" list...
 
11:55 AM
Would need to be a pretty big con to counteract staying on the practically dead and degrading current one.
I'd say even the security ones should be scrapped from bugs.php.net, if you've got a system that's unmaintained.. it's probably a bad idea to keep really sensitive information on it.
(handle them via a security email instead)
 
cmb
12:14 PM
@Derick cons: none :)
@MarkR I usually don't get a reply (~ 10% only) …
 
@cmb You don't get a reply when you message back to people who email security, or when you email it?
 
@MarkR email is also not so great when you want to make it public afterwards, to expose transparency after the fix
 
cmb
@MarkR when I sent mail to security@
 
@bwoebi I understand, but also as someone who has a lot of $ invested into PHP based projects, putting information which should be under lock and key, behind a door that's apparently falling down, is... discomforting.
 
@MarkR My problem isn't with moving off bugsnet, but rather the immediate jump to the conclusion that GH issues is the answer.
 
12:21 PM
@Derick Do you have alternatives in mind that are free (to the level PHP would need) and don't require running your own infrastructure?
@cmb That is... also quite troubling :S
 
@MarkR Mantis, and YouTrack, are two on the top of my head.
Jira too, but let's never mention that again.
 
It would seem entirely proper to offer them up in the RFC discussion then. Belief that there's a better option than going all-in on github would certainly be grounds to vote against.
 
@Derick youtrack is only free for small teams though, or is it for also free for OSS?
(though I guess, if Nikita asks … then we'd get it for free :-D)
 
 
1 hour later…
1:52 PM
@NikiC lgtm
 
2:37 PM
Has there been interest in translating the manual into Georgian?
In reference to the account request
I noticed we don't have a github repo for Georgian
 
@Tiffany Just my 2c but I wouldn't accept new translations at this point ... seems very unlikely that there's going to be an actively maintained Georgian translation
 
@Tiffany Someone is volunteering, but I don't know how we would accept it, there should at least be some of it translated before we even consider making it official
 
cmb
2:52 PM
IIRC, @salathe replied in the past to such requests that it would be preferable to gather a team first (due to the large amount of work); I think these mails never got a reply.
 
yeah... other thing is verifying the accuracy of the translation, which is where having a group would help
 
@Derick @Sara tag day again ?
 
I suppose so. Currently looking at bugs.php.net/bug.php?id=81458 first though.
 
> MagickCLAHEImage() should be declared size_t but instead is declared as a double. To change it now would break the ABI-- which we cannot do.
@RemiCollet did you put the frighteners on them?
 
3:11 PM
@Danack I only see "number_bins" being long in imagick, but double in IM7
I think moving "number_bins from interger to double in imagick isn't a real BC break...
 
php.net/manual/en/… — annoyingly, that doesn't define what "days" means :-/
 
@JoeWatkins congrats on the new gig :)
 
@Danack or do you mean the opposite (changing it to integer in IM) ?
 
@beberlei ta
 
@RemiCollet I meant them keeping a function signature as it is, to maintain BC, even though it's technically wrong. github.com/ImageMagick/ImageMagick/issues/4429
 
3:15 PM
@beberlei incoming: @NikiC making another comment about APM space usurping all php-src devs
 
@Danack so nothing need to be change ? imagick use it, pass it as a double to IM function which cast it to an integer... so everything is uggly, but seems fine ;)
 
@RemiCollet Yes. And it's also the day that github decided to fuck with me.
 
@RemiCollet yeah.....it just took me a while to figure out......and then was surprised by them caring about BC.
 
at least they should be able to track such needed changes, and apply all of them when possible (and with a soname change). But this project is so strange....
 
from a competition perspective i hope joe only works on php-src 100% of the time ;-)
 
3:21 PM
@beberlei :-P
 
@bwoebi Seriously
 
sweats profusely
 
@Girgias Don't tell me you're at DataDog as well :P
 
Not yet :)
Whenever Datadog asks how I could be happier at my job, I say I'd like to do more OSS work that's not directly related to my job. I haven't directly asked, and they haven't directly said no. Maybe it will happen.
Also, I say I'd like a Linux laptop :)
 
3:36 PM
@NikiC it may happen, but there's at least 8 months of dreadful university
 
3:51 PM
I started hiring R11 devs too... afraid you have to actually write PHP though :P
 
scream.gif
 
... and typescript
 
@Derick Are you able to reach americas.jump.php.net ? I'm getting timeouts there, and the EU jumphost won't take my verification code.
 
4:13 PM
@LeviMorrison i can hypothetically offer a linux laptop, significant OSS even, but i can't remotely match a datadog salary :P
 
plus the beuracratic nightmare of hiring someone in the US vs europe might cause a few nightmares
oh two times nightmare, you get the problem :p
 
Datadog is working on Linux laptops -- there is a group of people beta testing them now. I'm hoping I can get one soon. But yeah, OSS is the biggest benefit competitors provide that Datadog doesn't (today, anyway).
 
linux laptops are a solved problem!! :p
 
Yeah, but it's all the corporate requirements that aren't :D
 
4:18 PM
@Sara Let me check. I don't think it has my jump host creds
 
just making fun, i can imagine it poses problems
 
@Sara No, americas. doesn't respond for me either
 
Out of curiosity, what's the going rate for a C programmer nowadays? It feels like something of a niche now so higher salary.
 
@Sara Want to see if I can remove your TFA at europe?
 
@MarkR Depends on what they're doing in C, I suppose. Unlike PHP, C is applicable in some form in many, many domains...
 
4:25 PM
@LeviMorrison I was thinking the market would have been swinging towards C++ and more recently Rust.
 
Maybe. I much prefer Rust to C, that's for sure.
 
C's main advantage is the 100,000,000 existing LOC out there.
 
That's probably a low-ball estimate of LOC.
 
aye, add a couple of 0s
 
Granted. Point remains, C's main advantage is there is just so damned much of it.
 
4:46 PM
^
 
@Derick Yeah, give that a shot, I'll sit it back up.
 
@Tiffany @NikiC It looks like they have a team. That's a pleasant surprise. Are you having any further reservations?
 
@beberlei I guess that really just depends on the size of your company … once you're big enough, there are PEOs doing that for you, globally :-)
 
@Sara Done
 
5:02 PM
@Dharman making sure they understand the Docbook spec and how to build the manual locally since they'll need to test their changes
It's not like they'll be rewriting the manual, but knowing that they have a good enough understanding of the build process is helpful
 
@Derick Now I get permission denied (keyboard-interactive) after using my ssh key.
 
@Dharman Did they respond to you? I don't see their response on the mailing list
 
@Sara Probably not have a password set-up - Did you see Rasmus reply to you?
 
Buenos Tardes!
 
Buenas
close ;)
 
5:11 PM
@Derick I saw his reply saying he'd deleted the authenticator directory.... nothing about a password.
 
@Sara Not to use jump4 instead of "americas" ?
 
Wes
@JoeWatkins this was so painful to read i insta-grew a mustache, a coppola hat materialized on my head, and started doing this at superluminal speed media.giphy.com/media/irvoUylJbDCVdmBI9j/giphy.gif
 
@Sara Sorry, email is being slow - I did the same in both places, so you should be able to SSH Into one of them now?
 
@Derick Oh, I tried both.
 
You don't have a password set on europe.jump
 
5:13 PM
@Derick I'd be very surprised if I did. I don't tend to use it.
 
@Sara Where can I find your ssh key? you have one on jump2/europe
 
@Derick I actually just sent a copy to the systems@ list a couple minutes ago
 
I'm not going to retype it :-þ -- mail comes into another machine
 
Facebook okay?
I mean, it should be on main.php.net as well
Yep, you can copy/paste from there.
 
OK, I'll get it from there.
What's Facebook? :-D
 
5:18 PM
@Derick That reply is so.... meta.
 
groan (also +1)
 
Thank you, I'll be here till 8.3
 
@Sara The key on main matches the one in your authorized_keys file
 
hah
 
Right, like I said in the email, I'm reasonably sure my key passes, or I wouldn't have gotten the TFA prompt
 
5:20 PM
what's the problem then, did I miss something?
 
$ ssh -p 9022 pollita@europe.jump.php.net
Enter passphrase for key '/home/sgolemon/.ssh/id_rsa':
pollita@europe.jump.php.net: Permission denied (keyboard-interactive).

^^ That's generally problematic
 
I can't check jump4, only jump2
 
Happens on jump2 as well :)
 
jump2 = europe
but your key matches
 
Right, that's what I'm saying.
 
5:21 PM
So then what is the problem?
 
The problem is that I can't authenticate to the machine.
That's usually seen as a bad thing.... usually.
 
But your key matches :-þ
 
The key isn't the problem.
 
So I don't know where to look.
 
Something is requiring more something post-key auth.
 
5:22 PM
I doubt that...
 
/me points up and makes a face.
Objective reality disagrees with you. :p
 
You don't have a shell set
But neither do I
 
Well shit, hook a sister up then
 
Set up bash for you
 
nope
 
5:24 PM
I did do that :-þ
 
I mean, nope, didn't fix it.
$ ssh -p 9022 pollita@php-jump2.php.net
Enter passphrase for key '/home/sgolemon/.ssh/id_rsa':
pollita@php-jump2.php.net: Permission denied (keyboard-interactive).
 
Nov 2 18:24:26 php-jump2 sshd[5646]: error: PAM: Permission denied for pollita from 23.120.123.116
we're getting somewhere
YOu need to ssh in, and run authenticator
 
I suspect you have many such lines.
I need to log in.... to the ssh server I can't log in to.
 
yes
 
Right. Why didn't I think of that?
 
5:25 PM
I think you can do that, without logging in...
 
I can log in without logging in?
 
cmb
@Sara, @Derick, isn't that exactly the same issue @gabrielcaruso and I have for months now?
 
gimme a sec
@cmb One at a time.
 
@cmb Is the github repo I made this morning good enough? 'cause I'm willing to just give up on jump to download....
 
cmb
@Derick no need for me to access the machine, though :)
@Sara builder builds from tags directly :) (I think Remi needs downloads, though)
 
5:28 PM
@RemiCollet How does your test matrix work?
 
root@php-jump2:/etc/pam.d# grep -r authenticator
sshd:auth required pam_google_authenticator.so nullok
Should meain you can login without having set-up credentials...
 
@Derick Yep. Reading the docs agrees with you there.
 
@Sara Can you -v -v -v -v it?
and show me the output
 
Enter passphrase for key '/home/sgolemon/.ssh/id_rsa':
debug3: sign_and_send_pubkey: RSA SHA256:****
debug3: sign_and_send_pubkey: signing using rsa-sha2-512 SHA256:****
debug3: send packet: type 50
debug2: we sent a publickey packet, wait for reply
debug3: receive packet: type 51
Authenticated with partial success.
There's more, but that much at least says that the SSH key isn't the issue.
 
Hi all! If you have a second for me, please. I find myself in the situation of needing to declare variables in a local scope, without polluting global scope, but also to have access to global scope variables without knowing which one of them will be used. Sounds like a great thing for arrow function, but php 7.4 is excluded. It is possible to somehow mimic the behavior of arrow function somehow? Or is there any php special function like a __get, but outside object context? Thanks!
 
5:33 PM
Just use a use clause?
Otherwise access $GLOBALS
 
cmb
^ that
 
@Sara I know the key isn't the problem... what else does it say?
 
Nothing very useful....

Authenticated with partial success.
debug1: Authentications that can continue: keyboard-interactive
debug3: start over, passed a different list keyboard-interactive
debug3: preferred gssapi-with-mic,publickey,keyboard-interactive,password
debug3: authmethod_lookup keyboard-interactive
debug3: remaining preferred: password
debug3: authmethod_is_enabled keyboard-interactive
debug1: Next authentication method: keyboard-interactive
debug2: userauth_kbdint
debug3: send packet: type 50
 
hmm
@Sara try again, changed permissions
 
Same. Permission denied (keyboard-interactive)
 
5:36 PM
hm
I wonder what's different between yours and mine account
 
FWIW, there's no data needing preserving on that machine. You can nuke my account and start over.
 
@Girgias I have to render a template, not knowing how many global variables will be used. Only if I search with a preg_match for $%, but is not a safe solution...
 
nullok
Allow users to log in without OTP, if they haven't set up OTP yet.

PAM requires at least one SUCCESS answer from a module, and nullok causes this module to say IGNORE. This means that if this option is used at least one other module must have said SUCCESS. One way to do this is to add auth required pam_permit.so to the end of the PAM config.
I assume there are no other PAM modules in the auth list
 
@BoteaFlorin Then use $GLOBALS['var_name']
 
Wes
how do i do this on windows?
extension=my_giant_extension\php_giant_extension.dll
 
5:39 PM
@Wes A lot like that? Not sure what you're asking...
 
Wes
ie the extension is a folder with several files and i don't want to mix them up with other php extensions
 
@Wes Right. Relative paths work now (as do absolute paths), what you have there should "just work"™
 
Wes
oooh, nice
 
You might need to use forward-slashes, not sure...
The fix for using paths came somewhere during 5.x times iirc
 
@Sara GFound another permi difference, try again?
 
Wes
5:41 PM
i never knew about that :P
 
sogj
 
@Girgias I know the existence of this approch, but is not solving the problem... anyway... is there a performance issue if I have global $a, global $b for like 30 variables at the starting of a function block?
 
sigh, again not
 
@Derick Still fail. Did you see my note above about nullok though? chat.stackoverflow.com/transcript/message/53384091#53384091
 
cmb
@BoteaFlorin are you sure these are globals?
 
5:42 PM
ok, that was my fault - I typoed it as 60 instead of 600
 
hahah
still fail though
 
@cmb Oh god... that sounds monstrously evil and it'll probably work.
 
@BoteaFlorin ... You do know that global $a and $GLOBALS['a'] is literally the same thing?
 
@cmb I can ensure myself before
 
5:44 PM
Also how can you enumerate the globals in the function if you don't know them beforehand
Because if you do... then just have a use clause
 
@Sara pam permit is included a little further down
 
@Derick ah, damn. That looked promising...
$scope = get_defined_vars();
$closure = function(...$params...) use ($scope) {
   extract($scope);
   ...doYourTemplate...
);
 
@Girgias I have a function rendering a template. That template can have nested template function calls and so on. I can preg_match for $vars to obtain a list
 
@BoteaFlorin ^^ That'll probably do what you want, but yikes.
 
@BoteaFlorin where the variables are defined, can you return them?
Is there something on the receiving end?
Ah, nevermind
 
5:47 PM
Had to remember the name of the extract() function because I've scrubbed it from my heart.
 
ThW
Has anybody ever seen a day/week calendar that shows the clock change (daylight saving time) correctly?
 
DST is a social construct and should be removed
5
 
Wes
how is DST still a thing
 
All time zones are made up.
Humanity should embrace UTC and get over itself.
 
@Sara yes... but is at least doubling ram consumtion...right?
 
5:49 PM
...
 
@BoteaFlorin why does it matter?
 
@BoteaFlorin No. PHP variables are copy-on-write. You'll add a bit for some overhead, but you won't copy the contents.
 
As a Brit I approve of the choice of using UTC as the global time reference for the whole world
@BoteaFlorin benchmarks
 
@Girgias 😂
 
@Derick Thanks for you help, even if we haven't figured it out and at least 3 RMs are now locked out of access to downloads.php.net . I'm going to just use github for my uploads. /shrug
 
Wes
5:51 PM
UTC? I want stardate
 
@sara It's really not cool. Rasmus and Sascha set this up, but they never explained how it works.
 
@Wes Until you realize that there's no such thing as objective time...
@Derick It'll be a problem when we lose access to the other machines because nobody can bounce through the jumphosts anymore.
 
@Sara wait... I don't understand that part... I will take some study about that... but consider that function A will call function B and C, at an expected level of max 5-7... will that be ok? won't that x5 memory used at some point?
 
ThW
@Sara I doubt that I can humanity to agree to that before I have to deliver the application. :-)
 
@Sara We also realyl should use github for uploads.
I'm goign to write a sternly worded email
 
5:57 PM
@Sara awesome information you gave me. Thanks!
 
@BoteaFlorin This is super dated, but the fundamental copy-on-write concept is still valid and may help: blog.golemon.com/2007/01/youre-being-lied-to.html
@Derick I'd vote for it. :+1:
 
cmb
@Derick the biggest issue is that everybody with write access to the repo can replace release attachments. We would need a separate repo with only RM access.
 
@cmb We already have downloads.php.net
 
cmb
you wrote "We also realyl should use github for uploads." – did you mean "not use github"?
 
@cmb php-distributions repo is RM only, and a far more significant target
 
6:00 PM
yes, not use
 
@Derick Oh... then I revoke my vote.
 
cmb
I think it makes more sense to use sth that works, than something that doesn't. And sending mails to systems isn't particularly useful (something something black hole).
 
@Sara extract() accepts a second param, EXTR_REFS. Will that reduce the risk of memory hit, or increase it?
 
cmb
@Derick, tarballs are not yet at downloads; are you still uploading?
 
I've been debugging shit.
 
cmb
6:05 PM
I know, but you send a mail claiming the uploads were there. Anyway, now they are. Thanks! :)
 
so they're on the way now
(and forgot to press enter for scp)
 
cmb
ERROR: cannot verify downloads.php.net's certificate, issued by 'CN=R3,O=Let\'s Encrypt,C=US':
  Issued certificate has expired.
To connect to downloads.php.net insecurely, use `--no-check-certificate'.
 
Someone forgot to run certbot?
 
cmb
guess it's time for Joe to say that "we suck at infra" :)
 
@BoteaFlorin No, it will increase the memory risk. Please read the blog entry I pointed to.
 
6:09 PM
The certificate has not expired @cmb
I bet this is related to the Let's Encrypt root certificate (on your side)
 
cmb
@Derick possibly, but it's a MSFT VM; I won't fiddle with it :)
they might "drill a hole in my head" otherwise
 
can we still ask the bot whether or not we should consume <any alcoholic beverage> and then secretly publicly hope for it to say yes?
 
@cmb Not sure what I can do - just ignore the warning, I suppose.
 
cmb
@Derick yep; checked hash and it matches, so I think we're good
 
You can not, but I am more than happy to answer it for you @FélixAdriyelGagnon-Grenier
 
cmb
6:11 PM
heh
 
@PeeHaa excellent! I do hope you shall answer correctly :P
 
@cmb Would it be possible for you to update the ImageMagick used to build Imagick on windows.php.net to 7.1.0-13 ? the currently used one apparently has some issues.
 
cmb
@Danack I tried that a while ago, but I have no idea how that has been built. Are there binaries available for download?
 
@cmb yes, and I presume they are built following the instructions in here.
 
@FélixAdriyelGagnon-Grenier yes you should
 
6:43 PM
@FélixAdriyelGagnon-Grenier Why do you need a bot for that?
 
cmb
@Danack I downloaded ImageMagick-7.1.0-portable-Q8-x64.zip, but that only contains the exes, but no dlls. The dlls installers have the dlls, but no link libs. I guess I need to build myself; would be nice if configure.exe was scriptable, but it likely isn't. If there only was a portable alternative to autotools ;)
 
@Derick To avoid weirdness when the announcement goes out on Thursday, would you mind grabbing my tarballs and signatures from github.com/sgolemon/uploads and putting them in your downloads folder?
 
Is there a way how I can download them in one go?
nm, they're on the way @Sara
 
7:08 PM
For that repo, at this moment, the most practical would be to simply clone it.
This should NOT be a long term solution, obviously.
 
right
 
Ah, and I see the uploads are complete. Thanks!
 
Rasmus seems to want me to be the middle man. That's not going to be effective debugging this.
 
I wonder if I could generate a config on my local machine and have you drop it in place...
Again, not the right fix, but more viable than you being the only one who can connect. :)
 
@Derick the illusion of someone else than me taking the decision of day-drinking
 
7:13 PM
He suggested turning on standard password auth, but it involves restarting sshd on a remove...
@FélixAdriyelGagnon-Grenier It's after 5pm somewhere.
 
@PeeHaa <3
@Derick touché :)
 
@Sara Have some time to try that though?
 
@Derick Yeah, just installing the tool now
 
which tool?
 
google-authenticator it's what makes the config file
 
7:17 PM
oh
 
Does anyone know if the +FollowSymlinks in Apache would actually resolve/expand the symlink before calling the PHP file or if it would just be following it?
 
@Sara You don't want to try with normal password login?
 
Let's avoid restarting sshd if we can :)
 
i agree
anyway, I am ready to paste
 
Super secure going this in a public chat, yo (though I'll replace it immediately and they'd still need my private ket)
 
7:20 PM
Wanna send that key
nice
 
Delete it so at least google doesn't pick it up
 
Soon as Derick says he has it
 
And room owners can still see the deleted message
 
delete it
 
Again, I'll nuke the config once I'm in. :p
 
7:21 PM
cat ~/.ssh/id_rsa what does that say
xD
 
@ln-s MEOW
@Sara try it
 
HAH! You are haxxed
 
that worked
 
Hacked the gibson, yo.
I'm in.
 
I know!
 
7:22 PM
Ugh this is turning weird
 
And config file replaced.
 
@Tiffany Oh, they only replied to me. Can you see my reply now?
 
@Sara try logging in again then?
 
Yep, just did :)
 
if you restart sshd while connected it won't kill your active connections
at least on debian based distros
 
7:25 PM
Now, if we can get the americas box up, we'll be solid.
 
The americas box works fine for me
 
@ln-s It's still to be avoided, especially with brittle infra.
@Derick Oh, I thought you said you didn't have access...
 
yeah makes sense, just a colorful tip
 
@Sara yes, because it was trying to go through the proxy :D
 
heh
 
7:26 PM
@Sara I'll cipy your auth
 
That's a plan
And someone with access to DNS should fix the CNAME
 
@Sara OK, give jump4 a go?
 
Connected and good to go!
So we didn't fix the cause, but the symptom is sorted at least.
Next step, I'll put my tarballs in my own homedir
 
What's the issue?
 
@ln-s That configuring libpam_google_authenticator.so to work the way it fucking well ought to is hard.
 
7:31 PM
@Sara I pushed the change for DNS, but not sure how long that will last to update.
 
@Derick Sure, TTLs gonna TTL.
 
@Sara Can you try writing up how to do this for @cmb and @GabrielCaruso ?
 
I'll put that in my summary to systems@ then send a separate email for them (to avoid thread hijacks)
 
affirmative
 
7:44 PM
'kay, emailed Gabe and Chris
 
cc-ed systems?
 
systems@ got its own version
You'll end up seeing both
 
I may be a tiny bit over-proud of this code gist.github.com/tiffany-taylor/d0200ce6bd13b69fe4d138776e6b47c3
a little clever, but not too clever
 
ew, spaces ;-D
 
What's the point of if (!empty($sValue))? Why not simply if ($sValue)
 
7:49 PM
is that Hungarian Notation a code style your company picked?
 
because I don't want to make an attempt at figuring out what this customer's content contains
@Derick yes
 
I have strong feelings about it. None of them good.
(They made us to this in Uni)
 
I don't like it either, but rules are rules. It gets flagged in code review.
code standards are from PHP 4 days, before types or even type-hinting existed.
 
@Tiffany How about this version? 3v4l.org/UuatO
More accurately 3v4l.org/48rfT
 
@Tiffany Are you actually looking for feedback, or just strutting? :-) (Strutting is fine.)
 
8:10 PM
@Crell strutting
 
Then thumbsup.gif and I will not give feedback. :-)
 
though, feedback is welcome if it's a meaningful improvement. not sure if I'll use it though.
 
If looking for feedback then:
1/ Build directly into a string instead of making an intermediate array.
2/ This will make string like `foo</p><p>bar</p><p>baz` note the outer tags missing, is that intended?
 
I'm opening a PR shortly though.
@Sara yeah, from the calling method, I surround in paragraph tags.
it gets called like $sContent .= '<p>' . $this->formatColumnsWithValues($kTemp) . '</p>';
 
I'd probably have done it with a filter and then a map, then implode. This is exactly where pipes would have been useful. :-)
 
8:13 PM
I would do it with array_filter and foreach appending to a string.
 
There are extremely few foreach statements that cannot easily be replaced with a map, filter, or reduce.
And most of them could be, it's just not actually a benefit.
 
@Sara for #1, do you mean when I do the sprintf call? ... not sure why I didn't think of that, actually
 
array_map can't work in this case, can it?
 
there's some mental overhead with map, filter and reduce because I have to remember how they work
foreach is simple
 
@Tiffany True. But that's simply a matter of exposure. The more you use it, the less overhead it becomes.
 
8:30 PM
Ok, foreach introduces extra tag, so array_walk is the best choice. 3v4l.org/4GKCW
 
8:44 PM
it didn't introduce an extra tag when I tested it out... data was formatted as I expected
 
I meant foreach with appending to a string
foreach with implode is fine
 
ahhh
 
but yeah, if you are looking for a feedback I would definitely flag that if statement as an issue in a review
use array_filter to do the same
 
 
1 hour later…
cmb
10:14 PM
@Dharman no objections from my side; a team of 10 people make me jealous, though ;)
 
unless they count in binary
 
Fuck me, if I had that many people for French ;-;
 
11:05 PM
feeling super dumb and it didn't feel like it was worth a full post, i'm trying to get the string "<customer name>" to display and it just keeps rendering as a tag called <customer name>...echo htmlspecialchars("&lt;customer name&gt;");
 
11:57 PM
finally got it working, nevermind on that question!
 

« first day (4035 days earlier)      last day (913 days later) »