Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | beginning_of_week extended in both Time and Date so that to return week ↵ | gregolsen | 2011-11-25 | 2 | -0/+38 | |
| | | | | | | | | start based on start day that is monday by default | |||||
* | | Rely solely on active_model_serializer and remove the fancy constant lookup. | José Valim | 2011-11-23 | 1 | -0/+5 | |
| | | ||||||
* | | Add safe_constantize to ActiveSupport::Dependencies. | José Valim | 2011-11-23 | 1 | -15/+23 | |
| | | ||||||
* | | Optimize cache expansion by skipping rails cache id in nested keys. | José Valim | 2011-11-23 | 1 | -4/+4 | |
| | | ||||||
* | | Fix #3737 AS::expand_cache_key generates wrong key in certain situations ↵ | Olek Janiszewski | 2011-11-23 | 1 | -2/+13 | |
| | | | | | | | | | | | | | | (part 2) `nil` and `false` both expand to `""` (empty string), while `true` expands to `"true"`; `false` should expand to `"false"` | |||||
* | | Fix #3737 AS::expand_cache_key generates wrong key in certain situations | Olek Janiszewski | 2011-11-23 | 1 | -2/+10 | |
| | | | | | | | | | | `cache_key` method is never called when the argument is a 1-element array with something that responds to `cache_key` | |||||
* | | Merge pull request #3725 from marcandre/twz_eql | Yehuda Katz | 2011-11-22 | 2 | -2/+15 | |
|\ \ | | | | | | | Fix inconsistencies with Time{WithZone}#{hash,eql?} | |||||
| * | | Fix inconsistencies with Time{WithZone}#{hash,eql?} | Marc-Andre Lafortune | 2011-11-22 | 2 | -2/+15 | |
| | | | ||||||
* | | | tests predicates according to the boolean interpretation of their return ↵ | Xavier Noria | 2011-11-21 | 1 | -48/+48 | |
| | | | | | | | | | | | | | | | | | | value, not expecting specifically true or false Respecting Genuine assert_equal false Edition | |||||
* | | | Deprecate InstanceMethods namespace handling in ActiveSupport::Concern. | José Valim | 2011-11-21 | 1 | -5/+2 | |
| | | | | | | | | | | | | This avoids the unnecessary "yo dawg, I heard you like include, so I put a module that includes your module when it is included" approach when building extensions. | |||||
* | | | Revert "tests predicates according to the boolean interpretation of their ↵ | Xavier Noria | 2011-11-21 | 1 | -49/+49 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | return value, not expecting specifically true or false" Reason: there were some genuine tests for false when reading values, going to revise those ones. This reverts commit a539a5e3f3be68f027d3dfe43f88dc9f0642c743. | |||||
* | | | tests predicates according to the boolean interpretation of their return ↵ | Xavier Noria | 2011-11-21 | 1 | -49/+49 | |
| | | | | | | | | | | | | value, not expecting specifically true or false | |||||
* | | | cache entry: options[:compressed] is a regular flag, no need for !! | Xavier Noria | 2011-11-21 | 1 | -3/+3 | |
| |/ |/| | ||||||
* | | refactor test_multiple_of | Vijay Dev | 2011-11-16 | 1 | -4/+3 | |
| | | ||||||
* | | Don't marshal dump twice when using encryptor. | José Valim | 2011-11-09 | 1 | -2/+1 | |
| | | ||||||
* | | Deprecated ActiveSupport::MessageEncryptor#encrypt and decrypt. | José Valim | 2011-11-09 | 1 | -24/+36 | |
| | | ||||||
* | | Merge pull request #3549 from luckydev/rmethod | Aaron Patterson | 2011-11-07 | 1 | -0/+29 | |
|\ \ | | | | | | | remove_method tests and changes | |||||
| * | | added remove_method core_ext tests | Anand | 2011-11-07 | 1 | -0/+29 | |
| | | | ||||||
* | | | Removed argument throwing warnings when running tests. | Henrik Hodne | 2011-11-06 | 2 | -16/+16 | |
| | | | | | | | | | | | | | | | | | | | | | | | | The last argument to DateTime.new/civil is the Julian day number for when the calendar reform occured. DateTime ignores it if it's 0, and sets it to the default of DateTime::ITALY instead, so we could just leave it out. This also removes the warnings while running the ActiveSupport tests (there still are some for the setup though). | |||||
* | | | Added the test case for #3537 | Pavan Kumar Sunkara | 2011-11-06 | 1 | -0/+1 | |
|/ / | ||||||
* | | implements AS::Notifications.subscribed, which provides subscriptions to ↵ | Xavier Noria | 2011-11-05 | 1 | -0/+20 | |
| | | | | | | | | events while a block runs | |||||
* | | make class name consistent with the filename | Vijay Dev | 2011-11-05 | 1 | -1/+1 | |
|/ | ||||||
* | defines Module#qualified_const_(defined?|get|set) and String#deconstantize | Xavier Noria | 2011-10-29 | 3 | -0/+110 | |
| | | | | | | | | | | This commit also implements a faster version of #demodulize I was unable to isolate with git add --patch. Not a big fan of the name #deconstantize. It complements #demodulize getting rid of the rightmost constant, hence the name, but it is unrelated to the well-known #constantize. So unsure. Could not come with anything better, please feel free to rename. | |||||
* | let demodulize do less work, and add tests | Xavier Noria | 2011-10-29 | 1 | -0/+2 | |
| | | | | This is also faster on 1.9. | |||||
* | Revert "Merge pull request #3395 from bdurand/fix_file_store_cleanup" | José Valim | 2011-10-25 | 1 | -34/+0 | |
| | | | | | | | Tests were failing on Travis-CI. This reverts commit 79d01a8f16e20c556a086a2f07e3ccb4400f9819, reversing changes made to b838570bd69ff13d677fb43e79f10d6f3168c696. | |||||
* | Unused variable removed | Arun Agrawal | 2011-10-24 | 1 | -1/+1 | |
| | ||||||
* | Merge pull request #3395 from bdurand/fix_file_store_cleanup | José Valim | 2011-10-21 | 1 | -0/+34 | |
|\ | | | | | Fix ActiveSupport::Cache::FileStore.cleanup to actually work. | |||||
| * | Fix ActiveSupport::Cache::FileStore.cleanup to actually work. | Brian Durand | 2011-10-21 | 1 | -0/+34 | |
| | | ||||||
* | | Fix threading issues with BufferedLogger. | Brian Durand | 2011-10-21 | 1 | -0/+53 | |
|/ | ||||||
* | Checking blank if tag might coming nil or blank | Arun Agrawal | 2011-10-21 | 1 | -0/+5 | |
| | | | In log it should not show the empty array. | |||||
* | Ensure TaggegLogging is thread safe. | José Valim | 2011-10-19 | 1 | -1/+29 | |
| | ||||||
* | Added X-Request-Id tracking and TaggedLogging to easily log that and other ↵ | David Heinemeier Hansson | 2011-10-19 | 1 | -0/+34 | |
| | | | | production concerns | |||||
* | Merge pull request #3151 from zenprogrammer/pluralize_without_count | José Valim | 2011-10-17 | 1 | -4/+8 | |
|\ | | | | | Added include_count parameter to pluralize. | |||||
| * | Modified String#pluralize to take an optional count parameter. | Ryan Oblak | 2011-09-28 | 1 | -4/+8 | |
| | | ||||||
* | | Fixed test_nonexisting_method_with_arguments to revert commit 29a5aea | Yasuo Honda | 2011-10-06 | 1 | -1/+1 | |
| | | ||||||
* | | Revert "don't raise NoMethodError the tried method doesn't exists" | José Valim | 2011-10-06 | 1 | -1/+1 | |
| | | | | | | | | This reverts commit 29a5aeaae976bf8432d57ec996c7c81932a39de6. | |||||
* | | Merge pull request #2801 from jeremyevans/patch-1 | Jeremy Kemper | 2011-10-03 | 1 | -0/+3 | |
|\ \ | | | | | | | Fix obviously breakage of Time.=== for Time subclasses | |||||
| * | | Add tests for fixes to Time.=== | Jeremy Evans | 2011-09-07 | 1 | -0/+3 | |
| | | | ||||||
* | | | Rescuable spelling error | Christian Höltje | 2011-10-03 | 1 | -1/+1 | |
| | | | ||||||
* | | | Improved testing so that it tests the real functionality and not the private ↵ | Philippe Huibonhoa | 2011-10-02 | 1 | -4/+5 | |
| | | | | | | | | | | | | method. | |||||
* | | | Added fix so that file store does not raise an exception when cache dir does ↵ | Philippe Huibonhoa | 2011-09-29 | 1 | -0/+7 | |
| |/ |/| | | | | | not exist yet. This can happen if a delete_matched is called before anything is saved in the cache. | |||||
* | | Ensure that constantize just rescues NameError that applies to the constant ↵ | José Valim | 2011-09-23 | 1 | -0/+6 | |
| | | | | | | | | being currently loaded. | |||||
* | | Added ActiveSupport::Inflector.safe_constantize and String#safe_constantize; ↵ | Ryan Oblak | 2011-09-23 | 3 | -18/+63 | |
| | | | | | | | | refactored common constantize tests into ConstantizeTestCases | |||||
* | | Merge pull request #3096 from phuibonhoa/master | Santiago Pastorino | 2011-09-22 | 1 | -1/+10 | |
|\ \ | | | | | | | FileStore key_file_path does not properly limit filenames to 255 characters | |||||
| * | | Updated existing test that fails with this fix ↵ | Philippe Huibonhoa | 2011-09-21 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | (8d63678d1406c5518d437709af0fde717c0248d7). Before the fix the test was giving a false positive, because file_store.key_file_path would return an empty filename (i.e. test/tmp_cache/4D0/F4D rather than test/tmp_cache/4D0/F4D/xxxx…). Even though the fix referenced above divides the filename into directories to prevent it from being too long, it seems that 1000 characters will always raise an error, so reducing the key size to 900). | |||||
| * | | Fixed issue in file store where it could create a filename that was too long ↵ | Philippe Huibonhoa | 2011-09-21 | 1 | -0/+9 | |
| | | | | | | | | | | | | for the file system. (https://github.com/rails/rails/issues/3072) | |||||
* | | | Fix test to reflect use of to_param for keys | brainopia | 2011-09-18 | 1 | -8/+8 | |
|/ / | ||||||
* | | Fix Hash#to_query edge case with html_safe string on 1.8 ruby | brainopia | 2011-09-16 | 1 | -0/+9 | |
| | | ||||||
* | | Test deprecation warning when not using an options hash as second parameter. | Willem van Bergen | 2011-09-15 | 2 | -2/+14 | |
| | | ||||||
* | | Use an options hash to specify digest/cipher algorithm and a serializer for ↵ | Willem van Bergen | 2011-09-15 | 2 | -6/+6 | |
| | | | | | | | | MessageVerifier and MessageEncryptor. |