From ce29624c6df83699a01808d68c19a2492864ffe8 Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Mon, 12 Sep 2016 18:22:43 +0200 Subject: Fixing image upload limits, allowing webm, merge/unmerge events trigger timeline reload in UI, other small fixes --- .../javascripts/components/actions/timelines.jsx | 45 ++++++++++++++++++++-- .../javascripts/components/components/avatar.jsx | 2 +- .../javascripts/components/containers/root.jsx | 30 ++++++++------- app/channels/application_cable/connection.rb | 4 +- app/lib/feed_manager.rb | 6 ++- app/lib/formatter.rb | 2 +- app/models/account.rb | 9 +++-- app/models/media_attachment.rb | 15 +++++++- app/services/follow_service.rb | 1 + app/services/unfollow_service.rb | 2 + 10 files changed, 90 insertions(+), 26 deletions(-) (limited to 'app') diff --git a/app/assets/javascripts/components/actions/timelines.jsx b/app/assets/javascripts/components/actions/timelines.jsx index 4d4122ec7..383b727e9 100644 --- a/app/assets/javascripts/components/actions/timelines.jsx +++ b/app/assets/javascripts/components/actions/timelines.jsx @@ -1,6 +1,12 @@ -export const TIMELINE_SET = 'TIMELINE_SET'; -export const TIMELINE_UPDATE = 'TIMELINE_UPDATE'; -export const TIMELINE_DELETE = 'TIMELINE_DELETE'; +import api from '../api' + +export const TIMELINE_SET = 'TIMELINE_SET'; +export const TIMELINE_UPDATE = 'TIMELINE_UPDATE'; +export const TIMELINE_DELETE = 'TIMELINE_DELETE'; + +export const TIMELINE_REFRESH_REQUEST = 'TIMELINE_REFRESH_REQUEST'; +export const TIMELINE_REFRESH_SUCCESS = 'TIMELINE_REFRESH_SUCCESS'; +export const TIMELINE_REFRESH_FAIL = 'TIMELINE_REFRESH_FAIL'; export function setTimeline(timeline, statuses) { return { @@ -24,3 +30,36 @@ export function deleteFromTimelines(id) { id: id }; } + +export function refreshTimelineRequest(timeline) { + return { + type: TIMELINE_REFRESH_REQUEST, + timeline: timeline + }; +} + +export function refreshTimeline(timeline) { + return function (dispatch, getState) { + dispatch(refreshTimelineRequest(timeline)); + + api(getState).get(`/api/statuses/${timeline}`).then(function (response) { + dispatch(refreshTimelineSuccess(timeline, response.data)); + }).catch(function (error) { + dispatch(refreshTimelineFail(timeline, error)); + }); + }; +} + +export function refreshTimelineSuccess(timeline, statuses) { + return function (dispatch) { + dispatch(setTimeline(timeline, statuses)); + }; +} + +export function refreshTimelineFail(timeline, error) { + return { + type: TIMELINE_REFRESH_FAIL, + timeline: timeline, + error: error + }; +} diff --git a/app/assets/javascripts/components/components/avatar.jsx b/app/assets/javascripts/components/components/avatar.jsx index 6177683ba..267d02bcd 100644 --- a/app/assets/javascripts/components/components/avatar.jsx +++ b/app/assets/javascripts/components/components/avatar.jsx @@ -11,7 +11,7 @@ const Avatar = React.createClass({ render () { return ( -
+
); diff --git a/app/assets/javascripts/components/containers/root.jsx b/app/assets/javascripts/components/containers/root.jsx index eb031bdd4..e2baefa24 100644 --- a/app/assets/javascripts/components/containers/root.jsx +++ b/app/assets/javascripts/components/containers/root.jsx @@ -1,12 +1,12 @@ -import { Provider } from 'react-redux'; -import configureStore from '../store/configureStore'; -import Frontend from '../components/frontend'; -import { setTimeline, updateTimeline, deleteFromTimelines } from '../actions/timelines'; -import { setAccessToken } from '../actions/meta'; -import PureRenderMixin from 'react-addons-pure-render-mixin'; -import { Router, Route, createMemoryHistory } from 'react-router'; -import AccountRoute from '../routes/account_route'; -import StatusRoute from '../routes/status_route'; +import { Provider } from 'react-redux'; +import configureStore from '../store/configureStore'; +import Frontend from '../components/frontend'; +import { setTimeline, updateTimeline, deleteFromTimelines, refreshTimeline } from '../actions/timelines'; +import { setAccessToken } from '../actions/meta'; +import PureRenderMixin from 'react-addons-pure-render-mixin'; +import { Router, Route, createMemoryHistory } from 'react-router'; +import AccountRoute from '../routes/account_route'; +import StatusRoute from '../routes/status_route'; const store = configureStore(); const history = createMemoryHistory(); @@ -36,10 +36,14 @@ const Root = React.createClass({ disconnected: function() {}, received: function(data) { - if (data.type === 'update') { - return store.dispatch(updateTimeline(data.timeline, JSON.parse(data.message))); - } else if (data.type === 'delete') { - return store.dispatch(deleteFromTimelines(data.id)); + switch(data.type) { + case 'update': + return store.dispatch(updateTimeline(data.timeline, JSON.parse(data.message))); + case 'delete': + return store.dispatch(deleteFromTimelines(data.id)); + case 'merge': + case 'unmerge': + return store.dispatch(refreshTimeline('home')); } } }); diff --git a/app/channels/application_cable/connection.rb b/app/channels/application_cable/connection.rb index f7cbe5485..772b76a6e 100644 --- a/app/channels/application_cable/connection.rb +++ b/app/channels/application_cable/connection.rb @@ -10,7 +10,9 @@ module ApplicationCable protected def find_verified_user - if verified_user = env['warden'].user + verified_user = env['warden'].user + + if verified_user verified_user else reject_unauthorized_connection diff --git a/app/lib/feed_manager.rb b/app/lib/feed_manager.rb index 230e0789d..86acd39d9 100644 --- a/app/lib/feed_manager.rb +++ b/app/lib/feed_manager.rb @@ -17,7 +17,11 @@ class FeedManager def push(timeline_type, account, status) redis.zadd(key(timeline_type, account.id), status.id, status.id) trim(timeline_type, account.id) - ActionCable.server.broadcast("timeline:#{account.id}", type: 'update', timeline: timeline_type, message: inline_render(account, status)) + broadcast(account.id, type: 'update', timeline: timeline_type, message: inline_render(account, status)) + end + + def broadcast(account_id, options = {}) + ActionCable.server.broadcast("timeline:#{account_id}", options) end def trim(type, account_id) diff --git a/app/lib/formatter.rb b/app/lib/formatter.rb index ad0f022a7..959ede95e 100644 --- a/app/lib/formatter.rb +++ b/app/lib/formatter.rb @@ -35,7 +35,7 @@ class Formatter def link_mentions(html, mentions) html.gsub(Account::MENTION_RE) do |match| acct = Account::MENTION_RE.match(match)[1] - mention = mentions.find { |mention| mention.account.acct.eql?(acct) } + mention = mentions.find { |item| item.account.acct.eql?(acct) } mention.nil? ? match : mention_html(match, mention.account) end diff --git a/app/models/account.rb b/app/models/account.rb index 1767f72e2..6e96defe4 100644 --- a/app/models/account.rb +++ b/app/models/account.rb @@ -1,6 +1,9 @@ class Account < ApplicationRecord include Targetable + MENTION_RE = /(?:^|\s|\.|>)@([a-z0-9_]+(?:@[a-z0-9\.\-]+)?)/i + IMAGE_MIME_TYPES = ['image/jpeg', 'image/png', 'image/gif'] + # Local users has_one :user, inverse_of: :account validates :username, presence: true, format: { with: /\A[a-z0-9_]+\z/i, message: 'only letters, numbers and underscores' }, uniqueness: { scope: :domain, case_sensitive: false }, if: 'local?' @@ -8,12 +11,12 @@ class Account < ApplicationRecord # Avatar upload has_attached_file :avatar, styles: { large: '300x300#', medium: '96x96#', small: '48x48#' } - validates_attachment_content_type :avatar, content_type: /\Aimage\/.*\Z/ + validates_attachment_content_type :avatar, content_type: IMAGE_MIME_TYPES validates_attachment_size :avatar, less_than: 2.megabytes # Header upload has_attached_file :header, styles: { medium: '700x335#' } - validates_attachment_content_type :header, content_type: /\Aimage\/.*\Z/ + validates_attachment_content_type :header, content_type: IMAGE_MIME_TYPES validates_attachment_size :header, less_than: 2.megabytes # Local user profile validations @@ -35,8 +38,6 @@ class Account < ApplicationRecord has_many :media_attachments, dependent: :destroy - MENTION_RE = /(?:^|\s|\.|>)@([a-z0-9_]+(?:@[a-z0-9\.\-]+)?)/i - def follow!(other_account) self.active_relationships.where(target_account: other_account).first_or_create!(target_account: other_account) end diff --git a/app/models/media_attachment.rb b/app/models/media_attachment.rb index 0f631af57..7dddfd610 100644 --- a/app/models/media_attachment.rb +++ b/app/models/media_attachment.rb @@ -1,9 +1,12 @@ class MediaAttachment < ApplicationRecord + IMAGE_MIME_TYPES = ['image/jpeg', 'image/png', 'image/gif'] + VIDEO_MIME_TYPES = ['video/webm'] + belongs_to :account, inverse_of: :media_attachments belongs_to :status, inverse_of: :media_attachments - has_attached_file :file, styles: { small: '510x680>' } - validates_attachment_content_type :file, content_type: /\Aimage\/.*\z/ + has_attached_file :file, styles: lambda { |f| f.instance.image? ? { small: '510x680>' } : { small: { format: 'webm' } } }, processors: lambda { |f| f.video? ? [:transcoder] : [:thumbnail] } + validates_attachment_content_type :file, content_type: IMAGE_MIME_TYPES + VIDEO_MIME_TYPES validates_attachment_size :file, less_than: 4.megabytes validates :account, presence: true @@ -15,4 +18,12 @@ class MediaAttachment < ApplicationRecord def file_remote_url=(url) self.file = URI.parse(url) end + + def image? + IMAGE_MIME_TYPES.include? file_content_type + end + + def video? + VIDEO_MIME_TYPES.include? file_content_type + end end diff --git a/app/services/follow_service.rb b/app/services/follow_service.rb index 5ac758e69..45f145df3 100644 --- a/app/services/follow_service.rb +++ b/app/services/follow_service.rb @@ -30,6 +30,7 @@ class FollowService < BaseService end FeedManager.instance.trim(:home, into_account.id) + FeedManager.instance.broadcast(into_account.id, type: 'merge') end def redis diff --git a/app/services/unfollow_service.rb b/app/services/unfollow_service.rb index 4d9c2a9a7..feaf28ceb 100644 --- a/app/services/unfollow_service.rb +++ b/app/services/unfollow_service.rb @@ -16,6 +16,8 @@ class UnfollowService < BaseService from_account.statuses.find_each do |status| redis.zrem(timeline_key, status.id) end + + FeedManager.instance.broadcast(into_account.id, type: 'unmerge') end def redis -- cgit