Есть несколько способов сделать это. Это зависит от того, чего вы пытаетесь достичь.
Метод 1, Простой:
Если вы хотите просто обновить модель обучения с помощью всего отправленного (а затемпроверьте потолок), вы можете автоматизировать большую часть этого с привязкой модели маршрута.
В вашем web.php :
Route::put('training/{training}', 'TrainingController@update');
Затем в вашем TrainingController :
public function update(Request $request, Training $training){
if($this->doCeilingCheck($training))
$training->update($request->all());
else{ // return ceiling met }
Способ 2, не привязка :
Ваш маршрут будет таким же, как сейчас, и ваша функция больше не будет иметь введенную модель, но будет иметь аналогичный поток.
В вашем TrainingController :
public function update(Request $request, $id){
$training = Training::find($id);
if($this->doCeilingCheck($training)){
$training->update($request->all());
}else{ // return ceiling met }
Метод 3, поля фильтра:
Если вы не хотите передавать каждую переменнуюПосле обновления к вашей обновленной модели тренировок вы можете отфильтровать то, что вы хотите, и просто сохранить, как вы делали ранее. Это потенциально то, что вызывает у вас проблемы. Обновляете ли вы обучение ученика А пользователем, который не является учеником А? Таким образом, возможно, изменив fk_student на неправильного ученика, а также вызвав потолок, достигнутый из-за неподходящего ученика?:
Внутри вашего TrainingController после прохождения проверки if для потолка:
$trainings->date_seance = $request->get('date_seance');
$trainings->hour_start = $request->get('hour_start');
$trainings->hour_end = $request->get('hour_end ');
// Maybe you don't want to allow change to FK - just remove this next line
// $trainings->fk_student = $request->get('fk_student ');
$trainings->save();
Проверка потолка :
Я переместил это в отдельную функцию в том же контроллере, просто чтобы это немного легче понять. Вам не нужно этого делать - вы можете оставить код в update(), если хотите. Ключ в том, что нам нужно проверить, заплатил ли СТУДЕНТ НА ЭТОМ ОБУЧЕНИИ слишком много тренировок. Итак, мы хотим убедиться, что использует студента из рассматриваемого курса обучения ($training->fk_student), не студент, заполняющий форму ($request->get('fk_student')).
Опционально вниз в вашем TrainingController :
protected function doCeilingCheck($training){
$thisStudentsTrainings = Training::where('fk_student', $training->fk_student)->get();
$thisStudentsPayments = Payment::where('fk_student', $training->fk_student)->get();
if(count($thisStudentsTrainings) >= count($thisStudentsPayments) * 5) {
return false
}else{
return true;
}
}
Редактировать :
У вас есть жесткая проверка, чтобы проверить,студент коснулся своего потолка для # тренировок и платежей. Это будет хорошо работать для вашего store() метода, потому что количество будет меняться после добавления нового обучения. То есть вы еще не сделали 2-ю тренировку по методу store() до тех пор, пока не пройдете проверку if, поэтому потолок не достигнут. Однако в методе update() предел обучения может уже достигнут, и, таким образом, вы можете достичь ложного потолка , даже если вы не добавляете новое обучение или не изменяете сумму платежей илитренировки любым способом . В программном плане - вы можете обойти это, допустив +1 тренировку против вашего нормального потолка:
// Instead of the 5 you have in your code:
if(count($thisStudentsTrainings) >= count($thisStudentsPayments) * 6)
Однако, это хакерская, и с логической точки зрения, я бы спросил себя, в чем смысл этой проверки во время обновление ? Студент не меняет количество тренингов, они не меняют количество платежей. Таким образом, проверка, чтобы увидеть, если они находятся за потолком, вероятно, следует игнорировать при обновлении. Они просто меняются как на одинаковые числа - они меняют детали курса, а НЕ их количество. Я предлагаю полностью удалить эту проверку из метода update(), чтобы сделать эту работу, как вы и планировали.