Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix `content_tag_for` with array html option. | Semyon Perepelitsa | 2013-01-31 | 1 | -0/+6 |
| | | | | | | | | | | | | | | | | | It would embed array as string instead of joining it like `content_tag` does: content_tag(:td, class: ["foo", "bar"]){} #=> '<td class="foo bar"></td>' Before: content_tag_for(:td, item, class: ["foo", "bar"]){} #=> '<td class="item ["foo", "bar"]" id="item_1"></td>' After: content_tag_for(:td, item, class: ["foo", "bar"]){} #=> '<td class="item foo bar" id="item_1"></td>' | ||||
* | No need to pass a empty block to content_tag_for anymore | Rafael Mendonça França | 2013-01-02 | 1 | -11/+11 |
| | |||||
* | Make content_tag_for work without block | Rafael Mendonça França | 2013-01-02 | 1 | -0/+8 |
| | | | | This is version of #8640 for master | ||||
* | Fix warning: assigned but unused variable - result | kennyj | 2012-01-19 | 1 | -1/+1 |
| | |||||
* | Fix errors when running entire suite due to class name collision | Carlos Antonio da Silva | 2012-01-18 | 1 | -23/+23 |
| | | | | | | | | | The Post class is created everywhere in the test suite, and due to that when applying the Array() logic to refactor content_tag_for, some other change to the Post class was breaking record tag tests. The solution is to rename the class to not collide with others already defined in the test suite. | ||||
* | Cleanup Post model, no need to require fake_models | Carlos Antonio da Silva | 2012-01-18 | 1 | -17/+6 |
| | |||||
* | Do not mutate options hash | Carlos Antonio da Silva | 2012-01-18 | 1 | -0/+6 |
| | |||||
* | Cleanup persisted setup for model | Carlos Antonio da Silva | 2012-01-18 | 1 | -7/+7 |
| | |||||
* | Mimic AR models yielding when building new records, avoid using tap | Carlos Antonio da Silva | 2012-01-18 | 1 | -6/+8 |
| | |||||
* | No need for concat as well | Carlos Antonio da Silva | 2012-01-18 | 1 | -4/+4 |
| | |||||
* | Add test for content tag with prefix and extra html options | Carlos Antonio da Silva | 2012-01-18 | 1 | -4/+10 |
| | |||||
* | test helpers in erb using erb | lest | 2011-11-30 | 1 | -2/+4 |
| | |||||
* | Merge pull request #2872 from sikachu/content_tag_html_safe | Santiago Pastorino | 2011-09-05 | 1 | -0/+12 |
|\ | | | | | Make sure that result from content_tag_for with collection is html_safe | ||||
| * | Make sure that result from content_tag_for with collection is html_safe | Prem Sichanugrist | 2011-09-05 | 1 | -0/+12 |
| | | | | | | Thank you @spastorino for catching the empty test in b84cee0, as I totally forgot that I still have to make it html_safe. | ||||
* | | add blanks between methods | Vijay Dev | 2011-09-05 | 1 | -0/+3 |
|/ | |||||
* | Remove empty test | Santiago Pastorino | 2011-09-05 | 1 | -3/+0 |
| | |||||
* | Tests shouldn't produce warnings | Ray Baxter | 2011-09-04 | 1 | -0/+5 |
| | |||||
* | Make `content_tag_for` and `div_for` accepts the array of records | Prem Sichanugrist | 2011-09-04 | 1 | -2/+22 |
| | | | | | | | | | | | | | | | So instead of having to do this: @items.each do |item| content_tag_for(:li, item) do Title: <%= item.title %> end end You can now do this: content_tag_for(:li, @items) do |item| Title: <%= item.title %> end | ||||
* | remove warning: assigned but unused variable | Santiago Pastorino | 2011-06-08 | 1 | -2/+0 |
| | |||||
* | Require persisted? in ActiveModel::Lint and remove new_record? and ↵ | José Valim | 2010-02-21 | 1 | -0/+1 |
| | | | | destroyed? methods. ActionPack does not care if the resource is new or if it was destroyed, it cares only if it's persisted somewhere or not. | ||||
* | API change: content_tag_for outputs prefixed class name | Joshua Peek | 2009-10-08 | 1 | -1/+1 |
| | |||||
* | Moved shared form helper models into fake_models | Joshua Peek | 2009-10-03 | 1 | -0/+1 |
| | |||||
* | AMo conversion helper | Joshua Peek | 2009-07-21 | 1 | -1/+2 |
| | |||||
* | Update some tests and add a to_model to form helpers | Yehuda Katz | 2009-07-20 | 1 | -1/+1 |
| | |||||
* | Define ActiveModel API Compliance | Yehuda Katz | 2009-07-20 | 1 | -1/+1 |
| | | | | | | - Define to_model on AR - Define to_model on ActiveModel::APICompliant - Update test fixtures to be API Compliant - Start using to_model in AP | ||||
* | Move model naming into ActiveModel | Joshua Peek | 2009-06-17 | 1 | -0/+1 |
| | |||||
* | Temporarily modifies setup to call super directly. This can support more ↵ | Yehuda Katz and Carl Lerche | 2009-04-08 | 1 | -0/+1 |
| | | | | T::U runners. | ||||
* | Ensure that calling content_tag_for in a helper doesn't cause duplicate output. | Tom Lea | 2008-08-29 | 1 | -0/+8 |
| | | | | | Signed-off-by: Michael Koziarski <michael@koziarski.com> [#871 state:committed] | ||||
* | Check whether blocks are called from erb using a special __in_erb_template ↵ | Jeremy Kemper | 2008-06-19 | 1 | -13/+14 |
| | | | | variable visible in block binding. | ||||
* | Use output_buffer reader and writer methods exclusively instead of hitting ↵ | Jeremy Kemper | 2008-06-08 | 1 | -7/+2 |
| | | | | the instance variable so others can override the methods. | ||||
* | Work with @output_buffer instead of _erbout | Jeremy Kemper | 2008-06-02 | 1 | -7/+7 |
| | |||||
* | Introduce ActionView::TestCase for testing view helpers. | Joshua Peek | 2008-04-19 | 1 | -8/+3 |
| | |||||
* | Tests for div_for and content_tag_for helpers. Closes #11223 [thechrisoshow] | Pratik Naik | 2008-03-05 | 1 | -0/+59 |
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@8980 5ecf4fe2-1ee6-0310-87b1-e25e094e27de |