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
Fix Reg: #1464 #1470
Fix Reg: #1464 #1470
Conversation
- using - SystemURLs - removed sDocuemntRoot Var - moved string cleanup into the SystemURLs class - Checked for error from SystemURLs and dispaly error page
@@ -60,6 +62,13 @@ function mysql_failure($message) | |||
exit(); | |||
} | |||
} | |||
try { | |||
SystemURLs::init($sRootPath, $URL, dirname(dirname(__FILE__))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
@@ -60,6 +62,13 @@ function mysql_failure($message) | |||
exit(); | |||
} | |||
} | |||
try { | |||
SystemURLs::init($sRootPath, $URL, dirname(dirname(__FILE__))); | |||
$sRootPath = SystemURLs::getRootPath(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume this is only to maintain backward-compatibility without a full refactor?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep
are you sure this was from this PR? that error msg was from #1472 |
|
||
private static function isValidRootPath() | ||
{ | ||
if (stripos(self::$rootPath, "http") !== true ) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are you trying to do here? this function always returns false...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we are making sure that the rootPath does not have a http
in it
New DTO class for keeping URLs - fixed #1464