FIX: allow staff to remove tags from queued topics
This commit is contained in:
parent
a603660ba5
commit
7efdccdbc5
|
@ -25,10 +25,10 @@ class QueuedPostsController < ApplicationController
|
|||
update_params = params[:queued_post]
|
||||
|
||||
qp.raw = update_params[:raw] if update_params[:raw].present?
|
||||
unless qp.topic_id
|
||||
if qp.topic_id.blank? && params[:queued_post][:state].blank?
|
||||
qp.post_options['title'] = update_params[:title] if update_params[:title].present?
|
||||
qp.post_options['category'] = update_params[:category_id].to_i if update_params[:category_id].present?
|
||||
qp.post_options['tags'] = update_params[:tags] if update_params[:tags].present?
|
||||
qp.post_options['tags'] = update_params[:tags]
|
||||
end
|
||||
|
||||
qp.save(validate: false)
|
||||
|
|
|
@ -100,6 +100,23 @@ describe QueuedPostsController do
|
|||
expect(queued_topic.post_options['category']).to eq(changes[:category_id])
|
||||
expect(queued_topic.post_options['tags']).to eq(changes[:tags])
|
||||
end
|
||||
|
||||
it 'removes tags if not present' do
|
||||
queued_topic.post_options[:tags] = ['another-tag']
|
||||
queued_topic.save!
|
||||
|
||||
put "/queued_posts/#{queued_topic.id}.json", params: {
|
||||
queued_post: changes.except(:tags)
|
||||
}
|
||||
|
||||
expect(response.status).to eq(200)
|
||||
queued_topic.reload
|
||||
|
||||
expect(queued_topic.raw).to eq(changes[:raw])
|
||||
expect(queued_topic.post_options['title']).to eq(changes[:title])
|
||||
expect(queued_topic.post_options['category']).to eq(changes[:category_id])
|
||||
expect(queued_topic.post_options['tags']).to be_nil
|
||||
end
|
||||
end
|
||||
|
||||
context 'when it is a reply' do
|
||||
|
|
Loading…
Reference in New Issue