X-Cart forums

[intel352]

@geckoday: you mentioned you implemented a stripped down version. out of curiousity, what did you strip out?

[geckoday]

Originally Posted by intel352

might i ask, how many products do you have in your test store? i think with stores that have less urls to display, it's not so bad, but if you were to go to a product page that had 100+ urls, it might get a bit intensive (my employer's website has a ton of cross-referencing links over the entire site, since our site is based around music, and every CDG, Artist, and Song is a product link)

I have 16 products/page. I just loaded up some test products and tried 200 products/page. It wasn't a very good test as the page generation time and size of the HTML page (118K) was unacceptable even without the output filter. But for reference XDebug was showing about 1/10 second increase in CPU and elapsed page generation time went from about 4.75 seconds to 5.37 seconds with the filter. I think most of that time is due to the large size of the HTML. If I customized the templates to generate much more compact HTML I think the difference would be far less.

BTW, when you are testing you can add a query string variable benchmark=y (e.g. http://www.example.com/home.php?cat=262&benchmark=y) to your URL's and then if you list the page source you'll see page generation times at the end of the source.

[geckoday]

Originally Posted by intel352

@geckoday: you mentioned you implemented a stripped down version. out of curiousity, what did you strip out?

I wasn't ready to spend the time debugging as I have way too many other projects on my plate for the holiday season. So I went for bare bones get the job done simplicity and cut out a lot of the features. I really gutted it and borrowed some from the HTML catalog generation to simplify. Lofi, config stuff, category folders, image links and any pages other than category, product or manufacturers are all gone. Any links with extraneous query string parameters are ignored (e.g. sort= on category pages).

[B00MER]

What did one regular expression say to the other?

.+

[intel352]

When testing, assuming you're intending to apply this to a live site, please carefully test the effects of the SEO mod when in SSL mode (if you will be incorporating SSL into your store). This was an area that the mod suffered in during testing last I remember.

I appreciate all your feedback on this btw. If you reach a point where you're happy with the stripped-down mod, would you mind sending me a copy of your stripped implementation, so I can view the code changes? It might be a good idea to offer a SEO-Lite version as well, I'm thinking.

Email is intel352 [AT] gmail [DOT] com

[intel352]

Originally Posted by B00MER

What did one regular expression say to the other?

.+

lmfao

[geckoday]

Originally Posted by intel352

When testing, assuming you're intending to apply this to a live site, please carefully test the effects of the SEO mod when in SSL mode (if you will be incorporating SSL into your store). This was an area that the mod suffered in during testing last I remember.

I appreciate all your feedback on this btw. If you reach a point where you're happy with the stripped-down mod, would you mind sending me a copy of your stripped implementation, so I can view the code changes? It might be a good idea to offer a SEO-Lite version as well, I'm thinking.

Email is intel352 [AT] gmail [DOT] com

I have had it running live and taking orders for several weeks now. I have found one problem where x-cart adds a query string ?xid=xxxxxxxxxxx when leaving checkout pages and my .htaccess ignores it causing page not found. I'm going to update it to allow query strings then I'll send it along.

[intel352]

Great, thanks gecko

[JeremyL]

I have updated the install docs. They will be included in all new release downloads and can be found here

http://x-cart-seo.googlecode.com/svn/trunk/x-cart-seo%20docs/

[Jayk]

In the install docs, under files that need to be edited, you show:

/smarty.php
/auth.php
/smarty.php

I think the first one is supposed to be config.php.

Cheers,
Jason