Follow us on Twitter X-Cart on Facebook Wiki
Shopping cart software Solutions for online shops and malls

XC SEO v1.1.0 Released

 
Reply
   X-Cart forums > X-Cart 4 > Dev Questions
 
Thread Tools Search this Thread
  #61  
Old 10-25-2006, 05:33 AM
 
Viejo Viejo is offline
 

Advanced Member
  
Join Date: Sep 2006
Location: San Diego area
Posts: 34
 

Default

In re feedback: I have 1.1.1 installed on two carts we are close to taking live. (4.1.3 and 4.0.19) There have been no issues at all to date. If there is a speed hit it appears to be slight. Urls work as expected. Customizations are a little tricky but we have made them to both the tpls and the root .htaccess. I'll rate the mod as top-notch and invaluable.

Intel352: Please pm me if you want to see our sites or know more about the how and why of our category handling. I don't want to hijack the thread.
__________________
Versions 4.3.1 - 4.5
Reply With Quote
  #62  
Old 10-25-2006, 05:47 AM
 
intel352 intel352 is offline
 

X-Wizard
  
Join Date: Dec 2005
Posts: 1,071
 

Default Re: XC SEO v1.1.0 Released

Update: the change that is needed for include/func/func.core.php in 4.1.x, should be done in include/func.php for 4.0.x

I'm working up updating the docs
__________________
-Jon Langevin
WARNING: Unethical developer - NOT RECOMMENDED
See details here
Reply With Quote
  #63  
Old 10-25-2006, 07:06 AM
  jasonroy's Avatar 
jasonroy jasonroy is offline
 

Senior Member
  
Join Date: Sep 2006
Location: Tualatin, Oregon
Posts: 147
 

Default Re: XC SEO v1.1.0 Released

going through the install on 4.1.3 right now.

noticed this...

the doc for 1.1.1 - 1.1.5 upgrade says

Quote:

-----------
Find Code:
-----------
if (!preg_match('!(\.htm(l)?|/$)!',$_tmp["path"])) {
func_header_location($l_redirect);
}
------------
Replace With
------------
################################################## ##########
# X-CART-SEO Mod :: http://code.google.com/p/x-cart-seo/
################################################## ##########
if (!empty($active_modules["XC_SEO"])) {
func_header_location($l_redirect);
}else{
if (!preg_match('!(\.htm(l)?|/$)!',$_tmp["path"])) {
func_header_location($l_redirect);
}
}
# End SEO

The only line that resembles what you've specified to find in my get_language.php file says

Quote:
if (
($current_area == "C" || $current_area == "B") &&
!empty($HTTP_GET_VARS["sl"]) &&
!defined('IS_ROBOT') &&
!preg_match('/\.html?($|\?)|\/$/s', $l_redirect)
) {
func_header_location($l_redirect);
}

Either way I don't think it matters since it says its optional and I run a single language on my site.
__________________
rjroffroad.com
x-cart 4.1.3
+ez checkout
+remember anonymous carts
Reply With Quote
  #64  
Old 10-25-2006, 08:34 AM
 
intel352 intel352 is offline
 

X-Wizard
  
Join Date: Dec 2005
Posts: 1,071
 

Default Re: XC SEO v1.1.0 Released

Nice catch jason, I'll have to doublecheck against 4.1.3 (argh, 4.1.3 had a number of changes, lol...)

EDIT: btw, the point of that edit, is that the get_language file is NOT redirecting of the previous page had a .html extension. Which means it won't redirect back to an SEO'd url... :-\ Really I need a better workaround that what I implemented, just haven't come up with one yet
__________________
-Jon Langevin
WARNING: Unethical developer - NOT RECOMMENDED
See details here
Reply With Quote
  #65  
Old 10-25-2006, 09:11 AM
 
Scott DeToffol Scott DeToffol is offline
 

Senior Member
  
Join Date: Apr 2005
Location: Minneapolis, MN
Posts: 174
 

Default Re: XC SEO v1.1.0 Released

I've noticed that when moving from http to https that the urls are not reformatted. Is this as designed?
__________________
Scott DeToffol
Online Marketing and E-Commerce Consulting in Minneapolis

