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

Collection radio buttons and collection check boxes through FormBuilder render the provided block #8916

Merged
merged 1 commit into from
Jan 14, 2013
Merged

Conversation

josemotanet
Copy link
Contributor

In the case of having a form_for method being called, the block for each
collection would not be passed and thus the result expected was always the same.
This patch passes the block to the original method like it would be assumed.

…er render

the provided block.

In the case of having a form_for method being called, the block for each
collection would not be passed and thus the result expected was always the same.
This patch passes the block to the original method like it would be assumed.
@josemotanet
Copy link
Contributor Author

This fixes issue #8897.

/cc @rafaelfranca

rafaelfranca added a commit that referenced this pull request Jan 14, 2013
Collection radio buttons and collection check boxes through FormBuilder render the provided block.

Closes #8897
@rafaelfranca rafaelfranca merged commit 242f125 into rails:master Jan 14, 2013
@rafaelfranca
Copy link
Member

Thank you so much, and welcome to Rails Contributors

@josemotanet josemotanet deleted the collection-with-block branch January 14, 2013 15:01
sgerrand pushed a commit to sgerrand/rails that referenced this pull request Nov 2, 2013
The [following pull request](rails#8916) fixed
the block being passed to the appropriate helper method. However, the content
being passed into the block is generating repeated markup on the page due to
some weird ERb evaluation.

This commit tries to capture the block's generated output so the page isn't
flooded with markup.

[Rafael França + José Mota]

Closes rails#8936
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