| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|\ \ \ \
| | | | |
| | | | | |
Add documentation to ActionDispatch::Request
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Rails 4 upgrade notes about clashing named route selection changes
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Add support for generate scaffold password:digest
* adds password_digest attribute to the migration
* adds has_secure_password to the model
* adds password and password_confirmation password_fields to _form.html
* omits password from index.html and show.html
* adds password and password_confirmation to the controller
* adds unencrypted password and password_confirmation to the controller test
* adds encrypted password_digest to the fixture
|
|/ / / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
* adds password_digest attribute to the migration
* adds has_secure_password to the model
* adds password and password_confirmation password_fields to _form.html
* omits password entirely from index.html and show.html
* adds password and password_confirmation to the controller
* adds unencrypted password and password_confirmation to the controller test
* adds encrypted password_digest to the fixture
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
rework passing headers and env variables in integration tests
|
| | | | | | | |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Closes #6513.
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Also:
cleanup, use consistent syntax for `Http::Header` and test.
|
|/ / / / / / |
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | |
| | | | | |
| | | | | | |
vipulnsward/change_from_each_to_each_value_in_http_params
Change from each to each_value in http/parameters since we don't use key
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
fix typo in Active Record CHANGELOG [ci skip]
|
|/ / / / / / |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
The bug with `ActiveSupport::TimeZone.parse` described in #9678 was
unwittingly fixed in 005d910 so add some tests to prevent regression.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
:heart::heart::heart::heart::heart::heart::heart::heart::heart::heart::heart::heart: @pixeltrix
|
| | | | | | |
|
|\ \ \ \ \ \
| |_|_|/ / /
|/| | | | | |
Also quote extension name in disable_extension
|
|/ / / / /
| | | | |
| | | | | |
A patch was committed recently which quoted the extension name in the SQL for `enable_extension`. But the same wasn't done for `disable_extension`.
|
| | | | |
| | | | |
| | | | |
| | | | | |
Cache the value of "super" in a variable and use it instead.
|
| | | | | |
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
dependent: :destroy should call destroy_all
|
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Commit https://github.com/rails/rails/pull/9668 shows warning
when `delete_all` is invoked with `:dependent` option
`:destroy`.
Unfortunately invoking `Post.destroy_all` invokes
`post.comments.delete_all` as part of `has_many` callbacks.
This commit ensures that instead `post.comments.destroy_all` is
invoked and in the process no warning is generated.
See issue #9567 for details .
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Pass column to quote when copying a sqlite table.
To make quote escape binary data correctly it needs the column passed in.
|
| | |/ /
| |/| |
| | | |
| | | | |
To make quote escape binary data correctly it needs the column passed in.
|
|\ \ \ \
| |/ / /
|/| | | |
Fix typo in DependenciesTestHelpers module name
|
| | | | |
|
| |/ /
|/| | |
|
| | | |
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | | |
Show warning message if delete_all is firing callbacks
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
`post.comments.delete_all` will fire callbacks if :dependent option is
:destroy . It will be fixed in Rails 4.1 . In the meantime display
a warning . Look at #9567 for details .
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Using regexps as arguments without parentheses end up raising:
warning: ambiguous first argument; put parentheses or even spaces
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Make sure that the test case is run under test env
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* Unset $RAILS_ENV that got set by abstract_unit to trigger the default.
* split out environment setting since Ruby 1.9.3 doesn't support inline
ENV setting.
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
https://github.com/rails/rails/pull/3180\#issuecomment-14705821 has
mention of rendering going from 1-2ms to 4 seconds with this on,
which seems reasonable: debugging is slow.
|
| | | | |
|
|/ / / |
|
|\ \ \
| | | |
| | | | |
Fix test failure introduced in 3ed41e57
|
|/ / /
| | |
| | |
| | | |
I forgot to run the test suit after changing the task name. :bomb:
|
|\ \ \
| | | |
| | | | |
Add `rails test` command to run the test suite
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Also, print out deprecation warning for other rake tasks except
`rake test` and `rake` (default)
|