Ceon URI Mapping V5.0

[neekfenwick]

The parameter products_id is not strictly an integer because many, many years ago, it was determined that the products_id would be used to support carrying the attribute information related to the product.

Ah yes good point, you've set off a nasty tic in my left eye, remembering the get_prid() function and how the cart links back to the product page to 'edit' the product, and how I tried to reverse engineer how this stuff works from code with little documentation :) Now I look again, init_sanitize does check products_id with more complicated preg_match logic allowing the colon-separated format and /\d/ regex which, I just checked, an integer PHP variable does test OK with, as does a string type variable, so the ceon addon can put an integer type into $_GET['products_id'] without error. Anyway I'll comment on the github issue I raised.

[torvista]

Handling changed/old/outdated URLS?

After updating some urls with a new format or names etc., how does the old url get handled?

I see a lot of requests from search engines that end in page not found as it is a historical url.

[ChuckPhillips]

When working properly, CEON will automatically 301 redirect old URIs to the new URIs.

[torvista]

When working properly, CEON will automatically 301 redirect old URIs to the new URIs

That's what I feared...I have to understand how this bit is supposed to work...ugh.

[mc12345678]

That's what I feared...I have to understand how this bit is supposed to work...ugh.

The old, recognized URI is attempted against the database where it is identified as not being the current, active URI. The associated dynamic data is returned and then the active version of the URI with that data is sought. If not found (in either case) it is expected that the URI is identified as not found (404).

If a matching new address is found then a redirect occurs to identify that the provided URI has changed. This happens for any and all of addresses that remain in the database marked as old (not current).

[swguy]

Attempting to add a new manufacturer doubles up the content of the sidebar so that the core fields (manufacturer name, image, etc.) are prompted twice.

This is a defect in includes/ceon_uri_mapping_javascript.php l.150-187, I'm just not sure of the fix yet.

[schoolboy]

Yes... I noticed this some weeks ago. Both sets of fields need in-filling for it to parse the data to the dbase.

[torvista]

I don't have this issue, I imagine I fixed it long ago.
Review my fork

**UPDATED URL***
https://github.com/torvista/Ceon-URI-Mapping-V5.1.0/

[swguy]

Steve, yours works, thank you.
You are missing some fixes, I will PR to your repo.

[swguy]

I'm setting Steve's fork as the Github repo since it seems to be the most up to date.