В чем странность с загрузкой изображений?

Question

[Twitt]

Создал модель Image для загрузки файла в нее, вот ее код:

class Image
{
    public $file;
    
    public function hash() {
        return $this->file->hashName();
    }
    
    public function saveFile() {
        $name = $this->hash();
        if($this->file->move(public_path() . '/imgs', $name)) {
            return $name;
        }
    }
}

В контроллере мне нужно принять из реквеста этот файл, запихнуть в модель Image, в ней вызвать метод saveFile(), и передать другой модели для сохранения в бд название загруженной картинки (хэшированное, конечно же).
Оно то получается, только не так как надо..
В папку imgs загружается действительно хэшированное название файла, а в бд сохраняется то, что лежит в папке temp, почему то:

D:\OServer\OpenServer\userdata\temp\php1BD6.tmp

Экшн контроллера:

public function store(JournalStoreRequest $request) {
        $journal = new Journal;
        $image = new Image;
        $image->file = $request->file('image');
        $journal->image = $image->saveFile();
        if($journal->fill($request->all()) && $journal->save()) {
            redirect()->route('journals.index');
        }
    }

Почему в бд сохраняется не хэшированное название? При том, что здесь по идее как должно передаваться как раз это, хэшированное значение

$journal->image = $image->saveFile();

Answer 1

[d-virt]

Заменить на:

if($journal->fill($request->except('image')) && $journal->save()) {

И НЕ используйте (но осмысленно можно) $request->all()

Comments

[Twitt]

Точно, сработало! Спасибо большое
Не использовать в плане с загрузкой изображений, или вообще не использовать?

[d-virt]

Igor: Отметь, пожалуйста, как решение и есть кнопка мне нравится. В плане, $request->all() лучше вообще не использоваться, т.к. это не безопасно если в моделе не организованны fillable. Лучше всего использоваться $requset->only() или $request->except().
НО $request->all() можно использовать если Вы точно в этом уверены и у Вас в моделе прописаны fillable поля.
P.S. с $request->except() тоже надо быть аккуратным.

[Twitt]

d-virt: да, я успел уже отметить решением, понял, спасибо))
P.S. а если через фасад делать, ::Create (или Save, не помню ибо не пользовался еще), это тоже самое?

[d-virt]

Igor: Правильно ли я понимаю, что Вы имеете введу Model::create() or Model::save() ?

[Twitt]

d-virt: да, я видел что делали так в одних уроках как Model::create(), а в других туториалах делали как я. и я вот не знаю, на каком варианте остановиться, и отличаются ли они

[d-virt]

Можете спокойно делать вот так:

// создание модель
$model = Model::create($request->only([.....]));

// или так 
$model = new Model($request->only([....]));
$model->save();

// Редактирование модели 
$model = Model::findOrFail($id);
$model->fill([...])->save();

[d-virt]

Igor: Но я бы лучше сделал вот так:

class Image
{
    /**
     * @var UploadedFile
     */
    public $file;

    /**
     * Image constructor.
     * @param UploadedFile $file
     */
    public function __construct(UploadedFile $file)
    {
        $this->file = $file;
    }

    /**
     * @return string
     */
    public function hash() {
        return $this->file->hashName();
    }

    /**
     * @return null|string
     */
    public function saveFile() {
    
        $name = $this->hash();
    
        if($this->file->move(public_path() . '/imgs', $name)) {
            return $name;
        }
        
        return null;
    }
}

/**
 * @param JournalStoreRequest $request
 */
public function store(JournalStoreRequest $request) {

    // надеюсь что в валидации стоит поле image как обязательный и проверка на файл идет
    $image = new Image($request->file($this->getImageParamName()));
    $journal = new Journal(
        array_merge(
            $request->only([ '', '' ]),
            [ 'image' => $image->saveFile(), ]
        )
    );

    $journal->save() ? redirect()->route($this->redirectRoute()) : abort(505);
}

/**
 * @return string
 */
protected function redirectRoute() {
    return 'journals.index';
}

/**
 * @return string
 */
protected function getImageParamName() {
    return 'image';
}