Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Update psql adapter to rename a default pkey sequence during rename_table. | Robb Kidd | 2012-07-10 | 2 | -0/+27 |
| | |||||
* | Merge pull request #7025 from rustygeldmacher/select_options_valid_html | Rafael Mendonça França | 2012-07-10 | 2 | -5/+18 |
|\ | | | | | Select options valid html | ||||
| * | Fixed bug creating invalid HTML in select options | Rusty Geldmacher | 2012-07-10 | 2 | -5/+18 |
|/ | | | | | | | | | When a select tag is created for a field with errors, and that select tag has :prompt or :include_blank options, then the inserted first option will errantly have a <div class="field_with_errors"> wrapping it. See https://github.com/rails/rails/issues/7017 | ||||
* | Merge pull request #7015 from sikachu/3-2-stable-code-style | Xavier Noria | 2012-07-09 | 1 | -8/+8 |
|\ | | | | | Update coding convention from master | ||||
| * | Update coding convention from master | Prem Sichanugrist | 2012-07-09 | 1 | -8/+8 |
|/ | |||||
* | Merge pull request #6985 from sidonath/disable-query-cache-for-locks | Rafael Mendonça França | 2012-07-06 | 2 | -1/+17 |
| | | | | | | Disable query cache for lock queries Conflicts: activerecord/lib/active_record/connection_adapters/abstract/query_cache.rb | ||||
* | Since Rails 3.2, use layout false to render no layout | José Valim | 2012-07-06 | 1 | -1/+1 |
| | |||||
* | Merge pull request #6973 from route/wrapped_keys_in_log_for_3_2 | Rafael Mendonça França | 2012-07-05 | 2 | -1/+10 |
|\ | | | | | MIssed backport for 3.2 | ||||
| * | Show in log correct wrapped keys | Dmitry Vorotilin | 2012-07-05 | 2 | -1/+10 |
|/ | |||||
* | Fix NumberHelper options wrapping to prevent verbatim blocks being rendered ↵ | Mark J. Titorenko | 2012-07-02 | 1 | -90/+159 |
| | | | | | | | | | | instead of line continuations. While I'm at it, wrap long comment lines consistently. Conflicts: actionpack/lib/action_view/helpers/number_helper.rb There was just one conflict related to the addition of the :format option to number_to_percentage. | ||||
* | Merge pull request #6935 from frodsan/b1e509ad7a8c8264544f10f4666705cd806b5408 | Rafael Mendonça França | 2012-07-02 | 2 | -15/+31 |
|\ | | | | | Backport #3329 to 3-2-stable | ||||
| * | Backport #3329 to 3-2-stable | Francesco Rodriguez | 2012-07-02 | 2 | -15/+31 |
|/ | | | | | | | Fix bug with autosave collection association on new record with a marked for destroy record in autosave collection. Fixes #6918. | ||||
* | make sure the inflection rules are loaded when cherry-picking ↵ | Xavier Noria | 2012-06-29 | 3 | -0/+5 |
| | | | | active_support/core_ext/string/inflections.rb [fixes #6884] | ||||
* | Merge pull request #6878 from masarakki/master | Rafael Mendonça França | 2012-06-28 | 5 | -0/+41 |
| | | | | | | fix bug in limit of enum columns of mysql Closes #6432 | ||||
* | Merge pull request #6900 from cbandy/issue-6898 | Carlos Antonio da Silva | 2012-06-28 | 1 | -0/+2 |
| | | | | | | Require URI in ConnectionSpecification Conflicts: activerecord/lib/active_record/connection_adapters/connection_specification.rb | ||||
* | Merge pull request #6857 from rsutphin/as_core_ext_time_missing_require | Carlos Antonio da Silva | 2012-06-25 | 1 | -0/+1 |
| | | | | Missing require breaks Time.=== when selectively loading ActiveSupport core_exts in 3.2.4+ | ||||
* | Ensure Arel columns are typecasted properly when grouping with calculation | Carlos Antonio da Silva | 2012-06-25 | 2 | -5/+9 |
| | | | | | | | | Fix build issue with postgresql. Conflicts: activerecord/lib/active_record/relation/calculations.rb activerecord/test/cases/calculations_test.rb | ||||
* | Merge pull request #6842 from ernie/handle-non-strings-in-grouped-calculations | Rafael Mendonça França | 2012-06-24 | 2 | -6/+22 |
| | | | | | | Stop assuming strings for grouped calculations Conflicts: activerecord/lib/active_record/relation/calculations.rb | ||||
* | Remove waning of unused variable | Rafael Mendonça França | 2012-06-21 | 1 | -1/+0 |
| | |||||
* | Merge branch 'acapilleri-update_nested_attributes' | Rafael Mendonça França | 2012-06-21 | 2 | -5/+25 |
| | | | | | | | Closes #6675 Conflicts: activerecord/lib/active_record/attribute_methods/dirty.rb | ||||
* | Merge pull request #6758 from caironoleto/master | José Valim | 2012-06-18 | 1 | -13/+13 |
| | | | | | | Fixing load config in some tasks Conflicts: activerecord/lib/active_record/railties/databases.rake | ||||
* | Merge pull request #6649 from route/logger_in_metal_3_2 | Carlos Antonio da Silva | 2012-06-18 | 2 | -2/+21 |
|\ | | | | | Logger in metal backport for 3.2 | ||||
| * | Added test for case when view doesn't have logger method when using ↵ | Dmitry Vorotilin | 2012-06-16 | 1 | -0/+17 |
| | | | | | | | | ActionController::Metal controller. | ||||
| * | ActionController::Metal doesn't have logger method, check it and then delegate | Dmitry Vorotilin | 2012-06-16 | 1 | -2/+4 |
| | | |||||
* | | Merge pull request #6742 from steveklabnik/deprecate_composed_of | Rafael Mendonça França | 2012-06-18 | 5 | -7/+24 |
|\ \ | | | | | | | Deprecate composed of | ||||
| * | | Deprecating composed_of in ActiveRecord | Steve Klabnik | 2012-06-18 | 5 | -7/+24 |
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. This feature will be removed in Rails 4. | ||||
* | | Use strict_args_position! if available from Thor | José Valim | 2012-06-18 | 1 | -0/+1 |
| | | |||||
* | | Merge pull request #6764 from frodsan/patch-4 | Rafael Mendonça França | 2012-06-17 | 1 | -2/+2 |
|\ \ | | | | | | | bump AS deprecation_horizon to 4.0 | ||||
| * | | bump AS deprecation_horizon to 4.0 | Francesco Rodríguez | 2012-06-17 | 1 | -2/+2 |
|/ / | |||||
* | | Merge pull request #6756 from arunagw/build_fix_ap | Rafael Mendonça França | 2012-06-16 | 1 | -1/+1 |
|\ \ | | | | | | | Build fix actionpack | ||||
| * | | It should also include text/css => Build Fix | Arun Agrawal | 2012-06-16 | 1 | -1/+1 |
|/ / | |||||
* / | Merge pull request #6752 from steveklabnik/fix_5680 | Rafael Mendonça França | 2012-06-16 | 3 | -4/+21 |
|/ | | | | Respect absolute paths in compute_source_path. | ||||
* | Remove unneded tests. | Rafael Mendonça França | 2012-06-14 | 1 | -11/+1 |
| | | | | | | | Before b081f6b59fb3f15d12043072ad9b331ffd2bc56e, this test are asserting that update_attribute does the dirty tracking. Since we deprecated this method and update_column write in the database directly this tests will always fail. | ||||
* | registers the deprecation of update_attribute in the CHANGELOG | Xavier Noria | 2012-06-14 | 1 | -0/+8 |
| | |||||
* | Merge pull request #6739 from steveklabnik/3-2-stable | Xavier Noria | 2012-06-14 | 7 | -17/+41 |
|\ | | | | | Deprecate update_attribute | ||||
| * | Deprecate update_attribute. | Steve Klabnik | 2012-06-14 | 7 | -17/+41 |
| | | | | | | | | | | | | | | | | | | | | | | 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 deprecated in favor of update_column, and will be removed in Rails 4. See the discussion on rails-core here: https://groups.google.com/d/topic/rubyonrails-core/BWPUTK7WvYA/discussion | ||||
* | | adding a test for #6459 | Aaron Patterson | 2012-06-14 | 1 | -0/+10 |
|/ | |||||
* | ActionController::Caching depends on RackDelegation and ↵ | Santiago Pastorino | 2012-06-13 | 2 | -0/+35 |
| | | | | AbstractController::Callbacks | ||||
* | Merge pull request #6716 from frodsan/patch-2 | Rafael Mendonça França | 2012-06-12 | 1 | -1/+1 |
|\ | | | | | Fix Build | ||||
| * | Fix build | Francesco Rodríguez | 2012-06-12 | 1 | -1/+1 |
|/ | |||||
* | Merge branch '3-2-stable-rel' into 3-2-stable | Aaron Patterson | 2012-06-12 | 20 | -16/+58 |
|\ | | | | | | | | | | | | | | | | | | | * 3-2-stable-rel: updating changelogs bumping version numbers updating changelogs with security fixes updating changelogs Array parameters should not contain nil values. Additional fix for CVE-2012-2661 | ||||
| * | updating changelogs | Aaron Patterson | 2012-06-12 | 7 | -7/+11 |
| | | |||||
| * | bumping version numbers | Aaron Patterson | 2012-06-11 | 9 | -9/+9 |
| | | |||||
| * | updating changelogs with security fixes | Aaron Patterson | 2012-06-11 | 2 | -0/+6 |
| | | |||||
| * | updating changelogs | Aaron Patterson | 2012-06-11 | 5 | -0/+20 |
| | | |||||
| * | Array parameters should not contain nil values. | Aaron Patterson | 2012-06-11 | 2 | -2/+8 |
| | | |||||
| * | Additional fix for CVE-2012-2661 | Ernie Miller | 2012-06-11 | 2 | -3/+9 |
| | | | | | | | | | | | | | | | | While the patched PredicateBuilder in 3.1.5 prevents a user from specifying a table name using the `table.column` format, it doesn't protect against the nesting of hashes changing the table context in the next call to build_from_hash. This fix covers this case as well. | ||||
* | | Merge pull request #6715 from arunagw/add_asserts | Rafael Mendonça França | 2012-06-12 | 1 | -2/+2 |
| | | | | | | | | Need a assert here in tests | ||||
* | | Merge pull request #6676 from aurelian/master | Rafael Mendonça França | 2012-06-12 | 2 | -1/+5 |
| | | | | | | | | Don't assign the attributes if the list is empty | ||||
* | | Merge pull request #6682 from acapilleri/dup_validation_fix_backport_for_1_9_3 | Rafael Mendonça França | 2012-06-12 | 3 | -1/+16 |
|\ \ | |/ |/| | Dup validation fix backport for 1 9 3 |