aboutsummaryrefslogtreecommitdiffstats
path: root/activesupport/lib
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | extract PerformanceTest into rails-performance_tests gemYves Senn2013-01-104-672/+0
| | | | |
* | | | | adding missing requiresAaron Patterson2013-01-091-1/+3
| | | | |
* | | | | adding start / finish on the instrumenter, adding tests for the classAaron Patterson2013-01-091-6/+17
| | | | |
* | | | | CVE-2013-0156: Safe XML params parsing. Doesn't allow symbols or yaml.Jeremy Kemper2013-01-081-4/+23
| | | | |
* | | | | Revert "Merge branch 'master-sec'"Jeremy Kemper2013-01-081-23/+4
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 88cc1688d0cb828c17706b41a8bd27870f2a2beb, reversing changes made to f049016cd348627bf8db0d72382d7580bf802a79.
* | | | | Merge branch 'master-sec'Aaron Patterson2013-01-081-4/+23
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | * master-sec: CVE-2013-0156: Safe XML params parsing. Doesn't allow symbols or yaml. * Strip nils from collections on JSON and XML posts. [CVE-2013-0155] * dealing with empty hashes. Thanks Damien Mathieu
| * | | | CVE-2013-0156: Safe XML params parsing. Doesn't allow symbols or yaml.Jeremy Kemper2013-01-081-4/+23
| | | | |
* | | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-01-097-8/+25
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Conflicts: guides/source/getting_started.md
| * | | | Correct method notation for #acts_like? [ci skip]Doug Yun2013-01-071-4/+4
| | | | |
| * | | | Instance_variable_names are strings. That is the most important information IMOAkira Matsuda2013-01-081-1/+1
| | | | |
| * | | | prefer american spelling of 'behavior'Gosha Arinich2013-01-072-2/+2
| | | | |
| * | | | `Model.all` alone does nothingAkira Matsuda2013-01-041-1/+1
| | | | |
| * | | | Add documentation for BacktraceCleaner#remove_filters!Matthew Stopa2013-01-021-0/+3
| | | | |
| * | | | Add more documentation to TimeWithZoneMatthew Stopa2013-01-011-0/+14
| | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | Revert some warning removals related to Ruby 2.0Carlos Antonio da Silva2013-01-072-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These warnings were actually a bug in Ruby 2.0, the accessors should not be raising such warnings, they are only meant for ivars. - Revert "fix warnings in Ruby 2.0" This reverts commit 26702a6d3461f4a1c75165030b96886514ecb877. - Revert "Merge pull request #8282 from arunagw/warning_removed_for_ruby2" This reverts commit f63d6544e45e78cda29c0c56fbdf3d9e1f405340, reversing changes made to 3a890681fad8218305585036abed6d7463a44e41.
* | | | | Require 'active_support/core_ext/module/aliasing' in the infinite_comparable ↵Diogo Mafra2013-01-061-0/+1
| |_|_|/ |/| | | | | | | | | | | module
* | | | Remove unnecessary begin..rescue..end, use only rescueAkira Matsuda2013-01-063-25/+20
| | | |
* | | | deprecate `assert_blank` and `assert_present`.Yves Senn2013-01-051-0/+2
| | | | | | | | | | | | | | | | | | | | They don't add any benefits over `assert object.blank?` and `assert object.present?`
* | | | added marshal_load and marshal_dump for ProxyTestResult. Behavior of ↵SHIBATA Hiroshi2013-01-051-2/+10
| | | | | | | | | | | | | | | | method_missing with Marshal.dump and Marshal.load is changing in ruby 2.0.0 later.
* | | | Better error message for String#to_dateKelly Stannard2013-01-041-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I did this because to_date gives a very unhelpful error message if you do not pass in a correct date. In the process I think this cleans up the code nicely and even better it tends to be slightly faster than the current implementation. Benchmark https://gist.github.com/4440875
* | | | Set the default timezone after the initialization since the configurationRafael Mendonça França2013-01-041-14/+16
| | | | | | | | | | | | | | | | | | | | | | | | now lives in the application initializers. Fix #8711
* | | | Refactor infinite comparable definition a bitCarlos Antonio da Silva2013-01-041-15/+19
| | | |
* | | | Remove unnecessary requiresRafael Mendonça França2013-01-031-2/+0
| | | |
* | | | Added ability to compare date/time with infinitybUg2013-01-0410-7/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Date, DateTime, Time and TimeWithZone can now be compared to infinity, so it's now possible to create ranges with one infinite bound and date/time object as another bound. Ex.: @range = Range.new(Date.today, Float::INFINITY) Also it's possible to check inclusion of date/time in range with conversion. Ex.: @range.include?(Time.now + 1.year) # => true @range.include?(DateTime.now + 1.year) # => true Ability to create date/time ranges with infinite bound is required for handling postgresql range types.
* | | | Change docs to use update instead of update_attributesAmparo Luna + Guillermo Iguaran2013-01-031-1/+1
| | | |
* | | | remove meaningless AS::FrozenObjectErrorAkira Matsuda2013-01-022-4/+0
| | | |
* | | | remove unnecessary always-nil var from #compactGosha Arinich2013-01-011-1/+1
|/ / /
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-01-011-0/+21
|\ \ \ | |/ / |/| |
| * | copy edits [ci skip]Vijay Dev2013-01-011-4/+3
| | |
| * | Add documentation for TimeWithZone methodsMatthew Stopa2012-12-311-3/+19
| | |
| * | Add documentation for the TimeWithZone#dst? method.Matthew Stopa2012-12-311-0/+6
| |/ | | | | | | [ci skip]
* | Updated copyright notices for 2013Andrew Nesbitt2012-12-311-1/+1
| |
* | Alias refute methods to assert_not and perfer assert_not on testsRafael Mendonça França2012-12-311-1/+10
| |
* | Add active_support/testing/autorunRafael Mendonça França2012-12-311-0/+5
| | | | | | | | | | minitest/autorun load minitest/spec polluting the global namespace with the DSL that we don't want on Rails
* | Inherit from MiniTest::Unit::TestCase instead of MiniTest::SpecRafael Mendonça França2012-12-311-8/+2
| |
* | Revert "minitest provides "it" and "describe""Rafael Mendonça França2012-12-312-19/+42
|/ | | | | | | | | | This reverts commit 22bc12ec374b8bdeb3818ca0a3eb787dd3ce39d8. REASON: We will remove the MiniTest::Spec from Rails and we need these methods again Conflicts: activesupport/lib/active_support/test_case.rb
* Merge pull request #8641 from arunagw/warning_removed_unused_varSteve Klabnik2012-12-281-1/+2
|\ | | | | Remove 'assigned but unused variable' warning
| * Remove 'assigned but unused variable' warningArun Agrawal2012-12-281-1/+2
| |
* | Introduce assert_not to replace 'assert !foo'Jeremy Kemper2012-12-281-0/+16
|/
* Make test logs easier to read.Jeremy Kemper2012-12-261-11/+10
| | | | | | Tagging every message in tests makes the logs really wide. It's great for grepping, but annoying to open in an editor or a narrow terminal. Try out a different approach: spit out a heading before each test.
* deprecation warning when BufferedLogger is instantiatedYves Senn2012-12-251-1/+11
|
* ActiveSupport::BufferedLogger can be subclassedYves Senn2012-12-251-2/+6
|
* Remove warning of intance variable not initializedRafael Mendonça França2012-12-231-1/+5
|
* Move background jobs to the 'jobs' branch until fully baked. Not shipping ↵Jeremy Kemper2012-12-211-105/+0
| | | | with Rails 4.0.
* Merge pull request #8471 from kytrinyx/refactor-xml-to-hashSteve Klabnik2012-12-211-50/+100
|\ | | | | WIP Refactor xml conversion to hash
| * Refactor Hash.from_xml.Steve Klabnik + Katrina Owen2012-12-211-50/+100
| | | | | | | | | | | | | | | | | | | | | | | | Three basic refactors in this PR: * We extracted the logic into a method object. We now don't define a tone of extraneous methods on Hash, even if they were private. * Extracted blocks of the case statement into methods that do the work. This makes the logic more clear. * Extracted complicated if clauses into their own query methods. They often have two or three terms, this makes it much easier to see what they _do_. We took care not to refactor too much as to not break anything, and put comments where we suspect tests are missing. We think ActiveSupport::XMLMini might be a good candidate to move to a plugin in the future.
* | Merge pull request #7376 from ↵Rafael Mendonça França2012-12-211-5/+29
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | dmitriy-kiriyenko/fix-double-callback-in-same-statement Prevent callback from being set twice. Conflicts: activesupport/CHANGELOG.md
| * | Prevent callback from being set twice.Dmitriy Kiriyenko2012-12-201-5/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When you add one callack in two separate `set_callback` calls - it is only called once. When you do it in one `set_callback` call - it is called twice. This violates the principle of least astonishment for me. Duplicating callback is usually an error. There is a correct and obvious way to do anything without this "feature". If you want to do before_save :clear_balance, :calculate_tax, :clear_balance or whatever, you should better do before_save :carefully_calculate_tax def carefully_calculate_tax clear_balance calculate_tax clear_balance end And this even opens gates for some advanced refactorings, unlike the first approach. My assumptions are: - Principle of least astonishment is violated, when callbacks are either prevented from duplication, or not. - Duplicating callbacks is usually an error. When it is intentional - it's a smell of a bad design and can be approached without abusing this "feature". My suggestion is: do not allow duplicating callbacks in one callback call, like it is not allowed in separate callbacks call.
* | | BufferedLogger is deprecated too.Rafael Mendonça França2012-12-211-1/+1
| | |
* | | Revert "Make sure that ActiveSupport::Logger includes the Logger extensions ↵David Heinemeier Hansson2012-12-213-20/+29
| | | | | | | | | | | | | | | | | | from core_ext/logger" (some confusion over deprecation) This reverts commit d00f568a83a5159ed93618b1081bd17858536d1c.