Skip to content

do not support block handling in element define method [fix #112]#116

Merged
dtognazzini merged 2 commits intomasterfrom
removeElementBlockHandling
May 13, 2015
Merged

do not support block handling in element define method [fix #112]#116
dtognazzini merged 2 commits intomasterfrom
removeElementBlockHandling

Conversation

@ipmsteven
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread lib/ae_page_objects/core/dsl.rb Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is wrong. Before block referenced the variable local to the define_method block, but now it's referencing the parameter to element.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dtognazzini oh, yeah, I forgot to remove the &block here

@ipmsteven ipmsteven force-pushed the removeElementBlockHandling branch 2 times, most recently from 5ee00fd to 05cda3b Compare May 9, 2015 02:05
@ipmsteven ipmsteven force-pushed the removeElementBlockHandling branch 3 times, most recently from a74476d to 361f85f Compare May 13, 2015 01:32
@ipmsteven ipmsteven force-pushed the removeElementBlockHandling branch from 361f85f to 357c023 Compare May 13, 2015 19:35
dtognazzini added a commit that referenced this pull request May 13, 2015
do not support block handling in element define method [fix #112]
@dtognazzini dtognazzini merged commit b7ebc74 into master May 13, 2015
@dtognazzini dtognazzini deleted the removeElementBlockHandling branch May 13, 2015 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants