Rewrite callbacks for avoid using redirect_to :back (#2169)

* Change redirect_to to render (#2113)

* Rewrite callbacks for avoid using redirect_to :back

app/controllers/admin/blockchains_controller.rb

@@ -21,7 +21,7 @@ def create
       if @blockchain.save
         redirect_to admin_blockchains_path
       else
-        flash[:alert] = @blockchain.errors.full_messages
+        flash[:alert] = @blockchain.errors.full_messages.first
         render :show
       end
     end
@@ -31,8 +31,8 @@ def update
       if @blockchain.update(blockchain_params)
         redirect_to admin_blockchains_path
       else
-        flash[:alert] = @blockchain.errors.full_messages
-        redirect_to :back
+        flash[:alert] = @blockchain.errors.full_messages.first
+        render :show
       end
     end
 

app/controllers/admin/currencies_controller.rb

@@ -33,7 +33,7 @@ def update
         redirect_to admin_currencies_path
       else
         flash[:alert] = @currency.errors.full_messages.first
-        redirect_to :back
+        render :show
       end
     end
 

app/controllers/admin/deposits/coins_controller.rb

@@ -27,7 +27,7 @@ def update
         when 'collect_fee'
           deposit.collect!
         end
-        redirect_to :back, notice: 'The deposit was successful.'
+        redirect_to admin_deposit_index_path(currency.id), notice: 'Deposit succesfully updated.'
       end
 
       private

app/controllers/admin/deposits/fiats_controller.rb

@@ -21,7 +21,6 @@ def new
 
       def show
         @deposit = ::Deposits::Fiat.where(currency: currency).find(params[:id])
-        flash.now[:notice] = 'The recharge have been successful.' if @deposit.accepted?
       end
 
       def create
@@ -35,14 +34,16 @@ def create
       end
 
       def update
-        deposit = ::Deposits::Fiat.where(currency: currency).find(params[:id])
+        @deposit = ::Deposits::Fiat.where(currency: currency).find(params[:id])
         case params.fetch(:commit)
         when 'Accept'
-          deposit.charge!
+          @deposit.charge!
+          flash.keep[:notice] = "The recharge have been successful."
         when 'Reject'
-          deposit.reject!
+          @deposit.reject!
         end
-        redirect_to :back
+        @deposit.reload
+        render :show
       end
 
     private

app/controllers/admin/markets_controller.rb

@@ -33,7 +33,7 @@ def update
         redirect_to admin_markets_path
       else
         flash[:alert] = @market.errors.full_messages.first
-        redirect_to :back
+        render :show
       end
     end
 

app/controllers/admin/wallets_controller.rb

@@ -23,7 +23,7 @@ def create
       if @wallet.save
         redirect_to admin_wallets_path
       else
-        flash[:alert] = @wallet.errors.full_messages
+        flash[:alert] = @wallet.errors.full_messages.first
         render :show
       end
     end
@@ -33,8 +33,8 @@ def update
       if @wallet.update(wallet_params)
         redirect_to admin_wallets_path
       else
-        flash[:alert] = @wallet.errors.full_messages
-        redirect_to :back
+        flash[:alert] = @wallet.errors.full_messages.first
+        render :show
       end
     end
 

app/controllers/admin/withdraws/coins_controller.rb

@@ -32,7 +32,7 @@ def update
 
       def destroy
         @withdraw.reject!
-        redirect_to :back, notice: 'Succesfully updated.'
+        redirect_to admin_withdraw_path(currency.id, @withdraw.id), notice: 'Withdrawal succesfully updated.'
       end
 
       private
@@ -47,15 +47,15 @@ def process!
           @withdraw.accept!
           @withdraw.process!
         end
-        redirect_to :back, notice: 'Succesfully updated.'
+        redirect_to admin_withdraw_path(currency.id, @withdraw.id), notice: 'Withdrawal succesfully updated.'
       end
 
       def load!
         @withdraw.transaction do
           @withdraw.update!(txid: params.fetch(:txid))
           @withdraw.load!
         end
-        redirect_to :back, notice: 'Succesfully updated.'
+        redirect_to admin_withdraw_path(currency.id, @withdraw.id), notice: 'Withdrawal succesfully updated.'
       end
     end
   end

app/controllers/admin/withdraws/fiats_controller.rb

@@ -26,12 +26,12 @@ def update
           @withdraw.dispatch!
           @withdraw.success!
         end
-        redirect_to :back, notice: 'Withdraw successfully updated!'
+        redirect_to admin_withdraw_path(currency.id, @withdraw.id), notice: 'Withdrawal successfully updated!'
       end
 
       def destroy
         @withdraw.reject!
-        redirect_to :back, notice: 'Withdraw successfully destroyed!'
+        redirect_to admin_withdraw_path(currency.id, @withdraw.id), notice: 'Withdrawal successfully destroyed!'
       end
     end
   end