Проблема проверки передачи изображения. Глюк YII?

Question

[Александр Борисович]

Версия: 1.1.10

Простая форма заливки фотографии. (_form.php)

Вид:

 <div class="row">
  <?php echo $form->labelEx($model,'foto'); ?>
   <?php  echo $form->fileField($model,'foto',array()); ?>
  <?php echo $form->error($model,'foto'); ?>
 </div>

В модели 'foto' — это строка

Выбираем изображение и пробуем сохранить.
Вот так не работает:

В контроллере функция actionUpdate

if ( $model->foto!='')
   {
   
   $fname='123.jpg';
   $img=CUploadedFile::getInstance($model,'foto'); 
   $model->foto=$fname;
   
   }

Вот так работает:

if ($_FILES['Pilgrims']['name']['foto'])
   {
   
   $fname='123.jpg';
   $img=CUploadedFile::getInstance($model,'foto'); 
   $model->foto=$fname;
   
   }

Как так получается, что $model->foto пустое, но потом из него же через CUploadedFile файл спокойно сохраняется? При дампе $model->foto равно ' '

Answer 1

[sdevalex]

Поле foto должно быть виртуальным:

class Item extends CActiveRecord
{
  public $foto;
}

$model->foto = CUploadedFile::getInstance($model,'foto'); 
$model->photoFile = '123.jpg';

А в моделе:
public function afterSave()
{
$this->foto->saveAs('/path/to/upload/folder/'. $this->photoFile);
}

Comments

[Александр Борисович]

А есть объяснение зачем делать его виртуальным? оно же вроде указано в модели.

[sdevalex]

Вам в моделе нужно поле «Имя картинки» или «Путь к картинке». И такое поле есть, но дополнительно, для удобства и валидации, можно добавить виртуальное поле «UploadedFile» для того, чтобы была возможность обратиться к нему из разных мест.

Просто так удобно, можно сказать Best Practice.

[sdevalex]

Если у вас в таблице есть поле foto, то код нужно написать так.

class Item extends CActiveRecord
{
  public $fotoUploader;

  public function rules()
  {
       return array(
            array('fotoUploader', 'file', ..., 'on' => 'upload'),
       );
  }

  public function beforeSave()
  {
        $this->photo = md5(time()) . '.jpg'; //генерация имени
  }

  public function afterSave()
  {
     $this->foto->saveAs('/path/to/upload/folder/'. $this->foto);
  }
}

//...

$model->fotoUploader = CUploadedFile::getInstance($model,'foto'); 
$model->save();

[sdevalex]

Немного ошибся при копипасте… так надо afterSave() и beforeSave() реализовать.

  public function beforeSave()
  {
        $this->photo = md5(time()) . '.jpg'; //генерация имени
        return parent::beforeSave();
  }

  public function afterSave()
  {
     $this->fotoUploader->saveAs('/path/to/upload/folder/'. $this->foto);
  }

[sdevalex]

Также в afterSave() и beforeSave() нужно добавить проверку… чтобы он каждый раз после сохранения не пытался генерировать имя и сохранять картинку.

[Александр Борисович]

>Вам в моделе нужно поле «Имя картинки» или «Путь к картинке».
Может не понятно написали. Мы просто пытаемся залить фотку на сервер с компа. С дедовских времен аплоад работает в php так( кэп моуд он): браузер отдает картинку скрипту, а там глобальной переменной далее мы проверяем IF ($_FILES !=0) то работаем дальше с фоткой. ( кэп моуд офф)

Удивительно то, что переложив на YII казалось бы аналогично все. С определением существования if($_FILES['Pilgrims']['name']['foto'] !=0) все пашет как и должно пахать с дедовский времен.
Но обращаться к глобальной переменной $_FILES чтобы узнать о существовании файла залитого немного глупо, потому что написана модель, которая загадочный образом работает!!! из нее вытаскивается и файл и имя но!!! if ( $model->foto!='') не работает))) но работает $img=CUploadedFile::getInstance($model,'foto');

То есть файл как бы есть в модели, но if ( $model->foto!='') не работает)))

[sdevalex]

У вас не должно быть такого кода нигде…

if($_FILES['Pilgrims']['name']['foto'] !=0)

Делайте нормальную в моделе примерно так…

  public function rules()
  {
       return array(
            array('fotoUploader', 'required'),
            array('fotoUploader', 'file', 'types'=>'jpg, gif, png'),
       );
  }

P.S.
Возможно вы не заметили ошибку…

//было photo
$this->foto = md5(time()) . '.jpg';

[sdevalex]

Делайте нормальную валидацию в моделе*

[Александр Борисович]

ммм. нет. так и не поняли. Все пашет у нас. Дело в том что if ( $model->foto) хоть и существует, но в данном виде if ( $model->foto) выдаёт что нет

[sdevalex]

Вы использовали отдельную переменную с CUploadedFile и коллбеками как я писал выше?

[Александр Борисович]

Увы — нет. Просто и так работает, добивает мысль о том что из $model->foto все вытащить можно а if ( $model->foto) говорить что пусто.
Завтра попробуем. Спасибо. Какая то abuz у вас с кармой. щелкнул +. Стала 3.5 вместо 4 бред какой то.