TryGhost/Ghost
1
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2024-12-25 03:44:29 +03:00
Code Issues Projects Releases Wiki Activity

🐛 Fixed post scheduling for sites with transferred ownership (#17075)

Browse Source 
refs TryGhost/Ghost#3494

- By default, the post scheduler runs as user_id = 1, which is the
original owner of the site
- If ownership has been transferred to a different user, it's possible
that there is no user with id = 1
- In this case, the scheduler would fail to publish a post, because
updating the post using user_id = 1 failed a foreign key constraint in
the post_revisions table
- This commit fixes the issue by checking if the contextUser exists, and
if not, replacing it with the current owner of the site
This commit is contained in:
Chris Raible 2023-06-20 19:51:13 -07:00 committed by GitHub
parent 69fddf8abe
commit 2568c5898a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ghost/core/core/server/models/post.js
View File

@ -875,7 +875,11 @@ Post = ghostBookshelf.Model.extend({
revision_interval_ms: POST_REVISIONS_INTERVAL_MS revision_interval_ms: POST_REVISIONS_INTERVAL_MS
} }
}); });
const authorId = this.contextUser(options); let authorId = this.contextUser(options);
const authorExists = await ghostBookshelf.model('User').findOne({id: authorId}, {transacting: options.transacting});
if (!authorExists) {
authorId = await ghostBookshelf.model('User').getOwnerUser().get('id');
}
ops.push(async function updateRevisions() { ops.push(async function updateRevisions() {
const revisionModels = await ghostBookshelf.model('PostRevision') const revisionModels = await ghostBookshelf.model('PostRevision')
.findAll(Object.assign({ .findAll(Object.assign({