Open matzipan opened 9 years ago
@dphiffer any word on this? I would say it's a pretty straight-forward merge. I'm insisting because we're using wpackagist so it would be nicer to pull this from the wordpress repository instead of maintaining our own copy.
This was required for usecases where wp-content is not in a child directory of the wordpress directory (i.e. https://github.com/roots/bedrock)
The problem was that the old method returned a path like
/var/www/public/wp/app/uploads/...
whereas theneeded path was/var/www/public/app/uploads/...