Download & Extend

sms_clickatell should perhaps not use drupal_urlencode for messages

Project:SMS Framework
Version:6.x-2.x-dev
Component:Clickatell
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

I wanted to send an URL by SMS (with the clickatell gateway) and got the issue #254738: drupal_urlencode() turning 'http://' to 'http%3A/%252F'.
To fix it I just replaced the drupal_urlencode() to urlencode() (on line 130 of the sms_clickatell.module) and it works now fine.

By the way, I found strange that #254738: drupal_urlencode() turning 'http://' to 'http%3A/%252F' is set as "by design"... any idea why?

Comments

#1

Subscribing

#2

Version:6.x-1.x-dev» 6.x-2.x-dev

Hi,

We found the same issue :) Wouldn't rawurlencode() be even more appropriate?

Regards
Johan Heander

#3

Status:active» fixed

this has been changed to use rawurlencode

#4

Status:fixed» closed (fixed)

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

nobody click here