aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | implements new option :month_format_string for date select helpers [Closes ↵Xavier Noria2014-02-153-9/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | #13618]
* | | | | | | add factory methods for empty alias trackersAaron Patterson2014-02-143-23/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we know the alias tracker is empty, we can create one that doesn't use a hash with default block for counting. ActiveRecord::Base.establish_connection(adapter: 'sqlite3', database: ':memory:') ActiveRecord::Schema.define do create_table :posts, force: true do |t| t.integer :comments_count end create_table :comments, force: true do |t| t.integer :post_id end end class Post < ActiveRecord::Base; has_many :comments; end class Comment < ActiveRecord::Base; belongs_to :post, counter_cache: true; end 10.times { Comment.create!(post: Post.create!) } record = Post.first association_name = :comments Benchmark.ips do |x| reflection = record.class.reflect_on_association(association_name) association = reflection.association_class.new(record, reflection) x.report('assoc') do reflection.association_class.new(record, reflection) end x.report('reader') do association.reader;nil end x.report('combined') do reflection.association_class.new(record, reflection).reader;nil end end [aaron@higgins rails (tracker)]$ TEST=ips bundle exec ruby ../1bb5456b5e035343df9d/gistfile1.rb -- create_table(:posts, {:force=>true}) -> 0.0062s -- create_table(:comments, {:force=>true}) -> 0.0003s Calculating ------------------------------------- assoc 833 i/100ms reader 28703 i/100ms combined 839 i/100ms ------------------------------------------------- assoc 9010.3 (±3.8%) i/s - 44982 in 5.000022s reader 3214523.4 (±5.5%) i/s - 16016274 in 5.001136s combined 8841.0 (±5.8%) i/s - 44467 in 5.049269s [aaron@higgins rails (tracker)]$ TEST=ips bundle exec ruby ../1bb5456b5e035343df9d/gistfile1.rb -- create_table(:posts, {:force=>true}) -> 0.0060s -- create_table(:comments, {:force=>true}) -> 0.0003s Calculating ------------------------------------- assoc 888 i/100ms reader 29217 i/100ms combined 900 i/100ms ------------------------------------------------- assoc 9674.3 (±3.3%) i/s - 48840 in 5.054022s reader 2988474.8 (±6.9%) i/s - 14842236 in 4.998230s combined 9674.0 (±3.1%) i/s - 48600 in 5.028694s
* | | | | | | guarantee a list in the alias tracker so we can remove a conditionalAaron Patterson2014-02-143-6/+12
| | | | | | |
* | | | | | | stop exposing table_joinsAaron Patterson2014-02-141-4/+3
| | | | | | |
* | | | | | | make most parameters to the AliasTracker requiredAaron Patterson2014-02-142-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This helps with our sanity. The class is internal, we can refactor to a "nice" API later.
* | | | | | | make a singleton for AssociationScopeAaron Patterson2014-02-143-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AssociationScope no longer maintains state, so we're safe to keep a singleton and save on GC time
* | | | | | | pass the association and connection to the scope methodAaron Patterson2014-02-143-11/+7
| | | | | | |
* | | | | | | pass the tracker down the stack and construct it in the scope methodAaron Patterson2014-02-141-18/+18
| | | | | | |
* | | | | | | clean up add_constraints signatureAaron Patterson2014-02-141-8/+9
| | | | | | |
* | | | | | | remove the reflection delegateAaron Patterson2014-02-141-15/+14
| | | | | | |
* | | | | | | remove klass delegatorAaron Patterson2014-02-141-10/+11
| | | | | | |
* | | | | | | remove railties changes. fixes #14054Aaron Patterson2014-02-146-191/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Squashed commit of the following: commit 96991e8e919edfb20cc4120bca4e36ed51175d57 Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Fri Feb 14 11:29:24 2014 -0800 Revert "gems can be added or skipped from the template" This reverts commit 8beb42cfbc41753ae4dbb91e16abcd1fb7d00356. Conflicts: railties/lib/rails/generators/rails/app/app_generator.rb railties/test/generators/app_generator_test.rb commit 35599c0e657245ef14ac0f28c9189ad16acf40e6 Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Fri Feb 14 11:26:53 2014 -0800 Revert "oops, template replay needs to happen after bundle. :orz:" This reverts commit 9104702be61253f9448ca070a22fc86bb4299555. Conflicts: railties/lib/rails/generators/rails/app/app_generator.rb commit f519c3902c313db8e906a49251c91643b8e6499e Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Fri Feb 14 11:25:51 2014 -0800 Revert "only ask for these ivars if the target responds to them" This reverts commit 656d412546cd97d5660c634c2a41c799d3f9e211. commit aa524a9428e3e4c45fe221f10a66a08efb827ab5 Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Fri Feb 14 11:25:39 2014 -0800 Revert "refactor generator tests to use block form of Tempfile" This reverts commit 65251820ef0ab7f3cffb38130de3dd41af8d72be. commit 7d3740549fa4dfa62e3761f8d4bc6d6d441256e7 Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Fri Feb 14 11:25:25 2014 -0800 Revert "add a more restricted codepath for templates fixes #13390" This reverts commit 2875b4a66e38e4333da887a4afbed33358999298. commit 525df0af1001918986cdfce59539fd2d52c4f32c Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Fri Feb 14 11:25:11 2014 -0800 Revert "add a send so `apply` can be called. Fixes #13510" This reverts commit c5034d60dba0cd31a6a8c612ee35d63b8127793a.
* | | | | | | remove chain delegateAaron Patterson2014-02-141-5/+6
| | | | | | |
* | | | | | | remove scope_chain delegateAaron Patterson2014-02-141-3/+3
| | | | | | |
* | | | | | | Merge pull request #14061 from davejachimiak/add_verb_in_guidesRafael Mendonça França2014-02-141-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add verb to sanitization note
| * | | | | | | Add verb to sanitization noteDave Jachimiak2014-02-141-1/+1
|/ / / / / / /
* | | | | | | Merge pull request #14060 from kassio/masterRafael Mendonça França2014-02-143-6/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix path shown in mailer's templates
| * | | | | | | fix path shown in mailer's templatesKassio Borges2014-02-143-6/+6
|/ / / / / / /
* | | | | | | Merge pull request #14048 from pcreux/better-error-message-db-migrate-downCarlos Antonio da Silva2014-02-141-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add hint to error message of task db:migrate:down
| * | | | | | | Add hint to error message of task db:migrate:downPhilippe Creux2014-02-131-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #14057 from rajcybage/Ar_remove_unnecessary_fixturesCarlos Antonio da Silva2014-02-141-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove unused fixtures from sqlite3 test case
| * | | | | | | | remove unused fixtures from sqlite3 test casesRajarshi Das2014-02-141-1/+1
| | | | | | | | |
* | | | | | | | | Merge pull request #14059 from tbpgr/guides_active_support_core_extCarlos Antonio da Silva2014-02-141-0/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | [Guides] fix guide active_support_core_extensions. add Note to String#indent [ci skip]
| * | | | | | | | | fix guide active_support_core_extensions. add Note to String#indent [ci skip]tbpgr2014-02-141-0/+2
| |/ / / / / / / /
* | | | | | | | | Merge pull request #14058 from lantoli/masterRafael Mendonça França2014-02-141-1/+1
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | updated Travis build status image url
| * | | | | | | | updated Travis build status image urlLeo Antoli2014-02-141-1/+1
|/ / / / / / / /
* | | | | | | | remove more delegate methodsAaron Patterson2014-02-131-7/+9
| | | | | | | |
* | | | | | | | :scissors: whitespaceAaron Patterson2014-02-131-1/+0
| | | | | | | |
* | | | | | | | rm delegate methods that are not actually usedAaron Patterson2014-02-131-2/+2
| | | | | | | |
* | | | | | | | JoinHelper is never reused, so there is no need to separateAaron Patterson2014-02-133-42/+25
| | | | | | | |
* | | | | | | | No variant should also be picked up by variant.any if variant.none is not ↵David Heinemeier Hansson2014-02-132-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | defined (just like any other variant)
* | | | | | | | speed up the collection proxy reader method, but slow down the constructorAaron Patterson2014-02-131-1/+5
| | | | | | | |
* | | | | | | | Merge pull request #14047 from strzalek/variants-array-checkDavid Heinemeier Hansson2014-02-132-2/+10
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Check if variant array contains only symbols
| * | | | | | | | Check if variant array contains only symbolsŁukasz Strzałkowski2014-02-132-2/+10
| | | | | | | | |
* | | | | | | | | Merge pull request #14034 from ↵Yves Senn2014-02-134-4/+49
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | hdabrows/drop-correct-index-when-reverting-migration Drop the correct index after reverting a migration
| * | | | | | | | | Drop the correct index after reverting a migrationHubert Dąbrowski2014-02-134-4/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously when reverting a migration which added a named index it would instead drop a corresponding index with matching columns but without a name.
* | | | | | | | | | Merge pull request #14045 from arthurnn/kill_fibersGuillermo Iguaran2014-02-131-2/+4
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | | Dont use Enumarator on join_association
| * | | | | | | | | Dont use Enumarator on join_associationArthur Neves2014-02-131-2/+4
| | |_|_|/ / / / / | |/| | | | | | |
* | | | | | | | | Merge pull request #14043 from strzalek/variants-negotiationDavid Heinemeier Hansson2014-02-135-12/+51
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Variant negotiation
| * | | | | | | | | Variant negotiationLukasz Strzalkowski2014-02-135-12/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow setting `request.variant` as an array - an order in which they will be rendered. For example: request.variant = [:tablet, :phone] respond_to do |format| format.html.none format.html.phone # this gets rendered end
* | | | | | | | | | tests are responsible to clean up afterwards.Yves Senn2014-02-131-82/+114
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | remove created state after test execution, not before the next test. This prevents the leak of the `ex` table outside of a single test.
* | | | | | | | | Merge pull request #13945 from rails/json_cookie_serializer_improvementsGuillermo Iguaran2014-02-1316-123/+316
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Cookies serializer improvements
| * | | | | | | | | rm warning about variable shadowingGodfrey Chan2014-02-111-2/+2
| | | | | | | | | |
| * | | | | | | | | Migrate hash-based cookie values correctlyGodfrey Chan2014-02-111-2/+4
| | | | | | | | | |
| * | | | | | | | | Updated CHANGELOG, docs, guides and release notes.Godfrey Chan2014-02-115-26/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also added a `cookies_serializer.rb` initializer to the app template.
| * | | | | | | | | Added changelog entry for Flash changes [ci skip]Godfrey Chan2014-02-111-0/+4
| | | | | | | | | |
| * | | | | | | | | Missed FlashHash#replaceGodfrey Chan2014-02-111-1/+1
| | | | | | | | | |
| * | | | | | | | | Fixed broken flash testsGodfrey Chan2014-02-112-5/+5
| | | | | | | | | |
| * | | | | | | | | Fix AppGeneratorTest: serializer option was removed from session_storeGuillermo Iguaran2014-02-111-1/+1
| | | | | | | | | |
| * | | | | | | | | Remove serializer option from session_store.rb templateGuillermo Iguaran2014-02-111-1/+1
| | | | | | | | | |