Avoid no keyword submit...anyone please help

Hello;

Is there anyway to avoid submit no keyword to the search form? I see that even there is no keyword; but if you just press Search it still go and search (of course there is no results). Or when the users access direct to localhost/sphider/search.php?q=&search=1 it also shows there is no results.

So how can avoid this please? Cause it could save some bandwith too.... Like if there is no keywords or access to url search without any keywords then it just redirect you to search.php.

Thank you

Hello hawkeye.

In search.php search for:

$search_results = get_search_results($query, $start, $category, $type, $results, $domain);
require("$template_dir/$template/search_results.html";
break;

Delete these three rows and replace them with the following:

if($query !=''){;
$search_results = get_search_results($query, $start, $category, $type, $results, $domain);
require("$template_dir/$template/search_results.html";
break;
}


Happy coding

Tec

thanks a lot....it works great

Hello; sorry to bother you again. I just want to ask; I have added this

if($query !=''){;
$search_results = get_search_results($query, $start, $category, $type, $results, $domain);
require("$template_dir/$template/search_results.html";
break;
}

i want also if there is a space but no keywords in the search so it also do the same. I have tried

if($query !=''){;
$search_results = get_search_results($query, $start, $category, $type, $results, $domain);
require("$template_dir/$template/search_results.html";
break;
}elseif($query !=' '){;
$search_results = get_search_results($query, $start, $category, $type, $results, $domain);
require("$template_dir/$template/search_results.html";
break;
}

but it doesnt work...what should i do?

Thank you

if($query !=''){;
if($query !=' '){;
$search_results = get_search_results($query, $start, $category, $type, $results, $domain);
require("$template_dir/$template/search_results.html";
break;
}
}


Tec

very cool, thank you!

Something that may work also and be a little cleaner

IF ( !EMPTY( TRIM( $query ) ) ) {
$search_results = get_search_results($query, $start, $category,
$type, $results, $domain );
require("$template_dir/$template/search_results.html";
}

remove the periods

Did you check your snippet before publishing here?

My result:
Fatal error: Can't use function return value in write context in C:\Programme\xampp\htdocs\...\sphider\search.php on line 130

Line 130: IF ( !EMPTY( TRIM ($query )) ) {

Reason: empty() only checks variables as anything else will result in a parse error. In other words, the following will not work: empty(trim($query)).

Tec

rr1024

You were fishing answers?

my bad...sorry didn't test it
FIND
if (isset($_GET['query']))
$query = $_GET['query'];
REPLACE WITH
if (isset($_GET['query']))
$query = TRIM( $_GET['query'] );

Here you may consider triming all your $_GET vars....up to you of course

Then do
IF ( !EMPTY( $query ) ) {
$search_results = get_search_results($query, $start, $category,
$type, $results, $domain );
require("$template_dir/$template/search_results.html";
}

I'm not sure what you mean by fishing answers? do you mean fishing for answers...then no I wasn't

Thanks, rr1024

To the atention of Ando:

In my experience, the first part of rr1024's code seems enough to correct the bug reported by hawkeye. So:

In search.php search for

if (isset($_GET['query']))
$query = $_GET['query'];
REPLACE WITH
if (isset($_GET['query']))
$query = TRIM( $_GET['query'] );

thank you very much