действие удаления не работает после добавления дополнительного кода внутри него - PullRequest
Новая
       7

действие удаления не работает после добавления дополнительного кода внутри него

0 голосов
/ 04 сентября 2018

У меня есть действие удаления в моем контроллере, это код в pickups_controller.rb 

  def delete
    @pickup = Pickup.find(params[:id])
    if !@pickup.nil?
      @pickup.destroy
      render json: { success_message: "Success!, Pickup is deleted." }, status: :ok
    end
  end

Я вызываю действие удаления с помощью javascript json, нажимая кнопку с помощью assets / javascripts / pickups.js 

document.addEventListener("DOMContentLoaded", function(event) {



    var xhttp = new XMLHttpRequest();

    // delete the pickup you choose

    $('.removepickup.btn.btn-primary').on('click', function() {
        var pickup_div = $(this).parents('.removepickupparent');
        var pickup_id = pickup_div.attr('id');
        var x = "../deletepickup?id=" + pickup_id;
        $.ajax({
            type: "POST",
            url: x,
            success: function(data) {
                var success = data.success_message;
                $(".successr"+ pickup_id).text(success).show(0).delay(1000).hide(0);   
                setTimeout(function () {
                    location.reload();
                }, 1000);
            },

            error: function (xhr, ajaxOptions, thrownError){
                if(xhr.status==404) {
                    $(".errorl"+ pickup_id).text("Fail!, pickup Is Already Deleted Before").show(0).delay(1000).hide(0);
                    setTimeout(function () {
                         location.reload();
                    }, 2000);
                }
            }

        });
    });



    // when pressing on this button, it redirects you to create pickup page

    $('.addpickup.btn.btn-primary').on('click', function() {
        var success = "Redirecting to add pickup Page"
        $(".successp").text(success).show(0).delay(2000).hide(0);
        setTimeout(function () {
        $(location).attr('href', '../createpickup');
        }, 2000);

    });


});

функция работает отлично, но при добавлении 4-х строк дополнительного кода внутри действия удаления она не работает, вот код после добавления 4-х строк дополнительного кода внутри моего действия удаления, и действие не работает. 

  def delete
    @pickup = Pickup.find(params[:id])
    if !@pickup.nil?

      # the start of the extra code
      @trip = Trip.find(@pickup.trip_id)
      if !@trip.nil?
        @trip.seatsno = @trip.seatsno + 1
        @trip.save
      end
      # the end of the extra code

      @pickup.destroy
      render json: { success_message: "Success!, Pickup is deleted." }, status: :ok
    end
  end

какие-либо решения, пожалуйста? .. зная, что я все еще новичок в Ruby on Rails

Примечание:

Я использовал byebug, и при достижении первой строки в коде etra я получил эту ошибку в терминале локального сервера "request.env["action_dispatch.show_detailed_exceptions"] ||= show_detailed_exceptions?"

Ответы [ 2 ]

0 голосов
/ 04 сентября 2018

Используйте find_by вместо find метода. find' method raises the exception if a particular record is not found, while find_by` возвращает ноль.

Использование: 

find_by(id: params[:id])
0 голосов
/ 04 сентября 2018

Этот ответ является скорее предложением по рефакторингу, чем фактическим ответом, но он также может решить вашу проблему.

Вы можете изменить свое действие на это: 

def delete
  @pickup = Pickup.find(params[:id])
  # no need to test @pickup.nil? here because `find` method raise 
  # an ActiveRecord::RecordNotFound error if the record is not found
  # which should be caught by ApplicationController to render a 404
  if @pickup.destroy
    @pickup.trip.update_attributes(seatsno: @pickup.trip.seatsno + 1)
    render json: { success_message: "Success!, Pickup is deleted." }, status: :ok
  else
    render json: { error_message: "Error, Pickup could not be deleted." }, status: 409
  end
end

Еще лучше перенести задачу увеличения seatsno на модель Pickup: 

# app/models/pickup.rb
after_destroy :increment_trip_seatsno

def increment_trip_seatsno
  self.trip.update_attributes(seatsno: self.trip.seatsno + 1)
end

И снимите проблему с контроллера. Таким образом, каждый раз, когда запись Pickup уничтожается через Rails (консоль или другие места в вашем приложении), поездка будет обновляться соответствующим образом.

...