Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove deprecated separator argument from parameterize | Andrew White | 2016-11-14 | 5 | -44/+6 |
| | |||||
* | Remove deprecated method Numeric#to_formatted_s | Andrew White | 2016-11-14 | 3 | -11/+4 |
| | |||||
* | Remove deprecated method alias_method_chain | Andrew White | 2016-11-14 | 3 | -263/+4 |
| | |||||
* | Remove deprecated constant MissingSourceFIle | Andrew White | 2016-11-14 | 3 | -10/+4 |
| | |||||
* | Remove deprecated Module.qualified_const_get/set/defined? | Andrew White | 2016-11-14 | 5 | -190/+5 |
| | |||||
* | Update CHANGELOG | Andrew White | 2016-11-13 | 1 | -0/+48 |
| | |||||
* | Remove deprecated :prefix option | Andrew White | 2016-11-13 | 3 | -37/+1 |
| | |||||
* | Remove deprecated new_from_hash_copying_default | Andrew White | 2016-11-13 | 2 | -16/+0 |
| | |||||
* | Remove deprecated time marshal core_ext file | Andrew White | 2016-11-13 | 1 | -3/+0 |
| | |||||
* | Remove deprecated struct core_ext file | Andrew White | 2016-11-13 | 1 | -3/+0 |
| | |||||
* | Remove deprecated module method_transplanting file | Andrew White | 2016-11-13 | 1 | -3/+0 |
| | |||||
* | Remove deprecated local_constants | Andrew White | 2016-11-13 | 2 | -18/+0 |
| | |||||
* | Remove deprecated kernel debugger file | Andrew White | 2016-11-13 | 1 | -3/+0 |
| | |||||
* | Remove deprecated namespaced_key | Andrew White | 2016-11-13 | 2 | -14/+0 |
| | |||||
* | Remove deprecated set_cache_value | Andrew White | 2016-11-13 | 2 | -17/+0 |
| | |||||
* | Remove deprecated escape_key | Andrew White | 2016-11-13 | 2 | -14/+0 |
| | |||||
* | Remove deprecated key_file_path | Andrew White | 2016-11-13 | 2 | -14/+0 |
| | |||||
* | Minor corrections to #26905 | Andrew White | 2016-11-13 | 2 | -2/+2 |
| | |||||
* | Fix typo in constant reference | Andrew White | 2016-11-13 | 1 | -1/+1 |
| | |||||
* | It's a NAN not Infinity | Andrew White | 2016-11-13 | 1 | -1/+1 |
| | |||||
* | Use literal values in assertions | Andrew White | 2016-11-13 | 1 | -6/+4 |
| | | | | | Using the method you're testing to generate expected values can lead to bugs being masked. | ||||
* | Merge pull request #26933 from prathamesh-sonpatki/fix-26877 | Andrew White | 2016-11-13 | 2 | -1/+23 |
|\ | | | | | Fix an issue with JSON encoding of "Infinity" and "NaN" values | ||||
| * | Fix an issue with JSON encoding of "Infinity" and "NaN" values | Prathamesh Sonpatki | 2016-10-30 | 2 | -1/+23 |
| | | | | | | | | | | | | | | | | | | | | | | - When `as_json` returns `Infinity` or `NaN` as the value of any of the key, we don't used to call `as_json` on it as it was treated as primitive. - This used to pass `Infinity` or `NaN` to `JSON.generate` and Ruby used to throw an error for `Infinity/NaN not allowed in JSON.` - This patch changes the code to call `as_json` on these primitives so that they are converted to proper values before being passed to `JSON.generate`. - Fixes #26877. | ||||
* | | Merge pull request #27024 from vipulnsward/changelog-edits | Andrew White | 2016-11-13 | 4 | -15/+15 |
|\ \ | | | | | | | Changelog edits | ||||
| * | | Changelog edits | Vipul A M | 2016-11-12 | 4 | -15/+15 |
| | | | |||||
* | | | Merge pull request #26905 from bogdanvlviv/docs | Andrew White | 2016-11-13 | 42 | -75/+77 |
|\ \ \ | | | | | | | | | Add missing `+` around a some literals. | ||||
| * | | | Add missing `+` around a some literals. | bogdanvlviv | 2016-10-27 | 42 | -75/+77 |
| | | | | | | | | | | | | | | | | | | | | | | | | Mainly around `nil` [ci skip] | ||||
* | | | | removes unconventional "then" in case expression | Xavier Noria | 2016-11-13 | 1 | -1/+1 |
| | | | | |||||
* | | | | revises style in recent code (of mine) | Xavier Noria | 2016-11-13 | 1 | -5/+7 |
| | | | | | | | | | | | | | | | | | | | | This alternative case expressions read better for my taste, and look more uniform in a file where other similar case expressions are used (without dynamic clauses). | ||||
* | | | | Merge pull request #27019 from djpowers/patch-1 | Jon Moss | 2016-11-12 | 2 | -3/+3 |
|\ \ \ \ | | | | | | | | | | | Fix spelling in API documentation | ||||
| * | | | | Fix spelling in API docs | Dave Powers | 2016-11-12 | 2 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | [ci skip] | ||||
* | | | | | Add test for #25248 | Sean Griffin | 2016-11-12 | 1 | -0/+10 |
| | | | | | |||||
* | | | | | Merge pull request #25248 from bluesh55:features/skip-coffee | Sean Griffin | 2016-11-12 | 1 | -1/+6 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | Add --skip-coffee generating option | ||||
| * | | | | | Add --skip-coffee generating option | seunghwan oh | 2016-06-02 | 1 | -1/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usage `rails new awesome-project --skip-coffee` | ||||
* | | | | | | Merge pull request #27022 from kamipo/add_nodoc_to_table_structure | Andrew White | 2016-11-12 | 1 | -9/+2 |
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | Add `:nodoc` to `table_structure` | ||||
| * | | | | | Add `:nodoc` to `table_structure` | Ryuta Kamizono | 2016-11-12 | 1 | -9/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow up to #27008. `table_structure` is an internal method so it is better to hide it in the doc. And alias `table_structure` to `column_definitions` to remove the duplicated `columns` method in the sqlite3 adapter. | ||||
* | | | | | | significant speedup of AC::Parameters#permit | Xavier Noria | 2016-11-12 | 2 | -1/+8 |
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current implementation of AC::Parameters#permit builds permitted hashes and then calls permit! on them. This filtering is recursive, so we call permit! on terminal branches, but then ascendants call permit! on themselves when the recursion goes up the stack, which recurses all the way down again because permit! is recursive itself. Repeat this for every parent node and you get some scary O-something going on that I don't even want to compute. Instead, since the whole point of the permit recursion is to build permitted hashes along the way and at that point you know you've just come up with a valid filtered version, you can already switch the toggle on the spot. I have seen 2x speedups in casual benchmarks with small structures. As the previous description shows, the difference in performance is going to be a function of the nesting. Note that that the involved methods are private and used only by permit. | ||||
* | | | | | Merge pull request #27018 from ↵ | Vipul A M | 2016-11-11 | 1 | -0/+12 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | alexanderkustov/alexanderkustov/update-docs-ActionView better docs for ActionView::ViewPaths#prepend_view_path [ci skip] | ||||
| * | | | | | better docs for ActionView::ViewPaths#prepend_view_path | Alexander Kustov | 2016-11-12 | 1 | -0/+12 |
| | | | | | | | | | | | | | | | | | | | | | | | | ✂️ extra line | ||||
* | | | | | | adds a missing dot [ci skip] | Xavier Noria | 2016-11-11 | 1 | -1/+1 |
| | | | | | | |||||
* | | | | | | adds support for arbitrary hashes in strong parameters | Xavier Noria | 2016-11-11 | 4 | -3/+101 |
| |_|/ / / |/| | | | | |||||
* | | | | | Merge pull request #27008 from kirs/new-column-from-field | Rafael França | 2016-11-11 | 4 | -38/+48 |
|\ \ \ \ \ | |/ / / / |/| | | | | Refactor column initialization into `new_column_from_field` | ||||
| * | | | | Refactor column initialization into `new_column_from_field` | Kir Shatrov | 2016-11-11 | 4 | -38/+48 |
|/ / / / | | | | | | | | | | | | | that accepts results of SHOW FIELDS | ||||
* | | | | Merge pull request #26978 from matthewd/query-cache-pool | Matthew Draper | 2016-11-10 | 4 | -4/+73 |
|\ \ \ \ | | | | | | | | | | | Configure query caching (per thread) on the connection pool | ||||
| * | | | | Configure query caching (per thread) on the connection pool | Matthew Draper | 2016-11-06 | 4 | -2/+71 |
| | | | | | |||||
| * | | | | Try harder to avoid making a connection while releasing it | Matthew Draper | 2016-11-06 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | connected? doesn't mean what we need here. | ||||
* | | | | | Merge pull request #27006 from bogdanvlviv/ar_querying_guide | Jon Moss | 2016-11-10 | 1 | -0/+3 |
|\ \ \ \ \ | | | | | | | | | | | | | Update guides/source/active_record_querying.md | ||||
| * | | | | | Update guides/source/active_record_querying.md | bogdanvlviv | 2016-11-10 | 1 | -0/+3 |
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add info about updating locking column value Follow #26050 and #26871 [ci skip] | ||||
* | | | | | Merge pull request #26820 from y-yagi/add_bang_merge_to_parameters | Rafael França | 2016-11-10 | 3 | -0/+28 |
|\ \ \ \ \ | | | | | | | | | | | | | add `ActionController::Parameters#merge!` | ||||
| * | | | | | add `ActionController::Parameters#merge!` | yuuji.yaginuma | 2016-10-18 | 3 | -0/+28 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method has the same behavior as `Hash#merge!`, returns current `ActionController::Parameters`. |