Laravel attempt to read property "id" on null''

Question

0.00/5 (No votes)

See more:

I try to make a crud in Laravel. Im done with the create and read functions but update is still a huge issue and while it does work it wont update at all the message i got is

Laravel Attempt to read property "id" on null''

Here is my updaterequest

{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        return true;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        $bedrijf = $this->route('bedrijf');
        return [
            'name' => 'string|min:2|max:75|required|unique:bedrijven,name,'.$bedrijf->id
        ];
    }
}

My updatecontroller

public function update(BedrijfUpdateRequest  $request, Bedrijf $bedrijven)
   {
       $bedrijven->name = $request-> name;
       $bedrijven->save();
       return redirect()->route('bedrijven.index')->with ('meassage', 'bedrijven gewijzigd');

   }

the route

Route::get('admin/bedrijven/update',[BedrijfController::class, 'update'])
->name('bedrijven.update');

the page

@extends('layouts.layout')
@section ('content')
<h2 class="my-4 text-4xl font-semibold dark:text-gray-400">
				Bewerk Bedrijven
			</h2>

@if($errors-> any())
 @foreach ($errors->all() as $error)
<li>{{$error}}</li>

 @endforeach
 @endif

<div class="w-full max-w-xs">
  <form class="bg-white shadow-md rounded px-8 pt-6 pb-8 mb-4"
  action="{{route('bedrijven.update', ['bedrijven'=>$bedrijven->id])}}" method="POST">
  @method('PUT')    
  @csrf
    <div class="mb-4">
      <label class="block text-gray-700 text-sm font-bold mb-2" for="name" >
        Name
      </label>
      <input class="shadow appearance-none border rounded w-full py-2 px-3 text-gray-700 leading-tight focus:outline-none focus:shadow-outline"  @error('name') border-red-500 @enderror 
      id="name" type="text" placeholder="name" name="name" value = "{{ old('name', $bedrijven->name)}}">
</div>
<div class="mb-4">
      <label class="block text-gray-700 text-sm font-bold mb-2" for="body" >
        Description
      </label>
      <input class="shadow appearance-none border rounded w-full py-2 px-3 text-gray-700 leading-tight focus:outline-none focus:shadow-outline"  @error('name') border-red-500 @enderror 
      id="body" type="textfield" placeholder="body" name="body" value ="{{old('body',$bedrijven->body)}}">
</div>
    <div class="flex items-center justify-between">
      <button class="bg-blue-500 hover:bg-blue-700 text-white font-bold py-2 px-4 rounded focus:outline-none focus:shadow-outline" 
      type="submit">
       Voeg toe
      </button>
</div>
</form>
@endsection

What I have tried:

I tried to change the variables, i recreated the page twice

Posted 24-Jun-22 3:11am

shady morsi

Updated 28-May-23 10:26am

Add a Solution

Comments

Richard MacCutchan 24-Jun-22 10:56am

"I tried to change the variables, i recreated the page twice"
You need to find the line of code that causes the error. Use your debugger to step through the part of the code where it occurs. Once you get to the failing line you should be able to see which reference is the one that is null.

2 solutions

Solution 3

In my case, in my model I had defined a relationship as follows:

PHP

public function nationality(){
    return $this->belongsTo(Country::class, 'nationality', 'countryid');
}

After trying many things and having a nearly-identical relationship working, I changed the name of the method to nation

PHP

public function nation(){
    return $this->belongsTo(Country::class, 'nationality', 'countryid');
}

Posted 28-May-23 10:26am

Member 16016447

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Dave Kreskowiak · Accepted Answer · 2022-06-24T06:17:00

What this means is that your code is assuming an operation worked and returned a valid object, when it didn't.

For example:

... $this->route('bedrijf');

Did route find the 'bedrijf' index string? Did that index hold a valid object? Maybe, maybe not. Your code never checks to see if what was returned was valid before it tried to use (call a method or property) on whatever was returned.

Never assume a lookup like that worked.