Ceon URI Mapping (SEO)

[conor]

Hi,

i have the ".=>," added to character/string replacements config, but don't replace the "." in a "dr." by manufacturer mapping. When the "." exists in a product name is ok.

You're right, you've found a bug in the autogeneration algorithms for the Manufacturers and EZ-Pages. The replacement words aren't being used correctly for those two page types. I'll include a fix for this in the next version. In the meantime you can apply the fix yourself by changing a few lines in

admin/includes/functions/extra_functions/ceon_uri_mapping.php

Change lines 1231-1232 from:

                $category_and_product_path_array[$i] = String::regexpReplace(
                $category_and_product_path_array[$i], 

to:

                $ez_page_name = String::regexpReplace($ez_page_name, 

And change lines 1467-1468 from:

                $category_and_product_path_array[$i] = String::regexpReplace(
                $category_and_product_path_array[$i], 

to:

            $manufacturer_name = String::regexpReplace($manufacturer_name, 

That'll get things working as expected.

Sorry about the bug.. that one's been there for years and managed to slip past, guess not many use this functionality in EZ-Page/Manufacturer names!

All the best..

Conor
ceon

[kamelion0927]

Hi,



If you don't see the configuration options in the admin then you haven't installed the module properly. Start again, following the installation instructions from the beginning to the end. Don't skip any steps, such as going to the Ceon URI Mapping Config page as it is the step that creates the database tables and admin configuration option.

All the best..

Conor
ceon

Thanks to everyone that answered! I know it's some kind of user error - I just wanted to make sure I wasn't for nothing.

[lat9]

FWIW, here's the patch I made to /includes/functions/html_output.php to use the language parameter, if present, to form the zen_href_link function's URI. I've got pages that have other parameters and it seems to strip out the language= and retain the remaining, valid, parameters.

    // BEGIN CEON URI MAPPING 2 of 4
    $uri_mapping_being_used = false;
    
    if (defined('CEON_URI_MAPPING_ENABLED') && CEON_URI_MAPPING_ENABLED == 1) {
// lat9: start  
      // Check to see if the language is specified as a parameter
      $mapping_language_id = (int) $_SESSION['languages_id'];
      $pattern = '/language\=([a-z\_]+)/';
      if (preg_match($pattern, $parameters, $matches)) {
        $parm_lang_id = $matches[1];

        $lang_mapping_sql = "
            SELECT
              languages_id
            FROM
              " . TABLE_LANGUAGES . "
            WHERE
              code = '" . zen_db_input($parm_lang_id) . "';";

        $lang_mapping_result = $db->Execute($lang_mapping_sql);
        if (!$lang_mapping_result->EOF) {
          $parameters = str_replace('language=' . $parm_lang_id, '', $parameters);
          $mapping_language_id = $lang_mapping_result->fields['languages_id'];
        }
      }
// lat9: end         
      if (preg_match('/.*index.php\?main_page=([^&]+)/i', $page, $matches)) {

I then changed all occurrences in zen_href_link of $_SESSION['languages_id'] to $mapping_language_id so that the parameter language, if present, is used in any of the SQL queries needed to form the URI.

[conor]

Hi,

FWIW, here's the patch

Thanks, that code looks fine. As it only runs if the language ID is being specified in the link it'll never be run on the vast majority of sites (your site is very special/unique in this respect).

However, the fact that it provides additional functionality at the cost of a single preg_match check per link lookup means that it may well be worth including in the next version of the software.

What do others here think? Is an extra preg_match per zen_href_link call acceptable or is this functionality too "niche/specialised" to be included in the main software? (The alternative would be to distribute it as a FAQ/patch).

All the best..

Conor
ceon

[csicontact]

Sorry about the bug.. that one's been there for years and managed to slip past, guess not many use this functionality in EZ-Page/Manufacturer names!

All the best..

Conor
ceon

Oh, nothing. Thanks for the speedy bugfix, and special thanks for the writing of this fantastic module. Great work.

csicontact

[DivaVocals]

In my early days as a BA/PM, I was always taught that you program by rules not exceptions..

That said, this functionality IMO seems too specialized to keep in the main software, but will definitely be of use to others in similar specialized situations.. I say FAQ/patch (patch probably being the easiest for end users of your mod)

Hi,



Thanks, that code looks fine. As it only runs if the language ID is being specified in the link it'll never be run on the vast majority of sites (your site is very special/unique in this respect).

However, the fact that it provides additional functionality at the cost of a single preg_match check per link lookup means that it may well be worth including in the next version of the software.

What do others here think? Is an extra preg_match per zen_href_link call acceptable or is this functionality too "niche/specialised" to be included in the main software? (The alternative would be to distribute it as a FAQ/patch).

All the best..

Conor
ceon

[bangsters]

works like a charm.... although i have a question. it can translate EZpages as I see the URI mapping at the bottom. however, for define pages I cannot see any option, like for contact us, privacy, etc....

any suggestions?

[lat9]

... As it only runs if the language ID is being specified in the link it'll never be run on the vast majority of sites (your site is very special/unique in this respect).

Although I'm using the "language in header" mod, the code that it uses to display the language flags is the same as that used by the language sidebox that comes in the standard zen-cart. That said, anyone who has a multi-lingual site and uses the Ceon URI Mapping mod will run into this problem.

[DivaVocals]

This is discussed in the readme/FAQ that comes with the module.. You'll need to add these pages to the DB manually.. The readme gives details on how..

works like a charm.... although i have a question. it can translate EZpages as I see the URI mapping at the bottom. however, for define pages I cannot see any option, like for contact us, privacy, etc....

any suggestions?

[conor]

Hi,

Although I'm using the "language in header" mod, the code that it uses to display the language flags is the same as that used by the language sidebox that comes in the standard zen-cart. That said, anyone who has a multi-lingual site and uses the Ceon URI Mapping mod will run into this problem.

It's not a "problem" as it doesn't cause any actual issues, it simply results in the native text for a link to the current page not being used and a background redirect having to take place.

When the languages box is used the module displays the correct page with the correct URI for the applicable language. The links might be in the current language and have ?language=CODE in them but they instantly redirect to the URI for the appropriate page... the user never gets to see the current_language_page_name?language=CODE except when they hover over the link.

I think I'm leaning towards including your patch in the FAQs rather than as part of the main software as it would slow down the code for a feature which is rarely used (comparatively speaking). I'm not sure though.

Again I'd like more feedback on this... personally I like the idea of all the links on a page being "perfect" without any redirects being necessary but the additional processing time per link could add up on large sites.

More comments are welcome!

All the best..

Conor
ceon