I'd think you're doing the right thing for the wrong reasons. Allow me to explain.
First off, the issue is trivial and simple for any intermediate developer to solve.
This isn't a valid reason to close a question. It's a valid reason to downvote a question for sure, but not closure. Closure implies that the post is temporarily off-topic or has some temporary issue with it which can be fixed until such a time the question is reopened. Downvoting implies that the question isn't well-researched.
I also don't see any real long term value of a question like this for future developers. It's essentially a one-off.
This has more prudence with determining if a question is on-topic, but can be difficult to evaluate from a non-PHP developer's standpoint.
For example, even though I'm not versed in PHP - I'm actually quite allergic to it - I don't see anything in the question that automatically makes me think, "Oh, this is a poor question." The only thing that slightly annoys me is this line...
It was built by a freelance PHP developer I no longer have access too, and my PHP is not great.
...only because it's not germane to the actual question at hand.
So, to summarize: The question probably should be closed and edited to be made more on-topic if at all possible. Removing the above line would be a start. From there, it could be reopened if the community feels that it's a question they wish to answer.