Level 104
You need to merge the newly defined $filename variable into your $request array before you create the Product, e.g.:
$new_product = $request->merge(['filename' => $filename])->all();
Jul 25, 2017
10
Level 1
Filename not storing correctly in the database
I'm trying to upload a file but the file name doesn't save correctly in the database instead of getting something like '1500677031.jpeg' stored in the file column in the database I get this instead '/private/var/tmp/phpPw35Ox' and this causes the image to break in the view. here's my controller code:
public function store(UploadRequest $request)
{
if ($request->hasFile('product_photo'))
{
$product_photo = $request->file('product_photo');
$filename = time() . '.' . $product_photo->getClientOriginalExtension();
Img::make($product_photo)->resize(250, 250)->save(public_path('/uploads/product_photo/') . $filename);
}
$new_product = $request->all();
Products::create($new_product);
session()->flash('message', 'Product successfully added to inventory');
return redirect()->back();
}
Level 1
@tykus I get this error when I include that line Call to a member function all() on null
Level 42
I think you can try this: You should set field wise value in $new_product array like 'filename'
$new_product = array('filename'=>$filename);
Products::create($new_product);
Hope this work for you !!!
Level 1
@saurabhd thanks. but this also doesn't work. data doesn't event get persisted with this solution
Level 104
Sorry, the merge method actually returns nothing (void); so it is not chainable; do this instead:
$request->merge(['filename' => $filename]);
Products::create($request->all());
Level 1
@tykus this works now except filename in the db still = '/private/var/tmp/phpiwM0dR'
Level 104
What is the field on the database table?
Level 1
its product_photo in the database
Level 122
because you are passing the php temporary filename into the request instead of using the filename you created.
Just pass the parameters as an array to the create method instead of just throwing it the request->all() data
$product = Products::create([
'column1' => $request->field1,
'column2' => $request->field2,
'column3' => $request->field3,
'product_photo' => $filename,
]);
3 likes
Level 1
@sneapy thanks this worked for me now picture displays correctly.
Please or to participate in this conversation.