previous commit was creating the status regardless
of mix of video and images in status, just wasn't rendering the show action. I moved the validation before the status creationlolsob-rspec
parent
008c95b3b9
commit
d55cf10b4e
|
@ -13,6 +13,7 @@ class PostStatusService < BaseService
|
|||
# @option [Doorkeeper::Application] :application
|
||||
# @return [Status]
|
||||
def call(account, text, in_reply_to = nil, options = {})
|
||||
media = validate_media options[:media_ids]
|
||||
status = account.statuses.create!(text: text,
|
||||
thread: in_reply_to,
|
||||
sensitive: options[:sensitive],
|
||||
|
@ -20,7 +21,7 @@ class PostStatusService < BaseService
|
|||
visibility: options[:visibility],
|
||||
application: options[:application])
|
||||
|
||||
attach_media(status, options[:media_ids])
|
||||
attach_media(status, media)
|
||||
process_mentions_service.call(status)
|
||||
process_hashtags_service.call(status)
|
||||
|
||||
|
@ -33,7 +34,7 @@ class PostStatusService < BaseService
|
|||
|
||||
private
|
||||
|
||||
def attach_media(status, media_ids)
|
||||
def validate_media(media_ids)
|
||||
return if media_ids.nil? || !media_ids.is_a?(Enumerable)
|
||||
media = MediaAttachment.where(status_id: nil).where(id: media_ids.take(4).map(&:to_i))
|
||||
if media.length > 1
|
||||
|
@ -43,6 +44,11 @@ class PostStatusService < BaseService
|
|||
end
|
||||
end
|
||||
end
|
||||
return media
|
||||
end
|
||||
|
||||
def attach_media(status, media)
|
||||
return if media.nil?
|
||||
media.update(status_id: status.id)
|
||||
end
|
||||
|
||||
|
|
Loading…
Reference in New Issue