preg_replace help needed! lol

PoorDoggie · 05-11-2006, 01:07 PM

this dosen't seem to work:

PHP Code:

  $old_q = urldecode($_GET['q']);

    $spelling = "<b>Spelling Suggestion:</b> <a href=\"search.php?".preg_replace("/q=".urlencode($old_q)."/","q=".urlencode($result['ResultSet']['Result']),$_SERVER['QUERY_STRING'])."\">Did you mean <i><b>".$result['ResultSet']['Result']."</b></i></a><br /><br />";

This is a part of the spell checking script on 6YD search.

Can anyone help me? If $_GET['q'] is one word, then the spell check works a dream - ie: http://www.6yd.net/search.php?q=scool&m=web but when you have two words - ie: http://www.6yd.net/search.php?q=scool+lessons&m=web then it spells it correctly in the link's text, but it dosen't correct it in the url. The link and text are in the variable $spelling by the way.

Tom

mikesherov · 05-11-2006, 01:52 PM

do the urlencoding after of the preg_replace, not before! I believe that's right.

:-)

PoorDoggie · 05-11-2006, 01:54 PM

will have a go... stand by

nope - even as:

PHP Code:

  $old_q = $_GET['q'];

$spelling = "<b>Spelling Suggestion:</b> <a href=\"search.php?".preg_replace("/q=$old_q/","q=".urlencode($result['ResultSet']['Result']),$_SERVER['QUERY_STRING'])."\">Did you mean <i><b>".$result['ResultSet']['Result']."</b></i></a><br /><br />";

I still get the same results... The search.php page up on 6yd at the moment has the script above in it, not the origional.

I think it could have something to do with the fact that it is "word+word" in the querystring, but not in a variable or something... anyone got any ideas or a better way of going about it?

PoorDoggie · 05-12-2006, 12:06 PM

Please help... anyone!

sacx13 · 05-12-2006, 12:40 PM

I just tested on my computer and is working in this way:

preg_replace("/q=(.+?)&/",'q='.urlencode($result['ResultSet']['Result'])."&",$_SERVER['QUERY_STRING'])

replace your preg_replace with the above one.

Regards

mikesherov · 05-12-2006, 12:41 PM

Sorry, I meant you have to url_encode the $_GET[old_q] variable before the replace because the server string will have it encoded, and the get superglobal will have it decoded!

So simply change:

PHP Code:

  $old_q = $_GET['q'];

to

PHP Code:

  $old_q = urlencode($_GET['q']);

and instead of preg_replace, just use str_replace! It is a bit faster and should be used when you're doing a literal replacement instead of a pattern replacement.

PoorDoggie · 05-12-2006, 01:15 PM

aha! works - You are a lifesaver!

have some rep

Quote:

Originally Posted by mikesherov

Sorry, I meant you have to url_encode the $_GET[old_q] variable before the replace because the server string will have it encoded, and the get superglobal will have it decoded!

So simply change:

PHP Code:

  $old_q = $_GET['q'];

to

PHP Code:

  $old_q = urlencode($_GET['q']);

and instead of preg_replace, just use str_replace! It is a bit faster and should be used when you're doing a literal replacement instead of a pattern replacement.

05-11-2006, 01:07 PM	#1 (permalink)
PoorDoggie Soon to be RICHdoggie! Join Date: Jan 2005 Location: UK Posts: 2,390 316.50 NP$ (Donate)	preg_replace help needed! lol this dosen't seem to work: PHP Code: `$old_q = urldecode($_GET['q']); $spelling = "<b>Spelling Suggestion:</b> <a href=\"search.php?".preg_replace("/q=".urlencode($old_q)."/","q=".urlencode($result['ResultSet']['Result']),$_SERVER['QUERY_STRING'])."\">Did you mean <i><b>".$result['ResultSet']['Result']."</b></i></a><br /><br />";` This is a part of the spell checking script on 6YD search. Can anyone help me? If $_GET['q'] is one word, then the spell check works a dream - ie: http://www.6yd.net/search.php?q=scool&m=web but when you have two words - ie: http://www.6yd.net/search.php?q=scool+lessons&m=web then it spells it correctly in the link's text, but it dosen't correct it in the url. The link and text are in the variable $spelling by the way. Tom __________________ █ Exchange your long URL for a much smaller one! █ Free Anonymizer/Proxy █ Taz Martin's blog - Learn all about internet marketing, promoting your website and much more!

05-11-2006, 01:54 PM	#3 (permalink)
PoorDoggie Soon to be RICHdoggie! Join Date: Jan 2005 Location: UK Posts: 2,390 316.50 NP$ (Donate)	will have a go... stand by nope - even as: PHP Code: `$old_q = $_GET['q']; $spelling = "<b>Spelling Suggestion:</b> <a href=\"search.php?".preg_replace("/q=$old_q/","q=".urlencode($result['ResultSet']['Result']),$_SERVER['QUERY_STRING'])."\">Did you mean <i><b>".$result['ResultSet']['Result']."</b></i></a><br /><br />";` I still get the same results... The search.php page up on 6yd at the moment has the script above in it, not the origional. I think it could have something to do with the fact that it is "word+word" in the querystring, but not in a variable or something... anyone got any ideas or a better way of going about it? __________________ █ Exchange your long URL for a much smaller one! █ Free Anonymizer/Proxy █ Taz Martin's blog - Learn all about internet marketing, promoting your website and much more!

05-12-2006, 12:06 PM	#4 (permalink)
PoorDoggie Soon to be RICHdoggie! Join Date: Jan 2005 Location: UK Posts: 2,390 316.50 NP$ (Donate)	Please help... anyone! __________________ █ Exchange your long URL for a much smaller one! █ Free Anonymizer/Proxy █ Taz Martin's blog - Learn all about internet marketing, promoting your website and much more!

05-12-2006, 12:40 PM	#5 (permalink)
sacx13 NamePros Regular Join Date: Mar 2006 Posts: 394 211.38 NP$ (Donate)	Reg ex. I just tested on my computer and is working in this way: preg_replace("/q=(.+?)&/",'q='.urlencode($result['ResultSet']['Result'])."&",$_SERVER['QUERY_STRING']) replace your preg_replace with the above one. Regards __________________ █ Free Pending&Expired Domains Tracker and more ... █ Webmaster's Talks Revenue Forum █ Webmaster Web Tools █ Hostmaster Web Tools █ PhotoGraphy Portal

05-12-2006, 12:41 PM	#6 (permalink)
mikesherov NamePros Member Join Date: Apr 2005 Posts: 116 134.00 NP$ (Donate)	Sorry, I meant you have to url_encode the $_GET[old_q] variable before the replace because the server string will have it encoded, and the get superglobal will have it decoded! So simply change: PHP Code: `$old_q = $_GET['q'];` to PHP Code: `$old_q = urlencode($_GET['q']);` and instead of preg_replace, just use str_replace! It is a bit faster and should be used when you're doing a literal replacement instead of a pattern replacement.

05-11-2006, 01:52 PM	#2 (permalink)
mikesherov NamePros Member Join Date: Apr 2005 Posts: 116 134.00 NP$ (Donate)	do the urlencoding after of the preg_replace, not before! I believe that's right. :-)

Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)