Skip to content

Conversation

@crtlib
Copy link
Contributor

@crtlib crtlib commented Aug 3, 2013

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d4e682d on crtlib:patch-2 into d836654 on yiisoft:master.

@qiangxue
Copy link
Member

qiangxue commented Aug 3, 2013

Nope, this is not correct because it involves checking behaviors.

@qiangxue qiangxue closed this Aug 3, 2013
@crtlib
Copy link
Contributor Author

crtlib commented Aug 3, 2013

Sorry, I thought $checkBehavior was used in the foreach. But now I don't understand where it's used after all. Perhaps there should be:
} elseif ($checkBehavior) {

qiangxue added a commit that referenced this pull request Aug 3, 2013
@qiangxue
Copy link
Member

qiangxue commented Aug 3, 2013

Good catch. Just checked in. Thanks.

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.

3 participants