DOMPDF and loading routed images

I added the code to make things more clear.

Absolutely nobody here that has an idea on how so tackle this? Or am i the only one to ever load an image from storage to a pdf through an auth route?

@Qlic your problem is you put your code inside your route closure and attach middleware to it. Instead create a separate class or helper function that you can call from your route and your controller..

function getImage($foo, $bar, $whatever)
{
    // image getting code
    return $image;
}

Route::group(['middleware' => 'myMiddleware'], function() {

    Route::get('photo/{lead}/{type}/{name}/{plain?}',  ['as' => 'inventory.photo', function ($lead, $type, $name) {

        return getImage($foo, $bar, $whatever);

    }]);

}]);

{!! Html::image(route('inventory.photo', [getImage($foo, $bar, $whatever)]), 'Photo alt name') !!}

This is not a working code, just an example to show that if you use helper functions you can re-use your code in different parts of your application.

@SaeedPrez thank you for your help, i tried it using http://stackoverflow.com/a/32772686/2952393 as a guide. It's working for the images on the site itself, but unfortunally it's still not displaying the images inside the pdf.

Routes:

use App\Helpers\Helper;

Route::group(['middleware' => 'myMiddleware'], function() {

    Route::get('photo/{lead}/{type}/{name}',  ['as' => 'inventory.photo', function ($lead, $type, $name)  {

        return Helper::displayPhoto($lead, $type, $name);

    }]);

}]);

Helper:

<?php

namespace App\Helpers;

use Image;

class Helper
{
    public static function displayPhoto($lead, $type, $name)
    {
        switch($type) {
            case 1:
                $folder = 'roof';
                break;
            case 2:
                $folder = 'meterboard';
                break;
            case 3:
                $folder = 'access';
                break;
            case 4:
                $folder = 'converter';
                break;
            case 5:
                $folder = 'cable';
                break;
        }

        return Image::make(storage_path() . '/app/photo/'. $lead .'/'. $folder .'/'. $name)->response();
    }
}

Template:

<img src="{{ Helper::displayPhoto($foo, $bar, $bla) }}" alt="Test" />

Config\app.php:

'Helper'    => App\Helpers\Helper::class, // added to alisasses

composer.json:

"autoload-dev": {
        "classmap": [
            "tests/TestCase.php",
            "app/Libraries",
            "app/Helpers"
        ]
    },

Also did dump-autoload through ssh on the server.

After doing some further testing, here's what i found:

//1) From a regular template, calling the image route does return an image
{!! Html::image(route('inventory.photo', [$foo, $bar, $bla])) !!}  // shows image as expected

//2) From the same regular template, calling the image method does not return an image, or at least not as expected.
<img src="{{ Helper::displayPhoto($foo, $bar, $bla) }}" /> // src remains empty, however, when i place a return 'test'; inside the method, src does show test, so at least i know it's inside the method.

{{ dd(Helper::displayPhoto($foo, $bar, $bla)) }} // shows me image data

@Qlic I'm not really sure what error message you get (if any) or what the Image class is and why you need to use it. Can't you simply use something like..

$photo = '/app/photo/'. $lead .'/'. $folder .'/'. $name; 
return Storage::get($photo);

Also, have you checked what DOMPDF wants? I mean does it want the actual image or does it only want a path to the image?

@SaeedPrez i don't get any error messages, also i'm testing the method directly inside a regular template view to make sure wit works without a call to the route, so this is even before it's called in DOMPDF.

Also i tried the strorage get, same story, the img src remains empty, but when i diedump the call, it does contain data.

<?php

namespace App\Helpers;

use Image;
use Storage;

class Helper
{
    public static function displayPhoto($lead, $type, $name, $plain=null)
    {
        switch($type) {
            case 1:
                $folder = 'roof';
                break;
            case 2:
                $folder = 'meterboard';
                break;
            case 3:
                $folder = 'access';
                break;
            case 4:
                $folder = 'converter';
                break;
            case 5:
                $folder = 'cable';
                break;
        }

        if ($plain) {
            return Storage::get('photo/'. $lead .'/'. $folder .'/'. $name);
        }

        return Image::make(storage_path() . '/app/photo/'. $lead .'/'. $folder .'/'. $name)->response();
    }
}

Had the same problem. Get your route out of the route group where you use a middleware with auth.

@DivDax well yes, that works, however than everyone can still access my images, which is exactly what i want to prevent.

@Qlic as I wrote in my last reply, I believe DOMPDF wants the path to the image, not the content of it.

<img src="/path/to/image.jpg" alt="Test" />

well yes, that works, however than everyone can still access my images, which is exactly what i want to prevent.

Are you trying to get the image through a route? I thought that was the first problem we tackled, that you shouldn't do that and instead use helpers..

@SaeedPrez i read that, but i'm currently not even testing it on DOMPDF, i wanted to see wether or not calling both the route and method worked, so i created a plain blade template, which i call through a controller, so without DOMPDF.

When both of those work, than i'll be able to shift my focus to getting it to work in DOMPDF aswell.

• EDIT: It's working now, i altered my method to return base64 encoded data, and this works in both a regular template ánd in the DOMPDF template, so in the end, SaeedPrez you helped me alot by changing the image response to within a helper function.

<?php

namespace App\Helpers;

use Image;
use Storage;

class Helper
{
    public static function displayPhoto($lead, $type, $name, $plain=null)
    {
        switch($type) {
            case 1:
                $folder = 'roof';
                break;
            case 2:
                $folder = 'meterboard';
                break;
            case 3:
                $folder = 'access';
                break;
            case 4:
                $folder = 'converter';
                break;
            case 5:
                $folder = 'cable';
                break;
        }

        if ($plain) {
            return Image::make(storage_path() . '/app/photo/'. $lead .'/'. $folder .'/'. $name)->encode('data-url');
        }

        return Image::make(storage_path() . '/app/photo/'. $lead .'/'. $folder .'/'. $name)->response();
    }
}

Now i only have to add a fourht parameter to the images i want back as base64 encoded, and it works like a charm.

@Qlic Alright, glad it works.

The lesson here is to never put code in routes or in controllers, because it's not their responsibility and the code will not be reusable in other parts of your application.

Edit: @Qlic I meant alright, not already ☺