aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge pull request #10662 from ↵Carlos Antonio da Silva2014-05-111-9/+9
|\ \ \ | | | | | | | | | | | | | | | | take/change-test-name-for-ActiveRecord--Validations#valid- Refactor AR's validations_test.rb
| * | | Refactor AR's validations_test.rbTakehiro Adachi2013-12-031-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `:context` switch feature is implemented in ActiveRecord::Validations#valid? method, so we should rename the test names, and execute `valid?` in the test. Change test name in AR's validations_test.rb This test is testing save method's code
* | | | Merge pull request #15059 from marcioj/patch-1Robin Dupret2014-05-111-2/+2
|\ \ \ \ | | | | | | | | | | Replace :base with :in [ci skip]
| * | | | Replace :base with :in [ci skip]Marcio Junior2014-05-111-2/+2
|/ / / /
* | | | Tidy up implementation of #15010Andrew White2014-05-112-11/+7
| | | |
* | | | Make TimeZone#parse behave more like Time#parse.Ulysse Carion2014-05-113-1/+25
| | | | | | | | | | | | | | | | | | | | Namely, if the mday is omitted but any other upper components are, then instead of supplying the mday from the current time, it defaults to 1.
* | | | docs, make `ActionNotFound` public API. [ci skip]Yves Senn2014-05-111-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a follow up to #15058. This exception is regularly raised during development. This means it will enter the user realm. We should provide an API page to show that this exception is public API. /cc @schneems
* | | | Merge pull request #15058 from Gaurav2728/gaurav-doc_messageRichard Schneeman2014-05-111-3/+4
|\ \ \ \ | | | | | | | | | | adding complete message in documentation [ci skip]
| * | | | adding complete message in documentation [ci skip]Gaurav Sharma2014-05-111-3/+4
|/ / / /
* | | | Merge branch 'master' of github.com:rails/railsAndrew White2014-05-111-0/+59
|\ \ \ \
| * | | | pg guide, PostgreSQL legacy table view example. [ci skip]Yves Senn2014-05-111-0/+59
| | | | |
* | | | | Always use the provided port for protocol relative urlsAndrew White2014-05-113-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | There may be situations where you need to tunnel SSL connections over port 80 so we shouldn't remove it if it has been explicitly provided.
* | | | | ActionDispatch::Http::URL.normalize_port should not strip port for protocol ↵Guilherme Cavalcanti2014-05-112-2/+12
|/ / / / | | | | | | | | | | | | relative URL.
* | | | docs, fix typo in docs added by ed56e596a04673. [ci skip]Yves Senn2014-05-101-2/+2
| | | |
* | | | Merge pull request #15056 from josemarluedke/single-quotes-on-plugin-gemfileRafael Mendonça França2014-05-102-7/+7
|\ \ \ \ | | | | | | | | | | Change Gemfile’s double quotation marks in plugin generator
| * | | | Change Gemfile’s double quotes in plugin generatorJosemar Luedke2014-05-102-7/+7
|/ / / / | | | | | | | | | | | | | | | | Since the generated application’s Gemfile is using single quotation marks, it seems fair to use it on plugin’s Gemfile as well.
* | | | Merge pull request #15054 from mechanicles/documentation-consistencyRafael Mendonça França2014-05-101-3/+3
|\ \ \ \ | | | | | | | | | | Keep method's description/documentation consistent [ci skip]
| * | | | Keep method's description/documentation consistent [ci skip]Santosh Wadghule2014-05-111-3/+3
|/ / / /
* | | | feature detect for FNM_EXTGLOB for older Ruby. Fixes #15053Aaron Patterson2014-05-101-5/+21
| | | |
* | | | minor change in AR readme [ci skip]Vijay Dev2014-05-101-1/+1
| | | |
* | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2014-05-106-11/+49
|\ \ \ \
| * | | | copy edits [ci skip]Vijay Dev2014-05-102-14/+13
| | | | |
| * | | | [ci skip] add example to AR#assign_attributesschneems2014-05-091-0/+10
| | | | |
| * | | | [ci skip] doc ActiveSupport::TimeWithZone#to_sschneems2014-05-091-2/+5
| | | | | | | | | | | | | | | Current docs are wrong. Does not accept strftime inputs.
| * | | | [ci skip] doc Http::Headers methodsschneems2014-05-091-1/+17
| | | | |
| * | | | fix formatting errors [ci skip]Vijay Dev2014-05-091-6/+6
| | | | |
| * | | | copy edits [ci skip]Vijay Dev2014-05-082-3/+2
| | | | |
| * | | | [ci skip] document ActionDispatch::HTTP::Headersschneems2014-05-071-0/+6
| | | | |
| * | | | [ci skip] document type_cast_for_writeschneems2014-05-072-0/+5
| | | | |
| * | | | Fix Formatting in AR ReadmeGaurish Sharma2014-05-041-2/+2
| | | | | | | | | | | | | | | Don't make free form text appear in code blocks [ci skip]
* | | | | deprecate, join, preload, eager load of instance dependent associations.Yves Senn2014-05-106-4/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #15024. These operations happen before instances are created. The current behavior is misleading and can result in broken behavior.
* | | | | Merge pull request #15049 from arthurnn/fix_ciRafael Mendonça França2014-05-091-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Fix broken proc syntax for 1.9.3
| * | | | | Fix broken proc syntax for 1.9.3Arthur Neves2014-05-091-1/+1
|/ / / / /
* | | | | skip dealing with params if none are providedAaron Patterson2014-05-091-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This lets us avoid 1. A slow call to Hash#slice 2. An is_a? test 3. Extra hash allocations (from slice) 4. String allocations etc.
* | | | | don't mutate the options hash, so we don't have to dupAaron Patterson2014-05-091-4/+3
| | | | | | | | | | | | | | | | | | | | avoids extra hash allocations on each call
* | | | | use fnmatch to test for case insensitive file systemsAaron Patterson2014-05-091-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this is due to: https://bugs.ruby-lang.org/issues/5994
* | | | | use unless and || since these options are booleanAaron Patterson2014-05-091-1/+1
| | | | |
* | | | | Merge pull request #15045 from xanhast/patch-1Godfrey Chan2014-05-091-2/+4
|\ \ \ \ \ | | | | | | | | | | | | Documentation, add examples of using an Enum scope [ci skip]
| * | | | | Documentation, add examples of using an Enum scopeZac2014-05-091-2/+4
|/ / / / / | | | | | | | | | | When scanning the Enum API page I found it easy to miss that scopes are generated. Adding an example clarifies use and has more... *pop*...
* | | | | test, regression test for has_many with instance dependent scope.Yves Senn2014-05-092-0/+12
| | | | |
* | | | | Merge pull request #15044 from zzak/travis_ruby_too_wonRafael Mendonça França2014-05-091-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Only need MINOR version level to test Ruby 2.1.x on Travis
| * | | | | Only need MINOR version level to test Ruby 2.1.x on TravisZachary Scott2014-05-091-1/+1
|/ / / / / | | | | | | | | | | | | | | | cc http://blog.travis-ci.com/2014-04-28-upcoming-build-environment-updates/
* | | | | Merge pull request #15042 from arthurnn/revert_dirty_transactionsRafael Mendonça França2014-05-093-52/+0
|\ \ \ \ \ | | | | | | | | | | | | Reverts dirty track restore on transactions rollback
| * | | | | Reverts "Fix bugs with changed attributes tracking when transaction gets ↵Arthur Neves2014-05-093-52/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rollback" We are reverting these commits, because there are other caveats related to dirty attributes not being restored when a transaction is rollback. For instance, nested transactions cannot proper restore the dirty attributes state after a rollback. At the moment, the dirty attributes are scoped by the transaction. When we call `.save` on a record, the dirty attributes will be reset even if the transaction gets rollback. [related #13166] [related #15018] [related #15016] [closes #15019] This reverts commits * bab48f0a3d53a08bc23ea0887219e8deb963c3d9 * b0fa7cf3ff8432cc2eef3682b34763b7f8c93cc8. * 73fb39b6faa9de593ae75ad4e3b8e065ea0e53af * 37c238927fbed059de3f26a90d8923fb377568a5. * 8d8d4f1560264cd1c74364d67fa0501f6dd2c4fa Revert "Merge pull request #13166 from bogdan/transaction-magic"
* | | | | | just call the method and assert the return valueAaron Patterson2014-05-091-8/+1
|/ / / / / | | | | | | | | | | | | | | | do not test internals
* | | | | Merge pull request #15041 from arthurnn/update_rubyGuillermo Iguaran2014-05-091-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Use ruby 2.1.2 on travis
| * | | | | Use ruby 2.1.2 on travisArthur Neves2014-05-091-1/+1
|/ / / / /
* | | | | add tests for path based url_for callsAaron Patterson2014-05-081-2/+19
| | | | |
* | | | | don't manipulate the testclass, just add helpers to the instanceAaron Patterson2014-05-081-10/+11
| | | | |
* | | | | add a test for classes with optionsAaron Patterson2014-05-081-0/+7
| | | | |