Ошибки версии 1.32.2

[Foton]

10) Да, действительно, слегка перепутали, дополнительные значения становятся активными если выключить "Возможность бронирования половину дня" (Настройки->Объявления)
13) Проверил, заявка с кнопки приходит.

[Foton]

9). Опишите пожалуйста подробнее порядок действий, повторить пока не удалось
4). Тоже опишите подробнее, как нам это повторить

[Dmitry]

4)

4) начал готовить марафет для запуска (создал новость, при этом она почему-то легла в отдельный блок новостей, чем тот, что был в установленной с нуля версии), по-отключал часть лишних материалов и обнаружил упавшие версии и хтмл и хмл карты сайта:

без дебага пишет: "Ошибка 500 Array to string conversion"
c Включенным дебагом XML жалуется на:

.../protected/modules/sitemap/controllers/MainController.php(136)
SimpleXMLElement::addChild() expects parameter 2 to be string, array given
136 $elem->addChild('loc', $item['url'][$keyUrl]);

а обычная на:

Array to string conversion
/themes/dolphin/views/modules/sitemap/views/index.php(35)
35 echo '<a href="' . $value['url'] . '">';

Ошибка появилась после огромного количества манипуляций с настройками (штатными средствами с движком).
Что именно к ней привело даже представить не могу. Посмотрите пожалуйста на моей доске (адрес доски, данные для ftp и админки есть на почте службы поддержки - пока не отключал).

6)

6) Есть проблема со сбросом статуса "Спецпредложение!" сейчас наблюдаю на своей доске объявление которое должно было перестать быть спецом несколько дней назад, а на демо-сайте висит объявление с "Спецпредложение! Доступно по 29.12.2018"

да, у меня на доске остался 1 исходный демо-вариант: dacha-posutochno-v-badeevo
и на нем не сбрасывается (остальные либо отключал, либо уже удалил)
На "живом" (новом) объявлении (если все пойдет по плану), то уже врядли до запуска в паблик успею проверить. Если я поставлю до 12/06, то оно спадет в 00:00 или в 23:59 12/06 ?

9)

9) Опять же баннер
"Хотите продать быстрее?
Попробуйте применить платные услуги"
Даже я умудрился его отключить для гостей (добавив условие из другого места на сайте - правильное?): && !Yii::app()->user->isGuest
Неужели так сложно изначально сделать этот баннер только на объявлениях, которые принадлежат зарегистрированному пользователю (у меня уже не хватило сил найти условие для такой проверки).
Тем более, что попытка нажать на ссылку под чужим объявлением приводит к ошибке:
ОШИБКА404
Запрашиваемая Вами страница не существует.

Ну то, что баннер показывает гостям - это думаю не нужно описывать как посмотреть. Зайти на страницу с любым объявлением и будет под профилем. Это уже лишнее и очевидное!?

А теперь на демосайте залогиньтесь ПОЛЬЗОВАТЕЛЕМ (не админом!) и посмотрите
https://demo-pro.open-real-estate.info/ ... nset_hotel (созданное другим пользователем)
при клике по баннеру "это ваше объявление..." попадете на страницу https://demo-pro.open-real-estate.info/ ... idservices :

ОШИБКА404

Запрашиваемая Вами страница не существует.

Было бы еще веселей, если бы ему удалось оплатить чужое
Самое правильно решение - написать условие для показа этого баннера: показывать только зарегистрированному и только если оно этого же пользователя
от показа гостям я избавился так: && !Yii::app()->user->isGuest
Неплохо бы эту проверку + проверку владельца ли текущее объявление добавить в "коробку".


10)

10) Справочники - "обособленные" значения "Время заезда" и "Время выезда" редактирую значения (добавляю много новых!) , но отображается только 2 исходных, естественно, проверил и на демо-сайте - проблема та же.
....

Отключил "половину дня" - заработало. Спасибо.
Видите, даже Вы путаетесь
Может стоит ввести комментарий на страницах этих справочников!? Например, в шапке, что-то типа:
"Настройки времени на данной страницы используются только в случае если Отключена "Возможность бронирования половину дня" (Настройки->Объявления)"
Если опция активна, то Вы можете только изменить отображаемое значение в переводах


