Getting Error: NotReadableException in AbstractDecoder.php line 302
Asked Answered
S

2

0

I am following a turtorial on how to save photos and resize them as thumbnails using PHP, Laravel 5.1, MySQL, InterventionImage plugin, and am getting the following error when I try to add photos:

NotReadableException in AbstractDecoder.php line 302: Image source not readable

It seems that the problem is happening in the Photo code below in the makeThumbnail method. The original photo is stored in my flyers/photo folder, but the thumbnail file is never created and the photo is not stored in the database. I have followed all the steps and checked the routes, models, views controllers but everything seems to be identical to the tutorial so I am not sure what I am doing wrong.The error stops showing up if I dd() this function, but I have failed at fixing it after many attempts. The

Photo eloquent model:

<?php
namespace App;

use Intervention\Image\Facades\Image;
use Illuminate\Database\Eloquent\Model;
use Symfony\Component\HttpFoundation\File\UploadedFile;

class Photo extends Model
{
protected $table = 'flyer_photos';
protected $fillable = ['path' , 'name' , 'thumbnail_path'];
protected $baseDir = 'flyer/photos';

public function flyer(){
    // creates a new instance of a photo
    return $this->belongsTo('App\Flyer');
}

/**
 * Build a photo instance from a file upload
 */
public static function named($name){


    return (new static)->saveAs($name);

}

/**
 * Setting name, path, and thumbnail_path parameters for Photo instance
 */
protected function saveAs($name){
    //concatenate file name with current time to prevent duplicate entries in db
    $this->name = sprintf("%s-%s", time(), $name);
    $this->path = sprintf("%/%s", $this->baseDir, $this->name);
    $this->thumbnail_path =sprintf("%s/tn-%s", $this->baseDir, $this->name);

    return $this;

}

public function move(UploadedFile $file){

    // move the file to new location in flyer/photos
    $file->move($this->baseDir, $this->name);

    $this->makeThumbnail();

    return $this;
}

/**
 * Change sizing of thumbnail and save it
 */
protected function makeThumbnail() {
    //dd('error test');
    Image::make($this->path)
    ->fit(200)
    ->save($this->thumbnail_path);
}

Controller code:

<?php

namespace App\Http\Controllers;
use App\Flyer;
use Illuminate\Http\Request;
use App\Http\Requests\FlyerRequest;
use App\Http\Controllers\Controller;
use App\Http;
use App\Photo;
use Symfony\Component\HttpFoundation\File\UploadedFile;


class FlyersController extends Controller
{

/**
 * Auth checks to make sure you are logged in before making any adjustments
 */
public function __construct()
{
    $this->middleware('auth', ['except' => ['show']]);
}

/**
 * Display a listing of the resource.
 */
public function index()
{
    //
}

/**
 * Show the form for creating a new resource.
 */
public function create()
{
    flash()->overlay('Hello World', 'this is the message');

    return view('flyers.create');
}

/**
 * Store a newly created resource in storage.
 */
public function store(FlyerRequest $request)
{

    //persist the flyer
    Flyer::create($request->all());

    //flash messaging
    flash()->success('Success!', 'Your flyer has been created.');

    return view('pages.home');//temporary redirect the landing page

}

/**
 * Display the specified resource.
 */
public function show($zip, $street)
{
    //find the new flyer
    $flyer = Flyer::locatedAt($zip, $street);

    return view('flyers.show', compact('flyer'));

}

/**
 * Apply photo to the referenced flyer.
 */
public function addPhoto($zip, $street, Request $request){

    //confirmtion that the photo file will be in appropriate format types
    $this->validate($request, [
        'photo' => 'required|mimes:jpg,jpeg,png,bmp'
    ]);

    //build up our photo instance taking the file from dropzone plugin
    $photo = $this->makePhoto($request->file('photo'));

    //Save photo and associate it with the Flyer
    Flyer::locatedAt($zip, $street)->addPhoto($photo);
}

protected function makePhoto(UploadedFile $file){

    //get new photo object with current name
    return Photo::named($file->getClientOriginalName())->move($file);
}
}
Spiniferous answered 1/11, 2015 at 23:27 Comment(0)
I
0

It looks like your path is not being set correctly here:

$this->path = sprintf("%/%s", $this->baseDir, $this->name);

Shouldn't that be...

$this->path = sprintf("%s/%s", $this->baseDir, $this->name);
Incise answered 2/11, 2015 at 1:30 Comment(0)
A
0

I did the same tutorial , you should do like this

Image::make($this->path.$this->name)->resize(128, 128)->save($this->thumbnail_path.$this->name);

instead of doing this

Image::make($this->path)->fit(200)->save($this->thumbnail_path);

Addam answered 13/5, 2016 at 4:43 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.