Как правильно загружать изображения в Yii2?

Question

[Orbb]

Добрый день!

Собираю уже 3 сайт на Yii, часть функционала уже успешно закопал в компоненты и не дублируюсь, но вот с картинками беда.

Сейчас делаю так:

public function actionUpdate($id)
    {
        $model = $this->findModel($id);
        $currentData = $this->findModel($id);

        if ($model->load(Yii::$app->request->post()) && $model->save()) {
            $image = UploadedFile::getInstance($model, 'image_name');
            if ($image != null) {
                if ($currentData->image_name)
                    ImagesController::deleteImage($currentData->image_name);
                $model->image_name = ImagesController::saveImage($image);
            } else {
                $model->image_name = $currentData->image_name;
            }
            $model->save();
            return $this->redirect(['view', 'id' => $model->id]);
        }

        return $this->render('update', [
            'model' => $model,
        ]);
    }

Кому лень читать код:
Получаю данные, беру модель до сохранения, проверяю, есть ли фотка в файл инпуте, если есть - проверяю на старую, удаляю старую, заливаю новую, если нет, но есть уже фотка в бд - ставлю на место старую из модели до сохранения и заливаю обратно.

Вынес всю ответственность за сохранение и удаление в отдельный класс, но все равно приходится во всех моделях дублировать подобный код. Т.к. во-первых, изображение при Update пропадает (не стал делать хидден инпут), а во-вторых, перед обновлением получаю старые данные, которые уже есть в модели (можно заменить на clone - суть не изменится). Считаю все это корявой реализацией, т.к. постоянно везде копирую этот код (ладно если 1 модель с картинкой , а если их 10?)

Как правильно бы централизованно избавиться от этого всего и прекратить подобные извращения? Компонент? Реализовать отдельный роут под изображения?

Подскажите, пожалуйста:3

Comments

[Arman]

Максим Федоров,

эти переменные одинаковы

но это ведь два разных объекта)

[Orbb]

Arik, именно. Если присвоить currentData = model, то данных старых не будет. Я написал, что можно это сделать через clone, но мне просто уже так привычней

[Arman]

Orbb, а почему не как в доках? там создают свойство куда присваивается загруженная картинка, проверяется на валидность и т.д., дальше просто нужно beforeSave переназначит и указать путь(?) на картинку для БД. Тогда не нужно два объекта.

[Максим Федоров]

Arik, ваша правда, позор мне :)
удалил

[Orbb]

Максим Федоров, один со старыми данными, другой с теми, что пришли из формы:)

Arik, можно ссылочку? Видимо, проглядел :(

[Orbb]

Arik, но это все равно чуть-чуть не то. Нужен внешний контроллер, который сам будет валидировать и возвращать название изображения или удалять старое. Т.к. если моделей таких 20 - в каждом прописывать одинаковые правила - опять дубль :)

[Orbb]

Даже не нужен, а как правильно реализовать-то такое решение? виджетом? компонентом? особенно актуально это на боевых сайтах недвижимости, там же тонны фоток и моделей

[Arman]

Orbb, Загрузка файлов тут у класса есть реальное свойство в котором будет экземпляр UploadedFile, после валидации начнется сохранение и будет вызван метод beforeSave, который еще вызовет событие, вот к одному из них можно прицепиться и реализовать логику сохранение пути в псевдосвойство, дальше save уже сохранит в БД

[Arman]

Orbb, виджет наверно ближе к вью, компонент что-то большое и автономное, скорее поведение для контроллеров, после того как его прицепить у контроллеров появится новый метод который умеет принимать экземпляр формы, как то проходит по его свойствам/правилам, понимать какие свойства это файлы и указывать на эти места нужные экземпляры UploadedFile.

[Arman]

т.е. на выходе наверно хорошо получить

if ($model->load(Yii::$app->request->post())) {
    $this->linkUploadFiles($model);
    if($model->save()) {
        return $this->redirect(['view', 'id' => $model->id]);
    }
}

[Orbb]

спасибо, пойду читать и думать :)