Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | move compiled filters to their own class so that recursive calls don't | Aaron Patterson | 2012-10-18 | 1 | -41/+45 |
| | | | | need to check for the length of @filters | ||||
* | nil isn't a valid filter | Aaron Patterson | 2012-10-18 | 1 | -1/+0 |
| | |||||
* | prevent people from accidentally modifying the filter word | Aaron Patterson | 2012-10-18 | 1 | -1/+1 |
| | |||||
* | simplify filter enabled checking | Aaron Patterson | 2012-10-18 | 1 | -9/+4 |
| | |||||
* | if no filters are supplied, default to an empty list | Aaron Patterson | 2012-10-18 | 1 | -1/+1 |
| | |||||
* | initialize required instance variables for this mixin | Aaron Patterson | 2012-10-18 | 1 | -2/+7 |
| | |||||
* | There is already a Set of non-hidden action_names lying around. | thedarkone | 2012-10-18 | 1 | -8/+2 |
| | |||||
* | eagerly populate the http method loookup cache so we don't need locks | Aaron Patterson | 2012-10-18 | 1 | -1/+7 |
| | |||||
* | move cache inside the instance so we do not need locking | Aaron Patterson | 2012-10-18 | 1 | -9/+12 |
| | |||||
* | add test for fetch with a block | Aaron Patterson | 2012-10-18 | 1 | -2/+2 |
| | |||||
* | prefer composition over inheritence | Aaron Patterson | 2012-10-18 | 1 | -11/+15 |
| | |||||
* | Merge pull request #7975 from butcher/extend_date_select_helper_functionality | Rafael Mendonça França | 2012-10-18 | 1 | -0/+3 |
|\ | | | | | | | | | | | | | Extend date_select helper functionality. Conflicts: actionpack/CHANGELOG.md | ||||
| * | Extend date_select helper functionality. | Pavel Nikitin | 2012-10-18 | 1 | -0/+3 |
| | | |||||
* | | Permit string and float values in the multiparameter attributes | Rafael Mendonça França | 2012-10-18 | 1 | -1/+1 |
| | | |||||
* | | fix Digestor to be thread safe. | Aaron Patterson | 2012-10-17 | 1 | -8/+30 |
|/ | | | | | Add a lock on the cache so that we don't have concurrency issues when calculating the cache. | ||||
* | Ignore asset url query string or anchor when appending extensions and ↵ | Joshua Peek | 2012-10-15 | 1 | -1/+3 |
| | | | | computing public path | ||||
* | Extract compute_asset_extname and allow extname to be disabled | Joshua Peek | 2012-10-15 | 1 | -7/+15 |
| | |||||
* | Merge branch 'master' into asset-path-helper | Joshua Peek | 2012-10-15 | 2 | -6/+4 |
|\ | | | | | | | | | Conflicts: railties/test/application/configuration_test.rb | ||||
| * | Removed useless "extend SanitizeHelper::ClassMethods" | Andrey Samsonov | 2012-10-15 | 1 | -4/+0 |
| | | | | | | | | | | Since SanitizeHelper includes ActiveSupport::Concern, extending of it ClassMethods is no needed. | ||||
| * | remove 'then' from conditional statement | Nihad Abbasov | 2012-10-14 | 1 | -2/+4 |
| | | |||||
* | | :fire: Rails asset id support | Joshua Peek | 2012-10-15 | 6 | -166/+1 |
| | | |||||
* | | merge! default asset tag options | Joshua Peek | 2012-10-15 | 1 | -12/+12 |
| | | |||||
* | | Just check request instead of controller.request | Joshua Peek | 2012-10-15 | 1 | -13/+2 |
| | | |||||
* | | Allow asset url config to be undefined | Joshua Peek | 2012-10-13 | 1 | -2/+4 |
| | | |||||
* | | All asset url helpers should pass options | Joshua Peek | 2012-10-12 | 1 | -24/+24 |
| | | |||||
* | | Ensure AssetUrlHelper can be mixed into AC::Base | Joshua Peek | 2012-10-12 | 1 | -6/+13 |
| | | |||||
* | | Refactor AssetUrlHelper to make it friendly for plugins and extensions | Joshua Peek | 2012-10-12 | 8 | -345/+240 |
| | | | | | | | | | | | | | | | | Add asset_path/url helper for a consolidated entry point Expose compute_asset_path as a public API Expose compute_asset_host as a public API Move RAILS_ASSET_ID to its own module, AssetIdHelper Removed AV::AssetPaths | ||||
* | | Add asset_path and asset_url helpers | Joshua Peek | 2012-10-12 | 2 | -17/+54 |
|/ | |||||
* | Split asset url helpers into seperate AssetUrlHelper | Joshua Peek | 2012-10-12 | 5 | -419/+416 |
| | |||||
* | warning removed: shadowing outer local variable - value | Arun Agrawal | 2012-10-12 | 1 | -2/+2 |
| | |||||
* | update AC::Parameters#permit documentation [ci skip] | Francesco Rodriguez | 2012-10-11 | 1 | -0/+22 |
| | | | | bdd105d changes the behaviour of AC::Parameters#permit. | ||||
* | When executing permit with just a key that points to a hash, DO NOT allow ↵ | Santiago Pastorino | 2012-10-12 | 1 | -1/+4 |
| | | | | | | | | | | | | | | all the hash params.require(:person).permit(:projects_attributes) was returning => {"projects_attributes"=>{"0"=>{"name"=>"Project 1"}}} When should return => {} You should be doing ... params.require(:person).permit(projects_attributes: :name) to get just the projects attributes you want to allow | ||||
* | 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] |