aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix scaffold controller generator testsCarlos Antonio da Silva2012-12-081-8/+6
|
* Improve where.not docs [ci skip]Carlos Antonio da Silva2012-12-081-14/+8
| | | | | | | * Fix example queries * Remove doc entries of where.like/not_like. * Remove :chain from where.not related docs. To me that's an implementation detail and we don't expect people to use where(:chain).not.
* Merge pull request #8461 from frodsan/fix_assert_method_generatorsCarlos Antonio da Silva2012-12-081-1/+1
|\ | | | | Fix #assert_instance_method to also assert indented methods
| * Fix #assert_instance_method to also assert indented methodsFrancesco Rodriguez2012-12-081-1/+1
| |
* | Merge pull request #8465 from frodsan/update_am_action_callbacksRafael Mendonça França2012-12-085-19/+19
|\ \ | | | | | | use _action callbacks in actionmailer
| * | use _action callbacks in actionmailerFrancesco Rodriguez2012-12-085-19/+19
|/ /
* | Merge pull request #8395 from bmaland/attr_accessorsCarlos Antonio da Silva2012-12-081-23/+1
|\ \ | | | | | | Use attr_accessor for Rails.{application,cache,logger}
| * | Use attr_accessor for Rails.{application,cache,logger}Bjørn Arild Mæland2012-12-011-23/+1
| | |
* | | Merge pull request #8457 from caike/masterCarlos Antonio da Silva2012-12-081-4/+4
|\ \ \ | | | | | | | | | | | | Update AR Changelog with correct example using includes [ci skip]
| * | | Update AR Changelog with correct example using includesCaike Souza2012-12-081-4/+4
| | |/ | |/| | | | | | | These queries don't seem to work without the includes clause. [ci skip]
* | | Merge pull request #8462 from frodsan/update_guides_action_callbacksCarlos Antonio da Silva2012-12-086-21/+21
|\ \ \ | | | | | | | | Update guides to use _action callbacks [ci skip]
| * | | update guides to use _action callbacks [ci skip]Francesco Rodriguez2012-12-076-21/+21
| |/ /
* | | Merge pull request #8459 from frodsan/explicit!Carlos Antonio da Silva2012-12-082-19/+14
|\ \ \ | |/ / |/| | use :only instead of :except callback option in the controller template
| * | fix scaffold controller generator testsFrancesco Rodriguez2012-12-071-16/+12
| | |
| * | use :only instead of :except callback option in the controller templateFrancesco Rodriguez2012-12-071-3/+2
|/ / | | | | | | | | | | As [Security Guide](http://edgeguides.rubyonrails.org/security.html#whitelists-versus-blacklists) says, it's better to use `before_filter only: []` instead of `except: []` so we don't forget to turn the filter off for newly added actions.
* | Merge pull request #8456 from frodsan/actionize2Xavier Noria2012-12-075-32/+76
|\ \ | | | | | | Actionize2
| * | add tests to aliased _filter callbacksFrancesco Rodriguez2012-12-071-0/+46
| | |
| * | use `_action` instead of `_filter` callbacksFrancesco Rodriguez2012-12-075-34/+32
|/ /
* | Merge pull request #8455 from frodsan/actionizeXavier Noria2012-12-0714-28/+28
|\ \ | | | | | | Actionize: Use `_action` callbacks in documentation and code
| * | update documentation and code to use _action callbacksFrancesco Rodriguez2012-12-0713-25/+25
| | |
| * | fix prepend_before_filter documentation [ci skip]Francesco Rodriguez2012-12-071-3/+3
| | |
* | | Merge pull request #8454 from frodsan/deprecate_as_basic_objectJeremy Kemper2012-12-076-15/+29
|\ \ \ | | | | | | | | Rename ActiveSupport::BasicObject to ActiveSupport::ProxyObject
| * | | Rename ActiveSupport::BasicObject to ActiveSupport::ProxyObjectFrancesco Rodriguez2012-12-076-15/+29
| | | | | | | | | | | | | | | | | | | | AS::BasicObject is used for proxy classes. Let's give it a less concerning name. Also, it avoids the confusion with Ruby's Basic Object.
* | | | Let the scaffold example use the "set shared record" pattern to explain ↵David Heinemeier Hansson2012-12-071-6/+7
| |/ / |/| | | | | | | | callbacks
* | | Rollback where.like and where.not_likeCarlos Antonio da Silva2012-12-073-46/+4
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The real win with these chain methods is where.not, that takes care of different scenarios in a graceful way, for instance when the given value is nil. where("author.id != ?", author_to_ignore.id) where.not("author.id", author_to_ignore.id) Both where.like and where.not_like compared to the SQL versions doesn't seem to give us that much: Post.where("title LIKE 'ruby on%'") Post.where.like(title: 'ruby on%'") Post.where("title NOT LIKE 'ruby on%'") Post.where.not_like(title: 'ruby on%'") Thus Rails is adding where.not, but not where.like/not_like and others.
* | Fix READMEDavid Heinemeier Hansson2012-12-071-1/+1
| |
* | Rename all action callbacks from *_filter to *_actionDavid Heinemeier Hansson2012-12-073-69/+118
| |
* | Merge pull request #8452 from claudiob/explain_where_chain_parametersCarlos Antonio da Silva2012-12-072-2/+23
|\ \ | | | | | | Document the types of arguments accepted by AR#not
| * | Document the types of arguments accepted by AR#notclaudiob2012-12-072-2/+23
|/ / | | | | | | | | | | | | | | This commit stems from https://github.com/rails/rails/pull/8332#issuecomment-11127957 Since the formats in which conditions can be passed to `not` differ from the formats in which conditions can be passed to `like` and `not_like`, then I think it's worth adding rdoc and tests to show this behavior
* | revises a RDoc example to make it idiomaticXavier Noria2012-12-071-8/+8
| | | | | | | | | | | | The force flag suggests the original was probably copied from some db/schema.rb. Thanks to Josh Susser for spotting and reporting this.
* | let @version be always externally set in guides generation [fixes #8221]Xavier Noria2012-12-072-2/+2
| | | | | | | | | | | | | | | | | | | | Shelling out was there for authors convenience, but we are rather going to have the tag or SHA1 always in RAILS_VERSION and if the environment variable is blank, then just use "local" as a reminder that you are just working locally. The docs server has been updated to set the long SHA1 in RAILS_VERSION when generating edge guides.
* | Fix where.not with in clauseCarlos Antonio da Silva2012-12-072-4/+4
| | | | | | | | | | | | Arel::Nodes::In inherits from Arel::Nodes::Equality, so the case statement was always using the Equality operator for both scenarios, resulting in a not equal query instead.
* | Merge pull request #8445 from claudiob/match_rdoc_with_6ba0f97Carlos Antonio da Silva2012-12-071-4/+4
|\ \ | | | | | | Update #where rdoc to match 6ba0f97 [ci skip]
| * | Update #where rdoc to match 6ba0f97 [ci skip]claudiob2012-12-071-4/+4
|/ / | | | | | | | | | | | | | | This commit updates the rdoc of AR#where to match the changes applied in https://github.com/rails/rails/commit/6ba0f97 that is: * `where(nil)` has the same effect of `where('')`: a no op * `where` (no args) has the same effect of `where(:chain)`: to create a WhereChain
* | Revert "Invert precedence of content in ActionDispatch::Static"Andrew White2012-12-074-51/+12
| | | | | | | | This reverts commit c59734f756b79c39486c45273d2cc5d42cd0c864.
* | Move where with blank conditions test to the correct where tests fileCarlos Antonio da Silva2012-12-072-6/+6
| | | | | | | | This test does not belong to has many associations test.
* | Ensure there won't be any regression with where(nil) callsCarlos Antonio da Silva2012-12-072-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Consider this scenario: if params[:foo] conditions = { foo: true } end foos = Foo.where(conditions).order(:id) When params[:foo] is nil, this would call: foos = Foo.where(nil).order(:id) In this scenario, we want Foo.where(conditions) to be the same as calling Foo.all, otherwise we'd get a "NoMethodError order for WhereChain". Related to #8332.
* | Merge pull request #8332 from amatsuda/ar_where_chainCarlos Antonio da Silva2012-12-075-8/+200
|\ \ | | | | | | | | | | | | | | | | | | | | | Relation.where with no args can be chained with not, like, and not_like Conflicts: activerecord/CHANGELOG.md activerecord/lib/active_record/relation/query_methods.rb
| * | Relation.where with no args can be chained with not, like, and not_likeAkira Matsuda2012-11-305-8/+198
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | examples: Model.where.not field: nil #=> "SELECT * FROM models WHERE field IS NOT NULL Model.where.like name: 'Jeremy%' #=> "SELECT * FROM models WHERE name LIKE 'Jeremy%' this feature was originally suggested by Jeremy Kemper https://github.com/rails/rails/pull/5950#issuecomment-5591330 Closes #5950
* | | Stop shelling out more than necessaryCarlos Antonio da Silva2012-12-071-10/+10
| | |
* | | Unscope update_column(s) query to ignore default scopeCarlos Antonio da Silva2012-12-063-1/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When applying default_scope to a class with a where clause, using update_column(s) could generate a query that would not properly update the record due to the where clause from the default_scope being applied to the update query. class User < ActiveRecord::Base default_scope where(active: true) end user = User.first user.active = false user.save! user.update_column(:active, true) # => false In this situation we want to skip the default_scope clause and just update the record based on the primary key. With this change: user.update_column(:active, true) # => true Fixes #8436.
* | | Improve AR changelogCarlos Antonio da Silva2012-12-061-18/+17
| | |
* | | Remove method redefined warningsCarlos Antonio da Silva2012-12-061-1/+1
| | |
* | | Use CURRENT_TIMESTAMP since it has apparently better cross db supportCarlos Antonio da Silva2012-12-062-2/+2
| | | | | | | | | | | | | | | LOCALTIMESTAMP is not support by sqlite3, and travis was giving us these errors: https://travis-ci.org/rails/rails/jobs/3535241/#L570
* | | Run rake tasks all at once in rake dbs test for a speed upCarlos Antonio da Silva2012-12-061-12/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before: Finished tests in 56.245787s, 0.2133 tests/s, 0.0000 assertions/s. 12 tests, 0 assertions, 0 failures, 0 errors, 0 skips After: Finished tests in 42.401416s, 0.2830 tests/s, 0.0000 assertions/s. 12 tests, 0 assertions, 0 failures, 0 errors, 0 skips
* | | Merge pull request #8440 from steveklabnik/masterRafael Mendonça França2012-12-061-0/+3
|\ \ \ | | | | | | | | | | | | | | | | CHANGELOG for 78d5d6f [ci skip]
| * | | CHANGELOG for 78d5d6f.Steve Klabnik2012-12-061-0/+3
|/ / / | | | | | | | | | | | | It was pointed out by @giner that the CHANGELOG entry for https://github.com/rails/rails/commit/78d5d6f8688bb7c45ba9a3ef893682231130da3f wasn't included. Here it is.
* | | Merge pull request #8439 from joshsusser/fixesCarlos Antonio da Silva2012-12-063-5/+5
|\ \ \ | | | | | | | | Fixes issues in test_mysql2
| * | | Oracle needs table to check index existenceJosh Susser2012-12-061-3/+3
| | | |
| * | | convert time to stringJosh Susser2012-12-062-2/+2
| | | | | | | | | | | | why is this a Time to start with?
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-30 12:46:23 +0000