aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge pull request #6798 from kennyj/db-rake-structure-loadCarlos Antonio da Silva2012-06-205-19/+26
|\ \ | | | | | | Refactor db:structure:load task.
| * | Refactor db:structure:load task.kennyj2012-06-215-19/+26
| | |
* | | Merge pull request #4396 from kennyj/fix_4259Rafael Mendonça França2012-06-201-4/+8
|\ \ \ | |/ / |/| | | | | Fix GH #4259. When we execute schema dumper, we must remove table_name_prefix and table_name_suffix.
| * | Fix GH #4259. We must remove table_name_prefix and table_name_suffix, when ↵kennyj2012-01-101-4/+8
| | | | | | | | | | | | we execute schema dumper.
* | | Remove unneeded code since pluck is respecting joins nowRafael Mendonça França2012-06-191-12/+1
| | |
* | | Extract conditional to a method to avoid duplicationRafael Mendonça França2012-06-191-15/+19
| | | | | | | | | | | | Also use if/else block to not use short circuit return
* | | handle joins/includes correctly for pluck and calculation.Andrey Deryabin2012-06-191-1/+6
| | | | | | | | | | | | Fix #5990
* | | Refactor db:structure:dump task.kennyj2012-06-205-13/+41
| | |
* | | Refactor db:charset taskSimon Jefford2012-06-195-17/+25
| | | | | | | | | | | | | | | In a similar vein to Pat's work on create, drop etc, the db:charset task is now a one liner in databases.rake
* | | Refactor the conditionalsRafael Mendonça França2012-06-191-7/+7
| | |
* | | Validates_numericality_of is skipped when changing 0 to to non-empty stringAngelo capilleri2012-06-191-5/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This happens when A has_many many B and A accepts_nested_attributes B that has a numeric colum with initial 0 value. So a.update_attributes({:b_attributes => { :id => b.id, :numeric => 'foo' }}) passes the validation test but, the value of :numeric doesn't change. his commit forces that the update fails with the above conditions. Fixes #6393 Fixes #2331
* | | Refactor and use class_attributeCarlos Antonio da Silva2012-06-181-3/+5
| | |
* | | Remember the stored attributes in a config attribute.Joost Baaij2012-06-181-0/+12
| | | | | | | | | | | | | | | | | | This allows you to retrieve the list of attributes you've defined. Usable for e.g. selects in the view, or interators based on the attributes you wish to store in the serialized column.
* | | Refactor Relation#update a bit to avoid an extra Enumerator instanceCarlos Antonio da Silva2012-06-181-1/+1
| | |
* | | Make DatabaseTasks a module with real private methodsCarlos Antonio da Silva2012-06-181-13/+14
| | |
* | | Removing composed_of from ActiveRecord.Steve Klabnik2012-06-1810-343/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This feature adds a lot of complication to ActiveRecord for dubious value. Let's talk about what it does currently: class Customer < ActiveRecord::Base composed_of :balance, :class_name => "Money", :mapping => %w(balance amount) end Instead, you can do something like this: def balance @balance ||= Money.new(value, currency) end def balance=(balance) self[:value] = balance.value self[:currency] = balance.currency @balance = balance end Since that's fairly easy code to write, and doesn't need anything extra from the framework, if you use composed_of today, you'll have to add accessors/mutators like that. Closes #1436 Closes #2084 Closes #3807
* | | Change minimum (default) log level in PostgreSQL to warning.kennyj2012-06-181-1/+1
| | |
* | | Merge branch 'dp_docs'Carlos Antonio da Silva2012-06-181-0/+92
|\ \ \ | | | | | | | | | | | | Closes #6697
| * | | Documentation for where and where! methods on relations.David Paschich2012-06-181-0/+92
| | | | | | | | | | | | | | | | Based on examples seen in the Rails test suite. [ci skip]
* | | | fix invalid syntax and sqlite rake failing testsFrancesco Rodriguez2012-06-181-2/+3
| | | |
* | | | add :nodoc: to AR::Tasks and update to follow the coding conventionsFrancesco Rodriguez2012-06-184-208/+226
| | | |
* | | | check if sqlite3 file exists before removeFrancesco Rodriguez2012-06-181-3/+3
| | | |
* | | | Merge pull request #6761 from freelancing-god/db-rakeJon Leighton2012-06-176-156/+268
|\ \ \ \ | | | | | | | | | | Get logic out of db rake tasks, and into classes and objects
| * | | | Rails is the default (but now override able) source for environment and path.Pat Allan2012-06-172-15/+25
| | | | |
| * | | | Cleaning up after some warnings, adding slightly higher-level tests.Pat Allan2012-06-174-7/+13
| | | | |
| * | | | One line db:create/db:drop tasks.Pat Allan2012-06-172-55/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now isn't that better? And yes, I know that private has no impact on class methods - it's a visual distinction, not a technical one.
| * | | | db:drop and some of db:test:purge.Pat Allan2012-06-175-50/+63
| | | | |
| * | | | db:create for PostgreSQL pulled out into a class.Pat Allan2012-06-175-36/+42
| | | | |
| * | | | db:create for MySQL now much cleaner.Pat Allan2012-06-175-37/+110
| | | | |
| * | | | A beginning of sorts.Pat Allan2012-06-173-12/+21
| | | | | | | | | | | | | | | | | | | | Minimal implementation that supports db:create SQLite replacement
* | | | | Merge pull request #6759 from dylanahsmith/only-catch-standard-errorSantiago Pastorino2012-06-173-6/+6
|\ \ \ \ \ | | | | | | | | | | | | Avoid unnecessary catching of Exception instead of StandardError.
| * | | | | Avoid unnecessary catching of Exception instead of StandardError.Dylan Smith2012-06-173-6/+6
| | | | | |
* | | | | | Adding `load_config` task as dependency.Cairo Noleto2012-06-161-14/+14
|/ / / / / | | | | | | | | | | | | | | | Without this, some tasks won't load configurations like custom engines.
* | | | | Don't read csv file during executing db:fixtures:load.kennyj2012-06-171-1/+1
| | | | |
* | | | | no need to define methods in included hookJon Leighton2012-06-151-13/+13
| | | | |
* | | | | Fix deprecation warnings when using ↵Jon Leighton2012-06-151-3/+8
| | | | | | | | | | | | | | | | | | | | config.active_record.mass_assignment_sanitizer=
* | | | | Fix observers with AR::ModelJon Leighton2012-06-153-3/+6
| | | | |
* | | | | Fix config.active_record.whitelist_attributes with AR::ModelJon Leighton2012-06-152-3/+13
| | | | |
* | | | | Simplify AR configuration code.Jon Leighton2012-06-1521-260/+343
| | | | | | | | | | | | | | | | | | | | | | | | | Get rid of ActiveModel::Configuration, make better use of ActiveSupport::Concern + class_attribute, etc.
* | | | | refactorJon Leighton2012-06-151-11/+5
| |/ / / |/| | |
* | | | Merge pull request #6713 from etehtsea/activerecord-uuidRafael Mendonça França2012-06-142-3/+8
|\ \ \ \ | | | | | | | | | | | | | | | Add uuid datatype support to PostgreSQL adapter
| * | | | Add uuid type support to PostgreSQL adapterKonstantin Shabanov2012-06-142-3/+8
| | | | |
* | | | | Merge pull request #6738 from steveklabnik/masterXavier Noria2012-06-143-21/+6
|\ \ \ \ \ | |_|_|_|/ |/| | | | Remove update_attribute.
| * | | | Remove update_attribute.Steve Klabnik2012-06-143-21/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Historically, update_attribute and update_attributes are similar, but with one big difference: update_attribute does not run validations. These two methods are really easy to confuse given their similar names. Therefore, update_attribute is being removed in favor of update_column. See the thread on rails-core here: https://groups.google.com/forum/?fromgroups#!topic/rubyonrails-core/BWPUTK7WvYA
* | | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-06-141-1/+1
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fixed typo.chrismcc2012-06-111-1/+1
| | | | |
* | | | | Merge pull request #6676 from aurelian/masterRafael Mendonça França2012-06-121-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | Don't assign the attributes if the list is empty
| * | | | Don't assign the attributes if the list is empty.Aurelian Oancea2012-06-081-1/+1
| | | | |
* | | | | Merge pull request #6492 from pmahoney/fair-connection-pool2Rafael Mendonça França2012-06-111-37/+174
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fair connection pool2 Conflicts: activerecord/test/cases/associations/eager_test.rb
| * | | | | Make connection pool fair with respect to waiting threads.Patrick Mahoney2012-05-251-37/+174
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The core of this fix is a threadsafe, fair Queue class. It is very similar to Queue in stdlib except that it supports waiting with a timeout. The issue this solves is that if several threads are contending for database connections, an unfair queue makes is possible that a thread will timeout even while other threads successfully acquire and release connections. A fair queue means the thread that has been waiting the longest will get the next available connection. This includes a few test fixes to avoid test ordering issues that cropped up during development of this patch.