@Beni Grind ~HB9HNT @Mario Vavti @Harald Eilertsen Please ignore for now, my long text got somehow lost and was replaced with a commit message. I'll edit the MR again and rewrite the lost text.
Comment on IPv6 database connection to Postgres
navigium@grindcore.ch 2 days ago
@Hubzilla Support Forum @Mario Vavti @Harald Eilertsen Would anyone of you mind giving me feedback on this PR: #^https://framagit.org/hubzilla/core/-/merge_requests/2312
The usual disclaimer holds: I gave my best to describe what my intentions were but chances are that I have no idea what I'm doing and that my code is useless. The last time I wrote php code is more than 20 years ago.
The usual disclaimer holds: I gave my best to describe what my intentions were but chances are that I have no idea what I'm doing and that my code is useless. The last time I wrote php code is more than 20 years ago.
navigium@grindcore.ch 2 days ago
navigium@grindcore.ch 2 days ago
@Beni Grind ~HB9HNT @Mario Vavti @Harald Eilertsen Ok, I rewrote the text. Now it's ready for inspection. Sorry for the confusion.
I think it looks good in general. A few nitpicks, but nothing major. I leave the final judgement to Mario :)
Regarding the 20 years: My memory isn't that good to remember single functions. All I remember is to use dollar signs for variables and curly braces for blocks. Apart from this I sadly have to read documentation and search the web. I did check though whether we require PHP 8 before using those new constructs.