Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #5255 from carlosantoniodasilva/active-record-review | Piotr Sarnacki | 2012-03-03 | 8 | -68/+77 |
|\ | | | | | Refactor and cleanup in some ActiveRecord modules | ||||
| * | Refactor and cleanup in some ActiveRecord modules | Carlos Antonio da Silva | 2012-03-03 | 8 | -68/+77 |
| | | | | | | | | | | | | | | | | | | | | | | * Avoid double hash lookups in AR::Reflection when reflecting associations/aggregations * Minor cleanups: use elsif, do..end, if..else instead of unless..else * Simplify DynamicMatchers#respond_to? * Use "where" instead of scoped with conditions hash * Extract `scoped_by` method pattern regexp to constant * Extract noisy class_eval from method_missing in dynamic matchers * Extract readonly check, avoid calling column#to_s twice in persistence * Refactor predicate builder, remove some variables | ||||
* | | Merge pull request #5246 from lest/patch-2 | José Valim | 2012-03-03 | 1 | -1/+1 |
|\ \ | | | | | | | simplify namespace assignment in fields_for | ||||
| * | | simplify namespace assignment in fields_for | Sergey Nartimov | 2012-03-02 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #5260 from castlerock/remove_ordered_hash_usages | Santiago Pastorino | 2012-03-03 | 5 | -6/+6 |
|\ \ \ | | | | | | | | | remove usage of AS::OrderedHash from other places | ||||
| * | | | remove usages of AS::OrderedHash | Vishnu Atrai | 2012-03-03 | 5 | -6/+6 |
|/ / / | |||||
* | | | Merge pull request #4984 from raghunadhd/change_orderhash | Vijay Dev | 2012-03-03 | 3 | -3/+3 |
|\ \ \ | | | | | | | | | Refactored code | ||||
| * | | | Refactored code | Raghunadh | 2012-02-10 | 3 | -3/+3 |
| | | | | |||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-03-03 | 6 | -19/+65 |
|\ \ \ \ | |||||
| * | | | | copy edits [ci skip] | Vijay Dev | 2012-03-03 | 1 | -1/+1 |
| | | | | | |||||
| * | | | | Made documentation of config.assets.precompile slightly more explicit. | Guy Bolton King | 2012-03-02 | 1 | -1/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Having read the previous version of the documentation, I was surprised when a wildcarded CSS pattern in precompile picked up a whole load of SCSS files! I've amended lines 426 onwards to document the behaviour. | ||||
| * | | | | Removed max-stale from the setting the cache-headers in the response as ↵ | Anuj Dutta | 2012-03-02 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | max-stale is a cache request header. | ||||
| * | | | | Add documentation for IndexDefinition. | Waseem Ahmad | 2012-03-02 | 1 | -0/+3 |
| | | | | | |||||
| * | | | | Improve header hierarchy. | Joost Baaij | 2012-02-29 | 1 | -1/+3 |
| | | | | | |||||
| * | | | | Include a comment as to why ETag header should be removed from the response. | Joost Baaij | 2012-02-29 | 1 | -0/+1 |
| | | | | | |||||
| * | | | | Changed asset_host config to more specific | Rahul P. Chaudhari | 2012-02-29 | 1 | -2/+2 |
| | | | | | |||||
| * | | | | Some cleanup on the upgrading guide | Mike Gunderloy | 2012-02-28 | 1 | -14/+40 |
| | | | | | |||||
| * | | | | Merge pull request #85 from whilefalse/rails_32_engine_upgrade | Vijay Dev | 2012-02-28 | 1 | -0/+12 |
| |\ \ \ \ | | | | | | | | | | | | | Added guide to updating engines built in previous versions of Rails | ||||
| | * | | | | Added guide to updating engines built in previous versions of Rails | Steven Anderson | 2012-02-24 | 1 | -0/+12 |
| | | | | | | |||||
* | | | | | | Merge pull request #5258 from guilleiguaran/amo-model | Jon Leighton | 2012-03-03 | 1 | -1/+1 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Use public_send instead of send in ActiveModel::Model initialize | ||||
| * | | | | | | Use public_send instead of send in ActiveModel::Model initialize | Guillermo Iguaran | 2012-03-03 | 1 | -1/+1 |
|/ / / / / / | |||||
* | | | | | | Merge pull request #5256 from carlosantoniodasilva/active-model-model | José Valim | 2012-03-03 | 3 | -1/+81 |
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | Add docs with usage examples for ActiveModel::Model | ||||
| * | | | | | Add docs with usage examples for ActiveModel::Model | Carlos Antonio da Silva | 2012-03-03 | 3 | -1/+81 |
|/ / / / / | | | | | | | | | | | | | | | | | | | | | Also add test to ensure basic model does not explode when initialized with nil. | ||||
* | | | | | Merge pull request #5253 from guilleiguaran/amo-model | Jeremy Kemper | 2012-03-02 | 4 | -0/+44 |
|\ \ \ \ \ | | | | | | | | | | | | | Add ActiveModel::Model, a mixin to make plain Ruby objects work with AP out of box | ||||
| * | | | | | Add ActiveModel::Model, a mixin to make Ruby objects to work with AP inmediatly | Guillermo Iguaran | 2012-03-02 | 4 | -0/+44 |
|/ / / / / | |||||
* | | | / | Turn off verbose mode of rack-cache, we still have X-Rack-Cache to check ↵ | Santiago Pastorino | 2012-03-03 | 1 | -1/+1 |
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | | that info Closes #5245 | ||||
* | | | | only log an error if there is a logger. fixes #5226 | Aaron Patterson | 2012-03-02 | 3 | -2/+11 |
| | | | | |||||
* | | | | Can't cache url_options on a controller level | Piotr Sarnacki | 2012-03-02 | 1 | -11/+12 |
| | | | | | | | | | | | | | | | | | | | | It fails if routes from to railties are called in one context, for example: blog.posts_path and main_app.users_path | ||||
* | | | | Merge pull request #5243 from lest/patch-2 | José Valim | 2012-03-02 | 1 | -2/+2 |
|\ \ \ \ | | | | | | | | | | | don't pass unnecessary argument | ||||
| * | | | | don't pass unnecessary argument | Sergey Nartimov | 2012-03-02 | 1 | -2/+2 |
|/ / / / | |||||
* | | | | Merge pull request #5242 from rails/opt_routes | José Valim | 2012-03-02 | 9 | -48/+107 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Optimize routes generation in simple cases. If you pass to the route helper the same amount of arguments as the required segments, route generation will be optimized as a string interpolation. After this commit, `post_path(post)` is about 6.5 times faster, `post_url(post)` is about 5 times. | ||||
| * | | | | Optimize url helpers. | Sergey Nartimov + José Valim | 2012-03-02 | 6 | -29/+36 |
| | | | | | |||||
| * | | | | Optimize path helpers. | José Valim | 2012-03-02 | 5 | -29/+81 |
| | | | | | |||||
* | | | | | Fix #5238, rendered_format is not set when template is not rendered | Piotr Sarnacki | 2012-03-02 | 2 | -2/+8 |
| | | | | | |||||
* | | | | | Merge pull request #5241 from carlosantoniodasilva/output-safety | José Valim | 2012-03-02 | 1 | -3/+1 |
|\ \ \ \ \ | | | | | | | | | | | | | Stop SafeBuffer#clone_empty from issuing warnings | ||||
| * | | | | | Stop SafeBuffer#clone_empty from issuing warnings | Carlos Antonio da Silva | 2012-03-02 | 1 | -3/+1 |
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Logic in clone_empty method was dealing with old @dirty variable, which has changed by @html_safe in this commit: https://github.com/rails/rails/commit/139963c99a955520db6373343662e55f4d16dcd1 This was issuing a "not initialized variable" warning - related to: https://github.com/rails/rails/pull/5237 The logic applied by this method is already handled by the [] override, so there is no need to reset the variable here. | ||||
* | | | | | Merge pull request #5234 from benpickles/allow-empty-fieldsets | Piotr Sarnacki | 2012-03-02 | 2 | -2/+11 |
|\ \ \ \ \ | |/ / / / |/| | | | | Allow empty fieldsets to be created without a block. | ||||
| * | | | | Allow fieldsets to be created without a block. | Ben Pickles | 2012-03-01 | 2 | -2/+11 |
| | | | | | |||||
* | | | | | revert setting NOT NULL constraints in add_timestamps | Xavier Noria | 2012-03-01 | 1 | -2/+2 |
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 3dbedd2 added NOT NULL constraints both to table creation and modification. For creation the new default makes sense, but the generic situation for changing a table is that there exist records. Those records have no creation or modification timestamps, and in the general case you don't even know them, so when updating a table these constraints are not going to work. See a bug report for this use case in #3334. | ||||
* | | | | Merge pull request #5232 from claudiob/add_release_date_to_docs | Vijay Dev | 2012-03-01 | 6 | -12/+12 |
|\ \ \ \ | | | | | | | | | | | Add release dates to documentation | ||||
| * | | | | Add release dates to documentation | claudiob | 2012-03-01 | 6 | -12/+12 |
|/ / / / | | | | | | | | | | | | | Set "March 1, 2012" as the release date for 3.2.2, 3.1.4, 3.0.12 | ||||
* | | | | Merge branch 'master-security' | Aaron Patterson | 2012-03-01 | 5 | -29/+63 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master-security: Ensure [] respects the status of the buffer. delete vulnerable AS::SafeBuffer#[] use AS::SafeBuffer#clone_empty for flushing the output_buffer add AS::SafeBuffer#clone_empty fix output safety issue with select options Conflicts: actionpack/lib/action_view/helpers/tags/base.rb | ||||
| * | | | | Ensure [] respects the status of the buffer. | José Valim | 2012-02-29 | 2 | -16/+41 |
| | | | | | |||||
| * | | | | delete vulnerable AS::SafeBuffer#[] | Akira Matsuda | 2012-02-20 | 2 | -12/+0 |
| | | | | | |||||
| * | | | | use AS::SafeBuffer#clone_empty for flushing the output_buffer | Akira Matsuda | 2012-02-20 | 1 | -1/+1 |
| | | | | | |||||
| * | | | | add AS::SafeBuffer#clone_empty | Akira Matsuda | 2012-02-20 | 2 | -0/+15 |
| | | | | | |||||
| * | | | | fix output safety issue with select options | Sergey Nartimov | 2012-02-20 | 2 | -4/+11 |
| | | | | | |||||
* | | | | | Tap, tap goes the middleware | Jeremy Kemper | 2012-02-29 | 1 | -4/+1 |
| | | | | | |||||
* | | | | | Merge pull request #5203 from avakhov/anonymous-controller-layout | Aaron Patterson | 2012-02-29 | 2 | -0/+16 |
|\ \ \ \ \ | | | | | | | | | | | | | Fix layout lookup for anonymous controller | ||||
| * | | | | | Fix layout lookup for anonymous controller | Alexey Vakhov | 2012-02-28 | 2 | -0/+16 |
| | | | | | |