Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MISSING_VAR reports cfmail() attributes #482

Closed
eppinger opened this issue Oct 30, 2017 · 6 comments
Closed

MISSING_VAR reports cfmail() attributes #482

eppinger opened this issue Oct 30, 2017 · 6 comments
Assignees
Labels
Milestone

Comments

@eppinger
Copy link

sample:
cfmail( from="[email protected]", to="[email protected]" subject="#subjectText#" type="html" ){
}

reports
Variable from is not declared with a var statement., etc

@eppinger
Copy link
Author

same problem with this syntax:;
mail from="[email protected]", to="[email protected]" subject="#subjectText#" type="html"{}

@ryaneberly ryaneberly self-assigned this Nov 1, 2017
@ryaneberly ryaneberly added the bug label Nov 1, 2017
@ryaneberly
Copy link
Contributor

@eppinger or @TheRealAgentK, is that syntax with missing commas legit?
i.e. we should be able to parse:
cfmail( from="[email protected]", to="[email protected]" subject="#subjectText#" type="html" ){
}
as is?

Does this apply to ALL function calls?

@KamasamaK
Copy link
Collaborator

KamasamaK commented Nov 25, 2017

@ryaneberly This is another difference between ACF and Lucee/Railo. ACF requires the commas, but Lucee does not.

@ryaneberly
Copy link
Contributor

ryaneberly commented Nov 25, 2017 via email

@KamasamaK
Copy link
Collaborator

This appears to only apply to tags.

ryaneberly added a commit that referenced this issue Dec 16, 2017
@ryaneberly
Copy link
Contributor

I added tests to confirm, this seems to be fixed in dev with the latest cfparser

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants