I replaced the $_SERVER['DOCUMENT_ROOT'] occurences

[simon-dt]

Because $_SERVER['DOCUMENT_ROOT'] is evil and not to be built upon ;-)

It's in the tcpdf library, perhaps they have their own git repository where i should push a configurable document root solution.

[brendo]

Funnily enough it does have a git repo, it's just hosted on Sourceforge so collaboration isn't as seamless as it could be.

I have forward this pull request diff to the creator for inclusion.

[simon-dt]

Yeah, i couldn't find it on github either.

The commit i sent is a symphony-only fix, it uses symphony's DOCROOT define so the tcpdf creator has to do this in another way like, say, adding a variable to their config file ..

[brendo]

Doh, yeah oversight by me. The TCPDF library already has some checks in the config file, but you're correct, this particular commit doesn't need to be added to the actual library.