Why!!!
Why must some devs make life complicated!!!!!!

So, here I am enjoying my day (well enjoying the meetings that are taking me away from working) when I get a bug report that script X isn't sending out emails anymore.

Ok that's weird, this as far as I know uses the same email class as every other email being sent out from this project, and they all work.

Let's go have a quick dig...

function sendEmail(){
/*do a bunch of stuff*/
}
Is being used, well that's odd, it should be $emailService->send()
But what ever, it's probably an old wrapper for legacy sake since this script was written years ago. But nope, I almost cried, it's a wrapper for mail() isolated into this script.

Like for fucks sake, why in the hell would this be used when there's an entire fucking class that's tried and tested and only looses 1 email every few months, coz shit happens.

Errrrr.... sometimes i really wonder why people can't just do what they need to do the first time round.