Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | Improve PostgreSQL adapter schema-awareness | Paul Gallagher | 2011-06-10 | 3 | -27/+173 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * table_exists? scoped by schema search path unless schema is explicitly named. Added tests and doc to clarify the behaviour * extract_schema_and_table tests and implementation extended to cover all cases * primary_key does not ignore schema information * add current_schema and schema_exists? methods * more robust table referencing in insert_sql and sql_for_insert methods | |||||
* | | | | | Merge pull request #1784 from gazay/3-1-stable-shadowing-variables | José Valim | 2011-06-20 | 1 | -1/+1 | |
|\ \ \ \ \ | | | | | | | | | | | | | Removing shadowing outer local variables warnings | |||||
| * | | | | | Removed shadowing variable warning in activerecord railties | Alexey Gaziev | 2011-06-20 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | Add missing require 'set' | Daniel Azuma | 2011-06-20 | 1 | -0/+2 | |
|/ / / / / | ||||||
* | | | | | Merge pull request #1763 from grantneufeld/schema_column_exists_options_hash | José Valim | 2011-06-18 | 2 | -1/+13 | |
|\ \ \ \ \ | | | | | | | | | | | | | Fix default options argument on ActiveRecord::ConnectionAdaptors::Table#column_exists? | |||||
| * | | | | | Changed the default value for the options argument on ↵ | Grant Neufeld | 2011-06-18 | 2 | -1/+13 | |
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActiveRecord::ConnectionAdapters::Table#column_exists? from nil to an empty Hash {}. That method calls through to ActiveRecord::ConnectionAdapters::SchemaStatements##column_exists? expects options to be a Hash. When options was nil, an error would occur in cases where the column did exist because the called method attempted to perform a key lookup on options. | |||||
* / | | | | Fix inconsistencies by being polite to the wrapped body. Needed for ↵ | Steve Hodgkiss | 2011-06-18 | 4 | -0/+34 | |
|/ / / / | | | | | | | | | | | | | Rack::Sendfile to function properly. See issue #1761. | |||||
* | | | | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-06-18 | 4 | -13/+15 | |
|\ \ \ \ | ||||||
| * | | | | Update remove_index documentation | Lucia Escanellas | 2011-06-17 | 3 | -11/+13 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | * Changes should better reflect present code behavior * Related to issue: https://github.com/rails/rails/issues/1624 | |||||
| * | | | | Remove trailing whitespaces | Guillermo Iguaran | 2011-06-12 | 1 | -2/+2 | |
| |/ / / | ||||||
* | | | | Bump to mysql2 0.3.6 | Guillermo Iguaran | 2011-06-17 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Add require 'pathname' as it has not already been required in some circumstances | Jon Leighton | 2011-06-17 | 1 | -0/+1 | |
| | | | | ||||||
* | | | | Merge pull request #1636 from metaskills/upstream/3-1-stable | Aaron Patterson | 2011-06-17 | 1 | -0/+1 | |
|\ \ \ \ | | | | | | | | | | | Allow the connection pool's #table_exists? method to give the connections | |||||
| * | | | | Allow the connection pool's #table_exists? method to give the connections ↵ | Ken Collins | 2011-06-10 | 1 | -0/+1 | |
| |/ / / | | | | | | | | | | | | | #table_exists? method a chance. | |||||
* | | | | Perf fix - Use an instance variable instead of a class_attribute. Thanks ↵ | Jon Leighton | 2011-06-16 | 1 | -8/+4 | |
| | | | | | | | | | | | | | | | | @josevalim and @jhawthorn for the prompting. | |||||
* | | | | Bump mysql2 to 0.3.5 | Arun Agrawal | 2011-06-16 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Bump mysql2 up | Santiago Pastorino and José Ignacio Costa | 2011-06-16 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Pass mass-assignment options to nested models - closes #1673. | Andrew White | 2011-06-13 | 4 | -18/+276 | |
| | | | | ||||||
* | | | | Rename tests to reflect associations | Andrew White | 2011-06-13 | 1 | -30/+30 | |
| | | | | ||||||
* | | | | Add support for using an ARCONFIG environment variable to specify the ↵ | Jon Leighton | 2011-06-12 | 2 | -4/+12 | |
| | | | | | | | | | | | | | | | | location of the config.yml file for running the tests | |||||
* | | | | Oops, this should have been in the last commit :zap: | Jon Leighton | 2011-06-12 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Make assert_no_queries literally enforce that there are no queries. As in, ↵ | Jon Leighton | 2011-06-12 | 2 | -15/+4 | |
| | | | | | | | | | | | | | | | | not a single line of SQL should be sent to the database, not even stuff that is ignored by assert_queries. The argument being that if you write assert_no_queries, you really do not want there to be any queries. | |||||
* | | | | If we're going to use a global variable, let's at least namespace it | Jon Leighton | 2011-06-12 | 2 | -10/+10 | |
| | | | | ||||||
* | | | | Replace inline lambdas with named methods | Jon Leighton | 2011-06-12 | 1 | -29/+42 | |
| | | | | ||||||
* | | | | Refactor tests to be less brittle | Jon Leighton | 2011-06-12 | 3 | -28/+74 | |
| | | | | ||||||
* | | | | Move BEGIN and COMMIT into IGNORED_SQL rather than having them as a special ↵ | Jon Leighton | 2011-06-12 | 2 | -2/+1 | |
| | | | | | | | | | | | | | | | | case in assert_queries | |||||
* | | | | Don't wrap operations on collection associations in transactions when they ↵ | benedikt | 2011-06-12 | 2 | -3/+54 | |
|/ / / | | | | | | | | | | are not needed, so the connection adapter does not send empty BEGIN COMMIT transactions blocks to the database. | |||||
* | | | Ensure engine migrations are copied in the same sequence that they are required. | Brian Quinn | 2011-06-10 | 1 | -1/+1 | |
| | | | ||||||
* | | | don't reinitialize the arel_table unless the table_name changes | Damien Mathieu | 2011-06-09 | 1 | -1/+2 | |
| | | | ||||||
* | | | Print out information about whether the identity map is on or off when ↵ | Jon Leighton | 2011-06-09 | 3 | -6/+14 | |
| | | | | | | | | | | | | running unit tests | |||||
* | | | Update the RUNNING_UNIT_TESTS file | Jon Leighton | 2011-06-09 | 1 | -10/+5 | |
| | | | ||||||
* | | | Merge pull request #1591 from smartinez87/unused | José Valim | 2011-06-08 | 2 | -3/+3 | |
|\ \ \ | | | | | | | | | Fix "warning: assigned but unused variable" on AR tests | |||||
| * | | | Remove unused 'y' variable. | Sebastian Martinez | 2011-06-08 | 1 | -2/+2 | |
| | | | | ||||||
| * | | | Remove unused 'quoted_column_names' variable. | Sebastian Martinez | 2011-06-08 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | No need to create a new Arel::Table, as the arel_table method already ↵ | Rodrigo Navarro | 2011-06-08 | 1 | -1/+0 | |
|/ / / | | | | | | | | | | creates a new one | |||||
* | | | Do not hardcode :id as it was ignored by mass assignment filter | Raimonds Simanovskis | 2011-06-08 | 1 | -2/+2 | |
| | | | | | | | | | As a result test was failing on Oracle where ids are assigned by default from 1000 | |||||
* | | | remove warning: assigned but unused variable | Santiago Pastorino | 2011-06-08 | 2 | -2/+2 | |
| | | | ||||||
* | | | Allow polymorphic has_one to work when the association is set before the ↵ | Jon Leighton | 2011-06-08 | 2 | -3/+12 | |
| | | | | | | | | | | | | owner has been saved. Fixes #1524. | |||||
* | | | When you add a record to a polymorphic has_one, you should be able to access ↵ | Jon Leighton | 2011-06-08 | 3 | -3/+6 | |
| | | | | | | | | | | | | the owner from the associated record | |||||
* | | | fix typo | Alexandr Zykov | 2011-06-08 | 1 | -1/+1 | |
| | | | ||||||
* | | | This way asserting that updated_at was changed in touch look more obvious. | Dmitriy Kiriyenko | 2011-06-07 | 1 | -0/+1 | |
| | | | ||||||
* | | | Do not use default_scope in ActiveRecord::Persistence#touch. | Dmitriy Kiriyenko | 2011-06-07 | 2 | -1/+10 | |
| | | | ||||||
* | | | Fix a couple of helper requires that were missed. This was causing the tests ↵ | Jon Leighton | 2011-06-07 | 2 | -2/+2 | |
| | | | | | | | | | | | | to fail to run on 1.8.7. | |||||
* | | | Merge pull request #1474 from ganeshkumar/update_column_changes | José Valim | 2011-06-07 | 1 | -0/+9 | |
|\ \ \ | | | | | | | | | added tests for update_column | |||||
| * | | | added more tests for update_column | ganesh | 2011-06-04 | 1 | -0/+9 | |
| |/ / | ||||||
* | | | make sure AR isolated tests set the environment variable | Aaron Patterson | 2011-06-06 | 1 | -1/+2 | |
| | | | ||||||
* | | | oops! add parens so the block is properly bound | Aaron Patterson | 2011-06-06 | 1 | -1/+1 | |
| | | | ||||||
* | | | set the environment variable from the rake file | Aaron Patterson | 2011-06-06 | 18 | -18/+6 | |
| | | | ||||||
* | | | more oracle fixes | Aaron Patterson | 2011-06-06 | 1 | -1/+1 | |
| | | | ||||||
* | | | do not muck with the load path, that is the test task responsibility | Aaron Patterson | 2011-06-06 | 1 | -6/+0 | |
| | | |