2015-10-06 69 views
0

我正在开发我的应用程序,其中用户可以接受或拒绝来自其他用户的提议,但出现问题,因为当我创建例如5个工作提议,并尝试拒绝或接受例如第三次提议,改变了第一次提议的状态,我不知道发生了什么事情?我遵循这个主题Product orders between 2 users,所以也许我理解错了什么?请帮我Rails方法工作出错

我的模型:

#user 
has_many :jobs_given, class_name: 'Job' 
has_many :offers_recived, class_name: 'Offer', through: :jobs_given, source: :offers 

has_many :offers_made, class_name: 'Offer' 
has_many :jobs_take, class_name: 'Job', through: :offers_made, source: :job 

#job 
belongs_to :seller, class_name: 'User', foreign_key: :user_id 
has_many :offers 
has_many :buyers, class_name: 'User', through: :offers 

#offer 
belongs_to :job 
belongs_to :buyer, class_name: 'User', foreign_key: :user_id 
has_one :seller, class_name: 'User', through: :job 

def accept 
    self.status = true 
    save 
end 

def reject 
    self.status = false 
    save 
end 

我的控制器:

def create 
    @job = Job.find(params[:job_id]) 
    @offer = @job.offers.create(offer_params) 
    @offer.user_id = current_user.id 

    if @offer.save 
     redirect_to @job 
    else 
     redirect_to :back 
    end 
end 

def accept 
    offer = current_user.offers_recived.find(params[:id]) 
    offer.accept 
    flash[:success] = "Offer accepted!" 
    redirect_to :back 
end 

def reject 
    offer = current_user.offers_recived.find(params[:id]) 
    flash[:notice] = "Offer rejected!" 
    offer.reject 
    redirect_to :back 
end 

private 

def offer_params 
    params.require(:offer).permit(:pirce_offer, :status, :job_id, :user_id) 
end 
end 

我的观点:

#offers/form 
= simple_form_for([@job, @job.offers.build]) do |f| 
= f.input :pirce_offer 
= f.button :submit 

#offers/offer 
%p= offer.pirce_offer 

= link_to "Accept this offer", accept_job_offer_path(offer, @job), method: :post 
= link_to "Reject this offer", reject_job_offer_path(offer, @job), method: :post 

#jobs/show 
= render @job.offers 

= render 'offers/form' 

@EDIT

Parameters: {"authenticity_token"=>"YB0i2/JmroyZBAOOyCom5NDwjxrPsGRFV6Uucnp1dNKmIGgED3H4cB7f2fnutRKCh2OjyQICQghM4Zoy13zZrw==", "job_id"=>"5", "id"=>"1"} 
User Load (0.1ms) SELECT "users".* FROM "users" WHERE "users"."id" = ? ORDER BY "users"."id" ASC LIMIT 1 [["id", 1]] 
Offer Load (0.1ms) SELECT "offers".* FROM "offers" INNER JOIN "jobs" ON "offers"."job_id" = "jobs"."id" WHERE "jobs"."user_id" = ? AND "offers"."id" = ? LIMIT 1 [["user_id", 1], ["id", 1]] 
(0.1ms) begin transaction 
SQL (0.1ms) UPDATE "offers" SET "status" = ?, "updated_at" = ? WHERE "offers"."id" = ? [["status", "f"], ["updated_at", "2015-10-06 08:12:40.425741"], ["id", 1]] 
+0

嗯,看来你有一些调试的事情。什么'params'到达你的'accept'方法?什么生成它们?等等。 –

+0

我将它添加到我的问题 – exerq

回答

2

改变了第一次报价

永远记住,在计算的状态,这是你的故障如果事情不能正常工作。

系统是非常灵活和多变的程度,除非你有系统性的问题,没有理由不起作用。


ID

的问题是几乎可以肯定做的id你传递:

"job_id"=>"5", "id"=>"1" 

我不知道你的数据是什么样子,但是这是由此创建link_to

= link_to "Accept this offer", accept_job_offer_path(offer, @job), method: :post 

您的accept_job_offer_path会 - IMO - 分别具有[:job_id][:(offer)id]的参数。你以相反的方式发送它们。

试试这个:

= link_to "Accept this offer", accept_job_offer_path(@job, offer), method: :post 
= link_to "Reject this offer", reject_job_offer_path(@job, offer), method: :post 

这是假设你的数据结构是正确的。


数据

还有一些其他的问题,我可以看到:

current_user.offers_recived.find(params[:id]) 

你打电话current_user.offers_received不表明工作其收到的优惠。你当然希望在这个过程中包含这份工作?

此外,你可以干起来你的行动颇有几分:

#config/routes.rb 
resources :jobs do 
    match :offer, action: :update, via: [:post, :delete] #-> url.com/jobs/:job_id/offer 
end 

#app/controllers/jobs_controller.rb 
class JobsController < ApplicationController 
    def offer 
     @job = Job.find params[:job_id] 
     offer = current_user.offers_recived.find(params[:id]) 
offer.accept 
     if request.post? 
     offer.accept 
     elseif request.delete? 
     offer.reject 
     end 
    end 
end 
2

假设你已嵌套:offers嵌套在:jobs嵌套资源您的路径在您的"Accept this offer""Reject this offer"链接错误的方式。

这意味着你发送报价的id作为params[:job_id](除非它不会有一个id但因为你才刚刚建立了它),然后params[:id]@job的ID。这就是为什么它更新了错误的报价 - 这是因为它正在更新与ID匹配的报价@job

当你刚刚在表单中建立它时,你如何通过id找到要约可能是沿着这条路的下一步。