X-Cart Gold Version 4.1.9 for www.sundogtrading.com
X-Cart Pro Version 4.2.3 for www.heavyglare.com
Reply With Quote
  #66  
Old 10-25-2006, 09:23 AM
 
intel352 intel352 is offline
 

X-Wizard
  
Join Date: Dec 2005
Posts: 1,071
 

Default Re: XC SEO v1.1.0 Released

Yes, the thought process is that search engines *shouldn't* be accessing your HTTPS site, and additionally most stores force users back to the http domain when they are wanting to browse more products. Also, wanted to avoid any unforseen issues with https urls (i.e.- in the event that a site is on shared SSL, there can be problems...)

If you would like to argue against this thought process, by all means, be my guest Additionally, if you would like to modify the code so that it functions under HTTPS, to test in your environment, let me know how it goes. I could implement an option to toggle support for rewriting under HTTPS (with an "At your own risk" disclaimer)
__________________
-Jon Langevin
WARNING: Unethical developer - NOT RECOMMENDED
See details here
Reply With Quote
  #67  
Old 10-25-2006, 09:57 AM
 
Scott DeToffol Scott DeToffol is offline
 

Senior Member
  
Join Date: Apr 2005
Location: Minneapolis, MN
Posts: 174
 

Default Re: XC SEO v1.1.0 Released

Quote:
Update: the change that is needed for include/func/func.core.php in 4.1.x, should be done in include/functions.php for 4.0.x

I don't have include/functions.php, but I do have include/func.php. I made the change there and got a SQL error:

Quote:
You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'sql/xcseo.sql
sql/xcseo_lng_US.sql' at line 1
Backtrace :
/include/func.php:121
/include/func.php:80
/admin/patch_sql.php:179
/admin/patch.php:515
__________________
Scott DeToffol
Online Marketing and E-Commerce Consulting in Minneapolis

X-Cart Gold Version 4.1.9 for www.sundogtrading.com
X-Cart Pro Version 4.2.3 for www.heavyglare.com
Reply With Quote
  #68  
Old 10-25-2006, 10:04 AM
 
Scott DeToffol Scott DeToffol is offline
 

Senior Member
  
Join Date: Apr 2005
Location: Minneapolis, MN
Posts: 174
 

Default Re: XC SEO v1.1.0 Released

I'm still getting Google Base/Froogle errors with the following characters:




®
²
½
__________________
Scott DeToffol
Online Marketing and E-Commerce Consulting in Minneapolis

X-Cart Gold Version 4.1.9 for www.sundogtrading.com
X-Cart Pro Version 4.2.3 for www.heavyglare.com
Reply With Quote
  #69  
Old 10-25-2006, 10:05 AM
 
intel352 intel352 is offline
 

X-Wizard
  
Join Date: Dec 2005
Posts: 1,071
 

Default Re: XC SEO v1.1.0 Released

I apologize, please look at the specified files:

sql/xcseo.sql
sql/xcseo_lng_US.sql

And copy the SQL commands from there. The install instructions refer to those files so that the instructions are less redundant/shorter, since the SQL is the same across 4.0.x,4.1.x
__________________
-Jon Langevin
WARNING: Unethical developer - NOT RECOMMENDED
See details here
Reply With Quote
  #70  
Old 10-25-2006, 10:09 AM
 
intel352 intel352 is offline
 

X-Wizard
  
Join Date: Dec 2005
Posts: 1,071
 

Default Re: XC SEO v1.1.0 Released

Regarding your Froogle errors, please refer to the Install instructions for 4.0x, scroll to the Froogle modification instructions, and make sure that your Froogle file is edited as instructed.
__________________
-Jon Langevin
WARNING: Unethical developer - NOT RECOMMENDED
See details here
Reply With Quote
Reply
   X-Cart forums > X-Cart 4 > Dev Questions


Thread Tools Search this Thread
Search this Thread:

Advanced Search

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Forum Jump


All times are GMT -8. The time now is 03:04 AM.

   

 
X-Cart forums © 2001-2020