Skip to content

Conversation

@juavillo
Copy link

Hello.

First thanks for this module that works great!

I have a question.

My new URLS are like:
http://www.jmbarrachina.com/bolsas/bolsa-larsen

Is posible to accept also my old URLS like?
http://www.jmbarrachina.com/bolsas/100003322-bolsa-larsen.html

The ID of the product now is not the same, but the rewrite in product_lang is the same 'bolsa-larsen'

Thanks, best regards

Juan

Lebioda and others added 30 commits October 25, 2013 23:34
	- passed missing $id_shop to the getLangLink and createUrl methods
	- remove only root and current category from the URL

Tested on PrestaShop v1.5.6.1. Now working fine!
canonicalRedirection has to remove rewrite ids from _GET before
redirect, as it does in the FrontController for ordinary id_*
BIG Fixed: Configuration::get('PS_ROOT_CATEGORY') has to be used *after* parent constructor has been invoked
Fixed wrong PHP syntax (use of self:: in the constructor instead of Link::) causing "root" node not to be added to the excluded categories array ...

Static variables work differently :)
Better implementation. Do exactly what we want, not what is supposed to give the same result
hack to make smartblog category path work. Remember to deactivate html in the smartblog module configuration
works on prestashop 1.6.0.9
argh that semicolon...
Meta keywords and title can be passed to the function as an array or string.
This update ensures that the array is flatened to a single string prior to asking Tools:str2url to work with it.

If Tools::str2url recieves an array it throws our lots of 'Warning' messages when PHP debugging is enabled in PS.

merged from mrtwister76
mrtwister76/PrestaShop-modules-CleanURLs@0b50f2a
make getCategoryLink compatible with new prestashop 1.6.0.12
added addRoute function to the override on ps 1.6.0.11 var $regex has changed to '#^/'.$regexp.'$#u' changed back to '#^/'.$regexp.'(\?.*)?$#u' so now cleanurl cart, login, quickview and pagination, basically whenever there's a ? in the url works again
@drAlberT drAlberT deleted the dev branch October 25, 2016 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants