Jump to content


Corporate Sponsors


Latest News: (loading..)

- - - - -

search result bug fix


6 replies to this topic

#1 hubhuby

  • Community Member
  • 3 posts
  • Real Name:Huber

Posted 12 July 2008, 08:05

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'

#2 spooks

  • Community Member
  • 7,017 posts
  • Real Name:Sam
  • Gender:Male
  • Location:UK

Posted 12 July 2008, 09:19

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?
Sam

Remember, What you think I ment may not be what I thought I ment when I said it.

Contributions:


Auto Backup your Database, Easy way

Multi Images with Fancy Pop-ups, Easy way

Products in columns with multi buy etc etc

Disable any Category or Product, Easy way

Secure & Improve your account pages et al.

#3 spooks

  • Community Member
  • 7,017 posts
  • Real Name:Sam
  • Gender:Male
  • Location:UK

Posted 12 July 2008, 10:52

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

Remember, What you think I ment may not be what I thought I ment when I said it.

Contributions:


Auto Backup your Database, Easy way

Multi Images with Fancy Pop-ups, Easy way

Products in columns with multi buy etc etc

Disable any Category or Product, Easy way

Secure & Improve your account pages et al.

#4 zdavatz

  • Community Member
  • 51 posts
  • Real Name:Zeno R.R. Davatz
  • Gender:Male
  • Location:Zürich

Posted 10 September 2008, 13:31

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

Edited by zdavatz, 10 September 2008, 13:32.


#5 zdavatz

  • Community Member
  • 51 posts
  • Real Name:Zeno R.R. Davatz
  • Gender:Male
  • Location:Zürich

Posted 10 September 2008, 13:49

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

Sorry for the noise!

Zeno

#6 ezfindit

  • Community Member
  • 26 posts
  • Real Name:Chris
  • Gender:Male

Posted 24 November 2009, 05:21

View Posthubhuby, on 12 July 2008, 08:05, said:

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:
Chris

#7 kliksys

  • Community Member
  • 22 posts
  • Real Name:Dutch
  • Gender:Male
  • Location:Atlanta, GA

Posted 22 December 2011, 06:38

Thanks Sam...
Knowledge is useless, unless transferred.