Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix assertion order and :scissors: extra spaces | Carlos Antonio da Silva | 2014-05-13 | 2 | -4/+3 |
| | |||||
* | Merge pull request #15068 from josepjaume/patch-1 | Aaron Patterson | 2014-05-13 | 2 | -0/+8 |
|\ | | | | | Dup options hash to prevent modifications | ||||
| * | Dup options hash to prevent modifications | Josep Jaume Rey | 2014-05-13 | 2 | -0/+8 |
| | | | | | | | | `options[:default]` and `options[:raise]` can be mistakenly added to the `options` hash. This can be a problem if you're reusing the same object. | ||||
* | | pg, clarify default behavior for composite types. | Yves Senn | 2014-05-13 | 1 | -7/+94 |
| | | | | | | | | | | | | | | | | | | | | | | | | * by default composite types are mapped as `OID::Identity` and issue a warning * the user is advised to register his own `OID::Type` to make use of composite types Registering a new `OID::Type` does currently not allow to specify the type casting behavior when writing to the database. In order for it to work we need to use the values within `@attributes`. They are already being type casted and are ready to be written to the DB. See https://github.com/rails/rails/blob/57643c961feb24b662620d330e71103a830003e8/activerecord/lib/active_record/attribute_methods.rb#L460-L462 | ||||
* | | extract pg type map initialization process to `TypeMapInitializer`. | Yves Senn | 2014-05-13 | 2 | -65/+74 |
| | | |||||
* | | Get rid of unused method | Carlos Antonio da Silva | 2014-05-13 | 1 | -11/+0 |
| | | | | | | | | It's usage has been removed in 325c9d5e5235db4b5210d9db6c863835d1ac7eed. | ||||
* | | Simplify merge call on polymorphic helpers | Carlos Antonio da Silva | 2014-05-13 | 1 | -2/+2 |
| | | |||||
* | | test, move all pg array tests into `postgresql/array_test.rb`. | Yves Senn | 2014-05-13 | 2 | -51/+29 |
| | | |||||
* | | Merge pull request #15084 from arthurnn/fix_assert_redirected_to | Andrew White | 2014-05-13 | 2 | -1/+13 |
|\ \ | | | | | | | Fix regression on `assert_redirected_to`. | ||||
| * | | Fix regression on `assert_redirected_to`. | Arthur Neves | 2014-05-13 | 2 | -1/+13 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `assert_redirected_to` would fail if there is no controller set on a `ActionDispatch::IntegrationTest`, as _compute_redirect_to_location would be called on the controller to build the url. This regression was introduced after 1dacfbabf3bb1e0a9057dd2a016b1804e7fa38c0. [fixes #14691] | ||||
* | | | Merge pull request #15083 from ↵ | Yves Senn | 2014-05-13 | 1 | -11/+14 |
|\ \ \ | | | | | | | | | | | | | | | | | zuhao/refactor_actionmailer_i18n_with_controller_test Use with_translation helper to clean up I18n stored translations. | ||||
| * | | | Use with_translation helper to clean up I18n stored translations. | Zuhao Wan | 2014-05-13 | 1 | -11/+14 |
| |/ / | |||||
* | | | Merge pull request #15082 from zuhao/refactor_actionmailer_delivery_methods_test | Yves Senn | 2014-05-13 | 1 | -30/+42 |
|\ \ \ | | | | | | | | | Clean up deliveries after each test. | ||||
| * | | | Move state restoration from setup/teardown into affected test cases. | Zuhao Wan | 2014-05-13 | 1 | -30/+42 |
| | | | | |||||
* | | | | Merge pull request #15086 from zuhao/refactor_actionmailer_mail_layout_test | Yves Senn | 2014-05-13 | 1 | -10/+0 |
|\ \ \ \ | | | | | | | | | | | Do not clear deliveries in setup. | ||||
| * | | | | Remove unnecessary setup and teardown. | Zuhao Wan | 2014-05-13 | 1 | -10/+0 |
| | |/ / | |/| | | |||||
* | | | | docs, explain usage of `count` + `select`. Closes #15065. [ci skip] | Yves Senn | 2014-05-13 | 1 | -0/+8 |
| | | | | |||||
* | | | | Merge pull request #15085 from zuhao/refactor_actionmailer_log_subscriber_test | Yves Senn | 2014-05-13 | 1 | -3/+5 |
|\ \ \ \ | |/ / / |/| | | | A minor stylistic fix and cleanup of mail deliveries. | ||||
| * | | | A minor stylistic fix and cleanup of mail deliveries. | Zuhao Wan | 2014-05-13 | 1 | -3/+5 |
|/ / / | |||||
* | | | fix bad merge | Aaron Patterson | 2014-05-12 | 1 | -15/+0 |
| | | | |||||
* | | | avoid array allocation when extracting usernames and passwords | Aaron Patterson | 2014-05-12 | 1 | -1/+7 |
| | | | |||||
* | | | Ok. Enough models for today :rage: | Rafael Mendonça França | 2014-05-12 | 1 | -3/+6 |
| | | | |||||
* | | | Test with ruby-head to make @tenderlove happy | Rafael Mendonça França | 2014-05-12 | 1 | -0/+2 |
| | | | |||||
* | | | This is also a model | Rafael Mendonça França | 2014-05-12 | 1 | -1/+1 |
| | | | |||||
* | | | This model is a model so it should behaves like one | Rafael Mendonça França | 2014-05-12 | 1 | -7/+3 |
| | | | |||||
* | | | Merge pull request #15080 from ↵ | Rafael Mendonça França | 2014-05-12 | 1 | -3/+3 |
|\ \ \ | | | | | | | | | | | | | | | | | eileencodes/flip-unless-else-conditional-in-halting-method flip conditional to use if/else instead of unless/else | ||||
| * | | | flip conditional to use if/else instead of unless/else | eileencodes | 2014-05-12 | 1 | -3/+3 |
|/ / / | | | | | | | | | | Use if/else instead of unless/else so conditional reads better. | ||||
* | | | Revert "reduce conditionals in url_for" | Aaron Patterson | 2014-05-12 | 1 | -6/+3 |
| | | | | | | | | | | | | This reverts commit 9df60693d77813b362fb528fc62381ac8f1eec12. | ||||
* | | | :cry::gun: use an empty hash for magical speed. | Aaron Patterson | 2014-05-12 | 1 | -1/+5 |
| | | | | | | | | | | | | | | | | | | For some reason, if the hash is empty, we choose "optimized" routes. I am not sure how or why optimized routes differ from regular routes. But, if we make sure the hash is empty, we get a nice speed improvement. | ||||
* | | | this class is a model, so it should act like one | Aaron Patterson | 2014-05-12 | 1 | -0/+3 |
| | | | |||||
* | | | use the helper method builder to construct helper methods | Aaron Patterson | 2014-05-12 | 3 | -59/+118 |
| | | | |||||
* | | | probably should not check in a raise | Aaron Patterson | 2014-05-12 | 1 | -1/+0 |
| | | | |||||
* | | | undo optimized calls until the builder object is finished | Aaron Patterson | 2014-05-12 | 3 | -41/+23 |
| | | | |||||
* | | | remove suffix from the handler methods | Aaron Patterson | 2014-05-12 | 1 | -20/+19 |
| | | | |||||
* | | | push list handling to the builder object | Aaron Patterson | 2014-05-12 | 1 | -30/+30 |
| | | | |||||
* | | | push string handling to the builder object | Aaron Patterson | 2014-05-12 | 1 | -10/+8 |
| | | | |||||
* | | | push model and class handling to a helper builder object | Aaron Patterson | 2014-05-12 | 1 | -36/+48 |
| | | | |||||
* | | | adding a method to handle strings using the same api | Aaron Patterson | 2014-05-12 | 1 | -22/+32 |
| | | | |||||
* | | | fix url_for with classes | Aaron Patterson | 2014-05-12 | 1 | -1/+1 |
| | | | |||||
* | | | break each polymorphic type to it's own method | Aaron Patterson | 2014-05-12 | 2 | -25/+86 |
| | | | |||||
* | | | remove dead code | Aaron Patterson | 2014-05-12 | 1 | -12/+0 |
| | | | |||||
* | | | only test `persisted?` on the record once | Aaron Patterson | 2014-05-12 | 1 | -3/+7 |
| | | | | | | | | | | | | | | | also avoid the case of calling `persisted?` on something that doesn't respond to it | ||||
* | | | only send options if there are any options | Aaron Patterson | 2014-05-12 | 1 | -5/+7 |
| | | | |||||
* | | | add a test for the parameters sent to the url helper for new records | Aaron Patterson | 2014-05-12 | 1 | -0/+15 |
| | | | |||||
* | | | remove should_pop variable | Aaron Patterson | 2014-05-12 | 1 | -4/+1 |
| | | | |||||
* | | | only pop records when we get an array | Aaron Patterson | 2014-05-12 | 1 | -6/+7 |
| | | | |||||
* | | | remove unnecessary array allocations | Coraline Ada Ehmke + Aaron Patterson | 2014-05-12 | 1 | -5/+1 |
| | | | |||||
* | | | simplified route method name generation | Coraline Ada Ehmke + Aaron Patterson | 2014-05-12 | 2 | -27/+19 |
| | | | |||||
* | | | small refactoring | Coraline Ada Ehmke + Aaron Patterson | 2014-05-12 | 1 | -6/+2 |
| | | | |||||
* | | | pulling helpermethods up before refactoring | Coraline Ada Ehmke + Aaron Patterson | 2014-05-12 | 1 | -20/+46 |
| | | |