12)

12) Сам себе частично отвечу на вопрос про карту на странице "СВЯЖИТЕСЬ С НАМИ". Для темы Дельфин координаты зашиты в файле:
/protected/helpers/HMap.php
в разделе public function OSMMap()
тут широта и долгота:

Код: Выделить всё

var lonLat = new OpenLayers.LonLat( ' . $this->getOption('lng', 37.64) . ', ' . $this->getOption('lat', 55.76) . ') 

а тут уровень зума при запуске:

Код: Выделить всё

         var zoom = ' . $this->getOption('zoom', 8) . '; 

а не "слушается" настроек по выбору карты она потому, что на главной бывает отображаются 2 карты сразу и Яндекс с Гуглом возмущаются

//т.к. на главной странице объявления с картой + виджет с картой конфликтуют

Просьба ввести эти 3 параметра широта, долгота и зум в настройки (не у всех офис расположен в центре Москвы ), а еще лучше было бы добавить 4й параметр - отображать карту или совсем скрыть на этой странице

12). Ошибок не должно быть

Проверять Ваше решение (делая откат своего решения и в случае неудачи обратно) сейчас уже возможности нет.
Я правильно понимаю, что в "коробке" поправили и теперь заработало:
"12.1) Настройки центровки карты для формы контактов задаются в настройках темы."
Потому что, исходная редакция в 1.32.2 их игнорировала и показывало даже в разных темах разные координаты (по вбитым файле данным без какой-либо реакции на настройки)
А если уж ошибки будут, то внесу свои правки (с откатом на замену на ОпенМэпс) в список постоянный после обновления версии

13)

13) О заявке отправленной с кнопки в подвале сайта (см. скрин выше) не приходит уведомлений на почту никому, даже администратору. Администратору приходит только уведомление о регистрации. Причем способ отключения регистрации, который Вы посоветовали для кнопки "бронировать" в этом случае не помогает (регистрация происходит и письмо об этом приходит, а письма о заявке нет)

13) Проверил, заявка с кнопки приходит.

Проверить на демо сайте, к сожалению, возможности нет (не менять же адрес почты админу ). Возможно те самые изменения к невоспроизводимому пункту 20) решают эту проблему (у меня никаких правок связанных с уведомлениями на доске не было), но у меня ни один из вариантов (пробовал и зарегистрированным и незарегистрированным - во втором случае приходит только уведомление о регистрации) нажатия на кнопку не приводит к приходу письма на почту администратору о сути новой заявки (единственное место где остается след - в админке, но ничто не дает повода туда пойти и посмотреть её).
Так же проверил и в случае если администратор не залогинен - все равно не приходит на почту.
Если, конечно, у Вас сейчас совсем другая система нотификации, то может потому и приходит... И искать наличие проблем нотификации тогда уже имеет смысл только после очередного релиза.
!!! А почему на демо-сайте не отключена возможность смены адреса почты демо-пользователям? Я без проблем сменил (но вернул) и... у кого-то пропадет возможность проверить пользовательский интерфейс до следующего отката
хм... и админу можно сменить... но, пожалуй, воздержусь (тем более все равно не известно какой там из кодов в уведомлениях - тот что у меня или уже другой)

20)

20). Про базу я имел ввиду, что вы всё правильно поняли. Действительно странно что код так отличается

Ну значит дождусь релиза для проверки, то как бы не сделать ещё хуже


В целом по системе уведомлений на почту (13, 23, второе сообщение это темы): сейчас во всех этих случаях уведомления приходят?

[Dmitry]

Продвинулся на следующий шаг и тут же свеженькое попалось:


26.1)
разные вариации главной страницы не содержат Canonical
https://demo-pro.open-real-estate.info/ru
https://demo-pro.open-real-estate.info/ ... le&lang=ru
https://demo-pro.open-real-estate.info/ ... ap&lang=ru
https://demo-pro.open-real-estate.info/ ... ck&lang=ru

26.2)
а так же все версии для печати не содержат Canonical
https://demo-pro.open-real-estate.info/ ... rintable=1

