Download & Extend
Pathologic » Issues

Query fragment being encoded

Posted by cocoliso on March 31, 2009 at 7:06pm
Project:Pathologic
Version:6.x-2.0-beta19
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Hi,

I noticed that some characters are urlencoded when pathologic is enabled

for example:
cart/add/p30_q1?destination=cart

are transform to this:
cart/add/p30_q1%3Fdestination%3Dcart

which breaks the url redirection...

If I disable pathologic the url is not encoded and everything works as usual...any ideas?

Regards,
Sebastian

Login or register to post comments

Comments

#1

Posted by Garrett Albright on April 2, 2009 at 3:08pm
Title:url encoded characters in links» Query fragment being encoded

Hi… Just want to acknowledge that I've seen your message, but I can't quite work on it just yet. Thanks for trying Pathologic, though.

#2

Posted by cocoliso on April 9, 2009 at 9:42pm

Hey Garret thank for the reply and for making this module.
No rush :-)

#3

Posted by Garrett Albright on July 13, 2009 at 3:00am

Hi there. Still trying to use Pathologic and having this problem?

When the next dev release (*not* the beta release… the dev releases on the project page have a red background) is available, please download it and give it a try. It will hopefully fix this problem.

#4

Posted by jrosen on July 17, 2009 at 4:43pm

Hi Garrett, I am having this issue with the following URL:

/civicrm/profile/edit&gid=1&reset=1

becoming:
/civicrm/profile/edit%2526amp%3Bgid%3D1%2526amp%3Breset%3D1

I downloaded the latest dev release, but it doesn't seem to fix it. Can you post the new dev release you referred to or at least post a patch or which lines of code to change to fix this.

Thanks.

#5

Posted by Garrett Albright on July 20, 2009 at 3:45pm

Yeargh, okay. That's not how I was expecting query strings to look, but I guess it's legitimate. Thanks for noting it.

#6

Posted by Garrett Albright on July 21, 2009 at 2:56am

All right, I've committed what may be a fix. The packaging robot should package together a new dev release soon; please give it a try.

#7

Posted by Garrett Albright on June 3, 2010 at 3:08am
Status:active» fixed

#8

Posted by System Message on June 17, 2010 at 3:10am
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.