Search plus (исправить недоработки)

23.05.2023 05:48
#1
kit2m2
Joomshopping forum user no avatar
Name: Dmitriy
03.12.2010
Posts: 107
Quote
Search plus (исправить недоработки)

Здравствуйте.

Давно хотел написать про этот модуль Search plus, прошло уже несколько лет, но так и не исправили ошибку:

Не применяются выбранные в настройках модуля свои шаблоны!

Чтобы работало как нужно, для этого приходиться в файле - mod_jshopping_ajaxsearch.php заменять строку -
require(JModuleHelper::getLayoutPath('mod_jshopping_ajaxsearch'));

На строку -
require(JModuleHelper::getLayoutPath('mod_jshopping_ajaxsearch', $params->get('layout', 'default')));

Тогда шаблон выбранный в настройках модуля без проблем применяется... Если этого не сделать то на сайте всегда будет выведен шаблон - default

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

И еще:
Неплохо было бы добавить этому полю атрибут placeholder -
<input type="text" class="inputbox" onkeyup="ajaxSearch();" onfocus="ajaxSearch();" name="search" id="jshop_search" value="<?php print htmlspecialchars($search, ENT_QUOTES)?>"/>

Добавить атрибут placeholder с языковой константой, а лучше эту переменную вынести в настройки модуля, как модуле поиска Joomla

И чтобы совсем хорошо было, то еще заменить <input /> на <button></button>
Это -
<input class="button btn btn-primary" type="submit" value="<?php print \JText::_('JSHOP_GO')?>"/>

На это -
<button class="button btn btn-primary search-submit" type="submit"><span class="какой-то класс"><?php print \JText::_('JSHOP_GO')?></span></button>


 
27.05.2023 05:48
#2
kit2m2
Joomshopping forum user no avatar
Name: Dmitriy
03.12.2010
Posts: 107
Quote
Aw: Search plus (исправить недоработки)

Здравствуйте.
Будут исправления в модулях?

 
27.05.2023 17:35
#3
admin
(Support Team)
User admin
Name: Admin
05.08.2010
Posts: 25940
Quote
Aw: Search plus (исправить недоработки)

Новая версия
Search plus 2.9.4

-----
кроме
И чтобы совсем хорошо было, то еще заменить <input /> на <button></button>
Это -
<input class="button btn btn-primary" type="submit" value="<?php print \JText::_('JSHOP_GO')?>"/>

На это -
<button class="button btn btn-primary search-submit" type="submit"><span class="какой-то класс"><?php print \JText::_('JSHOP_GO')?></span></button>


Last change: 27.05.2023 17:36
 
30.05.2023 16:12
#4
kit2m2
Joomshopping forum user no avatar
Name: Dmitriy
03.12.2010
Posts: 107
Quote
Aw: Search plus (исправить недоработки)

Спасибо, что исправили.

 
12.06.2023 10:47
#5
kit2m2
Joomshopping forum user no avatar
Name: Dmitriy
03.12.2010
Posts: 107
Quote
Aw: Search plus (исправить недоработки)

Здравствуйте.

Нашел еще недоработки в вашем расширении коде контроллера - AjaxsearchController.php

Чтобы не было предупреждений:

Warning: Undefined variable $lang in /.../.../.../.../../.../components/com_jshopping/Controller/AjaxsearchController.php on line 48

Warning: Attempt to read property "lang" on null in /.../.../.../.../../.../components/com_jshopping/Controller/AjaxsearchController.php on line 48

Warning: Undefined variable $search_type in /.../.../.../.../../.../components/com_jshopping/Controller/AjaxsearchController.php on line 49

Warning: Undefined variable $include_subcat in /.../.../.../.../../.../components/com_jshopping/Controller/AjaxsearchController.php on line 49


Нужно в коде контроллера объявить переменные: $include_subcat, $search_type и $lang - следующим образом:

$module = \JModuleHelper::getModule('jshopping_ajaxsearch');
$moduleParams = new \JRegistry($module->params);
$moreResults = $moduleParams->get('more_results', 0);
$include_subcat = $moduleParams->get('include_subcat', 0);/*добавленная строка*/
$search_type = $moduleParams->get('searchtype', 'all');/*добавленная строка*/


И:

$lang = \JSFactory::getLang();/*добавленная строка*/
\JFactory::getLanguage()->load('mod_jshopping_ajaxsearch', JPATH_SITE, $lang->lang, true);



Last change: 12.06.2023 10:49
 
12.06.2023 12:10
#6
kit2m2
Joomshopping forum user no avatar
Name: Dmitriy
03.12.2010
Posts: 107
Quote
Aw: Search plus (исправить недоработки)

И еще языковая константы:

SHOW_NO_RESULTS = "Показать Нету результатов"
NO_RESULTS = "Нету результатов"

Так писать и говорить неграмотно! Лучше так:

SHOW_NO_RESULTS = "Показать - Ничего не найдено"
NO_RESULTS = "Ничего не найдено"


 
12.06.2023 14:58
#7
admin
(Support Team)
User admin
Name: Admin
05.08.2010
Posts: 25940
Quote
Aw: Search plus (исправить недоработки)

Спасибо за подробное описание ошибок.

Search plus 2.9.5

 


Copyrights MAXXmarketing GmbH. Alle Rechte vorbehalten
Durch die Nutzung dieser Website stimme ich zu, dass Cookies zur optimalen Gestaltung der Website sowie zur Analyse der Nutzung der Website verwendet werden. Weiterführende Informationen finden Sie hier. OK, einverstanden.