Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | Allow for deep directory path for view templates. | Andy Shipman | 2012-10-11 | 1 | -1/+1 | |
| | | ||||||
* | | refactor `ActionView::TestCase` internals to track rendered locals | Yves Senn | 2012-10-11 | 2 | -20/+33 | |
| | | | | | | | | | | this refactoring extracts the semi complex data structure of rendered locals per view into into a separate class | |||||
* | | recognizes when a partial was rendered twice. Closes #3675 | Yves Senn | 2012-10-11 | 2 | -9/+18 | |
| | | ||||||
* | | Merge pull request #7890 from AvnerCohen/new_hash_syntax | Vijay Dev | 2012-10-10 | 1 | -9/+9 | |
|\ \ | |/ |/| | Moving to new hash syntax | |||||
| * | moving to new hash syntax, for discussion before I take the time on full folders | AvnerCohen | 2012-10-09 | 1 | -9/+9 | |
| | | ||||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-10-11 | 2 | -2/+2 | |
|\ \ | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/persistence.rb railties/lib/rails/generators/rails/resource_route/resource_route_generator.rb | |||||
| * | | Closed unclosed <tt>, working on a complete solution for docrails, but ↵ | AvnerCohen | 2012-10-08 | 1 | -1/+1 | |
| | | | | | | | | | | | | testing push process first | |||||
| * | | Space between "every" and "time" | schneems | 2012-10-02 | 1 | -1/+1 | |
| | | | ||||||
* | | | Remove expansion config | Joshua Peek | 2012-10-10 | 1 | -2/+0 | |
| | | | ||||||
* | | | Remove old asset tag concatenation | Joshua Peek | 2012-10-10 | 4 | -378/+16 | |
| |/ |/| | | | | | Use sprockets, jammit, or some other asset bundler | |||||
* | | Fix nodoc comment | Rafael Mendonça França | 2012-10-07 | 1 | -16/+15 | |
| | | ||||||
* | | More Ruby 1.9 hash syntax. | Rafael Mendonça França | 2012-10-06 | 1 | -52/+47 | |
| | | ||||||
* | | We don't need to check blank? here. | Rafael Mendonça França | 2012-10-06 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | Also the blank? check introduced a bug. $ rails generate model Foo blank:boolean form_for(Foo.new(:blank => true)) => ArgumentError, "First argument in form cannot contain nil or be empty" | |||||
* | | Update some code styles. | Rafael Mendonça França | 2012-10-06 | 1 | -50/+48 | |
| | | | | | | | | | | * Uses the Ruby 1.9 hash syntax * Avoid escaping " inside string using the %{} syntax | |||||
* | | Accept :remote as symbol in link_to options | Riley Lynch | 2012-10-06 | 1 | -2/+5 | |
| | | | | | | | | | | Accept either :remote or 'remote' in both the html_options and (url_)options hash arguments to link_to. | |||||
* | | can't pass :locals to #assert_template without a view test case. Closes #3415 | Yves Senn | 2012-10-06 | 1 | -3/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | the documentation on #assert_template states that the :locals option is only available in view test cases: # In a view test case, you can also assert that specific locals are passed # to partials: I added a warning when it's passed in an inapropriate context to prevent a NoMethodError. | |||||
* | | We don't need the each call since grep take a block and yield each match | Rafael Mendonça França | 2012-10-06 | 1 | -1/+1 | |
| | | | | | | | | | | | | to the block. Thanks @tenderlove | |||||
* | | Allow parameter filters to match multi-parameter attributes | Rafael Mendonça França | 2012-10-05 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | This will make easier to permit date/time attributes generated by helpers like date_select. [Sven Schwyn + Rafael Mendonça França] | |||||
* | | Revert "Use flat_map { } instead of map {}.flatten" | Santiago Pastorino | 2012-10-05 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | This reverts commit abf8de85519141496a6773310964ec03f6106f3f. We should take a deeper look to those cases flat_map doesn't do deep flattening. irb(main):002:0> [[[1,3], [1,2]]].map{|i| i}.flatten => [1, 3, 1, 2] irb(main):003:0> [[[1,3], [1,2]]].flat_map{|i| i} => [[1, 3], [1, 2]] | |||||
* | | Use flat_map { } instead of map {}.flatten | Santiago Pastorino | 2012-10-05 | 1 | -2/+2 | |
| | | ||||||
* | | config.action_dispatch.rack_cache should set explicitly to enable Rack::Cache | Guillermo Iguaran | 2012-10-04 | 1 | -6/+1 | |
| | | ||||||
* | | Merge pull request #7847 from bloudermilk/recursive_permit | David Heinemeier Hansson | 2012-10-04 | 1 | -0/+5 | |
|\ \ | | | | | | | ActionController::Parameters#permit! is recursive | |||||
| * | | ActionController::Parameters#permit! is recursive | Brendan Loudermilk | 2012-10-04 | 1 | -0/+5 | |
| | | | ||||||
* | | | Do not set removed option | Rafael Mendonça França | 2012-10-04 | 1 | -1/+0 | |
|/ / | | | | | | | | | Sine page caching was remove from the core we don't have the page_cache_directory anymore | |||||
* | | Merge pull request #7833 from frodsan/extract_ap_pages_actions_caching | Rafael Mendonça França | 2012-10-04 | 4 | -400/+17 | |
|\ \ | | | | | | | Extract AP Page and Action caching from Rails | |||||
| * | | rename page_cache_extension option to default_static_extension | Francesco Rodriguez | 2012-10-03 | 2 | -11/+13 | |
| | | | ||||||
| * | | extract AP Page and Action caching to actionpack-deprecated_caching gem | Francesco Rodriguez | 2012-10-03 | 3 | -399/+14 | |
| | | | ||||||
* | | | Add nodoc to now public fragment_name_with_digest | Ryan Garver | 2012-10-04 | 1 | -0/+1 | |
| | | | ||||||
* | | | Merge branch 'master' into feature/public-fragment_name_with_digest | Ryan Garver | 2012-10-02 | 14 | -166/+202 | |
|\| | | ||||||
| * | | Failsafe exception returns text/plain. | Steve Klabnik | 2012-10-01 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | It's best to just return text/plain when something has gone terribly wrong. Fixes #5660. | |||||
| * | | Merge pull request #7708 from bdurand/optimize_log_subscribers | Rafael Mendonça França | 2012-10-01 | 2 | -8/+16 | |
| |\ \ | | | | | | | | | Optimize log subscribers to check if the log level is sufficient | |||||
| | * | | Optimize log subscribers to check if the log level is sufficient before ↵ | Brian Durand | 2012-09-30 | 2 | -8/+16 | |
| | | | | | | | | | | | | | | | | performing an operations. | |||||
| * | | | prefix TemplateAssertions ivars (#7459) | Yves Senn | 2012-10-01 | 2 | -23/+23 | |
| | | | | ||||||
| * | | | resource and resources do no longer modify passed options | Yves Senn | 2012-10-01 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | this is a patch for #7777. | |||||
| * | | | Whitespaces :scissors: [ci skip] | Rafael Mendonça França | 2012-09-30 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | `assert_template` no more passing with what ever string that matches. | Hugo Roque | 2012-09-29 | 1 | -5/+12 | |
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Given Im rendering an template `/layout/hello.html.erb`, assert_template was passing with any string that matches. This behavior allowed false passing like: assert_template "layout" assert_template "out/hello" Now the passing possibilities are: assert_template "layout/hello" assert_template "hello" fixing assert_template bug when template matches expected, but not ends with Cherry Pick Merge: Fixes issue #3849 assert_template false positive taking redundant test off prevening incorrect assert_template when rendering with repeated names in path updating CHANGELOG with bugfix: assert_template false passing | |||||
| * | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-09-28 | 8 | -119/+139 | |
| |\ \ | | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/asset_tag_helper.rb | |||||
| | * | | update image_tag output in examples to actual | Nihad Abbasov | 2012-09-25 | 1 | -4/+4 | |
| | | | | | | | | | | | | [ci-skip] | |||||
| | * | | update AC::ConditionalGet documentation [ci skip] | Francesco Rodriguez | 2012-09-22 | 1 | -27/+31 | |
| | | | | ||||||
| | * | | update AC::Caching documentation [ci skip] | Francesco Rodriguez | 2012-09-22 | 5 | -87/+103 | |
| | | | | ||||||
| | * | | remove AC::Parameters reference [ci skip] | Francesco Rodriguez | 2012-09-21 | 1 | -1/+1 | |
| | | | | ||||||
| | * | | fix typos in AC::StrongParameters documentation [ci skip] | Francesco Rodriguez | 2012-09-21 | 1 | -2/+2 | |
| | | | | ||||||
| * | | | Fix tests broken by adding a new instance variable to view test cases | Jeremy Kemper | 2012-09-27 | 1 | -0/+1 | |
| | | | | ||||||
| * | | | Is not needed to wrap name inside an array here | Santiago Pastorino | 2012-09-27 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Change key not found to param not found | Santiago Pastorino | 2012-09-27 | 1 | -6/+6 | |
| | | | | ||||||
* | | | | Move the CacheHelper#fragment_name_with_digest to be public so custom ↵ | Ryan Garver | 2012-09-27 | 1 | -11/+11 | |
|/ / / | | | | | | | | | | fragment caching can benefit from it. | |||||
* | | | Also includes ConstantLookup dependency for controller and mailer tests | Andy Lindeman | 2012-09-26 | 1 | -0/+1 | |
| | | | ||||||
* | | | Adds missing dependency to ActionView::TestCase::Behavior | Andy Lindeman | 2012-09-26 | 1 | -0/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * The module is needed for the `determine_constant_from_test_name` method. * Without it, the including class is required to also include `ActiveSupport::Testing::ConstantLookup` or a `NoMethodError` will be raised upon instantiation of that class. * Issue introduced in c0a24555f9e2749fb94efe1967cb9943db0b6a7e | |||||
* | | | Merge pull request #7444 from szimek/params_parser_raises_parsing_error | Aaron Patterson | 2012-09-26 | 1 | -5/+12 | |
|\ \ \ | | | | | | | | | Raise generic ParseError exception when ParamsParser fails parsing request params | |||||
| * | | | Make ActionDispatch::ParamsParser::ParseError#original_exception return the ↵ | Szymon Nowak | 2012-08-27 | 1 | -4/+10 | |
| | | | | | | | | | | | | | | | | original exception. |