aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge pull request #15084 from arthurnn/fix_assert_redirected_toAndrew White2014-05-132-1/+13
|\ \ \ \ \ | | | | | | | | | | | | Fix regression on `assert_redirected_to`.
| * | | | | Fix regression on `assert_redirected_to`.Arthur Neves2014-05-132-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 Senn2014-05-131-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 Wan2014-05-131-11/+14
| |/ / / / /
* | | | | | Merge pull request #15082 from zuhao/refactor_actionmailer_delivery_methods_testYves Senn2014-05-131-30/+42
|\ \ \ \ \ \ | | | | | | | | | | | | | | Clean up deliveries after each test.
| * | | | | | Move state restoration from setup/teardown into affected test cases.Zuhao Wan2014-05-131-30/+42
| | | | | | |
* | | | | | | Merge pull request #15086 from zuhao/refactor_actionmailer_mail_layout_testYves Senn2014-05-131-10/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Do not clear deliveries in setup.
| * | | | | | | Remove unnecessary setup and teardown.Zuhao Wan2014-05-131-10/+0
| | |/ / / / / | |/| | | | |
* | | | | | | docs, explain usage of `count` + `select`. Closes #15065. [ci skip]Yves Senn2014-05-131-0/+8
| | | | | | |
* | | | | | | Merge pull request #15085 from zuhao/refactor_actionmailer_log_subscriber_testYves Senn2014-05-131-3/+5
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | A minor stylistic fix and cleanup of mail deliveries.
| * | | | | | A minor stylistic fix and cleanup of mail deliveries.Zuhao Wan2014-05-131-3/+5
|/ / / / / /
* | | | | | fix bad mergeAaron Patterson2014-05-121-15/+0
| | | | | |
* | | | | | avoid array allocation when extracting usernames and passwordsAaron Patterson2014-05-121-1/+7
| | | | | |
* | | | | | Ok. Enough models for today :rage:Rafael Mendonça França2014-05-121-3/+6
| | | | | |
* | | | | | Test with ruby-head to make @tenderlove happyRafael Mendonça França2014-05-121-0/+2
| | | | | |
* | | | | | This is also a modelRafael Mendonça França2014-05-121-1/+1
| | | | | |
* | | | | | This model is a model so it should behaves like oneRafael Mendonça França2014-05-121-7/+3
| | | | | |
* | | | | | Merge pull request #15080 from ↵Rafael Mendonça França2014-05-121-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/elseeileencodes2014-05-121-3/+3
|/ / / / / / | | | | | | | | | | | | | | | | | | Use if/else instead of unless/else so conditional reads better.
* | | | | | Revert "reduce conditionals in url_for"Aaron Patterson2014-05-121-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 9df60693d77813b362fb528fc62381ac8f1eec12.
* | | | | | :cry::gun: use an empty hash for magical speed.Aaron Patterson2014-05-121-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 oneAaron Patterson2014-05-121-0/+3
| | | | | |
* | | | | | use the helper method builder to construct helper methodsAaron Patterson2014-05-123-59/+118
| | | | | |
* | | | | | probably should not check in a raiseAaron Patterson2014-05-121-1/+0
| | | | | |
* | | | | | undo optimized calls until the builder object is finishedAaron Patterson2014-05-123-41/+23
| | | | | |
* | | | | | remove suffix from the handler methodsAaron Patterson2014-05-121-20/+19
| | | | | |
* | | | | | push list handling to the builder objectAaron Patterson2014-05-121-30/+30
| | | | | |
* | | | | | push string handling to the builder objectAaron Patterson2014-05-121-10/+8
| | | | | |
* | | | | | push model and class handling to a helper builder objectAaron Patterson2014-05-121-36/+48
| | | | | |
* | | | | | adding a method to handle strings using the same apiAaron Patterson2014-05-121-22/+32
| | | | | |
* | | | | | fix url_for with classesAaron Patterson2014-05-121-1/+1
| | | | | |
* | | | | | break each polymorphic type to it's own methodAaron Patterson2014-05-122-25/+86
| | | | | |
* | | | | | remove dead codeAaron Patterson2014-05-121-12/+0
| | | | | |
* | | | | | only test `persisted?` on the record onceAaron Patterson2014-05-121-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | also avoid the case of calling `persisted?` on something that doesn't respond to it
* | | | | | only send options if there are any optionsAaron Patterson2014-05-121-5/+7
| | | | | |
* | | | | | add a test for the parameters sent to the url helper for new recordsAaron Patterson2014-05-121-0/+15
| | | | | |
* | | | | | remove should_pop variableAaron Patterson2014-05-121-4/+1
| | | | | |
* | | | | | only pop records when we get an arrayAaron Patterson2014-05-121-6/+7
| | | | | |
* | | | | | remove unnecessary array allocationsCoraline Ada Ehmke + Aaron Patterson2014-05-121-5/+1
| | | | | |
* | | | | | simplified route method name generationCoraline Ada Ehmke + Aaron Patterson2014-05-122-27/+19
| | | | | |
* | | | | | small refactoringCoraline Ada Ehmke + Aaron Patterson2014-05-121-6/+2
| | | | | |
* | | | | | pulling helpermethods up before refactoringCoraline Ada Ehmke + Aaron Patterson2014-05-121-20/+46
| | | | | |
* | | | | | push all is_a tests up, always pass options to the named routeAaron Patterson2014-05-121-28/+27
| | | | | |
* | | | | | pull nil checks upAaron Patterson2014-05-121-3/+5
| | | | | |
* | | | | | skip another Array is_a checkAaron Patterson2014-05-121-4/+6
| | | | | |
* | | | | | only do the Array is_a check onceAaron Patterson2014-05-121-4/+3
| | | | | |
* | | | | | we always convert this variable to a list, so we can remove the unconversionAaron Patterson2014-05-121-1/+0
| | | | | |
* | | | | | record as a hash is not possible, so rm branchAaron Patterson2014-05-121-2/+0
| | | | | |
* | | | | | stop popping in build_named_route_callAaron Patterson2014-05-121-4/+3
| | | | | |
* | | | | | remove extract_record since we have extracted the listAaron Patterson2014-05-121-12/+1
| | | | | |