aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | pushing specialization down to the optimized classAaron Patterson2013-01-301-23/+25
| | | | | |
* | | | | | use polymorphism to remove conditionalAaron Patterson2013-01-301-35/+27
| | | | | |
* | | | | | move conditionals to instanceAaron Patterson2013-01-301-6/+16
| | | | | |
* | | | | | pull stuff out of the caller and hide in the instanceAaron Patterson2013-01-301-4/+3
| | | | | |
* | | | | | moving more stuff on to the instanceAaron Patterson2013-01-301-18/+18
| | | | | |
* | | | | | move optimize_helper? to the helper instanceAaron Patterson2013-01-301-9/+12
| | | | | |
* | | | | | moving more stuff to the instanceAaron Patterson2013-01-301-6/+6
| | | | | |
* | | | | | moving some stuff to the initializerAaron Patterson2013-01-301-14/+38
| | | | | |
* | | | | | moved more evald codeAaron Patterson2013-01-301-8/+7
| | | | | |
* | | | | | factored out some of the dynamic codeAaron Patterson2013-01-301-19/+31
| | | | | |
* | | | | | Merge pull request #9124 from semaperepelitsa/content_tag_for_class2Carlos Antonio da Silva2013-01-303-1/+25
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix content_tag_for with array :class option
| * | | | | | Fix `content_tag_for` with array html option.Semyon Perepelitsa2013-01-313-1/+25
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It would embed array as string instead of joining it like `content_tag` does: content_tag(:td, class: ["foo", "bar"]){} #=> '<td class="foo bar"></td>' Before: content_tag_for(:td, item, class: ["foo", "bar"]){} #=> '<td class="item [&quot;foo&quot;, &quot;bar&quot;]" id="item_1"></td>' After: content_tag_for(:td, item, class: ["foo", "bar"]){} #=> '<td class="item foo bar" id="item_1"></td>'
* | | | | | Merge pull request #9118 from dazuma/remove_adapter_names_from_databases_rakeCarlos Antonio da Silva2013-01-301-9/+3
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Fix to databases.rake so third party adapters can register rake task implementations
| * | | | | Remove pre-emptive adapter name check from databases.rake so third party ↵Daniel Azuma2013-01-291-9/+3
|/ / / / / | | | | | | | | | | | | | | | adapters can register rake tasks
* | | | | reloading type map on extension changingAaron Patterson2013-01-292-2/+15
| | | | |
* | | | | fixing commentAaron Patterson2013-01-291-1/+1
| | | | |
* | | | | Changelog about BestStandardsSupport removalGuillermo Iguaran2013-01-291-5/+6
| | | | |
* | | | | Add 'X-UA-Compatible' => 'chrome=1' to default headersGuillermo Iguaran2013-01-292-2/+5
| | | | |
* | | | | Remove BestStandardsSupport middlewareGuillermo Iguaran2013-01-299-108/+3
| | | | |
* | | | | Fix typoRafael Mendonça França2013-01-291-1/+1
| | | | |
* | | | | Only search for enabled extension if the PostgreSQL version supportsRafael Mendonça França2013-01-291-3/+5
| | | | | | | | | | | | | | | | | | | | extensions
* | | | | 'minitest/parallel_each' might not always be availableAkira Matsuda + Koichi Sasada2013-01-291-1/+4
| | | | |
* | | | | Merge pull request #9104 from bemurphy/remove_bad_idea_parser_testCarlos Antonio da Silva2013-01-291-13/+0
|\ \ \ \ \ | | | | | | | | | | | | Remove yaml Proc param parser test
| * | | | | Remove yaml Proc param parser testBrendon Murphy2013-01-281-13/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I don't believe this test is exercising any explicit params_parser behavior that the other two Proc tests aren't already doing. Given that we now know it's a bad idea to load user input via YAML.load, somebody reading this test might get a dangerous idea about building out a YAML params parser.
* | | | | | Needs to explicitly require 'minitest/parallel_each'Akira Matsuda + Koichi Sasada2013-01-291-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Minitest has changed to require this since https://github.com/seattlerb/minitest/commit/728054b
* | | | | | s/ERb/ERB/Akira Matsuda2013-01-291-1/+1
|/ / / / /
* | | | | Send paths argument as an array rather than converting it internallyCarlos Antonio da Silva2013-01-281-3/+3
| | | | |
* | | | | Refactor to use each_key, remove extra spacesCarlos Antonio da Silva2013-01-287-11/+5
| | | | |
* | | | | add API to pg for enabling / disabling hstoreAaron Patterson2013-01-283-7/+54
| | | | |
* | | | | Merge pull request #9096 from tricknotes/fix-code-commentRafael Mendonça França2013-01-281-4/+4
|\ \ \ \ \ | | | | | | | | | | | | Fix code comment
| * | | | | Fix JavaScript syntax in code comment [ci skip]Ryunosuke SATO2013-01-291-4/+4
| | | | | |
* | | | | | revamps the RDoc of Object#tryXavier Noria2013-01-281-22/+30
| | | | | |
* | | | | | Move AS changelog entry to the top, improve AP changelog a bitCarlos Antonio da Silva2013-01-282-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | no need for \Z, \z is more conciseXavier Noria2013-01-281-1/+1
| | | | | |
* | | | | | Merge pull request #8830 from antoinelyset/masterXavier Noria2013-01-284-5/+13
|\ \ \ \ \ \ | | | | | | | | | | | | | | Improve String#squish whitespaces matching
| * | | | | | Improve String#squish whitespaces matchingAntoine Lyset2013-01-224-5/+13
| | | | | | |
* | | | | | | Merge pull request #9098 from firmhouse/add-head-requestsRafael Mendonça França2013-01-284-5/+12
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Make current_url? work with HEAD requests and remove caching_allowed? since it doesn't do anything
| * | | | | | | Added a Changelog entry for fixing HEAD requests.Michiel Sikkes2013-01-281-0/+5
| | | | | | | |
| * | | | | | | Make current_url? work with a HEAD methodMichiel Sikkes2013-01-282-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActionDispatch::Head was removed in favor of Rack::Head. But Rack::Head does not convert GET requests to HEAD requests so we need to do checking for HEAD requests ourselves.
| * | | | | | | Remove caching_allowed? from ActionController::CachingMichiel Sikkes2013-01-281-4/+0
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Where is this used? No other code references to this method and it isn't being tested anywhere. No tests fail when commented out.
* | | | | | | updates a dynamic method heredoc docXavier Noria2013-01-281-2/+2
| | | | | | |
* | | | | | | explains why the query cache checks arel.lockedXavier Noria2013-01-281-0/+2
| |/ / / / / |/| | | | |
* | | | | | Merge pull request #9032 from firmhouse/head-breaks-csrfSantiago Pastorino2013-01-282-2/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Make HEAD work / convert to GET once more
| * | | | | | Added request.head? to forgery protection codeMichiel Sikkes2013-01-221-2/+2
| | | | | | |
| * | | | | | Added a test that shows that a HEAD request does not normally pass CSRF ↵Michiel Sikkes2013-01-221-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | protection
* | | | | | | Use Encoding::UTF_8 constant :do_not_litter:Akira Matsuda2013-01-2810-15/+15
| | | | | | |
* | | | | | | Use already defined Encoding constants rather than creating one-trip StringsAkira Matsuda2013-01-283-3/+3
| | | | | | |
* | | | | | | add fetch to CookieJarAaron Patterson2013-01-272-0/+34
| | | | | | |
* | | | | | | Prevent Relation#merge from collapsing wheres on the RHSJon Leighton2013-01-279-22/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This caused a bug with the new associations implementation, because now association conditions are represented as Arel nodes internally right up to when the whole thing gets turned to SQL. In Rails 3.2, association conditions get turned to raw SQL early on, which prevents Relation#merge from interfering. The current implementation was buggy when a default_scope existed on the target model, since we would basically end up doing: default_scope.merge(association_scope) If default_scope contained a where(foo: 'a') and association_scope contained a where(foo: 'b').where(foo: 'c') then the merger would see that the same column is representated on both sides of the merge and collapse the wheres to all but the last: where(foo: 'c') Now, the RHS of the merge is left alone. Fixes #8990
* | | | | | | Fix typo: adaptors => adapters [ci skip]Carlos Antonio da Silva2013-01-271-1/+1
| | | | | | |