Да - они все предлагаются коробочным роботс.тхт быть закрытыми, но ссылки есть, вес растекается, так почему бы не сказать поисковикам, что это не плохая страница, а просто та же самая, но "в профиль".... (Canonical) и желающим можно для пущей убедительности сообщить
Яндексу в роботс (Гугл, к сожалению, с ним не знаком):
Clean-param: ls

[andipas]

Продвинулся на следующий шаг и тут же свеженькое попалось:

26.1) разные вариации главной страницы не содержат Canonical
https://demo-pro.open-real-estate.info/ru
https://demo-pro.open-real-estate.info/ ... le&lang=ru
https://demo-pro.open-real-estate.info/ ... ap&lang=ru
https://demo-pro.open-real-estate.info/ ... ck&lang=ru

26.2) а так же все версии для печати не содержат Canonical
https://demo-pro.open-real-estate.info/ ... rintable=1

Изменения в файлах //, ( - удалено, + добавлено )

Код: Выделить всё



//  protected/helpers/HSite.php

     public static function setCanonicalTagForIndex()
     {
-        if (Yii::app()->request->getParam('page') > 1) {
-            $canonicalUrl = Yii::app()->getBaseUrl(true);
-            $defaultLang = Lang::getDefaultLang();
-            if ($defaultLang != Yii::app()->language) {
-                $canonicalUrl .= '/' . Yii::app()->language;
-            }
+        $canonicalUrl = Yii::app()->createAbsoluteUrl('/');
+        if (substr_compare( $canonicalUrl, Yii::app()->request->url, -strlen( Yii::app()->request->url )) !== 0) {
             Yii::app()->clientScript->registerLinkTag('canonical', null, $canonicalUrl);
         }
     }

//  protected/modules/apartments/controllers/MainController.php

         if (!$printable && !$this->_isAPICall && issetModule('seo') && $apartment->seo && strpos(Yii::app()->request->url, $apartmentUrl) !== 0) {
             $this->redirect($apartmentUrl, true, 301);
         }
 
+        if($printable){
+            Yii::app()->clientScript->registerLinkTag('canonical', null, $apartmentUrl);
+        }

 

[Foton]

6). В файле protected/components/BeginRequest.php вставьте строку

Код: Выделить всё


self::clearSpecialOffers();


в двух местах после

Код: Выделить всё


self::clearGuestAdImagesTemp();


и после public static function updateStatusAd() {...} вставьте

Код: Выделить всё


public static function clearSpecialOffers()
    {
        $activeOffers = Apartment::model()->findAll('is_special_offer = 1 AND `is_free_to` IS NOT NULL AND DATE_FORMAT(`is_free_to`, "%Y-%m-%d") < DATE_FORMAT( NOW(), "%Y-%m-%d")');
        foreach ($activeOffers as $apartment) {
            $apartment->is_special_offer = 0;
            $apartment->is_free_to = NULL;
            $apartment->update(array('is_special_offer', 'is_free_to'));
        }
    }


значение const TIME_UPDATE измените на 21600
сброс будет происходить 4 раза в день, в зависимости от того когда была включена платная услуга, если спецпредложение было отмечено при редактировании объявления то сброс будет на следующий день.

[Dmitry]

✔
6) проверка до изменений : для двух новых объявлений (выставлял "спецпредложение" на сутки) сбросилось вроде адекватно.
"Куплен" статус был 2 разными способами (один присвоен админом, другой пользователем с баланса)
Может проблема в том как "заведен" демо-вариант (не выставлен какой-либо флаг)?

изменения внес (понаблюдаю сегодня за дачей в бадеево - может и она сбросится )

26) Спасибо за оперативность c Canonical (очень вовремя - как раз планирую запускать роботов для индексации)/
✔
26.1) Не могу проверить как это выглядит при установке в корень сайта, но в папке проблема у главной я бы даже сказал усугубилась - теперь главная выпадет из индекса совсем из-за того, что не хватает слеша в конце.
я для себя решил проблему так:
Yii::app()->clientScript->registerLinkTag('canonical', null, $canonicalUrl.'/');
Для "коробки" наверное потребуется добавить делать проверку и дописывать слэш в папке (а если в корне слэш берется от домена .ru/ , то там может быть не нужно, чтобы не вышло два слэша и главная не выпала из индекса) или более универсально - проверять заканчивается главная ORE на слэш и если нет дописывать. Хотя смотрю сейчас на демо сайте в "корне" стоит английская версия и она и "языковые" главные заканчиваются без слэша
✔
26.2) все отлично

