Merge branch 'fix84_attachmenturl' into 'master'

[fix] Attachment urls don't seems to be local if using different server in config

See merge request !83
This commit is contained in:
hannes 2017-02-05 18:30:22 +00:00
commit 65c18a252c
2 changed files with 15 additions and 2 deletions

View File

@ -84,7 +84,15 @@ class QvitterAction extends ApiAction
$siterootdomain = common_config('site','server');
$qvitterpath = Plugin::staticPath('Qvitter', '');
$apiroot = common_path('api/', StatusNet::isHTTPS());
$attachmentroot = common_path('attachment/', StatusNet::isHTTPS());
$attachmentconfig=common_config('attachments');
if(StatusNet::isHTTPS() && $attachmentconfig['sslserver']){
$attachmentroot ='https://'.$attachmentconfig['sslserver'].$attachmentconfig['path'];
} elseif(!StatusNet::isHTTPS() && $attachmentconfig['server']) {
$attachmentroot ='http://'.$attachmentconfig['server'].$attachmentconfig['path'];
} else {
$attachmentroot = common_path('attachment/', StatusNet::isHTTPS());
}
$instanceurl = common_path('', StatusNet::isHTTPS());
$favicon_path = QvitterPlugin::settings("favicon_path");

View File

@ -730,7 +730,12 @@ function isLocalURL(url) {
return true;
}
else {
return false;
if(url.substring(0,window.siteAttachmentURLBase.length) == window.siteAttachmentURLBase) {
return true;
}
else {
return false;
}
}
}