Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fixes #10432 add_column not creating array columns in PostgreSQL | Adam Anderson | 2013-06-04 | 1 | -0/+1 |
| | | | | | | When then PostgreSQL visitor was [added](https://github.com/rails/rails/commit/6b7fdf3bf3675a14eae74acc5241089308153a34) `add_column` was no longer receiving the column options directly. This caused the options to be lost along the way. | ||||
* | Remove behavior that deals with a string as third argument of `add_index`, ↵ | kennyj | 2013-06-02 | 1 | -28/+14 |
| | | | | | | because this was deprecated. Some testcases is failed, so I replaced nil to empty hash in add_reference. | ||||
* | Merge pull request #10619 from alno/top_level_array_in_postgres_json | Aaron Patterson | 2013-05-16 | 2 | -1/+3 |
|\ | | | | | Support array as root element in Postgresql JSON columns | ||||
| * | Support array as root element in JSON | Alexey Noskov | 2013-05-14 | 2 | -1/+3 |
| | | |||||
* | | tiny types should only be integers when the length is <= 1. fixes #10620 | Aaron Patterson | 2013-05-15 | 1 | -3/+9 |
| | | |||||
* | | Also support extensions in PostgreSQL 9.1, because this has been supported ↵ | kennyj | 2013-05-15 | 1 | -4/+4 |
|/ | | | | since 9.1. | ||||
* | Merge pull request #10556 from Empact/deprecate-schema-statements-distinct | Rafael Mendonça França | 2013-05-12 | 1 | -1/+2 |
| | | | | | | Deprecate SchemaStatements#distinct, and make SchemaStatements#columns_for_distinct nodoc. Conflicts: activerecord/CHANGELOG.md | ||||
* | Merge pull request #10572 from nertzy/dont-modify-options-hash-in-primary-key | Rafael Mendonça França | 2013-05-11 | 1 | -2/+1 |
| | | | | Don't modify args in TableDefinition#primary_key | ||||
* | Reject blank order_values within #columns_for_distinct, as the orders aren't ↵ | Ben Woosley | 2013-05-10 | 1 | -1/+1 |
| | | | | used at all on non-postgres adapters. | ||||
* | Fix that #exists? can produce invalid SQL: "SELECT DISTINCT DISTINCT" | Ben Woosley | 2013-05-10 | 2 | -10/+13 |
| | | | | | | | | | | | | The combination of a :uniq => true association and the #distinct call in #construct_limited_ids_condition combine to create invalid SQL, because we're explicitly selecting DISTINCT, and also sending #distinct on to AREL, via the relation#distinct_value. Rather than build a select distinct clause in #construct_limited_ids_condition, I set #distinct! and pass just the columns into the select statement. This requires introducing a #columns_for_distinct method to return the select columns but not the statement itself. | ||||
* | Revert "Merge pull request #10455 from ↵ | Aaron Patterson | 2013-05-07 | 1 | -6/+2 |
| | | | | | | | patricksrobertson/bigserial_id_not_identifying_pk" This reverts commit 3043d45eefc3776d5f3a9e7d212a01f99d869ef8, reversing changes made to ca0275d36b395631725c4583db5a45c06443fdb9. | ||||
* | Handle other pk types in PostgreSQL gracefully. | Patrick Robertson | 2013-05-07 | 1 | -2/+6 |
| | | | | | | | | | | | | | | In #10410 it was noted that you can no longer create PK's with the type of bigserial in PostgreSQL in 4.0.0.rc1. This is mostly because the newer adapter is checking for column type with the id column instead of just letting it pass through like it did before. Side effects: You may just create a PK column of a type that you really don't want to be your PK. As far as I can tell this was allowed in 3.2.X and perhaps an exception should be raised if you try and do something extremely dumb. | ||||
* | Changing method call according to coding conventions | Akshay Khole | 2013-05-05 | 1 | -1/+1 |
| | |||||
* | Make SchemaDumper emit "id: :uuid" when appropriate. Fixes #10451. | Brian Buchanan | 2013-05-03 | 1 | -1/+2 |
| | |||||
* | Add parameter :sslcompression to PostgreSQL adapter. | Lars Kanis | 2013-05-02 | 1 | -2/+2 |
| | | | | It is new in PostgreSQL-9.2 . | ||||
* | Improve docs for postgresql with uuid primary keys [ci skip] | Carlos Antonio da Silva | 2013-05-01 | 1 | -8/+7 |
| | | | | Introduced in 09ac1776abc0d3482f491f2d49f47bcb3d9a4ad7. | ||||
* | allow override of uuid_generate_v4() default by passing default: nil | Chad Moone | 2013-05-01 | 1 | -1/+30 |
| | | | | without this, it's not possible to use UUID primary keys without uuid-ossp installed and activated | ||||
* | maintain return value for recreate_database | Aaron Patterson | 2013-04-30 | 1 | -1/+2 |
| | |||||
* | mysql needs to reconnect after recreate. Thanks @mperham | Aaron Patterson | 2013-04-30 | 1 | -0/+1 |
| | |||||
* | StatementInvalid takes WrappedDatabaseException's place | Jeremy Kemper | 2013-04-28 | 2 | -2/+2 |
| | |||||
* | Move method used only in the test to the test code itself | Ivan Kataitsev | 2013-04-25 | 1 | -8/+0 |
| | |||||
* | fix typos | Vipul A M | 2013-04-21 | 1 | -1/+1 |
| | |||||
* | Fix loading of fixtures when the column type is a postgres array of strings. | Chris Constantine | 2013-04-18 | 1 | -1/+1 |
| | | | | - A string in an array of strings that has a quote char (') needs to have that quote char escaped if the array is getting wrapped in quote chars. | ||||
* | Revert "Merge pull request #10043 from cconstantine/master" | Rafael Mendonça França | 2013-04-18 | 1 | -15/+0 |
| | | | | | | | This reverts commit 521035af530482d6d9ad2dae568eaeb0ab188e1c, reversing changes made to 222011dbee842bbc60d3aaaa3145356b90a30fd1. Reason: This broke the tests | ||||
* | Merge pull request #10043 from cconstantine/master | Rafael Mendonça França | 2013-04-18 | 1 | -0/+15 |
|\ | | | | | DB with postgres string array column doesn't load fixtures well | ||||
| * | Fix loading of string arrays in postgres | Chris Constantine | 2013-04-08 | 1 | -0/+15 |
| | | |||||
* | | Fix #7619. 0x prefix must be added when assigning hexadecimal string into ↵ | kennyj | 2013-04-17 | 1 | -2/+4 |
| | | | | | | | | bit column in Postgresql, because solving ambiguity. | ||||
* | | Add OID::Bit for supporting bit string. | kennyj | 2013-04-16 | 2 | -2/+19 |
| | | |||||
* | | fix for the bytea/binary nil value bug | Matt Aimonetti | 2013-04-14 | 1 | -0/+1 |
|/ | |||||
* | Correctly parse bigint defaults in PostgreSQL | Erik Peterson | 2013-04-05 | 1 | -1/+1 |
| | |||||
* | Per #9999 revert the revert changing so columns are only tested for empty? ↵ | Sam | 2013-04-03 | 1 | -4/+4 |
| | | | | | | as opposed to blank? This is both faster and more correct, added tests to make sure this is not reverted again. | ||||
* | Merge pull request #10010 from pwnall/fix_ref_index_true | Rafael Mendonça França | 2013-04-01 | 1 | -1/+1 |
|\ | | | | | Make references with index:true pass Hash options to add_index | ||||
| * | Make references with index:true pass Hash options to add_index. | Victor Costan | 2013-04-01 | 1 | -1/+1 |
| | | |||||
* | | fix detect column type of enum | masarakki | 2013-04-01 | 1 | -2/+2 |
|/ | | | | enum includes text or blob or ... hooked by wrong regex | ||||
* | Refactor index algorithm lookup so that it only builds the available options ↵ | Carlos Antonio da Silva | 2013-03-30 | 1 | -4/+4 |
| | | | | | | | | | | | once This way the available options are only built for actually fetching the algorithm in case the option is given. The options are going to be necessary a second time only in case the option is given but does not exist, which is supposed to be due to a typo or something like that, so no problem. | ||||
* | Minor doc improvement about index algorithms [ci skip] | Carlos Antonio da Silva | 2013-03-30 | 1 | -3/+2 |
| | |||||
* | Do not calculate values if they are not going to be used | Carlos Antonio da Silva | 2013-03-30 | 1 | -7/+9 |
| | | | | | | | | | When building the indexes list in PostgreSQL, IndexDefinition objects are only created if the query for the related attributes really returns something matching the attributes. In case it does not, the variables for building the definition objects were being created but not used. | ||||
* | Make docs a little bit more consistent with colons [ci skip] | Carlos Antonio da Silva | 2013-03-30 | 1 | -9/+9 |
| | |||||
* | Fix code examples indent in new index options docs [ci skip] | Carlos Antonio da Silva | 2013-03-30 | 1 | -3/+3 |
| | |||||
* | Use new hash style in doc examples [ci skip] | Carlos Antonio da Silva | 2013-03-30 | 1 | -2/+2 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-03-30 | 4 | -7/+7 |
|\ | | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb activerecord/test/cases/adapter_test.rb guides/source/testing.md [ci skip] | ||||
| * | nodoc AR::ConnectionHandling for adapters [ci skip] | Francesco Rodriguez | 2013-03-15 | 4 | -7/+7 |
| | | |||||
| * | Unmatched parenthesis fix | Robin Dupret | 2013-03-15 | 1 | -1/+1 |
| | | |||||
* | | added comments to test_invalid_type in sqlite3 adapter test and added ↵ | Ranjay Krishna | 2013-03-28 | 2 | -4/+4 |
| | | | | | | | | valid_type? method to abstract_adapter.rb and removed unnecessary method from sqlite3 adapter | ||||
* | | Add support for FULLTEXT and SPATIAL indexes using the :type flag for MySQL. | Ken Mazaika | 2013-03-27 | 4 | -6/+25 |
| | | |||||
* | | Merge pull request #9928 from vipulnsward/fix_rename_auto_increment_mysql | Rafael Mendonça França | 2013-03-26 | 4 | -10/+14 |
|\ \ | | | | | | | respect auto_increment in rename_column for mysql | ||||
| * | | respect auto_increment in rename_column for mysql | Vipul A M | 2013-03-27 | 4 | -10/+14 |
| | | | |||||
* | | | Checks :algorithm argument for valid values | Dan McClain | 2013-03-26 | 1 | -1/+7 |
| | | | |||||
* | | | Adds support for concurrent indexing in PostgreSQL adapter | Dan McClain | 2013-03-25 | 5 | -14/+24 |
|/ / | | | | | | | | | | | | | | | Adds support for algorithm option in MySQL indexes Moves USING and algorithm options upstream The syntax is still specific to the Adapter, so the actual executed string happens in the corresponding adapter | ||||
* | | Fix typo | Rafael Mendonça França | 2013-03-25 | 1 | -1/+1 |
| | |