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

Вопросы и ответы по изменению функционала, смене дизайна, переносу сайта и т.п.

Модераторы: Xpycm, Koduc

Правила форума
ВНИМАНИЕ! Форум не является средством гарантированной поддержки клиентов и пользователей. Поэтому на быстрый ответ тут рассчитывать не нужно, как и на ответ вообще. Сотрудники отвечают по мере своих возможностей.

На форуме действует ограничение в 3 сообщения / сутки.
Если Вы хотите сказать "спасибо", то воспользуйтесь функцией "Повысить репутацию" - зелёная иконка "плюс" под ником ответившего.

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

Сообщение vanred79 » 29 май 2019, 15:44

Здравствуйте! У меня версия 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;
Подскажите пожалуйста можно ли убрать одну из них? Или обе эти одинаковые строки нужны в этом коде?
Вся недвижимость Астаны у нас на сайте
vanred79
Местная знаменитость
Местная знаменитость
 
Сообщения: 660
Зарегистрирован: 02 фев 2013, 18:42
Откуда: Казахстан
Очки репутации: 2

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

Сообщение andipas » 30 май 2019, 10:50

Здравствуйте! Одну можно убрать
Часто задаваемые вопросы. Для правки файлов notepad++, netbeans, phpStorm. Для правки CSS firebug.
Аватара пользователя
andipas
Разработчик
Разработчик
 
Сообщения: 695
Зарегистрирован: 28 дек 2011, 22:37
Очки репутации: 30

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

Сообщение vanred79 » 30 май 2019, 13:10

andipas писал(а):Здравствуйте! Одну можно убрать

Спасибо!
Вся недвижимость Астаны у нас на сайте
vanred79
Местная знаменитость
Местная знаменитость
 
Сообщения: 660
Зарегистрирован: 02 фев 2013, 18:42
Откуда: Казахстан
Очки репутации: 2


Вернуться в Вопросы и помощь по изменениям

Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 14

cron