sergiogonzalez/liferay-portal — pull request #308 by KocsisDaniel (2012-07-23 09:57:40+00:00)
Title: [TECH SUPPORT] LPS-28726

Hi Sergio,

this pr is to always extract URL params when we change the display language. Currently we do this only when the LOCALE_PREPEND_FRIENDLY_URL_STYLE != 0 and when it is 0 the params get lost that causes issues as it is described in the LPS. The responsible part of code is moved before the if statement so we always does it.

I made some research to find possible issues with that because I was concerned to put a specific part of the code into global. Based on my tests it seems OK, because the optional friendly URL parts or the parameters comes only after the page URL and the only difference between the 2 cases is how to generate this layout URL. But on the other hand I am not sure whether it is a real bug, or it was created in purpose in this way.

Please let me know if You have any concerns!
Daniel