Ultimate SEO v3 (and v2.200) [Support Thread]

[lat9]

Hi Cindy, many thanks for that explanation and for the time and effort you have put into this issue. I am often slow on the uptake so just checking that I have got this right


I appreciate it being a best guess given your inheritance of the plugin! .... and I do not see it as a plugin issue anyway.

Here is how I see it. USU worked like a charm in v1.5.1 with the /includes/init_includes/overrides/init_canonical.php included the file package and SitemapXML files were generated correctly using USU generated url's.

USU v2 does work in v1.5.5f in that it does create the user friendly URL's as it is designed to - the problem is that those URL's are not generated in the SitemapXML files - given that USU and SitemapXML versions were the same (as used in v1.5.1, and still valid) at the time of upgrade to v1.5.5f, it appears something is not correlating between ZC core files in v1.5.5f and USU and SitemapXML - given that USU is generating the files correctly then it appears something is not telling SitemapXML to produce USU generated URLs in its files. The only change in the process is changes to ZC core files in v1.5.5f - so what changed in the core of v1.5.5f to create this 'imbalance' ? If that change could be isolated it may (should) solve the issue - I really do not want to upgrade to v1.5.6 any time soon (still recovering from 155f upgrades).

I have converted all my v1.5.5f sites to CEON URI Mapping so to test your theory of removing /includes/init_includes/overrides/init_canonical.php from USU v2 I will need to look at how I can do that without creating chaos

I haven't checked it out but maybe I can have both plugins installed without creating conflict and switch between them - in your opinion is that likely to be possible?

cheers and thanks for your continuing help,
Mike

Mike, I don't believe that the two will work "in harmony" as both USU and Ceon require quite different changes to the site's .htaccess file and with Ceon's "override" of init_canonical.php.

[lat9]

Mike, I don't believe that the two will work "in harmony" as both USU and Ceon require quite different changes to the site's .htaccess file and with Ceon's "override" of init_canonical.php.

FWIW, I just installed Sitemap XML 3.9.5 and USU 3.0.0-beta3 on an Edit Orders/zc155f test site (w/ demo products) and I'm happy to report that the USU-generated links are used for the sitemap-generated files.

Here's, for example, the contents of the manufacturers' one:

<?xml version="1.0" encoding="UTF-8"?>
<?xml-stylesheet type="text/xsl" href="/zc155feo/includes/templates/template_default/css/gss.xsl"?>
<urlset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xsi:schemaLocation="http://www.sitemaps.org/schemas/sitemap/0.9 http://www.sitemaps.org/schemas/sitemap/0.9/sitemap.xsd"
        xmlns:image="http://www.google.com/schemas/sitemap-image/1.1"
        xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
<!-- generator="Zen-Cart SitemapXML" 3.9.5 -->
<!-- sitemapmanufacturers.xml created at 2019-02-23 08:38:45 -->
 <url>
  <loc>http://localhost/zc155feo/matrox-m-1.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>1.00</priority>
 </url>
 <url>
  <loc>http://localhost/zc155feo/microsoft-m-2.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>0.89</priority>
 </url>
 <url>
  <loc>http://localhost/zc155feo/warner-m-3.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>0.78</priority>
 </url>
 <url>
  <loc>http://localhost/zc155feo/fox-m-4.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>0.67</priority>
 </url>
 <url>
  <loc>http://localhost/zc155feo/logitech-m-5.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>0.56</priority>
 </url>
 <url>
  <loc>http://localhost/zc155feo/canon-m-6.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>0.44</priority>
 </url>
 <url>
  <loc>http://localhost/zc155feo/sierra-m-7.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>0.33</priority>
 </url>
 <url>
  <loc>http://localhost/zc155feo/gt-interactive-m-8.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>0.22</priority>
 </url>
 <url>
  <loc>http://localhost/zc155feo/hewlett-packard-m-9.html</loc>
  <lastmod>2003-12-23</lastmod>
  <changefreq>weekly</changefreq>
  <priority>0.11</priority>
 </url>
</urlset>

[jeking]

