@menelion sounds like doing a little bit of cleaning up will save you or the next dev some headaches.
@bhhaskin I'm very new at that client, but yes, I will push. Oh, there is also commented-out code, global vars, unnecessary repeated code, magic strings and numbers, all that we love and cherish.
@menelion I am sure it was written for 5.1 xD
@bhhaskin It was, indeed! Currently it's a mix of 7.0, 7.4 and 8.x (at least according to Docker Compose), but the code is… like this.
@menelion yeah, seems like your point is better, I'll agree, I just remembered I actually seen some nicely written php code, classes separated per source files, code not mixed with html text with tags
severe case of tech debt then, I guess?
@ag Exactly. People at my previous job were obsessed by clean code principles, and now I'm totally getting why it's so good.