Spatie Medialibrary

[DeveloperOnCall]

Description

In an effort to fix my hostname not identified issue I created a new Laravel install using the directions:
https://laravel-tenancy.com/docs/hyn/5.3/spatie-medialibrary

---

Actual behavior

When using the suggested method to attach media to a user -

// Switch the environment to use first hostname
        $environment = app(\Hyn\Tenancy\Environment::class);
        $hostname = Hostname::find(2);
        $environment->hostname($hostname);

// Retrieve your user model.
      $user = App\User::find(1);

// Add a media from url and attach it to your model.
    $user->addMediaFromUrl('http://url-to-your-media-to-attach.png')->toMediaCollection();

Seems to work without any problem, however - the image is placed in the storage/app/public/1 folder.

Expected behavior

This should put the new file into the tenant's disk.

---

Information

• hyn/multi-tenant version: 5.3
• laravel version: 5.7
• database driver and version: MySQL 5.x
• webserver software and version: Apache
• php version: 7.1.x

---

tenancy.php config

webserver.php config

---

Error log

[luceos]

$environment->hostname($hostname);

This does not set the tenant, this does:

$environment->tenant($hostname->website);

[DeveloperOnCall]

Modified to set the tenant according to your comment..

// Switch the environment to use first hostname
        $environment = app(\Hyn\Tenancy\Environment::class);
        $hostname = Hostname::find(2);
        $environment->hostname($hostname);
        $environment->tenant($hostname->website);


// MEDIA

// Retrieve your user model.
        $user = User::find(1);

// Add a media from url and attach it to your model.
        $user->addMediaFromUrl('http://profilepicturesdp.com/wp-content/uploads/2018/06/geek-profile-pictures.png')->toMediaCollection();

Image is still saving to the app/public/ folder.

Thanks

[luceos]

@bkintanar ?

[bkintanar]

Will check on this later this weekend. Thanks for reporting it.

[bkintanar]

@DeveloperOnCall

Here's my take on this. You need to override the PathGenerator.php. You can read more about it here: https://docs.spatie.be/laravel-medialibrary/v7/advanced-usage/generating-custom-urls#generating-custom-urls

Here's what I did with my setup. What you need to pay attention is the getPath method where we use the db name as part of the URL.

Afterwhich, you need to update the path_generator in the medialibrary.php config file to point to the file you just created below.

use Spatie\MediaLibrary\Models\Media;
use Spatie\MediaLibrary\PathGenerator\PathGenerator as SpatiePathGenerator;

class PathGenerator implements SpatiePathGenerator
{
    /*
     * Get the path for the given media, relative to the root storage path.
     */
    public function getPath(Media $media) : string
    {
        $prefix = $media->getConnection()->getDatabaseName();

        return $prefix.'/'.$media->id.'/';
    }

    /*
     * Get the path for conversions of the given media, relative to the root storage path.
     */
    public function getPathForConversions(Media $media) : string
    {
        return $this->getPath($media).'/conversions/';
    }

    /*
     * Get the path for responsive images of the given media, relative to the root storage path.
     */
    public function getPathForResponsiveImages(Media $media): string
    {
        return $this->getPath($media).'/responsive-images/';
    }
}

Note to self: add this to the docs. 😄

[bkintanar]

@DeveloperOnCall the above code still doesn't store it to the tenant disk, as that still needs more investigation on my end, what this does it stores the image to a subdirectory using the tenant connection name.

[lengletremy]

To secure a little more and not give the name of the database while separating the sites.

    /*
     * Get the path for the given media, relative to the root storage path.
     */
    public function getPath(Media $media): string
    {
        $prefix = $media->getConnection()->getDatabaseName();
        $website = Website::where('uuid', $prefix)->first();
        $hostname = preg_split('/(?=\.[^.]+$)/', Hostname::findOrFail($website->id)->fqdn);
        return $hostname[0] . '/' . $media->id . '/';
    }

[agustindidiego]

If you change medialibrary.disk_name config from 'public' to 'tenant' it works as expected. No need to override the PathGenerator

[jtomek]

If you change medialibrary.disk_name config from 'public' to 'tenant' it works as expected. No need to override the PathGenerator

This is gold.

[lengletremy]

@agustindidiego I may have missed some things or I do not master the package enough but I get this error using your method.
Storage path /var/www/atomsit/storage/app/tenancy/tenants/226878838bb94447b4db897f032d77c5 is not part of public path /var/www/atomsit/public

[bkintanar]

@RERE110 try running php artisan storage:link

[luceos]

Can someone summarize the findings of working with media library if possible and PR that into our docs somewhere? I think that would help other people out and we can move from there with possible integration paths.

[mauxtin]

What worked for me:

tenancy.disk => null,
medialibary.disk_name=>tenant,
filesystems.default => s3

// EventServiceProvider@boot
Event::listen(Switched::class, function (Switched $event) {
            config([
                'filesystems.disks.tenant' =>
                    [
                        'driver' => 's3',
                        'key'    => env('AWS_KEY'),
                        'secret' => env('AWS_SECRET'),
                        'region' => env('AWS_REGION'),
                        'bucket' => env('AWS_BUCKET'),
                        'url'    => env('AWS_URL'),
                        'root'   => env('AWS_TENANTS_ROOT').'/'.$event->website->uuid
                    ]
            ]);