Skip to content
This repository has been archived by the owner on Apr 11, 2018. It is now read-only.

bug for nested variable not usable in for tag #232

Merged
merged 1 commit into from
Jun 9, 2013
Merged

bug for nested variable not usable in for tag #232

merged 1 commit into from
Jun 9, 2013

Conversation

jinze
Copy link
Contributor

@jinze jinze commented May 27, 2013

There is no need to validate the property name of variable operand2, because the variable name will be instead as __loopIter.

The validation cause syntax like <% for item in pord.items %> throw an error, which are availiable in fact.

There is no need to validate the property "name" of variable 'operand2', because the variable name will be instead as '__loopIter'.

The validation cause syntax like '<% for item in pord.items %>' throw an error, which are availiable in fact.
@paularmstrong
Copy link
Owner

Can you write a unit test to assert this behavior? https://github.com/paularmstrong/swig/blob/master/CONTRIBUTING.md#checklist

paularmstrong added a commit that referenced this pull request Jun 9, 2013
@paularmstrong paularmstrong merged commit 7090154 into paularmstrong:master Jun 9, 2013
@paularmstrong
Copy link
Owner

This has been merged.

@jinze jinze deleted the patch-1 branch June 10, 2013 14:45
@jinze
Copy link
Contributor Author

jinze commented Jun 10, 2013

I'm sorry I ignored the previous comments, you are a conscientious maintainer.

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

Successfully merging this pull request may close these issues.

2 participants