Rails / Stripe - 付款未获得处理

时间:2016-09-20 12:01:30

标签: javascript ruby ruby-on-rails-4 model-view-controller stripe-payments

我正在使用Rails和Stripe构建一个事件应用来处理付款。我在预订模式中设置了一种预留方法来处理付款流程。

代码必须能够允许用户一次预订多个空格。

我没有收到任何错误,但是,每当我尝试处理任何类型的预订时,我都会收到错误消息“预订不成功”'从我的代码。我似乎无法理解为什么它会直接跳到此而不是完成付款和预订。

这是我的预订模式中的预留方法 -

    def reserve
    # Don't process this booking if it isn't valid
    return unless valid?

    # We can always set this, even for free events because their price will be 0.
    self.total_amount = quantity * event.price

    # Free events don't need to do anything special
    if event.is_free?
      save

    # Paid events should charge the customer's card
    else
      begin
        charge = Stripe::Charge.create(amount: total_amount, currency: "gbp", card: stripe_token, description: "Booking number #{Booking.id}", items: [{quantity: quantity}])
        self.stripe_charge_id = charge.id
        save
      rescue Stripe::CardError => e
        errors.add(:base, e.message)
        false
      end
     end 
  end
end

在我的bookings_controller中,我有以下新内容和创建操作 -

    def new

        @event = Event.find(params[:event_id])
        @booking = @event.bookings.new(quantity: params[:quantity])
        @booking.user = current_user

    end

    def create
    @event = Event.find(params[:event_id])
    @booking = @event.bookings.new(booking_params)
    @booking.user = current_user

        if @booking.reserve
            flash[:success] = "Your place on our event has been booked"
            redirect_to event_path(@event)
        else
            flash[:error] = "Booking unsuccessful"
            render "new"
        end
    end


    private

    def booking_params
        params.require(:booking).permit(:stripe_token, :quantity)
    end

是什么导致@ booking.reserve失败,而代码只是默认为flash错误消息?

这是我视图中的代码。我是否需要在Stripe javascript代码中添加任何内容?

booking.new.html.erb

<div class="col-md-6 col-md-offset-3" id="eventshow">
  <div class="row">
    <div class="panel panel-default">
        <div class="panel-heading">
            <h2>Confirm Your Booking</h2>
        </div>
                  <div class="calculate-total">
                              <p>
                                  Confirm number of spaces you wish to book here:
                                    <input type="number" placeholder="1"  min="1" value="1" class="num-spaces">
                              </p>
                                <p>
                                    Total Amount
                                    £<span class="total" data-unit-cost="<%= @event.price %>">0</span>
                                </p>
                          </div>





                <%= simple_form_for [@event, @booking], id: "new_booking" do |form| %>



                 <span class="payment-errors"></span>

                <div class="form-row">
                    <label>
                      <span>Card Number</span>
                      <input type="text" size="20" data-stripe="number"/>
                    </label>
                </div>

                <div class="form-row">
                  <label>
                  <span>CVC</span>
                  <input type="text" size="4" data-stripe="cvc"/>
                  </label>
                </div>

                <div class="form-row">
                    <label>
                        <span>Expiration (MM/YYYY)</span>
                        <input type="text" size="2" data-stripe="exp-month"/>
                    </label>
                    <span> / </span>
                    <input type="text" size="4" data-stripe="exp-year"/>
                </div>
            </div>
            <div class="panel-footer">    

               <%= form.button :submit %>


            </div> 

<% end %>
<% end %>

      </div>
  </div>
</div>  

<script type="text/javascript">
    $('.calculate-total input').on('keyup change', calculateBookingPrice);

function calculateBookingPrice() {
  var unitCost = parseFloat($('.calculate-total .total').data('unit-cost')),
      numSpaces = parseInt($('.calculate-total .num-spaces').val()),
      total = (numSpaces * unitCost).toFixed(2);

  if (isNaN(total)) {
    total = 0;
  }

  $('.calculate-total span.total').text(total);
}

  $(document).ready(calculateBookingPrice)

