Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | Skip test_setting_time_attribute because Oracle database | Yasuo Honda | 2012-06-28 | 1 | -0/+2 | |
|/ / / / | | | | | | | | | | | | | does not have TIME data type. | |||||
* | | | | Merge pull request #6886 from nertzy/faster_stringify_and_symbolize_keys | Carlos Antonio da Silva | 2012-06-27 | 1 | -1/+1 | |
|\ \ \ \ | | | | | | | | | | | Speed up Hash#transform_keys using Hash#each_key | |||||
| * | | | | Speed up Hash#transform_keys using Hash#each_key | Grant Hutchins | 2012-06-27 | 1 | -1/+1 | |
|/ / / / | | | | | | | | | See https://gist.github.com/3007749 for justification | |||||
* | | | | Merge pull request #6878 from masarakki/master | Rafael Mendonça França | 2012-06-27 | 5 | -0/+41 | |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | fix bug in limit of enum columns of mysql Closes #6432 | |||||
| * | | | | fix: limit of enum columns of mysql | Yamada Masaki | 2012-06-28 | 5 | -0/+41 | |
|/ / / / | ||||||
* | | | | Merge pull request #6873 from mauricio/fix-for-6825 | Rafael Mendonça França | 2012-06-27 | 3 | -16/+58 | |
|\ \ \ \ | |_|/ / |/| | | | Fixes #6825, adds tests covering cases and error possibilities | |||||
| * | | | Fixes #6825, adds tests covering cases and error possibilities, also changes ↵ | Mauricio Linhares | 2012-06-27 | 3 | -16/+58 | |
|/ / / | | | | | | | | | | SQLite3 driver to correctly generate a time column instead of datetime | |||||
* | | | Add changelog entry for polymorphic option in model generator | Carlos Antonio da Silva | 2012-06-27 | 1 | -0/+10 | |
| | | | | | | | | | | | | | | | Pull request #6856, merged in 52f6e47682003c83b0466bf5e140ee302498a226. [ci skip] | |||||
* | | | Merge pull request #6880 from kennyj/fix_20120628 | Carlos Antonio da Silva | 2012-06-27 | 2 | -2/+2 | |
|\ \ \ | | | | | | | | | Fix wrong testcase for db:test:prepare, and wrong environment in AR rake task. | |||||
| * | | | Fix wrong environment. env should be RAILS_ENV or test. | kennyj | 2012-06-28 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Fix wrong testcase. This is a testcase for db:test:prepare. | kennyj | 2012-06-28 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Merge pull request #6856 from lexmag/polymorphic_generators | Carlos Antonio da Silva | 2012-06-27 | 4 | -14/+42 | |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add polymorphic option to model generator For instance, $ rails g model Product supplier:references{polymorphic} generate model with `belongs_to :supplier, polymorphic: true` association and appropriate migration. | |||||
| * | | | | Add polymorphic option to model generator | Aleksey Magusev | 2012-06-27 | 4 | -14/+42 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For instance, $ rails g model Product supplier:references{polymorphic} generate model with `belongs_to :supplier, polymorphic: true` association and appropriate migration. Also fix model_generator_test.rb#L196 and #L201 | |||||
* | | | | | Merge pull request #6879 from yahonda/address_basic_inclusion_model_test_oracle | Rafael Mendonça França | 2012-06-27 | 1 | -1/+1 | |
|\ \ \ \ \ | |_|/ / / |/| | | | | Address test_basic_model test error with Oracle enhanced adapter | |||||
| * | | | | Address test_basic_model test error with Oracle enhanced adapter | Yasuo Honda | 2012-06-27 | 1 | -1/+1 | |
|/ / / / | | | | | | | | | | | | | which ids start from 1000 as a default. | |||||
* | | | | Merge pull request #6863 from bcardarella/asyn-actionmailer-guides | Xavier Noria | 2012-06-26 | 1 | -0/+31 | |
|\ \ \ \ | | | | | | | | | | | Guide update for Async ActionMailer | |||||
| * | | | | Guide update for Async ActionMailer | Brian Cardarella | 2012-06-26 | 1 | -0/+31 | |
| | | | | | ||||||
* | | | | | Refactor attribute method matcher to use Hash#fetch | Carlos Antonio da Silva | 2012-06-26 | 1 | -5/+3 | |
| | | | | | ||||||
* | | | | | Cleanup attribute methods a bit, use map! when accepting *args | Carlos Antonio da Silva | 2012-06-26 | 1 | -17/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we're dealing with a new array instance, it's safe to use map! and we avoid an extra array object. Also remove the symbolize_keys! from AttributeMethodMatcher, since it's an internal class that always receives symbol keys from the prefix/suffix methods implementations. | |||||
* | | | | | Use default + merge! instead of reverse_merge | Carlos Antonio da Silva | 2012-06-26 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Since we're not directly requiring the reverse_merge extension, we can avoid another hash creation by using default_hash.merge! instead. | |||||
* | | | | | Set hash value instead of merge a single key, and use flatten! if possible | Carlos Antonio da Silva | 2012-06-26 | 2 | -4/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There's no need to create two extra hashes with options.merge(another_hash), with the goal of setting only one value, so lets just set it. Also refactor validates_each to use _merge_attributes, like other validates_* helpers do. | |||||
* | | | | | Ensure Array#to_sentence does not modify given hash | Carlos Antonio da Silva | 2012-06-26 | 3 | -8/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Also simplify I18n logic for Array#to_sentence, doing only one lookup for all keys and using merge!, instead of one lookup for each option key. | |||||
* | | | | | Merge pull request #6870 from erichmenge/patch/base_test_oracle | Rafael Mendonça França | 2012-06-26 | 1 | -20/+8 | |
|\ \ \ \ \ | | | | | | | | | | | | | Setup each test with default ActiveRecord timezone settings to ensure ea... | |||||
| * | | | | | Setup each test with default ActiveRecord timezone settings to ensure each ↵ | Erich Menge | 2012-06-26 | 1 | -20/+8 | |
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | test starts with a clean slate. Fixes #6867. | |||||
* | | | | | Remove file added by mistake at 88230b7cf77181c2088cce3314493caa9b3ac1fb | Rafael Mendonça França | 2012-06-26 | 1 | -1/+0 | |
| | | | | | ||||||
* | | | | | Merge pull request #6859 from bogdan/deprecate_monkey_patch | José Valim | 2012-06-26 | 5 | -1/+23 | |
|\ \ \ \ \ | | | | | | | | | | | | | AS::Callbacks: deprecate monkey patch code | |||||
| * | | | | | AS::Callbacks: deprecate monkey patch code | Bogdan Gusiev | 2012-06-26 | 5 | -1/+23 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deprecate usage of filter object with #before and #after methods as around callback | |||||
* | | | | | | Merge pull request #6858 from amatsuda/av_no_such_method_update_details | José Valim | 2012-06-26 | 1 | -2/+1 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | :update_details method no more exists on @lookup_context | |||||
| * | | | | | | :update_details method no more exists on @lookup_context | Akira Matsuda | 2012-06-26 | 1 | -2/+1 | |
| |/ / / / / | | | | | | | | | | | | | | | | | | | the method has gone in this commit: 119e9e2dafb0cdc5b85613b730333679aef534af | |||||
* | | | | | | Merge pull request #6839 from bcardarella/async-actionmailer | Aaron Patterson | 2012-06-26 | 8 | -0/+86 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | Async actionmailer | |||||
| * | | | | | Some final syntax fixes | Brian Cardarella | 2012-06-24 | 2 | -3/+2 | |
| | | | | | | ||||||
| * | | | | | Better documentation for ActionMailer.async | Brian Cardarella | 2012-06-24 | 1 | -2/+2 | |
| | | | | | | ||||||
| * | | | | | Better documentation for ActionMailer.async | Brian Cardarella | 2012-06-24 | 1 | -0/+4 | |
| | | | | | | ||||||
| * | | | | | Forcing the message sending is no longer necessary | Brian Cardarella | 2012-06-24 | 2 | -16/+1 | |
| | | | | | | ||||||
| * | | | | | Support for custom queues on the mailer | Brian Cardarella | 2012-06-23 | 3 | -18/+30 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Credit goes to *Aaron Patterson* (tenderlove) | |||||
| * | | | | | Removed unecessary splatting | Brian Cardarella | 2012-06-23 | 1 | -1/+1 | |
| | | | | | | ||||||
| * | | | | | Use Delegator for a cleaner QueuedMessage class | Brian Cardarella | 2012-06-23 | 1 | -13/+8 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Credit goes to *Nicolás Sanguinetti* (foca) for this suggestion | |||||
| * | | | | | Updated Changelogs | Brian Cardarella | 2012-06-23 | 2 | -0/+5 | |
| |/ / / / | ||||||
| * | | | | Force message delivery despite async | Brian Cardarella | 2012-06-23 | 2 | -2/+16 | |
| | | | | | ||||||
| * | | | | Add commented out ActionMailer async config option | Brian Cardarella | 2012-06-23 | 1 | -0/+3 | |
| | | | | | ||||||
| * | | | | AsyncMailer documention | Brian Cardarella | 2012-06-23 | 2 | -0/+3 | |
| | | | | | ||||||
| * | | | | Asynchronous ActionMailer | Brian Cardarella | 2012-06-23 | 5 | -0/+66 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Any ActionMailer class can be set to render and delier messages using the new Rails Queue. Some of this work was borrowed (with permission) from Nick Plante's (zapnap) reqsue_mailer gem. | |||||
* | | | | | Use args.flatten! in query methods when applicable | Carlos Antonio da Silva | 2012-06-25 | 1 | -7/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | Try to use more destructive methods on *args when applicable, to avoid creating new objects. | |||||
* | | | | | Merge pull request #6544 from flexoid/issue-6431 | Piotr Sarnacki | 2012-06-25 | 3 | -6/+14 | |
|\ \ \ \ \ | | | | | | | | | | | | | exists?(false) returns false | |||||
| * | | | | | exists?(false) returns false | Egor Lynko | 2012-06-25 | 3 | -6/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | | `FinderMethods#exists?` finder method now returns *false* with the *false* argument | |||||
* | | | | | | Ensure Arel columns are typecasted properly when grouping with calculation | Carlos Antonio da Silva | 2012-06-25 | 2 | -9/+13 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Fix build issue with postgresql. | |||||
* | | | | | | 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+ | |||||
| * | | | | | | Add missing require. | Rhett Sutphin | 2012-06-25 | 1 | -0/+1 | |
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you selectively require core_exts (e.g., require 'active_support/core_ext/string'), it is possible for 'active_support/core_ext/time/calculations' to be required when `ActiveSupport::TimeWithZone` is not available. If this happens, the next call to Time.=== will fail with a NameError. | |||||
* | | | | | | Merge pull request #6855 from kennyj/refactor_database_tasks | Rafael Mendonça França | 2012-06-25 | 2 | -21/+46 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | Refactor and improve database tasks. | |||||
| * | | | | | Move to db:structure namespace, because these methods are global. | kennyj | 2012-06-26 | 1 | -8/+9 | |
| | | | | | |