Re: Ceon URI Mapping V5.0
Quote:
Originally Posted by
mc12345678
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.
Re: Ceon URI Mapping V5.0
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.
Re: Ceon URI Mapping V5.0
When working properly, CEON will automatically 301 redirect old URIs to the new URIs.
Re: Ceon URI Mapping V5.0
Quote:
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.
Re: Ceon URI Mapping V5.0
Quote:
Originally Posted by
torvista
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).
Re: Ceon URI Mapping V5.0
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.
Re: Ceon URI Mapping V5.0
Yes... I noticed this some weeks ago. Both sets of fields need in-filling for it to parse the data to the dbase.
Re: Ceon URI Mapping V5.0
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/
Re: Ceon URI Mapping V5.0
Steve, yours works, thank you.
You are missing some fixes, I will PR to your repo.
Re: Ceon URI Mapping V5.0
I'm setting Steve's fork as the Github repo since it seems to be the most up to date.