Fix #4695: Raise max video length to match Twitter's (2:20).
This commit is contained in:
@@ -1,6 +1,8 @@
|
|||||||
class Upload < ApplicationRecord
|
class Upload < ApplicationRecord
|
||||||
class Error < StandardError; end
|
class Error < StandardError; end
|
||||||
|
|
||||||
|
MAX_VIDEO_DURATION = 140
|
||||||
|
|
||||||
class FileValidator < ActiveModel::Validator
|
class FileValidator < ActiveModel::Validator
|
||||||
def validate(record)
|
def validate(record)
|
||||||
validate_file_ext(record)
|
validate_file_ext(record)
|
||||||
@@ -53,8 +55,8 @@ class Upload < ApplicationRecord
|
|||||||
end
|
end
|
||||||
|
|
||||||
def validate_video_duration(record)
|
def validate_video_duration(record)
|
||||||
if !record.uploader.is_admin? && record.media_file.is_video? && record.media_file.duration > 120
|
if !record.uploader.is_admin? && record.media_file.is_video? && record.media_file.duration > MAX_VIDEO_DURATION
|
||||||
record.errors.add(:base, "video must not be longer than 2 minutes")
|
record.errors.add(:base, "video must not be longer than #{MAX_VIDEO_DURATION.seconds.inspect}")
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -336,6 +336,7 @@ class UploadsControllerTest < ActionDispatch::IntegrationTest
|
|||||||
should_upload_successfully("https://twitter.com/noizave/status/875768175136317440")
|
should_upload_successfully("https://twitter.com/noizave/status/875768175136317440")
|
||||||
should_upload_successfully("https://pbs.twimg.com/media/DCdZ_FhUIAAYKFN?format=jpg&name=medium")
|
should_upload_successfully("https://pbs.twimg.com/media/DCdZ_FhUIAAYKFN?format=jpg&name=medium")
|
||||||
should_upload_successfully("https://pbs.twimg.com/profile_banners/2371694594/1581832507/1500x500")
|
should_upload_successfully("https://pbs.twimg.com/profile_banners/2371694594/1581832507/1500x500")
|
||||||
|
should_upload_successfully("https://twitter.com/zeth_total/1355597580814585856")
|
||||||
# XXX should_upload_successfully("https://video.twimg.com/tweet_video/EWHWVrmVcAAp4Vw.mp4")
|
# XXX should_upload_successfully("https://video.twimg.com/tweet_video/EWHWVrmVcAAp4Vw.mp4")
|
||||||
|
|
||||||
should_upload_successfully("https://www.weibo.com/5501756072/J2UNKfbqV")
|
should_upload_successfully("https://www.weibo.com/5501756072/J2UNKfbqV")
|
||||||
|
|||||||
Reference in New Issue
Block a user