Merge pull request #161 from tgriesser/published_at

Setting published_at when post changes to published status.
This commit is contained in:
Hannah Wolfe 2013-06-16 01:33:24 -07:00
commit 49f6cc92d4
3 changed files with 26 additions and 9 deletions

View File

@ -29,12 +29,12 @@
t.string('status');
t.string('language');
t.integer('author_id');
t.date('created_at');
t.dateTime('created_at');
t.integer('created_by');
t.date('updated_at');
t.integer('updated_by');
t.date('published_at');
t.integer('published_by');
t.dateTime('updated_at').nullable();
t.integer('updated_by').nullable();
t.dateTime('published_at').nullable();
t.integer('published_by').nullable();
}),
knex.Schema.createTable('users', function (t) {
@ -47,9 +47,9 @@
t.string('cover_picture');
t.text('bio');
t.string('url');
t.date('created_at');
t.dateTime('created_at');
t.integer('created_by');
t.date('updated_at');
t.dateTime('updated_at');
t.integer('updated_by');
}),
@ -91,9 +91,9 @@
t.string('key').unique();
t.text('value');
t.string('type');
t.date('created_at');
t.dateTime('created_at');
t.integer('created_by');
t.date('updated_at');
t.dateTime('updated_at');
t.integer('updated_by');
})

View File

@ -36,6 +36,12 @@
}
this.set('content_html', converter.makeHtml(this.get('content')));
if (this.hasChanged('status') && this.get('status') === 'published') {
this.set('published_at', new Date());
// This will need to go elsewhere in the API layer.
this.set('published_by', 1);
}
// refactoring of ghost required in order to make these details available here
// this.set('language', this.get('language') || ghost.config().defaultLang);
// this.set('status', this.get('status') || ghost.statuses().draft);

View File

@ -79,6 +79,10 @@
};
PostModel.add(newPost).then(function (createdPost) {
return new PostModel({id: createdPost.id}).fetch();
}).then(function (createdPost) {
should.exist(createdPost);
createdPost.has('uuid').should.equal(true);
@ -86,9 +90,16 @@
createdPost.get('title').should.equal(newPost.title, "title is correct");
createdPost.get('content').should.equal(newPost.content, "content is correct");
createdPost.get('slug').should.equal(newPost.title.toLowerCase().replace(/ /g, '-'), 'slug is correct');
should.equal(createdPost.get('published_at'), null);
// Set the status to published to check that `published_at` is set.
return createdPost.save({status: 'published'});
}).then(function(publishedPost) {
publishedPost.get('published_at').should.be.instanceOf(Date);
done();
}).then(null, done);
});
it('can delete', function (done) {