Перейти к содержанию

Classifieds 1.2.1


Рекомендуемые сообщения

Эээ, друг, шаблон мне нужен из АЦ и текстом под спойлером и в теге кода, а не эта картинка из кеша шаблона.



У меня это приложение не установлено, поэтому только с кодом шаблона смогу исправить.

Ссылка на комментарий
Поделиться на другие сайты


Исправил в шаблоне category_view
 спасибо

:yahoo:

 

Еще бы знать что именно

Ссылка на комментарий
Поделиться на другие сайты

Твоя картинка в тему, там видно массив $types который и не проверялся на пустоту. Так понял, он участвует в коде формирования некоего фильтра объявлений.

Еще бы знать что именно

Там над шаблоном кнопка есть для показа изменений, она расскажет.

Ссылка на комментарий
Поделиться на другие сайты


Там над шаблоном кнопка есть для показа изменений, она расскажет.
не возражаете, если я здесь покажу для всех, что именно надо добавить?
Ссылка на комментарий
Поделиться на другие сайты

soloas, нет.

Бага забавна тем, что поле Advert Type обязательное, однако при создании объявления оно не заполнено и на обязательность не проверяется. Это бага.

Эту багу тоже надо исправлять, а то твои пользователи смогут публиковать объявления без типа. Но это уже правкой шаблона не сделать, думаю.

Ссылка на комментарий
Поделиться на другие сайты

Если после установки Classifieds  v1.2.1 появилась следующая ошибка:

Warning: Invalid argument supplied for foreach() in /var/www/data/www/мойсайт.ru/cache/skin_cache/cacheid_1/skin_classifieds.php on line 388

Warning: Invalid argument supplied for foreach() in /var/www/data/www/мойсайт.ru/cache/skin_cache/cacheid_1/skin_classifieds.php on line 406

Лечится правкой шаблона   skin_classifieds/category_view

 

Заключаем следующие строки кода в тег <if>

<if test="count($types)">
                 <foreach loop="filterloop:$types as $type">
                 <if test="$filter == $type['type_id']">{$type['name']}</if>
                 </foreach>
</if>

и

<if test="count($types)">
             <foreach loop="filterloop:$types as $type">
             <li<if test="$filter == $type['type_id']"> class='selected'</if>>
                 <a href='{parse url="app=classifieds&amp;module=core&amp;do=view_category&amp;category_id={$category['category_id']}&amp;sort_key={$sort['key']}&amp;sort_order={$sort['order']}&amp;filter={$type['type_id']}" base="public" seotitle="{$category['seo_title']}" template="view_category"}'>{$type['name']}</a>
             </li>            
             </foreach>
</if>

Огромное спасибо spoken за ликбез и помощь. 



soloas, нет.

Бага забавна тем, что поле Advert Type обязательное, однако при создании объявления оно не заполнено и на обязательность не проверяется. Это бага.

Эту багу тоже надо исправлять, а то твои пользователи смогут публиковать объявления без типа. Но это уже правкой шаблона не сделать, думаю.

 

Я этим вопросом сразу он бы был актуален, если бы ты не помог решить проблему с проверкой массива на пустоту.

Сейчас понимаю, что он не критичен и пользователям при создании объявления не обязательно выставлять тип.

Вижу в этом только 2 недостатка: объявление не попадает в фильтр типов и нет ярлыка типа в текте самого объявления.

А так получилась совсем неплохая доска объявлений с хорошей структурой и отличной сортировкой.

 

Еще раз спасибо за помощь.

Ссылка на комментарий
Поделиться на другие сайты

Вижу в этом только 2 недостатка: объявление не попадает в фильтр типов и нет ярлыка типа в текте самого объявления.

На мой взгляд это критический недостаток. Его надо исправлять. Но так понял, что автор забил на это приложение. Без фильтра типа сложнее разделить объявления на "Покупка/Продажа, Предлагаю/Ищу" и т.п.

Ссылка на комментарий
Поделиться на другие сайты


Но так понял, что автор забил на это приложение.
С чего Вы взяли? ТП осуществляется через лс 

 

Since the popularity of this application has increased, support is now available via PM to make sure no issues are missed. Please feel free to PM me at any time with any issues or queries you may have.

Ссылка на комментарий
Поделиться на другие сайты

С чего Вы взяли?