[Dmitry]

Новое:

27) ✔
Сортировка "Тип недвижимости" (смена последовательности в Справочнике) влияет на отображение в соотв. после при поиске или заполнении объявлений, в сводной информации, но игнорируется при выводе в "Популярных направлениях" на главной странице в теме Dolphin

❌
28) Только сейчас оформив полностью страницы понял, что SEO-модуль работает неадекватно, а именно:
поле "Заголовок" используется 3 раза на странице в Title, H1 и хлебных крошках. Если по-уму максимально эффективно написать тайтл (до 70-100 символов), то в хлебных крошка и в H1 он смотрится слишком длинно, а из-за того что H1 идентичен тайтлу и крошкам это уже получается поисковый спам (3 раза одно и тоже). Особо проблемно это для страниц, где небольшая текстовая часть (доля переспама получается очень большой).
В некоторых места для себя я часть проблемы решил - изменив в городах и типах недвижимости код (используя не сео-тайтлы, а "чистое" название города и типа недвижимости Главная / Город / Гостиница ), но с H1 так просто "заплатку" не сделать (нужно дописывать отдельные поля для модуля SEO - и для городов из местоположения тоже).

[andipas]

26.1) В файле protected\helpers\HSite.php поправить функцию на:

Код: Выделить всё


    public static function setCanonicalTagForIndex()
    {
        $canonicalUrl = rtrim(Yii::app()->createAbsoluteUrl('/'), '/');
        if (substr_compare( $canonicalUrl, Yii::app()->request->url, -strlen( Yii::app()->request->url )) === 0) {
            Yii::app()->clientScript->registerLinkTag('canonical', null, $canonicalUrl);
        }
    }


27) Файл protected/modules/apartmentObjType/models/ApartmentObjType.php

вместо

Код: Выделить всё

$rows = Yii::app()->db->createCommand("SELECT * FROM {{apartment_obj_type}}")->queryAll();

нужно вставить:

Код: Выделить всё

$rows = Yii::app()->db->createCommand("SELECT * FROM {{apartment_obj_type}} ORDER BY sorter")->queryAll();

[Dmitry]

4) с картой сайта решили по "месту" (желательно исключить возможность появления подобной ошибки и в "коробке"). карта появилась, но в них обоих есть страницы с 404 и ссылки на неканонические страницы:
page/arenda (у меня отключена - выдает 404; единственное место, где она есть - в меню другой темы! убрал в неактивной теме и пропало, но думаю это не тоже не есть правильно)
search?objType=9 неканоническая
search?objType=... неканоническая
и на саму sitemap да она есть в меню, но её наверное нужно исключать, как минимум с html-версии

26.1) вариант из предыдущего сообщения привел к отключению каноникалс на главной (ORE стоит в папке). Видимо после rtrim (..., "/") canonicalUrl НЕ стал пустым (содержит папку) и не прошел проверку.
Если эта конструкция нормально работает при установке в корень домена, то для установленной в папку ORE (у себя) добавил

Код: Выделить всё

     else  
    {
      Yii::app()->clientScript->registerLinkTag('canonical', null, $canonicalUrl.'/');
    }

и у меня для главной теперь выдает адекватные

27) все в порядке - теперь последовательность типов недвижимости соответствует настройкам

28) не ошибка, а скорее юзабилити для "коробки" - в режиме редактирования администратором очень не хватает ссылки на просмотр объекта со стороны пользователя. Именно внутри страницы страницы редактирования (а то постоянно приходится либо запоминать номер объекта и с его помощью менять адрес другого объекта). Можно прямо рядом с "Уникальный номер объявления: 216" поставить "глазик" или номер сделать кликабельным.