Database table wont be updated

@Tray2 yes it does

@Danny971 Good then move the dd to this

dd($profilefamilydemographics);

And see if it finds any record, my guess is that it doesn't.

You really should use findOrFail() instead of just find;

Last tips, use camel case for your variable names, makes it much easier to read.

$profileFamilyDemographics

@Tray2 dd($profilefamilydemographics); does return data

@Snapey the identifier is to see which item is being change . I havent put validation as yet

@Danny971 yes I ser that, Im struggling with WHY?

@Snapey i put the identifier to be able to identify which item is being updated

@Snapey The identifier is used to identify the specific field that is being edited within the modal. It helps determine which field's value needs to be updated in the controller.

In the code, the identifier is passed as a data attribute (data-identifier="{{ $record->id }}") to the "Edit" button in the table. When the "Edit" button is clicked, the JavaScript code retrieves the identifier value and assigns it to the hidden field editId in the modal form.

Later, when the form is submitted, the SaveFamDemoEdits route is called with the id parameter, which represents the identifier of the field being edited. This identifier is then used in the controller to determine which field's value should be updated based on the switch statement:

public function SavefamdemoEdits(Request $req, $id){
 
        
         $ProfileFamilyDemographics = FamilyDemographicsModel::find($id);

       

         if($ProfileFamilyDemographics) {

            $identifier = $req->input('identifier');

            $firstnameCorrection  = $req -> input('name');
            $lastnameCorrection  = $req -> input('surname');
            $ageCorrection  = $req -> input('age');
            $genderCorrection  = $req -> input('gender');
           

            switch($identifier){
                case 'firstname':
                    $ProfileFamilyDemographics -> firstname = $firstnameCorrection;
                    break;
                case 'lastname':
                    $ProfileFamilyDemographics ->lastname = $lastnameCorrection;
                    break;
                case 'age' :
                    $ProfileFamilyDemographics -> age = $ageCorrection;
                    break;
                case 'gender':
                    $ProfileFamilyDemographics->gender = $genderCorrection;
                    break;
            }

            if($ProfileFamilyDemographics){
                $ProfileFamilyDemographics ->save();
             
                session() ->flash('notification', ['type' => 'success', 'message' => 'Profile updated  successfully']);
            } else {
             session()->flash('notification', ['type' => 'error', 'message' => 'An error occured while updating the profile ']);
            }
            return redirect() -> back();
            }
         }

@Danny971 are you sure your identifier is not null?

@krisi_gjika nope its not null but i figure out whats wrong

@Snapey because you can only edit one item at a time at a time .lets say we want to edit

Jon doe . we click edit next to jon doe and the modal appears with FirstName Jon last Name doe Age 8 gender Male and you can change any fields or all at once and click save cganges it suppose to update the db but in my case it doesnt

@Danny971 but you are trying to use identifier so that only one field gets updated?

switch($identifier){

@Snapey I used the identifer to get the id of the record we are trying to update

@Danny971 So many things.

Consider this update method.

public function update(Game $game, GameFormRequest $request)
{
  $game->update($request->validated);

  return redirect(route('games.index'));
}

It uses route model binding to fetch the record from the database, and a form request for the validation.

The route would look like this.

Route::put('games/{game}', [GamesController::class, 'update')->name('games.update');

And the form request class like this

class GameFormRequest extends FormRequest
{
  public function rules(): array
  {
    return [
      'title' => ['required', 'string'],
      'release_year' => ['required'],
      'blurb' => ['required', 'string'],
      'genre_name' => ['required', 'string'],
      'format_name' => ['required', 'string'],
      'platform_name' => ['required', 'string'],
    ];
  }
}

You really don't need to write any custom code like you have done, I suggest reading the docs on Route model binding, form requests and validation.

@Tray2 no luck

@Danny971 There is no shortcut. You actually have to learn this stuff.

@Danny971 Have you watched this free series https://laracasts.com/series/laravel-8-from-scratch if you haven't then I highly suggest that you do, it will tell you almost everything you need.

@Snapey yeah you're right . I knew this stuff been programming for a while and actually have a bachelors degree in com sci. I just run into a problem i posted it here. I actually understood the problem and know whats causing it. this is what causing it not to update ``` data-identifier="{{ $record->id }}"

. I am currently working on solving it . I changed it to data-identifier="firstname" and it worked it changed the Firstname however I am figuring out how to get the remaining three on board , namely : lastname,gender and age . I dont want to created a unique edit button for each based on the requirements so this is where I am now

@Danny971 any normal non computer science degree developer would present the form with the current values and allow the user to change whatever fields they need to change, the save ALL the fields together

Your advanced training must tell you there is a better ux way where the user has to indicate which field they want to change and then in the backend you use that flag to know which field to write ?

@Snapey so you saying that I should have an edit button for firstname, lastname,age , gender? IF yes then I would have accomplish that already . I am trying to have one edit button for each record

@Danny971 you need to recognise the difference between record and field.

This code;

            switch($identifier){
                case 'firstname':
                    $ProfileFamilyDemographics -> firstname = $firstnameCorrection;
                    break;
                case 'lastname':
                    $ProfileFamilyDemographics ->lastname = $lastnameCorrection;
                    break;
                case 'age' :
                    $ProfileFamilyDemographics -> age = $ageCorrection;
                    break;
                case 'gender':
                    $ProfileFamilyDemographics->gender = $genderCorrection;
                    break;
            }

is trying to decide which of the FIELDS to update based on identifier

The RECORD that you want to edit is already passed into the controller as $id because you have a modal for every single row in your view (very inefficient)

You seem to have grabbed a bunch of ideas from different places and smushed them into one solution

@Snapey a field refers has data about one aspect of the table , for example firstname and a record contains info about a particular person example ( firstname, lastname, age,gender ).

your statement : you have a modal for every single row in your view (very inefficient). I only have one modal for that. the modal has text field to edit first name ,last name, gender, age

your other statement: You seem to have grabbed a bunch of ideas from different places and smushed them into one solution. Nope i did not