aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge pull request #14329 from pch/digestor-lookup-fixRafael Mendonça França2014-03-1412-33/+130
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | Ensure LookupContext in Digestor selects correct variant
| * | | | Introduce #with_formats_and_variants to prevent problems with mutating ↵Łukasz Strzałkowski2014-03-144-5/+31
| | | | | | | | | | | | | | | | | | | | finder object
| * | | | Add mocked disable_cache for FixtureFinderŁukasz Strzałkowski2014-03-141-0/+4
| | | | |
| * | | | Set format in finderŁukasz Strzałkowski2014-03-141-3/+8
| | | | |
| * | | | Rename _setup_options to _options_for_digestŁukasz Strzałkowski2014-03-141-3/+3
| | | | |
| * | | | Disable LookupContext's cache when looking for templateŁukasz Strzałkowski2014-03-141-2/+6
| | | | |
| * | | | Don't pass hash as keys to #find methodŁukasz Strzałkowski2014-03-141-1/+1
| | | | |
| * | | | Add variants to Template classŁukasz Strzałkowski2014-03-143-12/+20
| | | | |
| * | | | Don't create addition vars, use options[] directlyŁukasz Strzałkowski2014-03-131-7/+2
| | | | |
| * | | | Don't pass variant in params, it's ignoredŁukasz Strzałkowski2014-03-131-1/+1
| | | | | | | | | | | | | | | | | | | | We're setting variant above, in request object directly
| * | | | Ensure LookupContext in Digestor selects correct variantPiotr Chmolowski2014-03-098-8/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to: #14242 #14243 14293 Variants passed to LookupContext#find() seem to be ignored, so I've used the setter instead: `finder.variants = [ variant ]`. I've also added some more test cases for variants. Hopefully this time passing tests will mean it actually works.
* | | | | Take in consideration guides CHANGELOGRafael Mendonça França2014-03-141-2/+2
| | | | |
* | | | | Match the whole stringRafael Mendonça França2014-03-141-1/+1
| |/ / / |/| | |
* | | | Force sass-rails ~> 4.0.2 to avoid sprockets compatibility errorRafael Mendonça França2014-03-141-1/+1
| | | | | | | | | | | | | | | | See https://github.com/rails/sass-rails/issues/191 for more information
* | | | Avoid duplicated conditionalsCarlos Antonio da Silva2014-03-141-2/+1
| |/ / |/| |
* | | Merge pull request #14378 from afshinator/masterGodfrey Chan2014-03-131-3/+3
|\ \ \ | | | | | | | | Fix probs in sections 5.6 and 5.9; [ci skip]
| * | | Fix probs in sections 5.6 and 5.9; [ci skip]Afshin Mokhtari2014-03-131-3/+3
| | | |
* | | | No need to binds be optionalRafael Mendonça França2014-03-131-2/+2
|/ / /
* | | Merge pull request #14372 from arthurnn/fix_query_cache_bindsRafael Mendonça França2014-03-134-12/+24
|\ \ \ | | | | | | | | Make select_all on query cache accept a Relation without binds.
| * | | Make select_all on query cache accept a Relation without binds.Arthur Neves2014-03-134-12/+24
| | | | | | | | | | | | | | | | | | | | [fixes #14361] [related #13886]
* | | | Merge pull request #14376 from laurocaetano/test-deprecation-warning-for-findGodfrey Chan2014-03-131-0/+6
|\ \ \ \ | | | | | | | | | | Add test for deprecation warning for passing an AR object to `find`.
| * | | | Add test for deprecation warning for passing an AR object to `find`.Lauro Caetano2014-03-131-0/+6
|/ / / /
* | | | Merge pull request #14375 from laurocaetano/test-deprecation-warning-for-existsAaron Patterson2014-03-131-0/+6
|\ \ \ \ | | | | | | | | | | Add test for deprecation warning for passing an AR object to `exists?`.
| * | | | Test deprecation warning for passing an ActiveRecord object toLauro Caetano2014-03-131-0/+6
| |/ / / | | | | | | | | | | | | `exists?`
* / / / require actions rather than create_fileAaron Patterson2014-03-131-1/+1
|/ / / | | | | | | | | | | | | | | | | | | thor's create_file seems to have a circular dependency on itself when used with our constant loading stuff. fixes #14319
* | | passing an instance of an AR object to `find` is deprecatedAaron Patterson2014-03-135-6/+13
| | | | | | | | | | | | please pass the id of the AR object by calling `.id` on the model first.
* | | passing an ActiveRecord object to `exists?` is deprecated.Aaron Patterson2014-03-135-4/+14
| | | | | | | | | | | | | | | Pass the id of the object to the method by calling `.id` on the AR object.
* | | make tests pass on Ruby 2.2Aaron Patterson2014-03-132-5/+5
| | | | | | | | | | | | | | | | | | Apparently we've been using a buggy feature for the past 6 years: https://bugs.ruby-lang.org/issues/9593
* | | Use Sqlite3 adapter in examplesJulian Simioni2014-03-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two bits of example code use sqlite as an adapter, which doesn't exist. Using the code verbatim will raise a LoadError exception: ActiveRecord::Base.establish_connection( "adapter" => "sqlite", "database" => "db.sqlite" ) # => LoadError: Could not load 'active_record/connection_adapters/sqlite_adapter'... Considering this is code a lot of people new to Rails might be running, it's especially confusing. Closes #14367 [ci skip]
* | | use the body proxy to freeze headersAaron Patterson2014-03-125-12/+57
| | | | | | | | | | | | | | | | | | avoid freezing the headers until the web server has actually read data from the body proxy. Once the webserver has read data, then we should throw an error if someone tries to set a header
* | | just ask the response for the commit status, we do not need to ask the jarAaron Patterson2014-03-121-1/+1
| | |
* | | only write the jar if the response isn't committedAaron Patterson2014-03-126-10/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when streaming responses, we need to make sure the cookie jar is written to the headers before returning up the stack. This commit introduces a new method on the response object that writes the cookie jar to the headers as the response is committed. The middleware and test framework will not write the cookie headers if the response has already been committed. fixes #14352
* | | Merge pull request #14366 from sukolsak/grammarLauro Caetano2014-03-121-1/+1
|\ \ \ | | | | | | | | Fix a grammatical error in the i18n guide
| * | | Fix a grammatical error in the i18n guide [ci skip]sukolsak2014-03-121-1/+1
| | | |
* | | | Merge pull request #14364 from why-el/better-docs-for-update-attributeArthur Nogueira Neves2014-03-121-0/+2
|\ \ \ \ | |/ / / |/| | | Enhance docs for update_attribute [skip ci]
| * | | Enhance docs for update_attribute [ci-skip]Mohamed Wael Khobalatte2014-03-121-0/+2
| | | |
* | | | use method_defined? to check whether or not a method is definedAaron Patterson2014-03-121-1/+1
| | | |
* | | | Merge pull request #14363 from arthurnn/usec_tests_mysql56Rafael Mendonça França2014-03-122-3/+10
|\ \ \ \ | |/ / / |/| | | Add more tests for mysql 5.6 microseconds + Fix tests
| * | | Change usec to 0 on tests that compare secondsArthur Neves2014-03-121-3/+3
| | | | | | | | | | | | | | | | | | | | Avoid rounding problems with `.usec` method rounding the seconds when the field doesn't persist the `.usec` piece.
| * | | Unit test for mysql quote time usecArthur Neves2014-03-121-0/+7
|/ / /
* | | Merge pull request #13040 from kamipo/case_sensitive_comparisonRafael Mendonça França2014-03-126-6/+62
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Only use BINARY for mysql case sensitive uniqueness check when column has a case insensitive collation. Conflicts: activerecord/CHANGELOG.md
| * | | Only use BINARY for mysql case sensitive uniqueness check when column has a ↵Ryuta Kamizono2013-11-266-6/+62
| | | | | | | | | | | | | | | | case insensitive collation.
* | | | Merge pull request #14359 from arthurnn/mysql2_56_franc_secRafael Mendonça França2014-03-125-2/+23
|\ \ \ \ | | | | | | | | | | MySQL 5.6 Fractional Seconds
| * | | | Changelog entry for mysql56 microsecondsArthur Neves2014-03-121-0/+4
| | | | |
| * | | | Test microsecond on mysql 5.6Arthur Neves2014-03-123-2/+11
| | | | |
| * | | | MySQL 5.6 and later supports microsecond precision in datetime.Tatsuhiko Miyagawa2014-03-121-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | You might want to branch it to include this only for 5.6, but passing these values to < 5.6 doesn't cause issues either.
* | | | | Merge pull request #14291 from laurocaetano/doc-for-original-fullpathYves Senn2014-03-121-0/+7
|\ \ \ \ \ | |/ / / / |/| | | | Add documentation for original_fullpath. [ci skip]
| * | | | [ci skip] Add documentation for original_fullpath.Lauro Caetano2014-03-111-0/+7
| | | | |
* | | | | Merge pull request #14353 from arthurnn/remove_small_mockRafael Mendonça França2014-03-111-3/+1
|\ \ \ \ \ | | | | | | | | | | | | Remove mocking on save, when not necessary
| * | | | | Remove mocking on save, when not necessaryArthur Neves2014-03-111-3/+1
|/ / / / /