drewcrawford / Phone-Pipe

97 stars 4 forks source link

Local var 'url' overrides parameter 'url' #2

Closed codeman38 closed 14 years ago

codeman38 commented 14 years ago

In the alert method, there's a local variable named url which overrides the parameter url that's used for passing a link to the notification server. Thus, every message ends up having the API URL as its link. This is clearly not right.

It's a simple fix: rename url to something else like posturl in lines 72 and 79.

drewcrawford commented 14 years ago

Thx! Fixed.