С долгого отсутствия новых версий.

Ссылка на комментарий
Поделиться на другие сайты

 

С долгого отсутствия новых версий.

 

Совсем не обязательно менять версию для того, чтобы пофиксить баги. Обновление компонента было в конце августа данного года.

Ссылка на комментарий
Поделиться на другие сайты

LvsF, странный подход. Бага, описанная выше, пофикшена?
Ссылка на комментарий
Поделиться на другие сайты


Бага, описанная выше, пофикшена?
Не знаю.

Нашел упоминание похожей ошибки (номера строк только отличаются, пост полуторагодичной давности). Автор написал про обновление данных баз. стиля + кешей файлов. Если не поможет, пишите ему и предоставляйте доступ в АЦ.

Ссылка на комментарий
Поделиться на другие сайты

LvsF, эти строки ошибок я уже пофиксил. Надо саму причину устранить - при публикации объявления не стоит radio button в выборе типа объявления и не проверяется обязательность выбора типа объявления.
Ссылка на комментарий
Поделиться на другие сайты

при публикации объявления не стоит radio button в выборе типа объявления

А зачем? Пусть юзер тыкает мышкой.

 

и не проверяется обязательность выбора типа объявления.

Тут 2 варианта:

- убрать звездочку в шаблоне;

- добавить проверку.

 

admin/applications_addon/other/classifieds/modules_public/listing/listing.php
Найти 

		$adtype = intval( $this->request['advert_type'] );
Ниже добавить 

		if ( ! $adtype )
		{
			$this->registry->output->showError( $this->lang->words['cfds_type_required'], '10CFDM2027' );
		}
Далее добавить фразу

Application - Classifieds

Pack Name - public_lang

Key - cfds_type_required

Default Text - Вы должны выбрать тип объявления
Ссылка на комментарий
Поделиться на другие сайты

LvsF, вот это верный вариант исправления. ИМХО, тип объявления это обязательное поле, поэтому проверка нужна.
Ссылка на комментарий
Поделиться на другие сайты


listing.php
внес указанные вами изменения в listing.php

ничего не произошло  :shake:  все как и было до...

Ссылка на комментарий
Поделиться на другие сайты

soloas, а ты попробуй объявление разместить без выбора типа, должен ошибку выдать.
Ссылка на комментарий
Поделиться на другие сайты

извините что вмешиваюсь в вашу беседу, но до установки вышеупомянутые фиксы возможно внести?

Ссылка на комментарий
Поделиться на другие сайты

  • 2 месяца спустя...


Далее добавить фразу


Application - Classifieds

Pack Name - public_lang

Key - cfds_type_required

Default Text - Вы должны выбрать тип объявления

ДД изменения внес, при попытку добавления выходит ошибка, куда добавлять данную фразу?

Ссылка на комментарий
Поделиться на другие сайты


обавления выходит ошибка,
Такая "Невозможно перевести защищенный пакет языков"?

 

__ipbmafia.ru/tutorials/article/62-redaktiruem-zaschischennye-iazykovye-pakety/

Ссылка на комментарий
Поделиться на другие сайты

Такая "Невозможно перевести защищенный пакет языков"?

 

__ipbmafia.ru/tutorials/article/62-redaktiruem-zaschischennye-iazykovye-pakety/

 

Нет, вот такая 

Ссылка на комментарий
Поделиться на другие сайты

Нет, вот та

Должна же оставаться страница добавления товара.

Ссылка на комментарий
Поделиться на другие сайты

По ссылке другой форум, проблема не ясна - о какой поддержке может быть речь? 

Ссылка на комментарий
Поделиться на другие сайты

Присоединяйтесь к обсуждению

Вы можете написать сейчас и зарегистрироваться позже. Если у вас есть аккаунт, авторизуйтесь, чтобы опубликовать от имени своего аккаунта.
Примечание: Ваш пост будет проверен модератором, прежде чем станет видимым.

Гость
Ответить в этой теме...

×   Вставлено с форматированием.   Вставить как обычный текст

  Разрешено использовать не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отображать как обычную ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставлять изображения напрямую. Загружайте или вставляйте изображения по ссылке.

  • Последние посетители   0 пользователей онлайн

    • Ни одного зарегистрированного пользователя не просматривает данную страницу
×
×
  • Создать...