coldfumonkeh / monkehTweets

A ColdFusion wrapper to interact with the Twitter API (with OAuth integration)
http://www.mattgifford.co.uk/
53 stars 30 forks source link

Local variable error #11

Closed cfgilles closed 10 years ago

cfgilles commented 10 years ago

Hi,

I have the latest code, I created an app (Twitter) and all my keys and tokens are there but I still get a:

Error Occurred While Processing Request Local variable find on line 564 must be grouped at the top of the function body.

Why is it still happening ?

Thanks

Gilles

Mark-Hetherington commented 10 years ago

Cold fusion versions before 9 require local variables to all be declared at the start of a function. I'm not sure the developers actively target earlier versions. On 15/01/2014 4:57 pm, "cfgilles" notifications@github.com wrote:

Hi,

I have the latest code, I created an app (Twitter) and all my keys and tokens are there but I still get a:

Error Occurred While Processing Request Local variable find on line 564 must be grouped at the top of the function body.

Why is it still happening ?

Thanks

Gilles

— Reply to this email directly or view it on GitHubhttps://github.com/coldfumonkeh/monkehTweets/issues/11 .

cfgilles commented 10 years ago

In base.cfc, I manage to change lines 564 and 565:

From

To

Now I get:

Error Occurred While Processing Request Variable I is undefined.

cfgilles commented 10 years ago

I meant...

From: cfset var find = '' cfset var replace = ''

To: cfset find = '' cfset replace = ''

coldfumonkeh commented 10 years ago

This has been updated and will be in the next commit (hopefully this evening). The vars have been moved to the top of the function to remedy this. Many thanks

cfgilles commented 10 years ago

Thanks, I managed to make it work.

Gilles

Gilles Tanguay Sent from my iPhone 5

On 2014-01-15, at 15:37, Matt Gifford notifications@github.com wrote:

This has been updated and will be in the next commit (hopefully this evening). The vars have been moved to the top of the function to remedy this. Many thanks

— Reply to this email directly or view it on GitHub.

coldfumonkeh commented 10 years ago

Fixed in revision 911c0f68793f6f0602a3ad6681c3ed2ee49de647