aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | extract a method for updating without a record objectAaron Patterson2013-12-121-6/+10
| | | | | |
* | | | | | extract methods out of the cache update methodAaron Patterson2013-12-121-14/+22
| | | | | |
* | | | | | pull a nil check up one frameAaron Patterson2013-12-121-6/+10
|/ / / / /
* | | | | docs guidelines: recommends wording in a way that avoids "you"s and "your"sXavier Noria2013-12-121-1/+15
| | | | |
* | | | | copy-edits 68abbac [ci skip]Xavier Noria2013-12-121-10/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Rewording to avoid "you"s. * Suggest as first natural alternative to refactor the block to evaluate to the returned value. * Removes the quotes around "filters", since that is a common work in our jargon.
* | | | | Merge pull request #13255 from strzalek/bump-builderRafael Mendonça França2013-12-122-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Bump up builder
| * | | | | More liberal builder dependencyŁukasz Strzałkowski2013-12-122-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Allowing us to get 3.2.x versions if needed.
* | | | | | Merge pull request #13300 from chancancode/add_nodoc_am_dirty_methodsAaron Patterson2013-12-121-2/+2
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Added :nodoc: for `attribute_changed?` and `attribute_was` [ci skip]
| * | | | | Added :nodoc: for `attribute_changed?` and `attribute_was` [ci skip]Godfrey Chan2013-12-121-2/+2
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | These methods were made "public" in 47617ecd so that `method_missing` can invoke them without going through `send`, but they aren't meant for consumption from outside of Rails.
* | | | | remove more nil checksAaron Patterson2013-12-121-6/+5
| | | | |
* | | | | remove nil checkAaron Patterson2013-12-122-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | | calling replace_keys could possibly do a nil check twice, this commit reduces it to once.
* | | | | remove the nil check from set_inverse_instanceAaron Patterson2013-12-124-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | methods that call set_inverse_instance with a record will not have to pay the cost of a nil check on every call
* | | | | make sure cached table name is a string. fixes #12582Aaron Patterson2013-12-123-1/+12
| | | | |
* | | | | Merge pull request #13229 from vipulnsward/fix-10635-testsRafael Mendonça França2013-12-122-2/+17
|\ \ \ \ \ | | | | | | | | | | | | Fix test-error introduced by #10635.
| * | | | | PR #10635 introduces rescue from ArgumentError thrown by ↵Vipul A M2013-12-122-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `Base64.strict_decode64`. This broke natural order of things for `StaleSessionCheck#stale_session_check!` which tried auto_loading a class based on `ArgumentError` message , and later retrying the `Marshal#load` of class, successfully allowing auto_loading. This PR tries to fix this behavior by forwarding `ArgumentError` 's not raised by `Base64.strict_decode64` , as is, ahead to `StaleSessionCheck#stale_session_check!`
* | | | | | Merge pull request #13293 from akshay-vishnoi/typoCarlos Antonio da Silva2013-12-128-12/+12
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | | Spelling and Grammar checks [ci skip]
| * | | | | Spelling and Grammar checksAkshay Vishnoi2013-12-128-12/+12
| | |_|/ / | |/| | |
* | | | | Merge pull request #13271 from ↵Rafael Mendonça França2013-12-121-0/+33
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | chancancode/warn_about_using_return_in_as_callbacks Warn about using `return` inside inline callback blocks [ci skip]
| * | | | | Warn about using `return` inside inline callback blocks [ci skip]Godfrey Chan2013-12-111-0/+33
| | | | | | | | | | | | | | | | | | | | | | | | Closes #12981
* | | | | | Copy-edit the parent commitRafael Mendonça França2013-12-121-6/+8
| | | | | |
* | | | | | Merge pull request #13268 from aayushkhandelwal11/masterRafael Mendonça França2013-12-121-22/+16
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Moving the common code of increment and decrement of cache file store in...
| * | | | | | Moving the common code of increment and decrement of cache file store into ↵Aayush khandelwal2013-12-121-22/+16
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | seprate function Removing the double checking for the value of id simplifying code for checking conditional code for key Removing the default values for parameters removing reduntant code and asiigning value of id in local variable removing wrongly added line break[ci ckip] reverting code
* | | | | | Merge pull request #13290 from strzalek/inline-syntax-docsRafael Mendonça França2013-12-123-0/+27
|\ \ \ \ \ \ | | | | | | | | | | | | | | Variants inline syntax documentation
| * | | | | | Variants inline syntax documentation [ci skip]Łukasz Strzałkowski2013-12-123-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Extend method documentation * Mention it in actionpack/CHANGELOG * Update release notes
* | | | | | | Merge pull request #13291 from strzibny/new_unique_constraintYves Senn2013-12-121-1/+5
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Translate new unique constraint error message for sqlite >= 3.8.2
| * | | | | | Translate new unique constraint for sqlite >= 3.8.2Josef Stribny2013-12-121-1/+5
| | | | | | |
* | | | | | | Merge pull request #13032 from kuldeepaggarwal/after_touch_documentationYves Senn2013-12-121-0/+49
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | [ci skip] added after_touch callback documentation [ci skip]
| * | | | | | | [ci skip] added after_touch callback documentationKuldeep Aggarwal2013-12-121-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] added more explanation for `after_touch` callback
* | | | | | | | Get rid of hack for freezing time on AR testsCarlos Antonio da Silva2013-12-122-23/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We can now make use of the existent #travel/#travel_to helper methods added to AS test case and available in all tests.
* | | | | | | | Changelog improvements [ci skip]Carlos Antonio da Silva2013-12-123-12/+14
| |_|/ / / / / |/| | | | | |
* | | | | | | Merge pull request #13288 from dvsuresh/fix_typoDamien Mathieu2013-12-121-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix typo in action_controller responder.rb [ci skip]
| * | | | | | | Fix typo in action_controller responder.rbdvsuresh2013-12-121-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #13287 from aayushkhandelwal11/typo_rectifiedYves Senn2013-12-113-5/+5
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | typos rectified lifecycle => life cycle
| * | | | | | | | typos rectified lifecycle => life cycleAayush khandelwal2013-12-123-5/+5
| | |_|_|/ / / / | |/| | | | | |
* | | | | | | | JSON: speed up date/time encodingJeremy Kemper2013-12-111-24/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Comparing dates & times with other objects using #== is slow. Internally, it hits #<=> which calls #to_datetime on objects that don't respond to it, so we incur the cost of raising an exception with a possibly-deep backtrace. Cost of #jsonify on a Time object: Calculating ------------------------------------- old 3644 i/100ms new 12652 i/100ms ------------------------------------------------- old 43373.8 (±3.5%) i/s - 218640 in 5.047017s new 173437.6 (±5.2%) i/s - 872988 in 5.047747s
* | | | | | | | Merge pull request #13284 from aayushkhandelwal11/typos_correctedGodfrey Chan2013-12-113-3/+3
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | s/everytime/every time/
| * | | | | | | typos rectified [ci skip]Aayush khandelwal2013-12-123-3/+3
| |/ / / / / /
* | | | | | | Merge pull request #13279 from misfo/session-fetch-that-behaves-like-hash-fetchRafael Mendonça França2013-12-113-14/+11
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Make ActionDispatch::Request::Session#fetch behave like Hash#fetch
| * | | | | | | Update CHANGELOG entry for Session#fetchTrent Ogren2013-12-111-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts the changes to CHANGELOG.md in commit 38f8872aa5fd8f0a1d0895e9eb41f73261acd040.
| * | | | | | | Make ActionDispatch::Request::Session#fetch behave like Hash#fetchTrent Ogren2013-12-112-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Session#fetch was mutating the session when given a default argument and/or a block. Since Session duck-types as a Hash, it should behave like one in these cases.
* | | | | | | | Merge pull request #13281 from waynn/patch-5Rafael Mendonça França2013-12-111-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | test description uses "disable" when it should be "disabled"
| * | | | | | | | test description uses "disable" when it should be "disabled"Waynn Lue2013-12-111-1/+1
|/ / / / / / / /
* | | | | | | | Merge pull request #13280 from waynn/patch-4Rafael Mendonça França2013-12-111-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | value is "disabled" not "disable"
| * | | | | | | | value is "disabled" not "disable"Waynn Lue2013-12-111-1/+1
| | | | | | | | |
* | | | | | | | | Remove not necessary file, move constants to the file they are usedCarlos Antonio da Silva2013-12-113-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | File 'empty_bool.rb' was introduced around 4 years ago in c10958fbddb22052e7cbe5fe6b825cda3cb26e48 to remove method redefined warning in AS test suite, however we do not have such need for reuse anymore, so we can safely move the constants back to the file where they are currently used and get rid of the extra file/require.
* | | | | | | | | Merge pull request #13264 from ↵Rafael Mendonça França2013-12-113-2/+16
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | laurocaetano/fix_dynamic_finder_with_reserved_words Prevent invalid code when using dynamic finders with reserved ruby word.
| * | | | | | | | Prevent invalid code when using dynamic finders with Ruby's reserved words.Lauro Caetano2013-12-113-2/+16
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The dynamic finder was creating the method signature with the parameters name, which may have reserved words and this way creating invalid Ruby code. Closes: #13261 Example: # Before Dog.find_by_alias('dog name') # Was creating this method def self.find_by_alias(alias, options = {}) # After Dog.find_by_alias('dog name') # Will create this method def self.find_by_alias(_alias, options = {})
* | | | | | | | Merge branch 'support-deprecated-finders'Rafael Mendonça França2013-12-118-48/+66
|\ \ \ \ \ \ \ \
| * | | | | | | | Move the parameter normalization to the initialize methodRafael Mendonça França2013-12-111-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | activerecord-deprecated_finders expects the parameters denormalized in its initialize method
| * | | | | | | | Revert the whole refactoring in the association builder classes.Rafael Mendonça França2013-12-118-47/+64
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | This is to get activerecord-deprecated_finders work again