Как в Laravel 5.x инкапсулировать загрузку файлов для разных сущностей или хотя бы в рамках разных действий в контроллере?

Question

[EVOSandru6]

Добрый день!

Есть форма _form.blade.php , которую тянут blade представления: create.blade.php и update.blade.php

...
    {{ Form::file('photo') }}
    <img src="/storage/images/articles/{{ $article->photo }}" style="height:50px" alt="">
...

Картинка улетает вместе с формой постом. В методе приведены выводы, которые я распечаю ниже:

class ArticlesController extends Controller
{
    public function update(ArticleRequest $request, Article $article)
    {
        $request = $this->saveImage($request);

        dump($request->all());

        dump($request->photo);

        $article->update($request->all());

        dd($article->photo);

        $this->syncTags($article, $request->input('tag_list'));

        return redirect()->back();
    }

private function saveImage(ArticleRequest $request)
    {

        if ($request->hasFile('photo')) {

            $file = $request->file('photo'); // ->isValid()

            $fileNameWithExt = $file->getClientOriginalName();
            $filename = pathinfo($fileNameWithExt, PATHINFO_FILENAME);
            $extension = $file->getClientOriginalExtension();
            $fileNameToStore = $filename . '_' . time() . '.' . $extension;
            $file->storeAs('public/articles', $fileNameToStore);

        } else {
            $fileNameToStore = 'noimage.jpg';
        }

        $request->photo = $fileNameToStore;

        return $request;

    }

}

Вывод dump и dd

1)
array:7 [▼
"_method" => "PATCH"
"_token" => "YPmvNLjhmoqKdxIt7RL2YbviiIjcfpEdqe0JW7ms"
"name" => "Sleep snow"
"content" => "Sleep snow in winter night with bears and foxes"
"published_at" => "2017-12-14"
"tag_list" => array:1 [▶]
"photo" => UploadedFile {#238 ▶}
]

2)
"logoct_1513334576.png"

3)
UploadedFile {#238 ▼
-test: false
-originalName: "logoct.png"
-mimeType: "image/png"
-size: 55490
-error: 0
#hashName: null
path: "/tmp"
filename: "phpulvqiE"
basename: "phpulvqiE"
pathname: "/tmp/phpulvqiE"
extension: ""
realPath: "/tmp/phpulvqiE"
aTime: 2017-12-15 10:42:56
mTime: 2017-12-15 10:42:56
cTime: 2017-12-15 10:42:56
inode: 21630383
size: 55490
perms: 0100600
owner: 33
group: 33
type: "file"
writable: true
readable: true
executable: false
file: true
dir: false
link: false
}


Сама картинка загружается, проблема в том, что $article->photo остается пустым.

В методе saveImage я хотел присвоить свойству photo имя картинки, но в итоге в свойство $article->photo попадает UploadedFile объект.

Почему так получается? Я же возвращаю $request в методе saveImage, где переопределено свойство photo.

Не нашел ни одного примера, где бы инкапсулировалась данная логика загрузки.

В Yii2 фреймворке такие вещи скармливаилсь поведениям. В Laravel предполагаю, что возможно должны быть как то задействованы middleware для таких случаев - если вообще вынести загрузку картинки и определения имени для сущности при обновлении и добавлении разных сущностей (Также удаление картинки при удалении сущности).

В свойстве $fillable элемент photo присутствует.

Также правило для картинки есть:

class ArticleRequest extends FormRequest{
...
public function rules()
    {
        return [
...
            'photo'=>'image|nullable|max:1999'
...
        ];
    }
...
}

Answer 1

[Andreo]

Я бы так не делал.
Лучшим на мой взгляд решением было бы как раз скармливание объекта UploadedFile в используемую модель. А в самой модели использовать сеттер этого атрибута для перехвата и преобразования значения.

Выглядеть в модели это будет приблизительно так:

public function setPhotoAttribute($value) {
    if ($value instanceOf UploadedFile) {
            $fileNameWithExt = $value->getClientOriginalName();
            $filename = pathinfo($fileNameWithExt, PATHINFO_FILENAME);
            $extension = $value->getClientOriginalExtension();
            $fileNameToStore = $filename . '_' . time() . '.' . $extension;
            $file->storeAs('public/articles', $fileNameToStore);
            return $this->attributes['photo'] = $fileNameToStore;
    }
}

Исходник нужно проверить. Накидал от руки :)

Comments

[EVOSandru6]

