Level 14
I did $table->string('image')->nullable(); in my migration
Apr 7, 2018
13
Level 14
Image store
Hi, I have image upload on my website, I',m trying to make it possible to have it nullable. Any ideas?
This is my store function:
public function store()
{
$this->validate(request(), [
'title' => 'required|unique:threads|max:255',
'body' => 'required|min:20',
'image' => 'image|nullable
]);
Thread::create([
'title' => request('title'),
'user_id' => Auth::user()->id,
'body' => request('body'),
'image' => request()->file('image')->store('frettir', 'public'),
]);
return redirect('/stjornbord')->with('flash', 'Fréttin var búin til!');
}
And this is my migration:
Schema::create('threads', function (Blueprint $table) {
$table->increments('id');
$table->integer('user_id');
$table->string('title');
$table->text('body');
$table->string('image')->nullable();
$table->timestamps();
});
Level 50
but you know it should 'image' => 'image|nullable' also work
Level 14
I don't understand :/ I have both 'image' => 'image|nullable' in my validation and also $table->string('image')->nullable(); in my migration
Level 50
try this
'image' => request()->file('image')->storeAs('uploads', 'frettir'),
Level 14
Now I'm getting Call to a member function storeAs() on null
Level 50
enctype="multipart/form-data" do you have this in your form?
Level 50
Controller strore method()
you can use this code. it is good idea to create methods for uploading image. call uploadImage method and pass the image
$thread = Thread::create([
'title' => request('title'),
'user_id' => Auth::user()->id,
'body' => request('body'),
]);
$thread->uploadImage($request->file('image'));
return redirect('/stjornbord')->with('flash', 'Fréttin var búin til!');
your Thread model . if image doesn't you just return nothing. or if image already exists first will be deleted. then image will always have unique name. all your images will be stored in public/uploads directory
Thread.php
public function uploadImage($image)
{
if ($image == null) {
return;
}
$this->removeImage();
$filename = str_random(10) . '.' . $image->extension();
$image->storeAs('uploads', $filename);
$this->image = $filename;
$this->save();
}
public function removeImage()
{
if ($this->image != null) {
Storage::delete('uploads/' . $this->image);
}
}
1 like
Level 122
put nullable first in validation. I think the rules are executed in order
Level 14
@rin4ik It did work to create a public function inside Thread model but I post an error message but still save the Thread to the database.
Undefined variable: thread
Level 14
@Snapey It didn't work
Level 50
@hjortur17 show your code please. you shouldn't get that error
Level 6
The problem is because you are trying to call a method that belongs to the File class in a null variable.
Try this when you create the thread.
Thread::create([
'title' => request('title'),
'user_id' => Auth::user()->id,
'body' => request('body'),
'image' => $request->hasFile('image') ? request()->file('image')->store('frettir', 'public') : null,
]);
Level 14
Thanks, everybody but @elcultivo had the right answer!
Please or to participate in this conversation.