mirror of
https://github.com/TryGhost/Ghost.git
synced 2024-12-22 10:21:36 +03:00
b335002269
closes #9727 - this is a short term fix to proof that Disqus comments on preview pages no longer appear on other threads - this is not a full solution to the problem - the private API still returns /404/, which is right now inconsistent, but not critical in any way - the url helper will now output the post preview url if you serve a draft/scheduled post - this should register unique page urls at Disqus and ensure uniquness for threads - i still don't understand why the cross posting happens at all, because we also pass an unique identifier to Disqus (the post ID) - it could be that comments, which are added on the preview page, won't appear on the published urls, because the published url !== preview url. I wasn't able to figure this out via testing or reading their docs
50 lines
2.1 KiB
JavaScript
50 lines
2.1 KiB
JavaScript
var schema = require('../schema').checks,
|
|
urlService = require('../../services/url');
|
|
|
|
// This cleans the url from any `/amp` postfixes, so we'll never
|
|
// output a url with `/amp` in the end, except for the needed `amphtml`
|
|
// canonical link, which is rendered by `getAmpUrl`.
|
|
function sanitizeAmpUrl(url) {
|
|
if (url.indexOf('/amp/') !== -1) {
|
|
url = url.replace(/\/amp\/$/i, '/');
|
|
}
|
|
return url;
|
|
}
|
|
|
|
function getUrl(data, absolute) {
|
|
if (schema.isPost(data)) {
|
|
/**
|
|
* @NOTE
|
|
*
|
|
* We return the post preview url if you are making use of the `{{url}}` helper and the post is not published.
|
|
* If we don't do it, we can break Disqus a bit. See https://github.com/TryGhost/Ghost/issues/9727.
|
|
*
|
|
* This short term fix needs a better solution than this, because this is inconsistent with our private API. The
|
|
* private API would still return /404/ for drafts. The public API doesn't serve any drafts - nothing we have to
|
|
* worry about. We first would like to see if this resolves the Disqus bug when commenting on preview pages.
|
|
*
|
|
* A long term solution should be part of the final version of Dynamic Routing.
|
|
*/
|
|
if (data.status !== 'published' && urlService.getUrlByResourceId(data.id) === '/404/') {
|
|
return urlService
|
|
.utils
|
|
.urlFor({relativeUrl: urlService.utils.urlJoin('/p', data.uuid, '/'), secure: data.secure}, null, absolute);
|
|
}
|
|
|
|
return urlService.getUrlByResourceId(data.id, {secure: data.secure, absolute: absolute, withSubdirectory: true});
|
|
}
|
|
|
|
if (schema.isTag(data) || schema.isUser(data)) {
|
|
return urlService.getUrlByResourceId(data.id, {secure: data.secure, absolute: absolute, withSubdirectory: true});
|
|
}
|
|
|
|
if (schema.isNav(data)) {
|
|
return urlService.utils.urlFor('nav', {nav: data, secure: data.secure}, absolute);
|
|
}
|
|
|
|
// sanitize any trailing `/amp` in the url
|
|
return sanitizeAmpUrl(urlService.utils.urlFor(data, {}, absolute));
|
|
}
|
|
|
|
module.exports = getUrl;
|