Благодарю, идея ясна. И этот вариант гораздо лучше, чем вариант реализации в контроллере. Интересует такой момент - возможно ли вынести реализацию метода вне модели, например в какой нибудь другой класс, чтобы данную функциональность можно было перекидывать на множество моделей. Ибо в предполагаемом классе должны также происходить удаление картинки при удалении модели. Мои соображения - базовая модель для моделей, сущности которых имеют картинку (не оч.), трейт (не оч.), возможное решение из коробки (пока такового не нашел) либо какой то сторонний вариант. Доп. соображения - в моделях, имеющих картинку хранится путь вроде того, что Вы написали выше: 'public/articles', остальное каким то оразом здорово бы скормить стороннему классу.

[Andreo]

Предпочтительнее, все таки, трейт на мой взгляд.
В трейте вы напишете три метода для работы с изображениями и их данными. Потом, подключая трейт, вы добавляете функционал для модели, который не расширяется громоздким базовым классом, и который скорее всего разрастется в будущем.
За пути в моделях не понял. У вас в модели хранится только имя файла...

[EVOSandru6]

Благодарю, имеется ввиду, что в трейте должна быть переменная $path, путь к которой должен браться из модели:

trait ImageManager()
{
  public function setPhotoAttribute($value) {
    if ($value instanceOf UploadedFile) {
            $fileNameWithExt = $value->getClientOriginalName();
            $filename = pathinfo($fileNameWithExt, PATHINFO_FILENAME);
            $extension = $value->getClientOriginalExtension();
            $fileNameToStore = $filename . '_' . time() . '.' . $extension;
// $this->path вместо явного 'public/articles'
            $file->storeAs(    $this->path      , $fileNameToStore); 
            return $this->attributes['photo'] = $fileNameToStore;
    }
  }
}

[EVOSandru6]

Интересует - сработает ли данная конструкция в трейте (содержащая $this->path) при условии, что данное свойство определено в модели Articles? Без прокидывания доп. публичных свойств из модели в трейт посредством $this->param возможно ли получить класс текущей модели?

[Andreo]

Не надо ничего прокидывать. Из трейта доступны все свойства и методы текущего объекта класса ($this), куда этот трейт был внедрен. Вы работаете с трейтом как и с обычным классом. Или другими словами, если у вас в модели есть атрибут path, то его спокойно можно использовать из любого метода в трейте и ваша конструкция заработает без проблем.

[EVOSandru6]

Андрулєс, Это похоже на вполне хорошее решение. Печалит лишь в трейте то, что нельзя переопределять методоы, а точнее в данном случае - события ( аля onCreate(), onDelete() ) в Модели. Было бы очень удобно, если бы например событие с одинаковым именем сначала выполнялось в трейте, а потом в основной модели. Laravel очень крутой фреймворк, но я удивлен, что здесь нет штуки Behavors (поведения) как в Yii2

[Andreo]

События модели можно определять в observers. Почитайте что это.
Их можно выражать и как обычный класс. И потом точно так же трейтом что-нибудь туда внедрять...

Answer 2

[hakkol]

Чтобы изменить свойство у $request нужно использовать merge
$request->merge(['photo' => $fileNameToStore]);

Comments

[EVOSandru6]

Благодарю, не помогло(

Все также вместо желаемой строки - объект.

$request = $this->saveImage($request);
        dump($request->all());

array:7 [▼
"_method" => "PATCH"
"_token" => "F8OlvOLb5wmPLy7sc3EzPXZmfCdZSi666ar0DB1A"
"name" => "Sleep snow"
"content" => "Sleep snow in winter night with bears and foxes"
"published_at" => "2017-12-14"
"tag_list" => array:1 [▶]
"photo" => UploadedFile {#238 ▶}
]

private function saveImage(ArticleRequest $request)
    {
        if ($request->hasFile('photo')) {

            $file = $request->file('photo'); // ->isValid()

            $fileNameWithExt = $file->getClientOriginalName();
            $filename = pathinfo($fileNameWithExt, PATHINFO_FILENAME);
            $extension = $file->getClientOriginalExtension();
            $fileNameToStore = $filename . '_' . time() . '.' . $extension;
            $file->storeAs('public/images', $fileNameToStore);
        } else {
            $fileNameToStore = 'noimage.jpg';
        }
        $request->merge(['photo' => $fileNameToStore]);
        return $request;
}

[hakkol]

EVOSandru6, Лучше сделайте так, чтобы метод saveImage возвращал имя фотографии, а в методе update уже перезайписывайте photo в $request

[EVOSandru6]

hakkol, Можно конечно в рамке одной сущности, но когда из более 10-ти, хочется найти какое то сквозное некописпастное решение. Смотрю в сторону Middlewars. Может там есть возможность такое провернуть.

[EVOSandru6]

hakkol, Если так сделаю, то массовое присвоение уже нельзя будет делать