Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

more pdf absolute pathes #1785

Merged
merged 1 commit into from
Mar 4, 2017
Merged

more pdf absolute pathes #1785

merged 1 commit into from
Mar 4, 2017

Conversation

trob
Copy link

@trob trob commented Sep 19, 2016

use COM_FABRIK_LIVESITE_ROOT for scheme,host,port (don't add port 80 as default to https)
handle subdirs (which may or maynot be included in relative pathes)
check for existing scheme to determine if abs. path (checking for base would add the base to external pathes, too)

handle subdirs (which may or maynot be included in relative pathes)
check for existing scheme to determine if abs. path (checking for base would add the base to external pathes)
@cheesegrits cheesegrits merged commit 22a3ca2 into Fabrik:master Mar 4, 2017
@trob trob deleted the pdf_abs_pathes branch May 9, 2017 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants