Как обновить существующую запись пользователя

Question

Я работаю над приложением, которое содержит soundcloud api.

У меня есть логин и модель пользователя, и я хотел бы добавить soundcloud_id и токен к существующему пользователю, но я не могу как-то обновить запись пользователей. что я делаю неправильно?

контроллер SoundCloud

class SoundcloudController < ApplicationController 


     def connect 
     # create client object with app credentials 
     client = Soundcloud.new(:client_id => ENV["SOUNDCLOUD_CLIENT_ID"], 
           :client_secret => ENV["SOUNDCLOUD_CLIENT_SECRET"], 
           :redirect_uri => "http://localhost:3000/soundcloud/oauth-callback", 
           :response_type => 'code') 

     # redirect user to authorize URL 
     redirect_to client.authorize_url(:grant_type => 'authorization_code', :scope => 'non-expiring', :display => 'popup') 
     end 

     def connected 
     # create client object with app credentials 
     client = Soundcloud.new(:client_id => ENV["SOUNDCLOUD_CLIENT_ID"], 
        :client_secret => ENV["SOUNDCLOUD_CLIENT_SECRET"], 
        :redirect_uri => "http://localhost:3000/soundcloud/oauth-callback") 

     # exchange authorization code for access token 
     access_token = client.exchange_token(:code => params[:code]) 
     client = Soundcloud.new(:access_token => access_token["access_token"]) 

     # make an authenticated call 
     soundcloud_user = client.get('/me') 
     unless User.where(:soundcloud_user_id => soundcloud_user["id"]).present? 
     #User.create_from_soundcloud(soundcloud_user, access_token) 
     UsersController.add_soundcloud_account(soundcloud_user, access_token) 
     end 
     sign_in_user = User.where(:soundcloud_user_id => soundcloud_user["id"]) 

     #create user sessions 
     #session[:user_id] = sign_in_user.first.id 
     redirect_to root_url, notice: "Signed in!" 
     end 

     def destroy 
     end 
    end 

контроллер пользователя

class UsersController < ApplicationController 


     def new 
     @user = User.new 
     end 

     #create a user and redirect to home 
     def create 
     @user = User.new(user_params) 
     if @user.save 
      session[:user_id] = @user.id 
      redirect_to '/' 
     else 
      redirect_to '/signup' 
     end 
     end 


     def self.add_soundcloud_account(soundcloud_user, access_token) 
     @current_user ||= User.find(session[:user_id]) 
     @current_user.soundcloud_user_id = soundcloud_user["id"] 
     @current_user.soundcloud_access_token = access_token["access_token"] 
     end 

     private 
     def user_params 
     params.require(:user).permit(:first_name, :last_name, :email, :password) 
     end 
    end 

Answer 1

Вам нужно позвонить save на @current_user и передать информацию сессии методу:

def self.add_soundcloud_account(user_id, soundcloud_user, access_token) 
    @current_user ||= User.find(user_id) 
    @current_user.soundcloud_user_id = soundcloud_user["id"] 
    @current_user.soundcloud_access_token = access_token["access_token"] 
    @current_user.save 
end 

Это называется например:

UsersController.add_soundcloud_account(session[:user_id], soundcloud_user, access_token) 

Однако я не уверен, что метод add_soundcloud_account принадлежит к контроллеру. Я бы предпочел увидеть его в Сервисе или, может быть, только в модели User.