3897 replies to this topic

[caravansplus]

FWR Media, on 02 August 2012 - 12:43 PM, said:

Sorry I didn't reply to your personal contact, caught me a bit busy. If the URLs are being generated fine then it is one of a few things. 1) .htaccess settings ( although I wouldn't expect a redirect loop from here ) 2) Admin settings - turn off the option to redirect index.php to root domain. 3) Your includes/configure.php settings are incorrect. 4) You are using path_standard and AcceptPathInfo is not set to On ( this can go in .htaccess ) 5) You have a very strnge server set up that I've never come across Is it a linux server?

RE: Redirect Loop

Hi Robert,

Thanks for the reply.  I hope I can eliminate at few options and give some more clues as to where the problem lies.

I'm on a Unix server, I dont think there is any problem there.  I have another site that uses the original SEO URL and works fine.

I have got the redirect index.php to root domain turned off in admin.

I have tried all the url path admin settings with the same result.  path_rewrite is my priority.

If I Enable SEO URLs 5, it will generate a nice url but I get the redirect loop message.  If I disable the module and then refresh the product_info page it will use the .htaccess rule

RewriteRule ^([a-z0-9/-]+)-p-([0-9]+).html$ product_info.php [NC,L,QSA]

to display the product_info page but goes to "no product found"

When I use a modified rule

RewriteRule ^([a-z0-9/-]+)-p-([0-9]+).html$  product_info.php?products_id=$2  [NC,L,QSA]

It will display the product_info page properly, but when I enable the module it has the redirect loop error again.

So the .htaccess redirect works but not when the module is enabled.

I think the problem is separate to the .htaccess settings.  Even when I strip all the rewrites from .htaccess I get the same redirect loop error.

If there is a problem in my includes/configure.php settings I can't see it, but am happy to show them to you.

Another clue may be when I first did the install the cache database table did not install and I had to run SQL query manually to insert the usu_cache table.  I have tried turning the enable cache setting off in admin.  Also when I try Reset USU5 Cache I get an error (but this might just be because the table is already empty).

I don't know if I have narrowed it down at all...

Thanks for your time.  Steve

[FWR Media]

This worries me: -

Quote

RewriteRule ^([a-z0-9/-]+)-p-([0-9]+).html$ product_info.php [NC,L,QSA]

to display the product_info page but goes to "no product found"

When I use a modified rule

RewriteRule ^([a-z0-9/-]+)-p-([0-9]+).html$ product_info.php?products_id=$2 [NC,L,QSA]

It will display the product_info page properly, but when I enable the module it has the redirect loop error again.

