Possible Bug When Upgrading from 4.5.2.018 to 4.5.3.002

37 views
Skip to first unread message

Juan Aguilar

unread,
Jan 5, 2016, 12:20:08 PM1/5/16
to Lucee
Hello,

Last week, in troubleshooting a separate issue, I upgraded from 4.5.2.018 to 4.5.3.002. This caused a problem with the Facebook SDK (https://github.com/TheRealAgentK/facebook-cf-sdk) which we use.

Here's part of the stacktrace:

missing

name declaration for property

at

lucee.transformer.cfml.script.AbstrCFMLScriptTransformer._propertyStatement(AbstrCFMLScriptTransformer.java:1270):1270

at

lucee.transformer.cfml.script.AbstrCFMLScriptTransformer.propertyStatement(AbstrCFMLScriptTransformer.java:1160):1160

at

lucee.transformer.cfml.script.AbstrCFMLScriptTransformer.statement(AbstrCFMLScriptTransformer.java:210):210

at

lucee.transformer.cfml.script.AbstrCFMLScriptTransformer.statement(AbstrCFMLScriptTransformer.java:201):201

at

lucee.transformer.cfml.script.AbstrCFMLScriptTransformer.statements(AbstrCFMLScriptTransformer.java:184):184



And, here's the offending code (from https://github.com/TheRealAgentK/facebook-cf-sdk/blob/master/sdk/FacebookApp.cfc#L30):


/**
* @description Facebook App Id
* @hint
*/

property
String appId;

I'd file this as a bug if someone wouldn't mind guiding me to the most appropriate way to do that.

Thanks,

Juan

Mark Drew

unread,
Jan 5, 2016, 12:22:03 PM1/5/16
to lucee
Another similar bug seems to have crept in (I saw this on the Slack channel) to do with non named properties in CFCs

Haven’t tried it myself but it looks like there will be a lot of broken code regarding this. 

Regards

MD 
--
Love Lucee? Become a supporter and be part of the Lucee project today! - http://lucee.org/supporters/become-a-supporter.html
---
You received this message because you are subscribed to the Google Groups "Lucee" group.
To unsubscribe from this group and stop receiving emails from it, send an email to lucee+un...@googlegroups.com.
To post to this group, send email to lu...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/lucee/58e9e584-38a7-4205-8874-08444d035a57%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Andrew Dixon

unread,
Jan 5, 2016, 4:11:05 PM1/5/16
to lu...@googlegroups.com
Hi Juan and Mark,

I found this as well, but I've not had time to create a repo case to raise the ticket, but it was cased by this pull request I believe:


If either of you have the time to create a repo case and raise the ticket that would be great!!!

Kind regards,

Andrew

Reply all
Reply to author
Forward
0 new messages