aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | fixing typo in from documentationSubba Rao Pasupuleti2012-06-281-2/+2
|/ /
* | Remove some aggregation tests related to composed_ofCarlos Antonio da Silva2012-06-282-21/+0
| | | | | | | | | | | | | | | | | | | | | | Since composed_of was removed in 051747449e7afc817c599e4135bc629d4de064eb, these tests were working "by mistake", due to the matching "address" string in the error message, but with a different error message than the expected multiparameter assignment error. Since "address" is not an attribute from Customer anymore, the error was "undefined method klass for nil", where nil was supposed to be the column object.
* | Move newly added multiparameter tests to base_test fileCarlos Antonio da Silva2012-06-282-35/+36
| | | | | | | | | | All other multiparameter assignment tests are in the same file, so it makes sense to have all of them here.
* | Bump minitest to 3.2.0Santiago Pastorino2012-06-281-1/+1
| |
* | Merge pull request #6890 from yahonda/address_test_setting_time_attribute_oracleCarlos Antonio da Silva2012-06-281-0/+2
|\ \ | | | | | | Skip test_setting_time_attribute for Oracle database
| * | Skip test_setting_time_attribute because Oracle databaseYasuo Honda2012-06-281-0/+2
|/ / | | | | | | does not have TIME data type.
* | Merge pull request #6886 from nertzy/faster_stringify_and_symbolize_keysCarlos Antonio da Silva2012-06-271-1/+1
|\ \ | | | | | | Speed up Hash#transform_keys using Hash#each_key
| * | Speed up Hash#transform_keys using Hash#each_keyGrant Hutchins2012-06-271-1/+1
|/ / | | | | See https://gist.github.com/3007749 for justification
* | Merge pull request #6878 from masarakki/masterRafael Mendonça França2012-06-275-0/+41
|\ \ | | | | | | | | | | | | fix bug in limit of enum columns of mysql Closes #6432
| * | fix: limit of enum columns of mysqlYamada Masaki2012-06-285-0/+41
|/ /
* | Merge pull request #6873 from mauricio/fix-for-6825Rafael Mendonça França2012-06-273-16/+58
|\ \ | | | | | | Fixes #6825, adds tests covering cases and error possibilities
| * | Fixes #6825, adds tests covering cases and error possibilities, also changes ↵Mauricio Linhares2012-06-273-16/+58
|/ / | | | | | | SQLite3 driver to correctly generate a time column instead of datetime
* | Add changelog entry for polymorphic option in model generatorCarlos Antonio da Silva2012-06-271-0/+10
| | | | | | | | | | Pull request #6856, merged in 52f6e47682003c83b0466bf5e140ee302498a226. [ci skip]
* | Merge pull request #6880 from kennyj/fix_20120628Carlos Antonio da Silva2012-06-272-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.kennyj2012-06-281-1/+1
| | |
| * | Fix wrong testcase. This is a testcase for db:test:prepare.kennyj2012-06-281-1/+1
| | |
* | | Merge pull request #6856 from lexmag/polymorphic_generatorsCarlos Antonio da Silva2012-06-274-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 generatorAleksey Magusev2012-06-274-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_oracleRafael Mendonça França2012-06-271-1/+1
|\ \ \ \ | |_|/ / |/| | | Address test_basic_model test error with Oracle enhanced adapter
| * | | Address test_basic_model test error with Oracle enhanced adapterYasuo Honda2012-06-271-1/+1
|/ / / | | | | | | | | | which ids start from 1000 as a default.
* | | Merge pull request #6863 from bcardarella/asyn-actionmailer-guidesXavier Noria2012-06-261-0/+31
|\ \ \ | | | | | | | | Guide update for Async ActionMailer
| * | | Guide update for Async ActionMailerBrian Cardarella2012-06-261-0/+31
| | | |
* | | | Refactor attribute method matcher to use Hash#fetchCarlos Antonio da Silva2012-06-261-5/+3
| | | |
* | | | Cleanup attribute methods a bit, use map! when accepting *argsCarlos Antonio da Silva2012-06-261-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_mergeCarlos Antonio da Silva2012-06-261-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 possibleCarlos Antonio da Silva2012-06-262-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 hashCarlos Antonio da Silva2012-06-263-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_oracleRafael Mendonça França2012-06-261-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 Menge2012-06-261-20/+8
|/ / / / | | | | | | | | | | | | | | | | | | | | test starts with a clean slate. Fixes #6867.
* | | | Remove file added by mistake at 88230b7cf77181c2088cce3314493caa9b3ac1fbRafael Mendonça França2012-06-261-1/+0
| | | |
* | | | Merge pull request #6859 from bogdan/deprecate_monkey_patchJosé Valim2012-06-265-1/+23
|\ \ \ \ | | | | | | | | | | AS::Callbacks: deprecate monkey patch code
| * | | | AS::Callbacks: deprecate monkey patch codeBogdan Gusiev2012-06-265-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_detailsJosé Valim2012-06-261-2/+1
|\ \ \ \ \ | | | | | | | | | | | | :update_details method no more exists on @lookup_context
| * | | | | :update_details method no more exists on @lookup_contextAkira Matsuda2012-06-261-2/+1
| |/ / / / | | | | | | | | | | | | | | | the method has gone in this commit: 119e9e2dafb0cdc5b85613b730333679aef534af
* | | | | Merge pull request #6839 from bcardarella/async-actionmailerAaron Patterson2012-06-268-0/+86
|\ \ \ \ \ | |/ / / / |/| | | | Async actionmailer
| * | | | Some final syntax fixesBrian Cardarella2012-06-242-3/+2
| | | | |
| * | | | Better documentation for ActionMailer.asyncBrian Cardarella2012-06-241-2/+2
| | | | |
| * | | | Better documentation for ActionMailer.asyncBrian Cardarella2012-06-241-0/+4
| | | | |
| * | | | Forcing the message sending is no longer necessaryBrian Cardarella2012-06-242-16/+1
| | | | |
| * | | | Support for custom queues on the mailerBrian Cardarella2012-06-233-18/+30
| | | | | | | | | | | | | | | | | | | | Credit goes to *Aaron Patterson* (tenderlove)
| * | | | Removed unecessary splattingBrian Cardarella2012-06-231-1/+1
| | | | |
| * | | | Use Delegator for a cleaner QueuedMessage classBrian Cardarella2012-06-231-13/+8
| | | | | | | | | | | | | | | | | | | | Credit goes to *Nicolás Sanguinetti* (foca) for this suggestion
| * | | | Updated ChangelogsBrian Cardarella2012-06-232-0/+5
| |/ / /
| * | | Force message delivery despite asyncBrian Cardarella2012-06-232-2/+16
| | | |
| * | | Add commented out ActionMailer async config optionBrian Cardarella2012-06-231-0/+3
| | | |
| * | | AsyncMailer documentionBrian Cardarella2012-06-232-0/+3
| | | |
| * | | Asynchronous ActionMailerBrian Cardarella2012-06-235-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 applicableCarlos Antonio da Silva2012-06-251-7/+12
| | | | | | | | | | | | | | | | | | | | Try to use more destructive methods on *args when applicable, to avoid creating new objects.
* | | | Merge pull request #6544 from flexoid/issue-6431Piotr Sarnacki2012-06-253-6/+14
|\ \ \ \ | | | | | | | | | | exists?(false) returns false
| * | | | exists?(false) returns falseEgor Lynko2012-06-253-6/+14
| | | | | | | | | | | | | | | | | | | | `FinderMethods#exists?` finder method now returns *false* with the *false* argument