</script>



<script type="text/javascript" src="https://js.stripe.com/v2/"></script>

<script type="text/javascript">
  Stripe.setPublishableKey('<%= STRIPE_PUBLIC_KEY %>');
  var stripeResponseHandler = function(status, response) {
    var $form = $('#new_booking');

    if (response.error) {
    // Show the errors on the form
    $form.find('.payment-errors').text(response.error.message);
    $form.find('input[type=submit]').prop('disabled', false);
    } else {
    // token contains id, last4, and card type
    var token = response.id;
    // Insert the token into the form so it gets submitted to the server
    $form.append($('<input type="hidden" name="booking[stripe_token]"     />').val(token));
    // and submit
    $form.get(0).submit();
    }
  };

  // jQuery(function($)  { - changed to the line below
  $(document).on("ready page:load", function () {

    $('#new_booking').submit(function(event) {
      var $form = $(this);

      // Disable the submit button to prevent repeated clicks
      $form.find('input[type=submit]').prop('disabled', true);

      Stripe.card.createToken($form, stripeResponseHandler);

      // Prevent the form from submitting with the default action
      return false;
    });
  });
</script>

我想知道关键线是否是这个?

  self.stripe_charge_id = charge.id

我是否需要在预订表中添加任何内容?这是架构 -

create_table "bookings", force: :cascade do |t|
t.integer  "event_id"
t.integer  "user_id"
t.string   "stripe_token"
t.datetime "created_at",   null: false
t.datetime "updated_at",   null: false
t.integer  "quantity"
t.integer  "total_amount"
  end

我对Rails相当陌生,这是我第一次尝试实施这样的付款流程。我确信经验丰富,这可能是直截了当的。任何援助将不胜感激。

问题更新 - 25/09/16

我已就我的问题做了一些工作,而且我发现这条线正在阻止整个&#39;保留&#39;继续进行的方法(因为它表明会这样做) -

  # Don't process this booking if it isn't valid
    return unless valid?

然后我遇到了一个未定义的方法&#39; *&#39;为零:class&#39;对于这一行 -

 # We can always set this, even for free events because their price will be 0.
    self.total_amount = booking.quantity * event.price

所以,我已经评论了这两条线,因为我不知道为什么他们不工作。我希望能够至少进入我可以处理付款的阶段,但现在我收到此错误 -

Stripe error - source or customer

我非常确定源代码是我的应用的正确选项,并且我已尝试为此添加代码,但它没有接受它。什么是正确的选择?

2 个答案:

答案 0 :(得分:0)

在bookings_controller :: create操作中,您正在使用

访问事件
@event = Event.find(params[:event_id])

我检查了new.html.erb,我认为你还没有传递事件ID来创建动作。请检查一下。

使用javascript传递event_id:

var eventID =“&lt;%= @ event.id%&gt;”

在您启动javascript的视图中添加以上行。

并且

$form.append($('<input type="hidden" name="event_id">').val(eventID));

在$('#new_booking')。submit function。

中添加以上行

答案 1 :(得分:0)

我在您的预备模型中看到了

 self.stripe_charge_id = charge.id

但是,根据您的迁移,您没有该字段。所以我建议你在迁移中创建该字段。

我还建议使用save!代替save,以便抛出可以通过救援声明获救的错误

 begin
   charge = Stripe::Charge.create(
     :amount => total_amount,
     :currency => "gbp",
     :source => stripe_token, # obtained with Stripe.js
     :description => "Booking created for amount #{total_amount}"
    )
   self.stripe_charge_id = charge.id
   save!
 rescue Stripe::CardError => e
   errors.add(:base, e.message)
   false
end

从您的错误中,您错误地调用了条带API。您必须提供一个source参数,而不是。您可以在https://stripe.com/docs/api/ruby#create_charge

了解更多信息