Osclass forums
Support forums => old => Topic started by: Nex23 on January 05, 2015, 07:45:04 pm
-
I have again a problem with my search.
My searches do not return any item.
The website is : http://xoor.ro (http://xoor.ro)
What can I do ? The install is fresh and installed with Softaculous .
-
Ok , I've figured out that the search problem appears after I change the default language to another language . What may be the problem ?
-
Hi,
Yes, searches are made over current selected language. Any ad with title/description marked as written in other language won't be taken into consideration.
***UPDATED with 5th version*** (February, 14 2017)
If you're interested in your searches covering all enabled languages, add this at the very bottom of your theme functions.php:
Notes:
1.- Take care not to leave blank lines after this.
2.- If your theme functions.php doesn't end with ?> skip first line of my code.
5th version (Alerts compatible -no duplicate ads, Multilanguage on User Alerts, solved problem with page language changing)
<?php
function cust_refined_pattern_search($params) {
if (@$params['sPattern'] != '') {
$mSearch = Search::newInstance();
$mSearch->addLocale('%');
$mSearch->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_user_dashboard() {
if (Params::getParam('page') == "user" && Params::getParam('action') == "alerts") {
$webUser = new CWebUser;
$aAlerts = Alerts::newInstance()->findByUser( Session::newInstance()->_get('userId'), false );
$user = User::newInstance()->findByPrimaryKey( Session::newInstance()->_get('userId'));
foreach($aAlerts as $k => $a) {
$array_conditions = (array)json_decode($a['s_search']);
$search = new Search();
$search->setJsonAlert($array_conditions);
if (osc_version() > 361) $search->notFromUser(Session::newInstance()->_get('userId'));
$search->addLocale('%');
$search->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
$search->limit(0, 3);
$aAlerts[$k]['items'] = $search->doSearch();
}
$webUser->_exportVariableToView('alerts', $aAlerts);
View::newInstance()->_reset('alerts');
$webUser->_exportVariableToView('user', $user);
}
}
osc_add_hook('before_html', 'cust_alerts_user_dashboard');
?>
4th version (Alerts compatible -no duplicate ads, Multilanguage on User Alerts, problem with page language changing, other bugs)
<?php
function cust_pattern_search_all_locales($params) {
if ($params['sPattern'] != '') {
$mSearch = Search::newInstance();
$mSearch->addLocale('%');
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_pattern_search_all_locales');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
?>
3rd version (Alerts compatible -no duplicate ads, No multilanguage on User Alerts)
<?php
function cust_pattern_search_all_locales($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addConditions("d.fk_c_locale_code LIKE '%'");
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_pattern_search_all_locales');
function cust_clean_duplicate_ads_user_alerts() {
$clean_alerts = View::newInstance()->_get('alerts');
if ($clean_alerts) {
foreach ($clean_alerts as $key => $alert) {
$outputArray = array();
$keysArray = array();
foreach ($clean_alerts[$key]['items'] as $innerArray) {
if (!in_array($innerArray['pk_i_id'], $keysArray)) {
$keysArray[] = $innerArray['pk_i_id'];
$outputArray[] = $innerArray;
}
}
$clean_alerts[$key]['items'] = $outputArray;
}
View::newInstance()->_exportVariableToView('alerts', $clean_alerts);
}
}
osc_add_hook('before_html', 'cust_clean_duplicate_ads_user_alerts');
?>
2nd version (Alerts compatible -but including duplicate ads under certain circumstances)
<?php
function cust_pattern_search_all_locales($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addConditions("d.fk_c_locale_code LIKE '%'");
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_pattern_search_all_locales');
?>
1st version (Alerts uncompatible)
<?php
function cust_pattern_search_all_locales($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addLocale('%');
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_pattern_search_all_locales');
?>
Regards
-
Yep , that worked for me . Thanks a lot !
There is another problem with the search algorithm .
Let's say I have: Blue T-Shirt and Black T-Shirt .
If I search after "T-Shirt" I will receive both results , if I search after "Black T-Shirt" I will receive again both results BUT they will be ordered by "Newest first" or by price , not by relevance !
And by this I mean if the Blue one is the newest added if I search for Black T-Shirt , the Blue one will come up first . How can I change that ? The blue one has two matches while the black one only one !!
-
***UPDATED with 10th version*** :o :) (February, 14 2017)
Add this at the very bottom of your theme functions.php:
Notes:
1.- Take care not to leave blank lines after this.
2.- If your theme functions.php doesn't end with ?> skip first line of my code.
10th version (Alerts compatible, Multilanguage on User Alerts), no Sorting by relevance on See By Price, Compatibility with other possible custom JOINS, ignore duplicates, fixed problem with page language changing and other bugs
<?php
function cust_refined_pattern_search($params) {
if (@$params['sPattern'] != '') {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addLocale('%');
$mSearch->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
if (@$params['sOrder'] != 'i_price') {
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT fk_i_item_id, MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_user_dashboard() {
if (Params::getParam('page') == "user" && Params::getParam('action') == "alerts") {
$webUser = new CWebUser;
$aAlerts = Alerts::newInstance()->findByUser( Session::newInstance()->_get('userId'), false );
$user = User::newInstance()->findByPrimaryKey( Session::newInstance()->_get('userId'));
foreach($aAlerts as $k => $a) {
$array_conditions = (array)json_decode($a['s_search']);
$search = new Search();
$search->setJsonAlert($array_conditions);
if (osc_version() > 361) $search->notFromUser(Session::newInstance()->_get('userId'));
$search->addLocale('%');
$search->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
$search->limit(0, 3);
$aAlerts[$k]['items'] = $search->doSearch();
}
$webUser->_exportVariableToView('alerts', $aAlerts);
View::newInstance()->_reset('alerts');
$webUser->_exportVariableToView('user', $user);
}
}
osc_add_hook('before_html', 'cust_alerts_user_dashboard');
?>
9th version (Alerts compatible, Multilanguage on User Alerts), no Sorting by relevance on See By (another little fix), Compatibility with other possible custom JOINS, Discard duplicatesproblem with page language changing, other bugs
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
$(this).prop('href', $(this).prop('href') + '?seeBy=1');
return true;
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
8th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (another little fix), Compatibility with other possible custom JOINS
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
$(this).prop('href', $(this).prop('href') + '?seeBy=1');
return true;
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
7th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (little fix), Compatibility with other possible custom JOINS
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
e.preventDefault();
window.location = $(this).prop('href') + '?seeBy=1';
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
6th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (little bug)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
e.preventDefault();
window.location = $(this).prop('href') + '/?seeBy=1';
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
5th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (incomplete)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Params::existParam('sOrder')) {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
?>
Well, first, I've realized that my previous solution to include all languages in the search could retrieve same ad several times, so I've added now a GROUP BY to prevent that happening.
And regarding your request for including all languages plus relevance ordering, here it's the solution (of course, use one or another):
4th version (Alerts compatible -no duplicate ads, Multilanguage on User Alerts)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '') {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
?>
3rd version (Alerts compatible -no duplicate ads, No multilanguage on User Alerts)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->addGroupBy("pk_i_id");
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_clean_duplicate_ads_user_alerts() {
$clean_alerts = View::newInstance()->_get('alerts');
if ($clean_alerts) {
foreach ($clean_alerts as $key => $alert) {
$outputArray = array();
$keysArray = array();
foreach ($clean_alerts[$key]['items'] as $innerArray) {
if (!in_array($innerArray['pk_i_id'], $keysArray)) {
$keysArray[] = $innerArray['pk_i_id'];
$outputArray[] = $innerArray;
}
}
$clean_alerts[$key]['items'] = $outputArray;
}
View::newInstance()->_exportVariableToView('alerts', $clean_alerts);
}
}
osc_add_hook('before_html', 'cust_clean_duplicate_ads_user_alerts');
?>
2nd version (Alerts compatible -but including duplicate ads under certain circumstances)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
1st version (Alerts uncompatible)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addField( sprintf("MATCH(d.s_title, d.s_description) AGAINST('%s') AS relevance", $params['sPattern']) );
$mSearch->addLocale('%');
$mSearch->order("relevance DESC, dt_pub_date", "DESC");
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
Regards
-
Hi, sorry to butt in but I always thought the search was done on solely the string that was being input by the user no matter what language.
So first it's good to see a better understanding of the search thanks to nex23 and great to have a solution however in this particular case I wonder if this should not just be an Admin option? ::)
I mean let admin decide whether a search will be language dependable or cross languages.
Can we make this into a plugin or even better how can we get this in the next updated Osclass? (Basic yet important functionality)
Just an example, we have lots of people not speaking the dutch language so they could be using english as their site language but living in The Netherlands I WANT them to be able to search on dutch terms having the site in english....
-
Well, you already have the functions here, it's just a matter of moving them to a proper plugin. ;) Myself, I haven't much time, so I find solutions and then it is "Add this at the very bottom of your theme functions.php (take care not to leave blank lines after this)" for me :D
Regards
-
I know teseo, I moved it towards guardians of code because I think this really should be core Admin.
Now I'm waiting for _Conejo to respond.... 8)
-
Just wanted to post a warning about this one, using code underneith (inside quote) did break my 'Alert Saving' in search but the earlier posted solution doesn't so maybe the 'relevance' part is causing this breaking? Anyway, if you want to use this code my advise would be to stick with the first solution offered:
function cust_pattern_search_all_locales($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addLocale('%');
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_pattern_search_all_locales');
Regards,
Eric
Well, first, I've realized that my previous solution to include all languages in the search could retrieve same ad several times, so I've added now a GROUP BY to prevent that happening.
And regarding your request for including all languages plus relevance ordering, here it's the solution (of course, use one or another):
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addField( sprintf("MATCH(d.s_title, d.s_description) AGAINST('%s') AS relevance", $params['sPattern']) );
$mSearch->addLocale('%');
$mSearch->order("relevance DESC, dt_pub_date", "DESC");
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
Regards
-
Hi,
No, none of them will work for alerts, it's just alerts aren't programmed to really retain all data on the search query. The first function won't keep "search in all languages" and the second will provoke an error (ORDER including a missing "relevance" field present in the original query but not in the stored alert).
No workaround for this I'm afraid, unless you were to modify a couple of core files.
Regards
-
..... :( So actually a nice fix independant locale search is not working with the saving of Alerts..... a sad day today! :-X
-
Well, took me a while, but in the end it was not impossible to solve. :)
Please check the second versions of both methods in the posts above and tell me if Alerts are working now (I've kept old versions, it might be useful to be able to compare them).
Regards
-
Nice! :)
I tested the latest with 'relevance' inthere and my Alert is being saved as expected but I am unable to test it with actually more than 1 language at the moment. For now I would say SUCCESS and big thank you for taking the time figuring it out, no more broken Alerts.
8)
-
EDIT: I have to stand correct still, the Alert IS working but in the search result the same ad was shown twice which is not good so I added the group by, that did the trick for the search results BUT now I have a working search and a working Alert where the Alert is showing the ad twice! ??? So this means that the Alert query also needs a group by somehow when watching your own Alerts in private user menu.
So there are still some minor issues to solve and I was wondering if any crons would be affected as well?
I used the following fixed code and I did a text search that should only return 1 ad (also has premium status):
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if (isset($params['sPattern'])) {
$mSearch->addJoinTable( null, sprintf("(SELECT fk_i_item_id, MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM oc_t_item_description ) des", $params['sPattern']), 'des.fk_i_item_id = oc_t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
This gave me a double result so I added this: $mSearch->addGroupBy("pk_i_id");
Now search is doing okay, the alert gets saved but is showing the double result :D
Regards,
Eric
-
You've ruined my saturday... >:( :D
Well, seems the problem of duplicate ads appears when an ad has non-empty description in at least two languages. Indeed adding a GROUP BY solves the issue on normal search pages, but with alerts there's the same problem as my first version of the relevance variant (there was that no new fields are stored on alerts table, here is the GROUP BY, same case).
So, the only solution I've found with no core modifications is to clean up duplicated ads on User Alerts view, adding a new function cust_clean_duplicate_ads_user_alerts(), check my 3rd version:
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778 (http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778)
Now, this seems to be working fine for User Dashboard, but I don't know if mailed alerts would still be affected. ??? I leave to you to check that, my Osclass internal crons don't like me... :D
Regards
-
:) Hi teseo.....today some testing time since I NEED this to work independant of used locale language.
I am thinking since this is by design there's no bug to report but more a work around making it independant.....
Wish me luck! :D
-
Dude!
You forgot the $mSearch->addGroupBy("pk_i_id"); again in the second third code sample (relevance) so when I copied the 3rd code it was showing me funny results again. :o
Haha.... after I added that code (again) the duplicate was gone.
Okay here's the deal, not working 100% yet.
My test setup = 2 active languages on front-end, all ads but one posted in 1 language, the other one is posted in 2nd language.
Now the search works for each chosen language which is perfect! Meaning it no longer matters in what language an ad is posted and whatever language is chosen by a user.
BUT when saving a search alert it is showing me the alert WITH the ads in the language the ads were posted.
Once I select the other language the alert is showing WITHOUT the ads ;D
Probably meaning.... there goes yet another saturday? :P
Nonetheless, I am pretty happy with the search working now independant of the chosen language either searching or posting ads.
Regards,
Eric
-
Teseo, this thread is about the same 'issue' and works flawlessly:
[/url]http://forums.osclass.org/general-help/search-doesn't-work-on-multi-language-site/?topicseen (http://forums.osclass.org/general-help/search-doesn't-work-on-multi-language-site/?topicseen)[/url]
Since there seems to be a nr. of corrections to be made which do not feel very stable I am thinking about using an class 'extend' and create a new search class which is missing the locale condition as mentioned in the thread. Using the new class also prevents future update issues, what do you think? ::)
Regards,
Eric
-
Hi,
Sorry, I won't take a look at that while a solution not involving core files modification is possible. And I think I've managed to get it, look now my 4th version:
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778 (http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778)
Problem was, again, that locale isn't stored on alerts table, so I had to "fool" the User Alerts page about the user's current language. Function cust_alerts_locale_hack_start retrieves the current one, stores it in a View variable and set it to '%'. Just after the alerts has been retrieved from database, cust_alerts_locale_hack_end restores correct language so the page won't always use English.
While at it, I found a way to avoid duplicates and retrieve 3 different ads every time, replacing the blessed "GROUP BY" :D by a DISTINCT inside the subquery. ;)
Next saturday I will be on the beach, don't count me in... 8) :D
Regards
-
Hi teseo,
so I've added now a GROUP BY to prevent that
you must have a 'group by' blindspot ;D
Your fourth solution...... sorry but it gave me doubles in a text search..... BUT when actually adding the group by
$mSearch->addGroupBy("pk_i_id"); in function cust_refined_pattern_search it looks okay. I thought the DISTINCT in sql was taking care of the doubles but it didn't and I haven't given it more thought.
So far I checked the Alert saving/display and it works afaik and I also tested a search (text) with again no duplicates.
IF I see a miss the coming weeks I'll let you know but besides the group by this one really looks good! 8)
P.s. my earlier referral to a core fix looked to be okay but also had doubles issue in the text search ;)
Thanks,
Eric
***UPDATED with 4th version*** :o :)
Well, first, I've realized that my previous solution to include all languages in the search could retrieve same ad several times, so I've added now a GROUP BY to prevent that happening.
And regarding your request for including all languages plus relevance ordering, here it's the solution (of course, use one or another):
4th version (Alerts compatible -no duplicate ads, Multilanguage on User Alerts)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '') {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM oc_t_item_description ) des", $params['sPattern']), 'des.fk_i_item_id = oc_t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
?>
3rd version (Alerts compatible -no duplicate ads, No multilanguage on User Alerts)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addJoinTable( null, sprintf("(SELECT fk_i_item_id, MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM oc_t_item_description ) des", $params['sPattern']), 'des.fk_i_item_id = oc_t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->addGroupBy("pk_i_id");
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_clean_duplicate_ads_user_alerts() {
$clean_alerts = View::newInstance()->_get('alerts');
if ($clean_alerts) {
foreach ($clean_alerts as $key => $alert) {
$outputArray = array();
$keysArray = array();
foreach ($clean_alerts[$key]['items'] as $innerArray) {
if (!in_array($innerArray['pk_i_id'], $keysArray)) {
$keysArray[] = $innerArray['pk_i_id'];
$outputArray[] = $innerArray;
}
}
$clean_alerts[$key]['items'] = $outputArray;
}
View::newInstance()->_exportVariableToView('alerts', $clean_alerts);
}
}
osc_add_hook('before_html', 'cust_clean_duplicate_ads_user_alerts');
?>
2nd version (Alerts compatible -but including duplicate ads under certain circumstances)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addJoinTable( null, sprintf("(SELECT fk_i_item_id, MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM oc_t_item_description ) des", $params['sPattern']), 'des.fk_i_item_id = oc_t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
1st version (Alerts uncompatible)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addField( sprintf("MATCH(d.s_title, d.s_description) AGAINST('%s') AS relevance", $params['sPattern']) );
$mSearch->addLocale('%');
$mSearch->order("relevance DESC, dt_pub_date", "DESC");
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
Regards
-
Don't want to be a pain but I think I found another 'issue' which is related to this new function:
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
It seems that the _set('userLocale','%'); code is affecting the function 'osc_get_current_user_locale' (probably other locale functions as well) which is used to determine the user's chosen locale. Meaning, the new function is actually disrupting getting to know what language was chosen by the user which in my case causes a failure in my header where I show the chosen language. It can be a while but then all of a sudden the user locale is set to % and I can see it being returned by the function 'osc_get_current_user_locale'.
So the locale is having multi purpose use....1 when querying the items, 2 with Alerts and 3 keeping track of default/chosen language in Osclass. It does make sense but it also means you need to find a way to keep chosen language stick to 1 language and have the query locale changed to % (as well as the alerts).
EDIT:
I didn't look at the other function yet, the 'cust_alerts_locale_hack_end' which I think you use to set the locale to the chosen language again. Having said so I tend to think now that either the used hook is incorrect or is not working a 100% seeing my locale get 'lost' (which I have not been able to pin point yet).
Regards,
Eric
-
I'm afraid that's not the worst news... :(
so I've added now a GROUP BY to prevent that
you must have a 'group by' blindspot ;D
If you [still :D] can recall, the goal was to eliminate that GROUP BY because it isn't stored for alerts. I thought I had achieved that replacing it by that SELECT DISTINCT(fk_i_item_id) in the JOIN subquery, but obviously not. :( See, this would be the perfect query:
SELECT DISTINCT(oc_t_item.pk_i_id) as nodups, oc_t_item.*, oc_t_item.s_contact_name as s_user_name
FROM (oc_t_item)
LEFT JOIN oc_t_item_description as d ON d.fk_i_item_id = oc_t_item.pk_i_id
INNER JOIN (SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('test') AS relevance FROM oc_t_item_description ) des ON des.fk_i_item_id = oc_t_item.pk_i_id
WHERE MATCH(d.s_title, d.s_description) AGAINST('test' IN BOOLEAN MODE)
AND ( d.fk_c_locale_code LIKE '%' )
AND oc_t_item.b_enabled = 1 AND oc_t_item.b_active = 1 AND oc_t_item.b_spam = 0 AND (oc_t_item.b_premium = 1 || oc_t_item.dt_expiration >= '2015-04-27 17:08:26')
ORDER BY des.relevance DESC, dt_pub_date DESC
But, no way, you may add fields for the SELECT at the end of the chain:
SELECT oc_t_item.*, oc_t_item.s_contact_name as s_user_name, DISTINCT(oc_t_item.pk_i_id) as nodups
But not at the start, and DISTINCT needs to be placed right there, just after SELECT. :(
So at this point we'd need to return to GROUP BY for search page query and erasing of duplicates on alert pages, just as one of my earlier versions did.
When I have a little time I'll do that and study that last issue you discovered.
Regards
-
LOL.... well don't worry about it, it's fun to learn a bit more every time and since this is pretty the 'heart' of the system after this your/our understanding will only be better.
Interesting findings! (and no hurry)
Thanks,
Eric
-
@teseo,
I had another thought regarding the multiple languages and the 'issue' we are trying to fix here well actually trying to break the design and have all ads in searches/alerts no matter what language was used when posting a new ad.
HOW ABOUT; simply setting the locale to the default preventing new ads to be saved based on the locale in use? So, no matter what locale is selected by the user, when posting a new ad we simply always use ONE locale which will be used when saving new ads. this way you do not have to make any changes to the search/alerts and we follow the design of Osclass with regard of the architecture.
Incorrect thought, with more than 1 language in use we still need the fix of getting ads in other locale but the above would make sure all ads are posted in only one language keeping the system more consistent if you want to use 1 main language yet show all in searches.
I was thinking about a simple hook before new ad is saved that changes the locale to the (default) one you want.
Do you think this is possible? ::)
Regards,
Eric
-
Some thoughts trying to have an overview of what happens.....
Posting & Locales
1) In good working original Osclass new ads are posted using the active locale.
2) In case someone changes language locale and starts editing his existing ad posted using another language locale when saving the edited ad is saved again using the new locale. In this case the ad is being duplicated and having differences where the owner probably doesn't know about!
Search
3) Search is checking locale mainly when doing a text search
4) In case you simply press search without criteria the system shows ALL ads independant of locales
SO, my thought is that posting and editing of ads should always stick to the locale which was used when posting the ad to prevent the creation of duplicates and creating two different ad versions in two different locales!!
IMO this could be considered a bug or a feature with nasty side effects. ::)
Next, to ignore the locales in text search (and maybe other criteria using locale) we need to simply take it out of the search query. This could be an Admin option and/or a core change OR this might be achieved with the search changes done by teseo using functions.php.
The latter was causing me issues BUT I also had an issue in my theme I wasn't aware of when posting new ads. One locale function being used in my header prevented it to work again in the body of the page causing my newly posted ads to be posted without the locale in the table! Causing my system to have ads with 1 locale, ad duplicates with different locale and different ad properties and ads without a locale.
As for the latter, I was able to fix my theme issue so my new ads are posted the correct way again and I will test the last solution again teseo created to see if that makes any difference as for working or not working 100%.
But most of all I want the edit of ads not create duplicates..... I am not sure if this should be considered a bug and I hope _Conejo is reading along and can shed a light on this.
All other thoughts very welcome here! :)
Regards,
Eric
-
The search is very important to all websites and with osclass there is a problem like you both said.
I was trying to integrate the codes provided (version 4 and 7) but both are breaking my website languagage.
Which one I should use if I only want to bring up the most relevant ads and then all other if there is no other filter used?
Thanks in advance for your replays.
-
Hi,
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778
Last version (7th).
Add that at the very bottom of your theme functions.php:
Notes:
1.- Take care not to leave blank lines after this.
2.- If your theme functions.php doesn't end with ?> skip first line of my code.
Regards
-
Hi,
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778
Last version (7th).
Add that at the very bottom of your theme functions.php:
Notes:
1.- Take care not to leave blank lines after this.
2.- If your theme functions.php doesn't end with ?> skip first line of my code.
Regards
I've done that but for some reason after I hit subscribe to this search my browser cookies are breaking and the browser does not read my translated .po but bring back US language.
If I open my website in incognito window then it looks fine.
-
Sorry, I couldn't reproduce anything of that. Maybe your theme is incompatible with this function somehow... ??? Have you tried with another theme?
Regards
-
No but its a good idea and I will test...the point is that you might be right partly about the theme.
When i was building my website at 1st i had a problem with the translation and some plugins.Because the plugins was not compatible to work with other than EN language. So I did cheat the system by translating the original EN files within the files.Basically the file says EN but inside is an other language so that might cause the problem as well...im not quite sure but i will do some testing with other languages.
-
When you cheat the System, the System will get back at you sooner or later. ;D :D
Well, let's hope you find out what's happening. :)
Regards
-
I've tested version 7 on bender theme and its same result. After you subscribe to alert... not really sure how or why the language changes to english. Is it possible the system to avoid the original .po files and read directly from the source?
However i've tested your 3th version and it seem to be working fine and not mixing my language but...
1. There is no sorting by relevance
2. When unregistered user subscribe to search and the validation link is clicked from the email there is an error message "There was an error. Please contact us". Further investigation reveals that the subscription went fine but the alert is inactive and the admin have to activate it manually. (not important at the moment)
My question is - Can you combine 3th version to search by relevance too?
Im really eager to provide my users with relevant results of their searches.
-
remove
-
remove
-
remove
-
hello friends , which version works in Portuguese language Brazil -My search does not work
-
***UPDATED with 9th version*** :o :) (19 April 2016)
Add this at the very bottom of your theme functions.php:
Notes:
1.- Take care not to leave blank lines after this.
2.- If your theme functions.php doesn't end with ?> skip first line of my code.
9th version (Alerts compatible, Multilanguage on User Alerts), no Sorting by relevance on See By (another little fix), Compatibility with other possible custom JOINS, Discard duplicates
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
$(this).prop('href', $(this).prop('href') + '?seeBy=1');
return true;
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
8th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (another little fix), Compatibility with other possible custom JOINS
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
$(this).prop('href', $(this).prop('href') + '?seeBy=1');
return true;
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
7th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (little fix), Compatibility with other possible custom JOINS
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
e.preventDefault();
window.location = $(this).prop('href') + '?seeBy=1';
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
6th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (little bug)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
e.preventDefault();
window.location = $(this).prop('href') + '/?seeBy=1';
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
5th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (incomplete)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Params::existParam('sOrder')) {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
?>
Well, first, I've realized that my previous solution to include all languages in the search could retrieve same ad several times, so I've added now a GROUP BY to prevent that happening.
And regarding your request for including all languages plus relevance ordering, here it's the solution (of course, use one or another):
4th version (Alerts compatible -no duplicate ads, Multilanguage on User Alerts)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '') {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
?>
3rd version (Alerts compatible -no duplicate ads, No multilanguage on User Alerts)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->addGroupBy("pk_i_id");
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_clean_duplicate_ads_user_alerts() {
$clean_alerts = View::newInstance()->_get('alerts');
if ($clean_alerts) {
foreach ($clean_alerts as $key => $alert) {
$outputArray = array();
$keysArray = array();
foreach ($clean_alerts[$key]['items'] as $innerArray) {
if (!in_array($innerArray['pk_i_id'], $keysArray)) {
$keysArray[] = $innerArray['pk_i_id'];
$outputArray[] = $innerArray;
}
}
$clean_alerts[$key]['items'] = $outputArray;
}
View::newInstance()->_exportVariableToView('alerts', $clean_alerts);
}
}
osc_add_hook('before_html', 'cust_clean_duplicate_ads_user_alerts');
?>
2nd version (Alerts compatible -but including duplicate ads under certain circumstances)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
1st version (Alerts uncompatible)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addField( sprintf("MATCH(d.s_title, d.s_description) AGAINST('%s') AS relevance", $params['sPattern']) );
$mSearch->addLocale('%');
$mSearch->order("relevance DESC, dt_pub_date", "DESC");
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
Regards
Hello, I'm trying to use a script to find the 9 version, everything works fine, except "Subscribe to this search", when I try to "Subscribe to this search" have lost site language, how can I fix it?
-
when I try to "Subscribe to this search" have lost site language
What do you mean by "lost site language"? ??? Please try to be more specific.
-
Hi,
Hello, I'm trying to use a script to find the 9 version, everything works fine, except "Subscribe to this search", when I try to "Subscribe to this search" have lost site language, how can I fix it?
I think I've fixed now this problem (and others that I found while reviewing the code). Now there's new versions for "multilanguage search" only:
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113711/#msg113711 (5th version)
And "multilanguage search with sorting by relevance":
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778 (10th version).
I strongly recommend updating to anyone using these functions.
Regards
-
Hi,
Hello, I'm trying to use a script to find the 9 version, everything works fine, except "Subscribe to this search", when I try to "Subscribe to this search" have lost site language, how can I fix it?
I think I've fixed now this problem (and others that I found while reviewing the code). Now there's new versions for "multilanguage search" only:
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113711/#msg113711 (5th version)
And "multilanguage search with sorting by relevance":
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778 (10th version).
I strongly recommend updating to anyone using these functions.
Regards
***UPDATED with 10th version*** :o :) (February, 14 2017)
Add this at the very bottom of your theme functions.php:
Notes:
1.- Take care not to leave blank lines after this.
2.- If your theme functions.php doesn't end with ?> skip first line of my code.
10th version (Alerts compatible, Multilanguage on User Alerts), no Sorting by relevance on See By Price, Compatibility with other possible custom JOINS, ignore duplicates, fixed problem with page language changing and other bugs
<?php
function cust_refined_pattern_search($params) {
if (@$params['sPattern'] != '') {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addLocale('%');
$mSearch->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
if (@$params['sOrder'] != 'i_price') {
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT fk_i_item_id, MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_user_dashboard() {
if (Params::getParam('page') == "user" && Params::getParam('action') == "alerts") {
$webUser = new CWebUser;
$aAlerts = Alerts::newInstance()->findByUser( Session::newInstance()->_get('userId'), false );
$user = User::newInstance()->findByPrimaryKey( Session::newInstance()->_get('userId'));
foreach($aAlerts as $k => $a) {
$array_conditions = (array)json_decode($a['s_search']);
$search = new Search();
$search->setJsonAlert($array_conditions);
if (osc_version() > 361) $search->notFromUser(Session::newInstance()->_get('userId'));
$search->addLocale('%');
$search->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
$search->limit(0, 3);
$aAlerts[$k]['items'] = $search->doSearch();
}
$webUser->_exportVariableToView('alerts', $aAlerts);
View::newInstance()->_reset('alerts');
$webUser->_exportVariableToView('user', $user);
}
}
osc_add_hook('before_html', 'cust_alerts_user_dashboard');
?>
9th version (Alerts compatible, Multilanguage on User Alerts), no Sorting by relevance on See By (another little fix), Compatibility with other possible custom JOINS, Discard duplicatesproblem with page language changing, other bugs
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->addGroupBy(DB_TABLE_PREFIX.'t_item.pk_i_id');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
$(this).prop('href', $(this).prop('href') + '?seeBy=1');
return true;
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
8th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (another little fix), Compatibility with other possible custom JOINS
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
$(this).prop('href', $(this).prop('href') + '?seeBy=1');
return true;
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
7th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (little fix), Compatibility with other possible custom JOINS
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$query_elements = (array) json_decode($mSearch->toJson());
$mSearch->addJoinTable(count($query_elements['tables_join']), sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
e.preventDefault();
window.location = $(this).prop('href') + '?seeBy=1';
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
6th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (little bug)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Session::newInstance()->_get('seeBy')) {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
Session::newInstance()->_drop('seeBy');
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
// Detect See By link
if (Session::newInstance()->_get('seeBy') == '') Session::newInstance()->_set('seeBy', (Params::getParam('seeBy') ? 1 : 0));
Params::unsetParam('seeBy');
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
function cust_seeBy_link_add_identifier() { ?>
<script type="text/javascript">
$(".see_by a").on('click', function(e) {
e.preventDefault();
window.location = $(this).prop('href') + '/?seeBy=1';
})
</script>
<?php }
osc_add_hook('footer', 'cust_seeBy_link_add_identifier');
?>
5th version (Alerts compatible - no duplicate ads, Multilanguage on User Alerts), no Sorting by relevance on See By (incomplete)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '' && !Params::existParam('sOrder')) {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
?>
Well, first, I've realized that my previous solution to include all languages in the search could retrieve same ad several times, so I've added now a GROUP BY to prevent that happening.
And regarding your request for including all languages plus relevance ordering, here it's the solution (of course, use one or another):
4th version (Alerts compatible -no duplicate ads, Multilanguage on User Alerts)
<?php
function cust_refined_pattern_search($params) {
if ($params['sPattern'] != '') {
$mSearch = Search::newInstance();
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_alerts_locale_hack_start() {
if (osc_get_osclass_section() == "alerts") {
View::newInstance()->_exportVariableToView('aUserLocale', Session::newInstance()->_get('userLocale'));
Session::newInstance()->_set('userLocale', '%');
}
}
osc_add_hook('init', 'cust_alerts_locale_hack_start');
function cust_alerts_locale_hack_end($conditions) {
if (osc_get_osclass_section() == "alerts") {
Session::newInstance()->_set('userLocale', View::newInstance()->_get('aUserLocale'));
}
return $conditions;
}
osc_add_filter('sql_search_item_conditions', 'cust_alerts_locale_hack_end');
?>
3rd version (Alerts compatible -no duplicate ads, No multilanguage on User Alerts)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->addGroupBy("pk_i_id");
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
function cust_clean_duplicate_ads_user_alerts() {
$clean_alerts = View::newInstance()->_get('alerts');
if ($clean_alerts) {
foreach ($clean_alerts as $key => $alert) {
$outputArray = array();
$keysArray = array();
foreach ($clean_alerts[$key]['items'] as $innerArray) {
if (!in_array($innerArray['pk_i_id'], $keysArray)) {
$keysArray[] = $innerArray['pk_i_id'];
$outputArray[] = $innerArray;
}
}
$clean_alerts[$key]['items'] = $outputArray;
}
View::newInstance()->_exportVariableToView('alerts', $clean_alerts);
}
}
osc_add_hook('before_html', 'cust_clean_duplicate_ads_user_alerts');
?>
2nd version (Alerts compatible -but including duplicate ads under certain circumstances)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addJoinTable( null, sprintf("(SELECT DISTINCT(fk_i_item_id), MATCH(s_title, s_description) AGAINST('%s') AS relevance FROM %st_item_description ) des", $params['sPattern'], DB_TABLE_PREFIX), 'des.fk_i_item_id = '.DB_TABLE_PREFIX.'t_item.pk_i_id', 'INNER' );
$mSearch->addLocale('%');
$mSearch->order("des.relevance DESC, dt_pub_date", "DESC");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
1st version (Alerts uncompatible)
<?php
function cust_refined_pattern_search($params) {
$mSearch = Search::newInstance();
if ($params['sPattern'] != '') {
$mSearch->addField( sprintf("MATCH(d.s_title, d.s_description) AGAINST('%s') AS relevance", $params['sPattern']) );
$mSearch->addLocale('%');
$mSearch->order("relevance DESC, dt_pub_date", "DESC");
$mSearch->addGroupBy("pk_i_id");
}
}
osc_add_hook('search_conditions', 'cust_refined_pattern_search');
?>
Regards
-
Hi,
Hello, I'm trying to use a script to find the 9 version, everything works fine, except "Subscribe to this search", when I try to "Subscribe to this search" have lost site language, how can I fix it?
I think I've fixed now this problem (and others that I found while reviewing the code). Now there's new versions for "multilanguage search" only:
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113711/#msg113711 (5th version)
And "multilanguage search with sorting by relevance":
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778 (10th version).
I strongly recommend updating to anyone using these functions.
Regards
Thank you. How about location translation???
-
Hey teseo
You are my HERO :)
Thank you very much. Your Code is helping me a lot.
Great Job 8), thx
-
Hi,
Hello, I'm trying to use a script to find the 9 version, everything works fine, except "Subscribe to this search", when I try to "Subscribe to this search" have lost site language, how can I fix it?
I think I've fixed now this problem (and others that I found while reviewing the code). Now there's new versions for "multilanguage search" only:
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113711/#msg113711 (5th version)
And "multilanguage search with sorting by relevance":
http://forums.osclass.org/3-5-x/search-problem-oc-3-5-3/msg113778/#msg113778 (10th version).
I strongly recommend updating to anyone using these functions.
Regards
Hello,
I copy the code into function.php but the search does not work on a multilingual site, where no title is specified, does not search in this language. How to start your function?
-
Hello, there is a solution for the osclass version 3.8 ?
My search finds ads only in the language in which they were published.
For example: the customer has published Apple iPhone 7, he used the English language of the site.
I use Russian version, I write Apple iPhone 7 in the search box, and I don’t find anything in the Russian version.
I turn webpage to English version, I write the same text "Apple iPhone 7". and an ad finds.
How to fix this issue?
Thanks
-
That is by design, you have to do custom changes in many places to make it work differently. I think there was one older post about instructions, not sure if it is still functional.