Как я могу очистить этот вызов AJAX от Padrino?

Question

Я пишу приложение, которое будет главным образом пользовательским интерфейсом, вызывающим бэкэнд-сервер через AJAX. Очень мало загрузок страниц.

Так, например, когда я создаю поездку, мой JS просто отправляет объект JSON в Padrino (через POST), а Padrino сохраняет объект поездки (через ActiveRecord) и возвращает ответ JSON.

Кажется, что это работает, но я хочу не только очистить код, но и хочу дезинформировать представленные значения.

Вот мой POST код (trips controller)

post :index, :provides => :json do 
    response = {} 
    response[:trip] = {} 

    begin 
     @trip = Trip.new 
     @trip.title = params[:trip][:title] 
     @trip.description = params[:trip][:title] 

     if @trip.save 
      response[:status] = "Success" 
      response[:trip] = {:title => @trip.title, :description => @trip.description} 
      response[:message] = "#{@trip.title} successfully saved" 
     else 
      response[:status] = "Error" 
      response[:message] = "Error saving trip" 
     end 
    rescue 
    response[:status] = "Error" 
    response[:message] = "Error saving trip" 
    end 

    response.to_json 
end 

В настоящее время существует только два поля (название и описание), но будет около 4-8, когда сделано. Мне не нравится, как я строю новый объект поездки.

Я попытался с помощью:

@trip = Trip.build(params[:trip]) 

, но это не сработало.

Вот мой JS код, который отправляет POST:

// Save new trip 
$("#new_trip_save_btn").click(function() { 
    var self = this; 
    var new_trip = get_new_trip(); 

    $.ajax({ 
     data: {trip:new_trip}, 
     dataType: "json", 
     url: "/trips", 
     type: "post", 
     success: function(res) { 
      console.log(res) 
     } 
    }); 
}); 

...... 

var get_new_trip = function() { 
    var self = this; 
    var trip = {}; 
    trip.title = $("#new_trip_title").val(); 
    trip.description = $("#new_trip_description").val(); 
    trip.departure_date = $("#new_trip_departure").val(); 
    trip.return_date = $("#new_trip_return").val(); 

    return trip; 
} 

Так что я могу сделать, чтобы очистить код (удалить избыточность в действие POST) и убедитесь, что текст продезинфицировать перед сохранением.

Благодаря

Answer 1

первую очередь, вы должны защитить вашу модель от массовых заданий с использованием attr_accessible и attr_protected ака mass assignment.

Затем я настоятельно рекомендую использовать «формы», чтобы ваш сайт мог работать без включенного javascript.

Таким образом, используя код unobtrusive, код javascripts также может быть приятнее.

// Live watch events on form submit 
$(document).on('submit', 'form[data-remote]', function(e){ 
    e.preventDefault(); 
    self = $(this); 
    $.ajax({ 
    url: self.attr('action'), 
    data: self.serializeArray(), 
    type: self.attr('method'), 
    dataType: 'json', 
    success: function(res){ console.log(res) } 
    }) 
}); 

Здесь форма:

/* Here the form, for dates I suggest to use a calendar */ 
/* like https://github.com/arshaw/fullcalendar */ 
- form_for @trip, url(:trip, :index), :'data-remote' => true do |f| 
    == f.error_messages 
    == f.text_field :title 
    == f.text_area :description 
    == f.text_field :departure_date 
    == f.text_field :return_data 
    == f.submit 'Send' 

Здесь контроллер:

provides :json # if all your actions in your controller respond to #json it is more dry 

get :index do 
    @trip = Trip.new(params[:trip]) 

    if @trip.save 
    render :success => true, :attributes => @trip.to_json 
    else 
    render :success => false, :errors => @trip.errors.full_messages 
    end 
end