My own File model conflicting with Illuminate's file facade

use \Illuminate\Support\Facades\File as IlluminateFile 

@robertvrabel I’d ask why you’re trying to interact with the file system from within an Eloquent model?

@martinbean Where would the proper place for that go? I debated putting it into a Controller. But I want to use that code from multiple different controllers. So it didn't make sense for me to initiate my FilesController class within another controller. Wasn't sure where it would go.

I'm pretty new to laravel, so if you have any suggestions on where code like this should go so I can reuse it all over would be great.

If its helpful, here is the code I have within the App\File model to save the file to the server and save it to the DB.

    /**
     * Save the file, so the reference location is in one spot.
     *
     * @param UploadedFile $file
     * @return int
     */
    public function saveFile(UploadedFile $file)
    {
        // Format filename
        $this->attributes['filename'] = $this->formatFileName($file->getClientOriginalName());

        // Save to storage
        Storage::disk($_SERVER['APP_ENV'])->put($this->dir . $this->attributes['filename'], IlluminateFile::get($file));

        // Return false if the Save Fails
        if(!$this->save()){
            return false;
        }

        // File saved, return true
        return true;
    }

I agree, if your going to reuse the method it definitely doesn't belong in the controller.

If the file is under the app directory and namespace it differently (as you mentioned different folders) it will not be an issue. Just be careful in your use statement that you reference the correct class. Also you can import the illuminate file by simply use File;

@robertvrabel It’s probably best being wrapped up in a service class. You would instantiate the service class in your controller, pass the request parameters to it, and then it would handle both the file uploading and saving to the database via the model.

public function create(Request $request)
{
    $service = new CreateFileService;
    $service->execute($request->all());
}

class CreateFileService
{
    public function execute(array $data)
    {
        // Handle file upload
        // Save record to database
    }
}

This way, your logic’s wrapped up in a single class that can then be re-used.