Страница 1 из 1

Небольшой вопрос по коду

СообщениеДобавлено: 29 май 2019, 15:44
vanred79
Здравствуйте! У меня версия ORE 1.3. Изучая файл \protected\modules\userads\controllers\MainController.php заметил дублирование кода в функции actionCreate.
В этом куске кода:
Код: Выделить всё
if(isset($_POST[$this->modelName])){

            $model->attributes=$_POST[$this->modelName];

            if(param('useUseradsModeration', 1) && strtotime($model->date_created) >= strtotime("-7 days")){
                $model->active = Apartment::STATUS_MODERATION;
            } else {
                $model->active = Apartment::STATUS_ACTIVE;
            }
            $model->owner_active = Apartment::STATUS_ACTIVE;

            if(($model->getStrByLang('address') && $model->city) && (param('useGoogleMap', 1) || param('useYandexMap', 1))){
                $city = null;
                if($model->city_id){
                    $city = ApartmentCity::model()->findByPk($model->city_id);
                    if($city){
                        $cityName = $city->name;
                    } else {
                        $cityName = null;
                    }
                }

                $coords = Geocoding::getCoordsByAddress($model->address, $cityName);
                if(isset($coords['lat']) && isset($coords['lng'])){
                    $model->lat = $coords['lat'];
                    $model->lng = $coords['lng'];
                }
            }

            $model->scenario = 'savecat';
            $model->owner_active = Apartment::STATUS_ACTIVE;

            if($model->save()){
                Yii::app()->user->setState('updateApartmentId', $model->id);
                //print_r($_POST);
                $redir=array('update', 'id'=>$model->id, 'show' => 'photo-gallery');
                if(isset($_POST['files'])){
                $i=0;
                foreach($_POST['files'] as $fi){
                    $redir=$redir+array('file['.$i.']'=>$fi);
                    $i++;
                    }
                }
                
                $this
->redirect($redir);
            }
        }

2 раза повторяется эта строка $model->owner_active = Apartment::STATUS_ACTIVE;
Подскажите пожалуйста можно ли убрать одну из них? Или обе эти одинаковые строки нужны в этом коде?

Re: Небольшой вопрос по коду

СообщениеДобавлено: 30 май 2019, 10:50
andipas
Здравствуйте! Одну можно убрать

Re: Небольшой вопрос по коду

СообщениеДобавлено: 30 май 2019, 13:10
vanred79
andipas писал(а):Здравствуйте! Одну можно убрать

Спасибо!