aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | Merge pull request #13611 from robinboening/plugin_generator_testsYves Senn2014-01-061-0/+6
| | |\ \ | | | | | | | | | | Add missing tests for invalid names in `rails plugin new` generator
| | | * | Add missing tests for invalid names in `rails plugin new`Robin Böning2014-01-061-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | * Test for: Invalid plugin name, because of reserved rails word. * Test for: Invalid plugin name because of constant name that is already in use.
| | * | | Fix typo in AMo docs [ci skip]Carlos Antonio da Silva2014-01-061-1/+1
| | | | |
| | * | | quick formatting pass through CHANGELOGS. [ci skip].Yves Senn2014-01-064-17/+24
| | | | |
| | * | | update 4.1 release notes with deprecation commit link. [ci skip]Yves Senn2014-01-061-1/+2
| | | | |
| | * | | Merge pull request #13607 from huoxito/actionview-versionGuillermo Iguaran2014-01-051-0/+1
| | |\ \ \ | | | | | | | | | | | | Require actionview/version
| | | * | | Require actionview/versionWashington Luiz2014-01-061-0/+1
| | |/ / / | | | | | | | | | | | | | | | just like all the other modules do require their version file
| | * | | Merge pull request #13604 from randomecho/docs-getting-startedRafael Mendonça França2014-01-051-3/+3
| | |\ \ \ | | | |_|/ | | |/| | Ruby is a proper noun here, not part of code fragment [ci skip]
| | | * | Ruby is a proper noun here, not part of code fragment [ci skip]Soon Van2014-01-051-3/+3
| | |/ / | | | | | | | | | | | | | | | | Last line of note also relates heavily to first part and parenthesis broke reading flow.
| | * | Allow an absolute controller path inside a module scopeAndrew White2014-01-054-2/+24
| | | | | | | | | | | | | | | | Fixes #12777
| | * | Add CHANGELOG entry for #10634 [ci skip]Andrew White2014-01-051-0/+17
| | | |
| | * | Merge pull request #10634 from teleological/time_advance_gregorianAndrew White2014-01-052-4/+13
| | |\ \ | | | | | | | | | | Maintain proleptic gregorian in Time#advance
| | | * | Maintain proleptic gregorian in Time#advanceRiley Lynch2013-05-152-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Time#advance uses Time#to_date and Date#advance to calculate a new date. The Date object returned by Time#to_date is constructed with the assumption that the Time object represents a proleptic gregorian date, but it is configured to observe the default julian calendar reform date (2299161j) for purposes of calculating month, date and year: Time.new(1582, 10, 4).to_date.to_s # => "1582-09-24" Time.new(1582, 10, 4).to_date.gregorian.to_s # => "1582-10-04" This patch ensures that when the intermediate Date object is advanced to yield a new Date object, that the Time object for return is contructed with a proleptic gregorian month, date and year.
| | * | | Merge pull request #13598 from zirni/remove_ar_warningsRafael Mendonça França2014-01-052-2/+2
| | |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Remove method redefined warnings for test suite Closes #13526
| | | * | | Remove method redefined warnings for test suiteMatthias Zirnstein2014-01-052-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | has_many definitions with "name" as singular and as plural e.g. has_many :welcome_posts_with_comment has_many :welcome_posts_with_comments Ruby mentions it with: lib/active_record/associations/builder/collection_association.rb:65: warning: method redefined; discarding old welcome_posts_with_comment_ids lib/active_record/associations/builder/collection_association.rb:65: warning: previous definition of welcome_posts_with_comment_ids was here lib/active_record/associations/builder/collection_association.rb:75: warning: method redefined; discarding old welcome_posts_with_comment_ids= lib/active_record/associations/builder/collection_association.rb:75: warning: previous definition of welcome_posts_with_comment_ids= was here
| | * | | | Switched to use `display:none` in extra_tags_for_form method.Gaelian Ditchburn2014-01-056-8/+14
| | | |_|/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The use of `display:inline` with the content_tag call in the extra_tags_for_form method potentially causes display issues with some browsers, namely Internet Explorer. IE's behaviour of not collapsing the line height on divs with ostensibly no content means that the automatically added div containing the hidden authenticity_token, utf8 and _method form input tags may interfere with other visible form elements in certain circumstances. The use of `display:none` rather than `display:inline` fixes this problem. Fixes #6403
| | * | | Unique the segment keys array for non-optimized url helpersAndrew White2014-01-053-1/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Rails 3.2 you only needed pass an argument for dynamic segment once so unique the segment keys array to match the number of args. Since the number of args is less than required parts the non-optimized code path is selected. This means to benefit from optimized url generation the arg needs to be specified as many times as it appears in the path. Fixes #12808
| | * | | Show full route constraints in error messageAndrew White2014-01-054-4/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When an optimized helper fails to generate, show the full route constraints in the error message. Previously it would only show the contraints that were required as part of the path. Fixes #13592
| | * | | Simplify arg parameterizationAndrew White2014-01-051-6/+1
| | | | |
| | * | | Use a custom route vistor for optimized route generationAndrew White2014-01-054-31/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using a Regexp to replace dynamic segments in a path string is fraught with difficulty and can lead to odd edge cases like #13349. Since we already have a parsed representation of the path it makes sense to use that to generate an array of segments that can be used to build an optimized route's path quickly. Tests on a simple route (e.g. /posts/:id) show a speedup of 35%: https://gist.github.com/pixeltrix/8261932 Calculating ------------------------------------- Current Helper: 5274 i/100ms New Helper: 8050 i/100ms ------------------------------------------------- Current Helper: 79263.6 (±3.7%) i/s - 395550 in 4.997252s New Helper: 153464.5 (±4.9%) i/s - 772800 in 5.047834s Tests on a more complex route show even an greater performance boost: https://gist.github.com/pixeltrix/8261957 Calculating ------------------------------------- Current Helper: 2367 i/100ms New Helper: 5382 i/100ms ------------------------------------------------- Current Helper: 29506.0 (±3.2%) i/s - 149121 in 5.059294s New Helper: 78815.5 (±4.1%) i/s - 398268 in 5.062161s It also has the added benefit of fixing the edge cases described above. Fixes #13349
| | * | | Merge pull request #13590 from kuldeepaggarwal/fix-date-helper-docRafael Mendonça França2014-01-041-1/+1
| | |\ \ \ | | | |/ / | | |/| | provide correct example of `datetime_select` helper [ci skip]
| | | * | provide correct example of `datetime_select` helper [ci skip]Kuldeep Aggarwal2014-01-041-1/+1
| | | | |
| | * | | Add preview_path to autoload_paths in after_initializeAndrew White2014-01-043-7/+55
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only config.autoload_paths is frozen, so add the preview_path to ActiveSupport::Dependencies.autoload_paths directly in an after_initialize block. Also protect against a blank preview_path being added to autoload_paths which can cause a serious slowdown as Dir[] tries to load all *_preview.rb files under / Fixes #13372
| | * | Merge pull request #13587 from kud86/change_truncate_examplesRobin Dupret2014-01-041-0/+3
| | |\ \ | | | | | | | | | | Fixed truncate documentation. [ci skip]
| | | * | Fixed documentation. [ci skip]Konstantin Wlasow2014-01-041-0/+3
| | |/ /
| | * | Fix mergefail in changelogJon Leighton2014-01-041-2/+0
| | | | | | | | | | | | | | | | The line was duplicated
| | * | Add deprecation I accidentally missedJon Leighton2014-01-041-1/+1
| | | |
| | * | Merge pull request #13584 from tjschuck/cannot_cannot_be_can_notXavier Noria2014-01-0314-71/+71
| | |\ \ | | | | | | | | | | Change all "can not"s to the correct "cannot"
| | | * | Change all "can not"s to the correct "cannot".T.J. Schuck2014-01-0314-71/+71
| | | | |
| | * | | Merge pull request #13583 from arthurnn/mem_cache_store_fixGuillermo Iguaran2014-01-033-10/+12
| | |\ \ \ | | | | | | | | | | | | mem_cache_store requires dalli, so only accept dalli/client
| | | * | | Raise if MemCacheStore doenst receive a Dalli objArthur Neves2014-01-033-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | :mem_cache_store should receive a list of hosts or a dalli client, otherwise raise it. Also adding a changelog.
| | | * | | mem_cache_store requires dalli, so only accept dalli/clientArthur Neves2014-01-032-14/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | :mem_cache_store require dalli, rescue Dalli exceptions, and follow Dalli API. Memcached gem, for instance, doesnt work anymore, as the API are different. As we already require one client, we should make sure that client works, and not accept others, and if someone wants to use another memcache client they can write their own store adapter.
| | * | | | Minor typo fix [ci skip]Robin Dupret2014-01-031-1/+1
| | | |/ / | | |/| |
| | * | | Merge pull request #10134 from ↵Rafael Mendonça França2014-01-033-6/+10
| | |\ \ \ | | | |/ / | | |/| | | | | | | | | | | | derikson/collection_proxy_select_with_multiple_args Change CollectionProxy#select to take the same arguments as ActiveRecord::select
| | | * | Changed ActiveRecord::Associations::CollectionProxy#select to take multiple ↵Dan Erikson2013-04-083-6/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | arguments. This makes the arguments the same as ActiveRecord::QueryMethods::select.
| | * | | No need to use fixed size font [ci skip]Rafael Mendonça França2014-01-031-2/+2
| | | | |
| | * | | Merge pull request #13579 from prathamesh-sonpatki/issue-13552Rafael Mendonça França2014-01-031-2/+6
| | |\ \ \ | | | | | | | | | | | | | | | | | | Fix documentation for end_year option of date_helper [ci skip]
| | | * | | Fix documentation for end_year option of date_helper [ci skip]Prathamesh Sonpatki2014-01-031-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - While editing an existing record, end_year is equal to current selected year plus 5 by default. - While editing an existing record, start_year is equal to current selected year value minus 5 by default. - Fixes #13552 Acked-by: Prathamesh Sonpatki <csonpatki@gmail.com> Acked-by: Prathamesh Sonpatki <csonpatki@gmail.com>
| | * | | | Whitespace [ci skip]Rafael Mendonça França2014-01-031-1/+1
| | | | | |
| | * | | | Building new records with enum scopes now works as expectedGodfrey Chan2014-01-033-12/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, this would give an `ArgumentError`: class Issue < ActiveRecord::Base enum :status, [:open, :finished] end Issue.open.build # => ArgumentError: '0' is not a valid status Issue.open.create # => ArgumentError: '0' is not a valid status PR #13542 muted the error, but the issue remains. This commit fixes the issue by allowing the enum value to be written directly via the setter: Issue.new.status = 0 # This now sets status to :open Assigning a value directly via the setter like this is not part of the documented public API, so users should not rely on this behavior. Closes #13530.
| | * | | | Deprecate unused `symbolized_base_class` and `symbolized_sti_name`.Yves Senn2014-01-033-0/+10
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These methods were only used for the `IdentityMap` which was removed. They are no longer used internally and should be removed without replacement. As they were not `:nodoc:`'ed it's better to deprecate them before removal.
| | * | | Merge pull request #13578 from schneems/schneems/database_url-env-defaultRafael Mendonça França2014-01-0212-19/+12
| | |\ \ \ | | | | | | | | | | | | Use DATABASE_URL by default
| | | * | | Use DATABASE_URL by defaultschneems2014-01-0212-19/+12
| | |/ / / | | | | | | | | | | See https://github.com/rails/rails/pull/13463#issuecomment-31480799 for full conversation.
| | * | | Merge pull request #13573 from kuldeepaggarwal/doc-fix-default-valuesRafael Mendonça França2014-01-021-2/+2
| | |\ \ \ | | | | | | | | | | | | provide correct information [ci skip]
| | | * | | provide correct information [ci skip]Kuldeep Aggarwal2014-01-021-2/+2
| | | | | |
| | * | | | Merge pull request #13528 from jonleighton/maintain_test_schemaJon Leighton2014-01-0217-58/+173
| | |\ \ \ \ | | | | | | | | | | | | | | Automatically maintain test database schema
| | | * | | | Automatically maintain test database schemaJon Leighton2014-01-0217-58/+173
| | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Move check from generated helper to test_help.rb, so that all applications can benefit * Rather than just raising when the test schema has pending migrations, try to load in the schema and only raise if there are pending migrations afterwards * Opt out of the check by setting config.active_record.maintain_test_schema = false * Deprecate db:test:* tasks. The test helper is now fully responsible for maintaining the test schema, so we don't need rake tasks for this. This is also a speed improvement since we're no longer reloading the test database on every call to "rake test".
| | * / / / Reverse 821525e and wrap run_generator callAndrew White2014-01-022-4/+14
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | Ruby 2.1.0 includes the json gem 1.8.1 by default so we need bundler 1.5.1 for `bundle install` to work. To fix this reverse the downgrade to 1.3.5 and wrap the `run_generator` call with a block that resets `THOR_DEBUG`.
| | * | | Adding missing requireRafael Mendonça França2014-01-021-0/+1
| | | | |
| | * | | Enable fast_finish in the travis configurationRafael Mendonça França2014-01-021-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since rbx and jruby builds are being allowed to fail lets receive the build information early enabling this option See http://about.travis-ci.org/blog/2013-11-27-fast-finishing-builds/
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-07 23:51:57 +0000