Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | Fix assumption of primary key name in PredicateBuilder subquery. | Ernie Miller | 2011-08-20 | 2 | -1/+11 | |
|/ / / / | ||||||
* | | | | prevent sql injection attacks by escaping quotes in column names | Aaron Patterson | 2011-08-16 | 4 | -3/+20 | |
| | | | | ||||||
* | | | | In 1efd88283ef68d912df215125951a87526768a51, ConnectionAdapters was put ↵ | Jon Leighton | 2011-08-16 | 3 | -14/+26 | |
| | | | | | | | | | | | | | | | | under eager_autoload. Due to the requires in that file, this caused ConnectionSpecification to be loaded, which references ActiveRecord::Base, which means the database connection is established. We do not want to connect to the database when Active Record is loaded, only when ActiveRecord::Base is first referenced by the user. | |||||
* | | | | Revert "Merge pull request #2543 from htanata/unused_require_in_ar_test" | Santiago Pastorino | 2011-08-16 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | | | This reverts commit 87152f2604e73b218df90befda576f0acfed0bbf, reversing changes made to 0d3615f04c79f6e90d8ab33fdfc920b8faac9cb8. | |||||
* | | | | Remove unused require. | Hendy Tanata | 2011-08-16 | 1 | -1/+0 | |
| | | | | ||||||
* | | | | Bump arel dependency | Jon Leighton | 2011-08-15 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Use new SelectManager#source method | Jon Leighton | 2011-08-15 | 2 | -2/+2 | |
| | | | | ||||||
* | | | | Use new SelectManager#projections= method | Jon Leighton | 2011-08-15 | 3 | -4/+4 | |
| | | | | ||||||
* | | | | use update.key instead of update.ast.key. make better use of select manager. | Jon Leighton | 2011-08-15 | 3 | -26/+18 | |
| | | | | ||||||
* | | | | Use a SelectCore rather than a full SelectManager | Jon Leighton | 2011-08-15 | 2 | -4/+6 | |
| | | | | ||||||
* | | | | Refactor building the update manager | Jon Leighton | 2011-08-15 | 4 | -16/+13 | |
| | | | | ||||||
* | | | | Support for multi-table updates with limits, offsets and orders | Jon Leighton | 2011-08-15 | 4 | -2/+77 | |
| | | | | ||||||
* | | | | Support updates with joins. Fixes #522. | Jon Leighton | 2011-08-15 | 5 | -4/+34 | |
| | | | | ||||||
* | | | | Ensure empty has_many :through association preloaded via joins is marked as ↵ | Jon Leighton | 2011-08-15 | 2 | -3/+10 | |
| | | | | | | | | | | | | | | | | loaded. Fixes #2054. | |||||
* | | | | Fix private methods which are delegated to. This previously worked because ↵ | Jon Leighton | 2011-08-15 | 2 | -13/+13 | |
|/ / / | | | | | | | | | | Module#delegate previously ignored method visibility. | |||||
* | | | Merge branch 'master' of github.com:lifo/docrails | Xavier Noria | 2011-08-13 | 2 | -1/+5 | |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: RELEASING_RAILS.rdoc actionpack/lib/sprockets/railtie.rb actionpack/test/template/sprockets_helper_test.rb activerecord/test/cases/calculations_test.rb railties/guides/source/3_1_release_notes.textile railties/guides/source/active_resource_basics.textile railties/guides/source/command_line.textile | |||||
| * | | | Document exclamation point on dynamic finders | Florent Guilleux | 2011-08-13 | 1 | -0/+4 | |
| | | | | ||||||
| * | | | Rubygems => RubyGems | Sukeerthi Adiga | 2011-08-13 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Quote these dates to prevent intermittent test failure. Suppose local time ↵ | Jon Leighton | 2011-08-13 | 2 | -4/+4 | |
| | | | | | | | | | | | | | | | | is 00:50 GMT+1. Without the quoting, the YAML parser would parse this as 00:50 UTC, into the local time of 01:50 GMT+1. Then, it would get written into the database in local time as 01:50. When it came back out the UTC date from the database and the UTC date of two weeks ago would be compared. The former would be 23:50, and the latter would be 00:50, so the two dates would differ, causing the assertion to fail. Quoting it prevents the YAML parser from getting involved. | |||||
| * | | | accept option for recreate db for postgres (same as mysql now) | artemk | 2011-08-13 | 1 | -2/+4 | |
| | | | | ||||||
| * | | | initializing @open_transactions in the initialize method | Aaron Patterson | 2011-08-13 | 1 | -4/+2 | |
| | | | | ||||||
| * | | | add a migrate class method and delegate to the new instance | Aaron Patterson | 2011-08-13 | 1 | -1/+4 | |
| | | | | ||||||
| * | | | Ensure that .up and .down work as well. | Christopher Meiklejohn | 2011-08-13 | 1 | -2/+13 | |
| | | | | ||||||
| * | | | Support backwards compatible interface for migration down/up with rails 3.0.x. | Christopher Meiklejohn | 2011-08-13 | 2 | -0/+25 | |
| | | | | ||||||
| * | | | remove extra require for 'stringio' as it is required in helper.rb | Vishnu Atrai | 2011-08-13 | 1 | -1/+0 | |
| | | | | ||||||
| * | | | magic comment test only if encoding_aware?. | Arun Agrawal | 2011-08-13 | 1 | -3/+4 | |
| | | | | ||||||
| * | | | Resolve warnings by instantizing @attrubtes as nil | Brian Cardarella | 2011-08-13 | 1 | -0/+1 | |
| | | | | ||||||
| * | | | default writing the schema file as utf-8 | Aaron Patterson | 2011-08-13 | 1 | -1/+2 | |
| | | | | ||||||
| * | | | dump IO encoding value along with schema.rb so the file can be reloaded. ↵ | Aaron Patterson | 2011-08-13 | 2 | -3/+16 | |
| | | | | | | | | | | | | | | | | fixes #1592 | |||||
| * | | | Revert "Merge pull request #2309 from smasry/master" | Jon Leighton | 2011-08-13 | 2 | -6/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 9d396ee8195e31f646e0b89158ed96f4db4ab38f, reversing changes made to fa2bfd832c1d1e997d93c2269a485cc74782c86d. Reason: the change broke the build. | |||||
| * | | | Reverse order fix when using function for ActiveRecord::QueryMethods Fixes #1697 | Samer Masry | 2011-08-13 | 2 | -1/+6 | |
| | | | | ||||||
| * | | | Fixed failing query when performing calculation with having based on select. | Dmitriy Kiriyenko | 2011-08-13 | 2 | -0/+8 | |
| | | | | ||||||
| * | | | Add a proxy_association method to association proxies, which can be called ↵ | Jon Leighton | 2011-08-13 | 4 | -6/+27 | |
| | | | | | | | | | | | | | | | | by association extensions to access information about the association. This replaces proxy_owner etc with proxy_association.owner. | |||||
| * | | | remove deprication warning: ambiguous first argument; put parentheses or ↵ | Vishnu Atrai | 2011-08-13 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | even spaces | |||||
| * | | | remove deprication warning for ruby 1.9.3-head for unused variables | Vishnu Atrai | 2011-08-13 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Revert "allow select to have multiple arguments" | Piotr Sarnacki | 2011-08-13 | 2 | -11/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 04cc446d178653d362510e79a22db5300d463161. I reverted it because apparently we want to use: select([:a, :b]) instead of select(:a, :b), but there was no tests for that form. | |||||
| * | | | allow select to have multiple arguments | Sławosz Sławiński | 2011-08-13 | 2 | -3/+11 | |
| | | | | ||||||
| * | | | Simplify the test by using id and name. `id` will be the only real sort ↵ | Franck Verrot | 2011-08-13 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | criteria in any case as it's unique. | |||||
| * | | | Revert "Explicitly included hashes in sentence regarding SQL-injection-safe ↵ | Xavier Noria | 2011-08-13 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | forms" Reason: The hash form is secure, and preferred over the array form if possible. This reverts commit 6dc749596c328c44c80f898d5fa860fff6cab783. | |||||
| * | | | make the warning clear about the effect of using validates_associated on ↵ | Vijay Dev | 2011-08-13 | 1 | -9/+1 | |
| | | | | | | | | | | | | | | | | both sides on an association. | |||||
| * | | | Removed trailing slash of 'Download and installation' Github URL as per ↵ | ejy | 2011-07-28 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | convention | |||||
| * | | | Explicitly included hashes in sentence regarding SQL-injection-safe forms | Pete Campbell | 2011-07-28 | 1 | -3/+3 | |
| | | | | ||||||
* | | | | Perf: don't mess around with thread local vars unless we actually need to | Jon Leighton | 2011-08-13 | 1 | -18/+23 | |
| | | | | ||||||
* | | | | Fix default scope thread safety. Thanks @thedarkone for reporting. | Jon Leighton | 2011-08-13 | 3 | -15/+53 | |
| | | | | ||||||
* | | | | Merge pull request #2514 from myronmarston/fix_active_record_observer_disabling | José Valim | 2011-08-13 | 2 | -1/+13 | |
|\ \ \ \ | | | | | | | | | | | Allow ActiveRecord observers to be disabled. | |||||
| * | | | | Allow ActiveRecord observers to be disabled. | Myron Marston | 2011-08-12 | 2 | -1/+13 | |
| | | | | | | | | | | | | | | | We have to use Observer#update rather than Observer#send since the enabled state is checked in #update before forwarding the method call on. | |||||
* | | | | | Work around for lolruby bug. (Read on for explanation.) | Jon Leighton | 2011-08-13 | 2 | -5/+14 | |
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We were experiencing CI test failures, for example: * 3-1-stable: http://travis-ci.org/#!/rails/rails/builds/79473/L407 * master: http://travis-ci.org/#!/rails/rails/builds/79507/L80 These failures only happened on 1.8.7-p352, and we were only able to reproduce on the Travis CI VM worker. We even tried creating a new 32 bit Ubuntu VM and running the tests on that, and it all worked fine. After some epic trial and error, we discovered that replacing the following: fuu = Marshal.load(Marshal.dump(fuu)) with: marshalled = Marshal.dump(fuu) fuu = Marshal.load(marshalled) seemed to prevent the failure. We have NO IDEA why this is. If anyone has some great insight to contribute then that is welcome. Otherwise, hopefully this will just help us get the CI green again. Many thanks to @joshk for help with sorting this out. | |||||
* | | / | add the gem requirement for sqlite3 | Aaron Patterson | 2011-08-10 | 1 | -0/+2 | |
| |_|/ |/| | | ||||||
* | | | Okay, the new incompatible arel is now called 2.2 and the sun shines upon ↵ | Jon Leighton | 2011-08-09 | 1 | -1/+1 | |
| | | | | | | | | | | | | thee once more | |||||
* | | | Remove TODO comment I didn't mean to commit | Jon Leighton | 2011-08-09 | 1 | -2/+0 | |
| | | |