That is incorrect ( on any version of server I've ever seen ).

RewriteRule ^([a-z0-9/-]+)-p-([0-9]+).html$ product_info.php [NC,L,QSA]

Cannot match a standard product_info.php url, what your new RewriteRule did was pretty much nothing. QSA means Query String Attached.

Have you asked the people who run your server because in years of this contrib being available I've never heard anything like this.

[theantiquestore]

I have updated php on my sever and now I am seeing some errors. I think I have fixed them all but I cant seem to figure this one out.

error:

PHP Notice:  Undefined property: SEO_URL::$keep_in_memory in /home/xxx/public_html/includes/classes/seo.class.php on line 1957

Line 1957 has this:

if ( $this->keep_in_memory || $local_memory ) {

Any help would be appreciated (sorry if its already been addressed and I missed it)

[FWR Media]

theantiquestore, on 05 August 2012 - 06:22 PM, said:

I have updated php on my sever and now I am seeing some errors. I think I have fixed them all but I cant seem to figure this one out.

error:

PHP Notice:  Undefined property: SEO_URL::$keep_in_memory in /home/xxx/public_html/includes/classes/seo.class.php on line 1957

Line 1957 has this:

if ( $this->keep_in_memory || $local_memory ) {

Any help would be appreciated (sorry if its already been addressed and I missed it)

That is Chemos old 2002 code not USU5.

[theantiquestore]

FWR Media, on 05 August 2012 - 07:00 PM, said:

That is Chemos old 2002 code not USU5.

Ok, I'll take some time and upgrade. Thanks rob!

[theantiquestore]

well i upgraded and now I get this error and the site wont load:

Fatal error: Uncaught exception 'SQLiteException' with message 'SQLiteDatabase::__construct() [<a href='sqlitedatabase.--construct'>sqlitedatabase.--construct</a>]: unable to open database: /home/track/public_html/includes/modules/ultimate_seo_urls5/cache_system/sqlite/usu_cache.db' in /home/track/public_html/includes/modules/ultimate_seo_urls5/cache_system/sqlite.php:99 Stack trace: #0 /home/track/public_html/includes/modules/ultimate_seo_urls5/cache_system/sqlite.php(99): SQLiteDatabase->__construct('/home/track/pub...', 438, 'unable to open ...') #1 /home/track/public_html/includes/modules/ultimate_seo_urls5/cache_system/sqlite.php(66): Sqlite_Cache_Module::createDatabase() #2 [internal function]: Sqlite_Cache_Module::i() #3 /home/track/public_html/includes/modules/ultimate_seo_urls5/main/bootstrap.php(515): call_user_func(Array) #4 /home/track/public_html/includes/modules/ultimate_seo_urls5/main/bootstrap.php(86): Usu5_Bootstrap->cacheSystem() #5 /home/track/public_html/includes/modules/ultimate_seo_urls5/main/usu5.ph in /home/track/public_html/includes/modules/ultimate_seo_urls5/cache_system/sqlite.php on line 99

[theantiquestore]

forgot to add, that the sqlite is at 777

[theantiquestore]

ok, my program wasnt allowing me to copy an empty folder. I created the folders and changed them to 777.

Also, where is the manufacturer files? I have 4 manufacturer places and need to duplicate the code so that I will have urls for all 4 of them.

Edited by theantiquestore, 05 August 2012 - 11:12 PM.

[FWR Media]

theantiquestore, on 05 August 2012 - 11:01 PM, said:

Also, where is the manufacturer files? I have 4 manufacturer places and need to duplicate the code so that I will have urls for all 4 of them.

extras/module_template.php/Usu_Template.php

Is a blank template for devs, look for xxx XXX

catalog/includes/modules/ultimate_seo_urls5/page_modules/index.php has code to produce manufacturers name although it differs from the template as it has more than one dependency.

[theantiquestore]

FWR Media, on 06 August 2012 - 06:28 AM, said:

extras/module_template.php/Usu_Template.php

Is a blank template for devs, look for xxx XXX

catalog/includes/modules/ultimate_seo_urls5/page_modules/index.php has code to produce manufacturers name although it differs from the template as it has more than one dependency.

got the manufacturers to work, now to figure out how to get the htaccess to work with the old urls (from chemos USEO contrib).

[theantiquestore]

I am also getting the redirect loop when trying to set it to rewrite. (although it looks like the url would be correct). I have the force www to false.

Here is what I have in my .htaccess:

RewriteCond %{HTTP_HOST} ^mysite.com$ [NC]
RewriteRule ^(.*)$ http://www.mysite.com/$1 [R=301,L]
RewriteBase /
RewriteRule ^(.*)-p-(.*).html$ product_info.php?products_id=$2&%{QUERY_STRING}
RewriteRule ^(.*)-c-(.*).html$ index.php?cPath=$2&%{QUERY_STRING}
RewriteRule ^(.*)-m-([0-9]+).html$ index.php?manufacturers_id=$2&%{QUERY_STRING}
RewriteRule ^(.*)-m2-([0-9]+).html$ index.php?manufacturers2_id=$2&%{QUERY_STRING}
RewriteRule ^(.*)-m3-([0-9]+).html$ index.php?manufacturers3_id=$2&%{QUERY_STRING}
RewriteRule ^(.*)-m4-([0-9]+).html$ index.php?manufacturers4_id=$2&%{QUERY_STRING}
RewriteRule ^(.*)-pi-([0-9]+).html$ popup_image.php?pID=$2&%{QUERY_STRING}
RewriteRule ^(.*)-t-([0-9]+).html$ articles.php?tPath=$2&%{QUERY_STRING}
RewriteRule ^(.*)-a-([0-9]+).html$ article_info.php?articles_id=$2&%{QUERY_STRING}
RewriteRule ^(.*)-pr-([0-9]+).html$ product_reviews.php?products_id=$2&%{QUERY_STRING}
RewriteRule ^(.*)-pri-([0-9]+).html$ product_reviews_info.php?products_id=$2&%{QUERY_STRING}
RewriteRule ^(.*)-i-([0-9]+).html$ information.php?info_id=$2&%{QUERY_STRING}

Did I do something wrong?

[FWR Media]

@theantiquestore

Got a link? PM me if you want kept secret.

Is it all looping or just the ones you created?

Edited by FWR Media, 06 August 2012 - 05:17 PM.

[theantiquestore]

FWR Media, on 06 August 2012 - 05:16 PM, said:

@theantiquestore

Got a link? PM me if you want kept secret.

Is it all looping or just the ones you created?

PM sent.

It is all looping.

[theantiquestore]

I also need to have the number 8 show up for 8 track tape, is there a way to turn off the filtering of 1 letter words?

[FWR Media]

PHP 5.4.5 compatible

Just to confirm that I have just tested USU5 PRO on osCommerce 2.3.2 with the server running PHP 5.4.5 and all works as intended.

[theantiquestore]

ok, have it resolved, my htaccess was incorrect. It now looks like this:

Options +SymLinksIfOwnerMatch
<IfModule mod_rewrite.c>
  RewriteEngine On

  RewriteBase /
  RewriteRule ^([a-z0-9/-]+)-p-([0-9]+).html$ product_info.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-c-([0-9_]+).html$ index.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-m-([0-9]+).html$ index.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-pi-([0-9]+).html$ popup_image.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-pr-([0-9]+).html$ product_reviews.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-pri-([0-9]+).html$ product_reviews_info.php [NC,L,QSA]
  # Articles contribution
  RewriteRule ^([a-z0-9/-]+)-t-([0-9_]+).html$ articles.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-au-([0-9]+).html$ articles.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-a-([0-9]+).html$ article_info.php [NC,L,QSA]
  # Information pages
  RewriteRule ^([a-z0-9/-]+)-i-([0-9]+).html$ information.php [NC,L,QSA]
  # Links contribution
  RewriteRule ^([a-z0-9/-]+)-links-([0-9_]+).html$ links.php [NC,L,QSA]
  # Newsdesk contribution
  RewriteRule ^([a-z0-9/-]+)-n-([0-9]+).html$ newsdesk_info.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-nc-([0-9]+).html$ newsdesk_index.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-nri-([0-9]+).html$ newsdesk_reviews_info.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-nra-([0-9]+).html$ newsdesk_reviews_article.php [NC,L,QSA]
  # Manufacturers
  RewriteRule ^([a-z0-9/-]+)-m2-([0-9]+).html$ index.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-m3-([0-9]+).html$ index.php [NC,L,QSA]
  RewriteRule ^([a-z0-9/-]+)-m4-([0-9]+).html$ index.php [NC,L,QSA]

ETA: I have the rewrite set in the URI format in the admin.

Edited by theantiquestore, 06 August 2012 - 07:07 PM.

[marcot]

Hello FWR,

It seems that the url validation isn't working for categories in categories.

first we have this breadcrumb

categorie1 (21) >> product listing (71)> product info
lets say with url:  productname-c-21_71.html

we have moved categorie1 into a new categorie because of the many products (more sorted out)
so now we have this breadcrumb

categorie1 (86) >> categorie2 (21) >> product listing (71) > product info
with url:  productname-c-86_21_71.html

same as:

categorie-c-70.html
categorie-c-86_70.html


but now both urls are available (this gives duplicate content in google)
any solution would be nice..

regards Marco

[FWR Media]

marcot, on 07 August 2012 - 12:40 PM, said:

Hello FWR,

It seems that the url validation isn't working for categories in categories.

That doesn't make sense Marco you mention productname-c- which never happens it can only be categoryname-c-

If you have categories in two locations there is nothing USU5 can do except create the URLs you have asked it to produce, you HAVE created pure duplicate content, 2 pages the same with two different links, there's nothing USU5 can do about it.

If you really must have categories in two locations then you need the same canonical element for both pages which links to one of the URLs which thereby notifies the search engines that the two pages should be taken as a single page.

[marcot]

I dont understand it exactly,

but why is google complaining about duplicated content? double title's and descriptions..?

live preview: http://www.mangelot.nl/budget-webhosting-1-core-c-86_21_71.html
goes to the same page as http://www.mangelot.nl/budget-webhosting-1-core-c-21_71.html
so generates duplicate content both url have a own canonical link etc..

notice also the breadcrumb (categorie 86 "shared hosting" is missing in breadcrumb)
is there an simple solution that if page -c-21_71.html is called this redirects to -c-86_21_71.html?

regards Marco

[FWR Media]

marcot, on 08 August 2012 - 06:55 AM, said:

I dont understand it exactly,

but why is google complaining about duplicated content? double title's and descriptions..?

Firstly this is not a subject for this support thread as it supports issues with USU5. you should ask this in the general forums.

Having said that I'll answer the question.

Google is complaining quite correctly as it IS duplicate content.

marcot, on 08 August 2012 - 06:55 AM, said:

live preview: http://www.mangelot.nl/budget-webhosting-1-core-c-86_21_71.html
goes to the same page as http://www.mangelot.nl/budget-webhosting-1-core-c-21_71.html
so generates duplicate content both url have a own canonical link etc..

notice also the breadcrumb (categorie 86 "shared hosting" is missing in breadcrumb)
is there an simple solution that if page -c-21_71.html is called this redirects to -c-86_21_71.html?

regards Marco

If these seperate pages have canonical links that point to themselves then the canonical link is simply compounding the fact that you have duplicate content.

No you can't have a permanent 301 redirect from one to the other, your site is stating to the search engines that these pages exist ( they exist on your website ) so to permanently redirect them simply doesn't make sense ( and would be hurtful ).

As I said before, if you are going to create duplicate content like this then both pages need to have a canonical element that is THE SAME URI for both pages ( e.g. both -c-86_21_71.html ).