Mission Impossible ? [Laravel 6 + InterventionImage + validation]

Why are you trying to validate after you process the request ?

Agreed, that was totally wrong :)

But now that I moved the conditional out of the validateTestProfile() function, into the store() method of the controller, right before TestProfile::create( $this->validateTestProfile() );, nothing seems to have changed. The validation still doesn't seem to kick in.

Okay, getting there :)

I created an additional validator function, just for the image, so that the conditional will return false in case the image is the wrong format/filesize. Here's the updated store() method:

/**
   * Store a newly created TestProfile in storage.
   *
   * @param Request $request
   *
   * @return Application|RedirectResponse|Redirector
   */
  public function store(Request $request)
  {
  if( request()->hasFile('main_picture') ) {
      
      $this->validateMainImage(); // will return false if the image is not the correct format/filesize
      
      $extension = request()->file('main_picture')->getClientOriginalExtension(); // get file extension
      $filename_to_store = time() . '_' . Str::random(40) . '.' . $extension; // filename to store
      
      // "Make" the resized image
      $resized = Image::make( request()->file('main_picture') )
                      ->resize(300, 300, function( $constraint ) {
                        $constraint->aspectRatio(); // preserve image's aspect ratio
                      });
      
      $save_resized = $resized->save('uploads/testprofiles/main_picture/' . $filename_to_store);
      
      $filename_resized = $save_resized->basename;
      
      
      $request['main_picture'] = $filename_resized;
    }
  
    TestProfile::create( $this->validateTestProfile() );
    
    return redirect( route('testprofiles.index') );
  }

  /* ... 
      ...
     ... */

protected function validateMainImage() {
    return request()->validate([
      'main_picture' => 'required|image|mimes:jpeg,jpg,png,gif|max:1024',
    ]);
  }

Now, the only thing that's left, is saving the proper filename of the image in the database, as it's still getting saved as a temporary system path, e.g. /private/var/tmp/phpPGy8PH

May i suggest to simplify it a bit ? Can you try something like that for example warning not tested

    public function store(Request $request)
    {
        $request->validate([
            'name' => 'required',
            'main_picture' => 'required|image|mimes:jpeg,jpg,png,gif',
        ]);

        $extension = request()->file('main_picture')->getClientOriginalExtension();
        $filename_to_store = time() . '_' . Str::random(40) . '.' . $extension; 

        $resized = Image::make( request()->file('main_picture') )
            ->resize(300, 300, function( $constraint ) {
                $constraint->aspectRatio();
            })->stream();        
        
        Storage::disk('local')->put($filename_to_store, (string) $resized);

        TestProfile::create([
            'name' => $request->input('name'),
            'main_picture' => $filename_to_store
        ]);

        return redirect( route('testprofiles.index') );
    }

@wingly Sir, you are a legend. Got it working on the first try (not a blind copy/paste of your code, I promise 😀 )

I was just wondering though, what's the purpose of using Storage::disk('local')when saving the processed image? How could I then get ahold of the newly-created image in a .blade file? I've already created a symlink with php artisan storage:link, can I just use the asset() helper method for that purpose?

Lol great! You can use Storage::url('file.jpg') check the docs to dig a bit deeper https://laravel.com/docs/7.x/filesystem#file-urls

That does help indeed. Thank you for that. But what if I'm storing the file under a much deeper folder structure, for instance

Storage::disk('local')->put('uploads/testprofiles/main-pictures/' . $filename_to_store, (string) $resized );

I'd then need to append the same uploads/testprofiles/main-pictures/path each and every time I'd need to get the URI of the stored images in my view files, right?

In other words,

<img src="{{ Storage::url('uploads/testprofiles/main-pictures/') . $testprofile->main_picture }}" alt="...">

Unless there's a better way of writing the path, either in the controller, or in the views, it seems to me a bit un-optimized way of using them.

Sorry for keeping on asking noob questions... 🤓

Then you simply store in your database the uploads/testprofiles/main-pictures/filename

I must be really tired. It's such a simple approach, yet it never crossed my mind. Once again, thanks so much for your help @wingly. No more noob questions from me, at least not today 😊

Have a great day (evening/night) my friend. Stay safe 🙏