From 8c0b19012bae203f41b75cf40d6baf121f67fdea Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Sat, 17 Sep 2016 16:36:10 +0200 Subject: [PATCH] Fix #41, debug #42 --- .../components/reducers/notifications.jsx | 22 +++++++++++++++---- app/controllers/api/follows_controller.rb | 2 +- app/services/follow_remote_account_service.rb | 8 ++++++- app/views/api/follows/show.rabl | 2 +- 4 files changed, 27 insertions(+), 7 deletions(-) diff --git a/app/assets/javascripts/components/reducers/notifications.jsx b/app/assets/javascripts/components/reducers/notifications.jsx index 6ba45329235..bdd2de993a2 100644 --- a/app/assets/javascripts/components/reducers/notifications.jsx +++ b/app/assets/javascripts/components/reducers/notifications.jsx @@ -7,6 +7,23 @@ import Immutable from 'immutable'; const initialState = Immutable.List(); +function notificationFromError(state, error) { + let n = Immutable.Map({ + message: '' + }); + + if (error.response) { + n = n.withMutations(map => { + map.set('message', error.response.statusText); + map.set('title', `${error.response.status}`); + }); + } else { + n = n.set('message', `${error}`); + } + + return state.push(n); +}; + export default function meta(state = initialState, action) { switch(action.type) { case COMPOSE_SUBMIT_FAIL: @@ -15,10 +32,7 @@ export default function meta(state = initialState, action) { case REBLOG_FAIL: case FAVOURITE_FAIL: case TIMELINE_REFRESH_FAIL: - return state.push(Immutable.fromJS({ - message: action.error.response.statusText, - title: `${action.error.response.status}` - })); + return notificationFromError(state, action.error); case NOTIFICATION_DISMISS: return state.clear(); default: diff --git a/app/controllers/api/follows_controller.rb b/app/controllers/api/follows_controller.rb index 604e6556563..dbd44cc546d 100644 --- a/app/controllers/api/follows_controller.rb +++ b/app/controllers/api/follows_controller.rb @@ -7,7 +7,7 @@ class Api::FollowsController < ApiController raise ActiveRecord::RecordNotFound end - @follow = FollowService.new.(current_user.account, params[:uri]) + @account = FollowService.new.(current_user.account, params[:uri]).try(:target_account) render action: :show end end diff --git a/app/services/follow_remote_account_service.rb b/app/services/follow_remote_account_service.rb index c4330d7cf3b..0e23ed7d2b9 100644 --- a/app/services/follow_remote_account_service.rb +++ b/app/services/follow_remote_account_service.rb @@ -13,8 +13,10 @@ class FollowRemoteAccountService < BaseService account = Account.find_remote(username, domain) if account.nil? + Rails.logger.debug "Creating new remote account for #{uri}" account = Account.new(username: username, domain: domain) elsif account.subscribed? + Rails.logger.debug "Already subscribed to remote account #{uri}" return account end @@ -29,7 +31,10 @@ class FollowRemoteAccountService < BaseService feed = get_feed(account.remote_url) hubs = feed.xpath('//xmlns:link[@rel="hub"]') - return nil if hubs.empty? || hubs.first.attribute('href').nil? || feed.at_xpath('/xmlns:feed/xmlns:author/xmlns:uri').nil? + if hubs.empty? || hubs.first.attribute('href').nil? || feed.at_xpath('/xmlns:feed/xmlns:author/xmlns:uri').nil? + Rails.logger.debug "Cannot find PuSH hub or author for #{uri}" + return nil + end account.uri = feed.at_xpath('/xmlns:feed/xmlns:author/xmlns:uri').content account.hub_url = hubs.first.attribute('href').value @@ -49,6 +54,7 @@ class FollowRemoteAccountService < BaseService return account rescue Goldfinger::Error, HTTP::Error + Rails.logger.debug "Error while fetching data for #{uri}" nil end diff --git a/app/views/api/follows/show.rabl b/app/views/api/follows/show.rabl index e5b52b828b7..0f24ddb4d58 100644 --- a/app/views/api/follows/show.rabl +++ b/app/views/api/follows/show.rabl @@ -1,2 +1,2 @@ -object @follow.target_account +object @account extends('api/accounts/show')