Merge pull request #7030 from nextcloud/proper_og_image
Use the new direct endpoint for the OpenGraph image
This commit is contained in:
commit
c2e82148e5
1 changed files with 6 additions and 1 deletions
|
@ -373,15 +373,20 @@ class ShareController extends Controller {
|
|||
$shareTmpl['previewMaxY'] = $this->config->getSystemValue('preview_max_y', 1024);
|
||||
$shareTmpl['disclaimer'] = $this->config->getAppValue('core', 'shareapi_public_link_disclaimertext', null);
|
||||
$shareTmpl['previewURL'] = $shareTmpl['downloadURL'];
|
||||
$ogPreview = '';
|
||||
if ($shareTmpl['previewSupported']) {
|
||||
$shareTmpl['previewImage'] = $this->urlGenerator->linkToRouteAbsolute( 'files_sharing.PublicPreview.getPreview',
|
||||
['x' => 200, 'y' => 200, 'file' => $shareTmpl['directory_path'], 't' => $shareTmpl['dirToken']]);
|
||||
$ogPreview = $shareTmpl['previewImage'];
|
||||
|
||||
// We just have direct previews for image files
|
||||
if ($share->getNode()->getMimePart() === 'image') {
|
||||
$shareTmpl['previewURL'] = $this->urlGenerator->linkToRouteAbsolute('files_sharing.publicpreview.directLink', ['token' => $token]);
|
||||
$ogPreview = $shareTmpl['previewURL'];
|
||||
}
|
||||
} else {
|
||||
$shareTmpl['previewImage'] = $this->urlGenerator->getAbsoluteURL($this->urlGenerator->imagePath('core', 'favicon-fb.png'));
|
||||
$ogPreview = $shareTmpl['previewImage'];
|
||||
}
|
||||
|
||||
// Load files we need
|
||||
|
@ -411,7 +416,7 @@ class ShareController extends Controller {
|
|||
\OCP\Util::addHeader('meta', ['property' => "og:site_name", 'content' => $this->defaults->getName()]);
|
||||
\OCP\Util::addHeader('meta', ['property' => "og:url", 'content' => $shareTmpl['shareUrl']]);
|
||||
\OCP\Util::addHeader('meta', ['property' => "og:type", 'content' => "object"]);
|
||||
\OCP\Util::addHeader('meta', ['property' => "og:image", 'content' => $shareTmpl['previewImage']]);
|
||||
\OCP\Util::addHeader('meta', ['property' => "og:image", 'content' => $ogPreview]);
|
||||
|
||||
$this->eventDispatcher->dispatch('OCA\Files_Sharing::loadAdditionalScripts');
|
||||
|
||||
|
|
Loading…
Reference in a new issue