search result bug fix

[hubhuby]

using the advanced search, then clicking on a product will pass the parameter 'manufacturers_id=', if you dont select a manufacturer. this will prevent the category of the product being selected and other things going wrong, like the breadcrumb trail being 'home >> product model nr', instead of the whole category path.

try it on the demo shop.

 

fix:

in 'product_listing.php' in 'includes\modules'

 

find

			if (isset($HTTP_GET_VARS['manufacturers_id'])) {

 

and replace it with

			if ((isset($HTTP_GET_VARS['manufacturers_id'])) && tep_not_null($HTTP_GET_VARS['manufacturers_id'])) {

 

simple!

you have do replace that twice, for the links 'PRODUCT_LIST_NAME' and 'PRODUCT_LIST_IMAGE'

[spooks]

But manufacturers is only set if you've selected one, if you select a category its not, on what circumstance do you have manufacturers set but null?

[spooks]

Ignore me, I did'nt read your post properly.

[zdavatz]

When I search with the advanced mode and then I click onto the product link, then I get "product not found"

 

In the URL-field of the Browser I see the following link after clicking on the product:

 

http://organicbeauty.ch/product_info.php/m...products_id/940

 

If I shorten above URL to

 

http://organicbeauty.ch/product_info.php/products_id/940

 

then I get the product details.

 

Any hints on this problem?

 

Above solution did not work for me. I recieved a blank page. The search result was not even delivered.

 

Thanks

Zeno

[zdavatz]

All BS what I say! It works. I forgot to close the first bracket!

 

Sorry for the noise!

 

Zeno

[ezfindit]

using the advanced search, then clicking on a product will pass the parameter 'manufacturers_id=', if you dont select a manufacturer. this will prevent the category of the product being selected and other things going wrong, like the breadcrumb trail being 'home >> product model nr', instead of the whole category path.

try it on the demo shop.

 

fix:

in 'product_listing.php' in 'includes\modules'

 

find

			if (isset($HTTP_GET_VARS['manufacturers_id'])) {

 

and replace it with

			if ((isset($HTTP_GET_VARS['manufacturers_id'])) && tep_not_null($HTTP_GET_VARS['manufacturers_id'])) {

 

simple!

you have do replace that twice, for the links 'PRODUCT_LIST_NAME' and 'PRODUCT_LIST_IMAGE'

 

Worked perfectly! Thank you sooooo much! Spent several hours trying all sorts of changes with no luck. :( Then I started reading every Topic here that dealt with Advanced Search to see if anyone else had encountered this issue. Sure enough here it was.

 

Thank you!!

 

:thumbsup: :thumbsup: :thumbsup:

[kliksys]

Thanks Sam...