Jump to content

Recommended Posts

On 19/04/2017 at 1:59 AM, LeeFoster said:

@@bonbec,

 

Change line 46 in cm_nb_store_search from 

 

$script .= '<script src="' . tep_href_link('ext/modules/content/header/store_search/content_searches.min.js', null, $request_type) . '"></script>';

 

to 

 

$script .= '<script src="' . tep_href_link('ext/modules/content/navbar/store_search/content_searches.min.js', null, $request_type) . '"></script>';

Hi @LeeFoster

I am still experiencing issues with this Add On. For feedback, I think there are a number of remaining coding issues with it, including:

1. The content_search.min.js  in the ext /modules /content /navbar/store_search directory points to a  "header" location for content_search.php, not the file included with this Add On, therefore requiring that the original header search on which this version was based is installed. Pointing it to the right file breaks it for some reason. So it does not seem to work without the header version of the content_search.php file from the original contribution.

2. The navbar content_search.php file above requires the header add on version of the language file, not the language file included with this Add On.

3. The defined terms in the content_search.php file all reference MODULE_CONTENT_HEADER_...  not what appears in the language file with this Add On.

I think this Add On has great potential. But it needs some work. The search function itself works fine.

Peter

Edited by peterpil19

Full-time I am the COO of a large retail company in Australia.

In my spare time (what little I have), I enjoy buying, selling, and trading ancient coins and antiquities as a hobby: https://www.ancientcointraders.com.

I am enjoying learning about web-design.

 

osCommerce responsive is the best version of osCommerce.

Download it here: https://github.com/gburton/Responsive-osCommerce

Please donate so the team can continue their work on that project: https://pledgie.com/campaigns/31724

Share this post


Link to post
Share on other sites

@peterpil19

If you are trying to convert this add-on from a Header module to a Navbar module, see my post here:

And, more importantly, the post right below it.

HTH

Malcolm


Get the latest Responsive osCommerce CE (community edition) here .

Share this post


Link to post
Share on other sites

@ArtcoInc

Thanks,

LeeFoster's is an existing Add On which is supposed to do exactly that: https://apps.oscommerce.com/dS9oo&modular-navbar-search-w-images-bootstr

I tried your Add On (the second attachment with the missing file) but it would not function for some reason - I got a white screen below the navbar! Not having luck with this image search add on.  Even the original header one would not work with images showing when I installed it!

Peter


Full-time I am the COO of a large retail company in Australia.

In my spare time (what little I have), I enjoy buying, selling, and trading ancient coins and antiquities as a hobby: https://www.ancientcointraders.com.

I am enjoying learning about web-design.

 

osCommerce responsive is the best version of osCommerce.

Download it here: https://github.com/gburton/Responsive-osCommerce

Please donate so the team can continue their work on that project: https://pledgie.com/campaigns/31724

Share this post


Link to post
Share on other sites

Hi Malcom,

Modified:)

I haven't tried it yet on a clean install. If I get a chance to do so I will let you know if there are any issues.

Peter


Full-time I am the COO of a large retail company in Australia.

In my spare time (what little I have), I enjoy buying, selling, and trading ancient coins and antiquities as a hobby: https://www.ancientcointraders.com.

I am enjoying learning about web-design.

 

osCommerce responsive is the best version of osCommerce.

Download it here: https://github.com/gburton/Responsive-osCommerce

Please donate so the team can continue their work on that project: https://pledgie.com/campaigns/31724

Share this post


Link to post
Share on other sites

@peterpil19

 

On 10/16/2017 at 5:09 PM, peterpil19 said:

Hi Malcom,

Modified:)

I haven't tried it yet on a clean install. If I get a chance to do so I will let you know if there are any issues.

Peter

Have you had a chance to test this in a clean install?

Malcolm

Edited by ArtcoInc

Get the latest Responsive osCommerce CE (community edition) here .

Share this post


Link to post
Share on other sites

Hi Malcom,

Not yet.

I got the other add on working by making some changes,  but even then, without the photos working.  I am under the pump at the moment but plan to test it when I get a chance. When I get time I might actually just put together a simple search box in navbar contribution as I do not believe one exists. I do not think it is a difficult task to put one together.

Peter 


Full-time I am the COO of a large retail company in Australia.

In my spare time (what little I have), I enjoy buying, selling, and trading ancient coins and antiquities as a hobby: https://www.ancientcointraders.com.

I am enjoying learning about web-design.

 

osCommerce responsive is the best version of osCommerce.

Download it here: https://github.com/gburton/Responsive-osCommerce

Please donate so the team can continue their work on that project: https://pledgie.com/campaigns/31724

Share this post


Link to post
Share on other sites

Thanks for the add-on!

I have something odd happening, the search appears to work, I see a listing of the products being populated when I type.

However, when I click on the item I want to visit, I get a HTTP Forbidden error returned.

I assume it may be something in my .htaccess ?

I have cross site scripting disabled maybe that is doing it?  I'm really not sure what would cause the forbidden.  The URL looks correct.

Share this post


Link to post
Share on other sites
4 hours ago, ArtcoInc said:

@phi148

Is this on a live site? If so, do you have a link?

M

It is on a live site, so I disabled it until I can find out why I'm getting the 403 forbidden errors.

Share this post


Link to post
Share on other sites
[Sat Jul 21 20:35:39.370816 2018] [:error] [pid 14534:tid 139761074984704] [client xx.xx.xx.xx:62135] client denied by server configuration: /home/xxx/public_html/ext/modules/content/header/store_search/content_searches.php, referer: https://www.mysite.com/

I get the above error in my server logs. When I click on the item that appears in the search box.

Share this post


Link to post
Share on other sites

I found the problem.  "Mod_Evasive" must be turned off in the apache settings.  However, this is a bad thing, as mod_evasive helps to prevent DDOS attacks!

Share this post


Link to post
Share on other sites

I really like this search feature, except for 1 thing.  It auto-selects the first item in the search results that appear below the search window.  I'd rather not have any of them autoselected, so if the user just clicks "enter" it will automatically do an advanced_search rather than always picking the first item below the search box.

Any idea how to change the code to do that?

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×