Как сделать multiple upload files в Yii2?

Question

[asferot]

Как сделать multiple upload files в yii2? Находил варианты решения этой проблемы, но там она решалась через модель, а мне необходимо через контроллер.
Вот мой код

public function actionAdd_damage(){
        $postDamage = new Damage_list();
        if($postDamage->load(Yii::$app->request->post()) && $postDamage->validate()){
            $imageDamage = UploadedFile::getInstance($postDamage, 'photo');
            $postDamage->photo = 'uploads/'.md5($imageDamage->baseName).'.'.$imageDamage->extension;
            if($postDamage->save()){
                $imageDamage->saveAs($postDamage->photo);
                Yii::$app->getSession()->setFlash('message', 'Post published successfull');
                return $this->redirect(['site/table']);
            }
            else{
                Yii::$app->getSession()->setFlash('message', 'Failed to post');
            }
        }
        return $this->render('add_damage', ['postDamage' => $postDamage]);
    }

<?=$form->field($postDamage, 'photo[]')->fileInput('multiple'=>true)?>

private $date;
    private $description;
    private $photo;
    private $compence;
    private $come;

    public function rules()
    {
        return [
            [['description','compence', 'come'], 'required'],
            ['description', 'string', 'length' => [5, 250]],
            [['photo'], 'file', 'extensions' => 'png, jpg, pdf', 'maxFiles' => 10, 'skipOnEmpty'=>false]
        ];
    }

Answer 1

[Дмитрий]

Добрый день.

а мне необходимо через контроллер

Контролер потому и называется контролером что он должен контролировать, а не выполнять что-либо...))) Это грубое объяснение, но...
Что именно у Вас не получается, дополните свой вопрос.
И замените строку

$imageDamage = UploadedFile::getInstance($postDamage, 'photo');

на

$imageDamage = UploadedFile::getInstances($postDamage, 'photo');

Comments

[asferot]

Я читал, что нужно писать foreach, но это объясняется как писать в модели, а как в контроллере объяснений нет. Про это $imageDamage = UploadedFile::getInstances($postDamage, 'photo'); спасибо, учту. Как нужно правильно описать foreach в моем случае?

[Дмитрий]

Я читал, что нужно писать foreach, но это объясняется как писать в модели, а как в контроллере объяснений нет.

asferot, всё так же как и в модели, только использовать $model, а не $this.
Но зачем Вам перегружать контролер лишним? Делайте в модели, так будет правильней.

[Дмитрий]

asferot, в документации показано как не перегружать контролер лишним кодом.
То есть, в модели создаётся метод, который валидирует и загружает файл.

public function upload()
    {
        if ($this->validate()) { 
            foreach ($this->imageFiles as $file) {
                $file->saveAs('uploads/' . $file->baseName . '.' . $file->extension);
            }
            return true;
        } else {
            return false;
        }
    }

В данном случае $this обозначает, что Вы обращаетесь к свойству класса, который принадлежит этому же классу.
А в контролере надо обращаться к свойству модели через $imageDamage(в Вашем случае)

[asferot]

slo_nik, у меня содержимое в activerecord, а не в Model

private $date;
    private $description;
    private $photo;
    private $compence;
    private $come;

    public function rules()
    {
        return [
            [['description','compence', 'come'], 'required'],
            ['description', 'string', 'length' => [5, 250]],
            [['photo'], 'file', 'extensions' => 'png, jpg, pdf', 'maxFiles' => 10, 'skipOnEmpty'=>false]
        ];
    }

[Дмитрий]

asferot, вот как-то так

$imageDamage = UploadedFile::getInstances($postDamage, 'photo');
foreach($imageDamage as $file){
   $file->saveAs($postDamage->photo)
}

[asferot]

slo_nik, благодарю)

[Дмитрий]

asferot, ну и что? и если в ActiveRecord, то зачем приватные свойства?

[Дмитрий]

asferot, получилось?

[asferot]

slo_nik, с публичными свойствами, данные не пишутся в бд почему-то. Пока не получилось, пытаюсь сделать

[Дмитрий]

asferot, удалите их вообще, если модель наследуется от ActiveRecord.
Получается, что Вы их переопределяете пустыми значениями.

[asferot]

slo_nik, удалил) все ровно как и было. спасибо

[Дмитрий]

asferot, что как и было?

[asferot]

slo_nik, ну то есть, все работает как и было с private. Но multi upload пока не удается соорудить

[Дмитрий]

asferot, вот здесь Вы валидируете $postDamage->validate().
В этой строке опять пытаетесь валидировать $postDamage->save()
Сделайте так $postDamage->save(false)

[Дмитрий]

asferot, не получается потому, что не надо всё валить в одну кучу.
Что именно не получается?
Показывайте код.

[asferot]

slo_nik,

public function actionAdd_damage(){
        $postDamage = new Damage_list();
        if($postDamage->load(Yii::$app->request->post()) && $postDamage->validate()){
            $imageDamage = UploadedFile::getInstances($postDamage, 'photo');
            $postDamage->photo = 'uploads/'.md5($imageDamage->baseName).'.'.$imageDamage->extension;
            if($postDamage->save()){
                foreach($imageDamage as $file){
                    $file->saveAs($postDamage->photo);
                }
                Yii::$app->getSession()->setFlash('message', 'Post published successfull');
                return $this->redirect(['site/table']);
            }
            else{
                Yii::$app->getSession()->setFlash('message', 'Failed to post');
            }
        }
        return $this->render('add_damage', ['postDamage' => $postDamage]);
    }

[Дмитрий]

asferot, запись в базу идёт, а изображения не грузятся?

[asferot]

slo_nik, запись не пройдет пока изображения не загрузятся. То есть, заполняю все формы, загружаю 3 картинки, нажимаю "Добавить" и просто страница обновляется и обнуляется форма с загрузкой файлов

[Дмитрий]

asferot, верните публичное свойство photo в модель, если Вы его удалили.

foreach($imageDamage->photo as $file){
     $path = 'uploads/'.md5($file->baseName).'.'.$file->extension;
      $file->saveAs($path);
}

[Дмитрий]

asferot, подправил последний код.

[Дмитрий]

asferot, а путь лучше задать через псевдоним @web или @frontend

[Дмитрий]

asferot, ошибки?
И не надо всё сваливать в кучу. У Вас там сейчас мешанина несусветная. Зачем в контролер всё сваливать?

[Дмитрий]

asferot, вот набросал Вам рабочий пример.
Только свои значения подставьте.
В контролере

public function actionUpload()
    {
        $model = new UploadsFiles();
        if($model->load(Yii::$app->request->post()) && $model->validate()){
            $model->imagesFile = UploadedFile::getInstances($model, 'imagesFile');
            foreach ($model->imagesFile as $file){
                $model->newName = md5($file->baseName);
                $file->saveAs('uploads/' . $model->newName . '.' . $file->extension);
            }
        }
        return $this->render('upload', ['model' => $model]);
    }

В модели

public $imagesFile;
    public $newName;

       public function rules()
    {
        return [
            [['imagesFile'], 'file', 'extensions' => 'png, jpg', 'maxFiles'  => 0, 'maxSize' => 1024*1024*1, 'message' => 'No file']
        ];
    }

Параметр maxFiles установленный в 0 указывает что количество загружаемых изображений не ограничено(только настройками сервера).
В форме

echo $form->field($model, 'imagesFile[]')->fileInput(['multiple' => true]);

Answer 2

[asferot]

slo_nik, так он перестал записывать вообще в бд

Comments

[Дмитрий]

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

[Дмитрий]

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