FWIW, I just installed Sitemap XML 3.9.5 and USU 3.0.0-beta3 on an Edit Orders/zc155f test site (w/ demo products) and I'm happy to report that the USU-generated links are used for the sitemap-generated files.

That's great to know. I'm also curious about the canonical. Are those USU links too?

[shrimp-gumbo-mmmhhh]

1) I started with completely fresh install script
2) Loaded the original usu-3.0.0-beta2 (this version has the known issue of not working correctly when "Enable Automatic Redirects" is set to TRUE) Install successful
3) Went to Admin -> Catalog/Products
4) Selected a Category
5) Was given the message "

[shrimp-gumbo-mmmhhh]

Well, I had a ton of information on there that got lost.... basically there is a bug in /includes/classes/usu.php

change:

$check = $GLOBALS['db']->Execute(
"SELECT pt.products_type

to:

$check = $GLOBALS['db']->Execute(
"SELECT p.products_type


And you will not get the error message I tried to post.



#10 zen_catalog_href_link() called at [/home/com/public_html/site/admin/category_product_listing.php:861]
--> PHP Fatal error: 1054:Unknown column 'pt.products_type' in 'field list' :: SELECT pt.products_type
FROM product_types pt
INNER JOIN products p
ON p.products_id = pt.type_id
WHERE pt.type_id = 35259
LIMIT 1 ==> (as called by) /home/com/public_html/site/includes/classes/usu.php on line 457 <== in /home/com/public_html/site/includes/classes/db/mysql/query_factory.php on line 171.



Was there a quick and simple fix for the Enable Automatic Redirects that I could make on this version of my code?

[lat9]

That's great to know. I'm also curious about the canonical. Are those USU links too?

Yep, the canonical links are also the USU-generated ones.

[shrimp-gumbo-mmmhhh]

Okay, got the usu-3.0.0-beta3 and moved over and everything works fine.... you had already fixed that issue.

Thank you!

[lat9]

Well, I had a ton of information on there that got lost.... basically there is a bug in /includes/classes/usu.php

change:

$check = $GLOBALS['db']->Execute(
"SELECT pt.products_type

to:

$check = $GLOBALS['db']->Execute(
"SELECT p.products_type


And you will not get the error message I tried to post.



#10 zen_catalog_href_link() called at [/home/com/public_html/site/admin/category_product_listing.php:861]
--> PHP Fatal error: 1054:Unknown column 'pt.products_type' in 'field list' :: SELECT pt.products_type
FROM product_types pt
INNER JOIN products p
ON p.products_id = pt.type_id
WHERE pt.type_id = 35259
LIMIT 1 ==> (as called by) /home/com/public_html/site/includes/classes/usu.php on line 457 <== in /home/com/public_html/site/includes/classes/db/mysql/query_factory.php on line 171.



Was there a quick and simple fix for the Enable Automatic Redirects that I could make on this version of my code?

Actually, that function should be changed as follows (GitHub issue: https://github.com/lat9/usu/issues/15):

    protected function getInfoPage($products_id)
    {
        // -----
        // Quick return if the zen_get_info_page function exists, noting that when
        // run in the zc156 and earlier admin that it isn't!
        //
        if (function_exists('zen_get_info_page')) {
            return zen_get_info_page($products_id);
        }
        
        // -----
        // If the function doesn't exist, emulate its output.
        //
        $check = $GLOBALS['db']->Execute(
            "SELECT pt.type_handler
               FROM " . TABLE_PRODUCTS . " p
                    INNER JOIN " . TABLE_PRODUCT_TYPES . " pt
                        ON pt.type_id = p.products_type
              WHERE p.products_id = " . (int)$products_id . "
              LIMIT 1"
        );
        return ($check->EOF) ? 'product_info' : ($check->fields['type_handler'] . '_info');
    }

The change needed for the automatic redirects is already "baked into" USU v3.0.0-beta3.

[lat9]

v3.0.0-beta4 is now available for testing: https://github.com/lat9/usu/releases/tag/v3.0.0-beta4

[jeking]

Yep, the canonical links are also the USU-generated ones.