ActivityLogger::performedOn() 必须是 Illuminate\Database\Eloquent\Model 的实例，给定 int，

Question

为什么update 不返回模型实例，有些可以帮助我。

控制器

public function cancel_bill_cash_store(Request $request){

            $sales_cancel=Sales::where('bill_number',$request->bill_number)
                    ->update(["cancel_bill" =>1,
                       "bill_status"=>$request->bill_status,

                      ]);

    if ($sales_cancel) {

                          activity('Sales')
                          ->performedOn($sales_cancel)
                          ->causedBy($user)
                          ->log('Cancelled Bill by ' . $user->name . ' with IP '.request()->ip());

                          toastr()->success(' Bill- '.$request->bill_number.' Imeshakuwa Cancelled Vizuri!, 
                          Device Zimerudi Store');
                                 return redirect()->back();
                            }
                          else{
                          toastr()->error('Error: Kuna Kitu Kimekwenda Wrong !. Otherwise check Support.');
                             return redirect()->back();
                           }
       }

Answer 1

我建议采用“分而治之”的方法。将你的函数一分为二，一个只处理更新和日志，另一个处理查询：

public function cancel_bill(Sales $sales_cancel, $bill_status) {
    $sales_cancel->cancel_bill = 1;
    $sales_cancel->bill_status = $bill_status;
    $result = $sales_cancel->save();
    if ($result) {   
        activity('Sales')
           ->performedOn($sales_cancel)
            ->causedBy($user)
            ->log('Cancelled Bill by ' . $user->name . ' with IP '.request()->ip());
    }
    return $result;
}

public function cancel_bill_cash_store(Request $request) {
    $sales_cancel=Sales::where('bill_number',$request->bill_number)
        ->first();
    if ($sales_cancel && $this->cancel_bill($sales_cancel, $request->bill_status)) {
       toastr()->success(' Bill- '.$request->bill_number.' Imeshakuwa Cancelled Vizuri!, 
                          Device Zimerudi Store');
                                 return redirect()->back();
       } else {
         toastr()->error('Error: Kuna Kitu Kimekwenda Wrong !. Otherwise check Support.');
         return redirect()->back();
    }
}

这为每个功能赋予了明确的职责，一个负责更新模型，另一个负责根据结果创建响应。

编辑：如果您有很多行需要更新，您可以这样做：

function cancel_bill_cash_store(Request $request) {
    Sales::where('bill_number', $request->bill_number)
        ->get()
        ->each(function ($sales_cancel) use ($request) {
            if ($sales_cancel && $this->cancel_bill($sales_cancel, $request->bill_status)) {
                toastr()->success(
                    ' Bill- ' . $request->bill_number . ' Imeshakuwa Cancelled Vizuri!, 
                          Device Zimerudi Store');
            } else {
                toastr()->error('Error: Kuna Kitu Kimekwenda Wrong !. Otherwise check Support.');
            }
        });


    return redirect()->back();
}

这将更新每一行并记录相应的事件。