aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge pull request #15755 from sgrif/sg-dirty-refactorRafael Mendonça França2014-06-163-9/+26
|\ \ \ \ \ | | | | | | | | | | | | Refactor in-place dirty checking to use the attribute object
| * | | | | Refactor in-place dirty checking to use the attribute objectSean Griffin2014-06-163-9/+26
| | | | | |
* | | | | | Add important plugins as one of the tasks of the releaseRafael Mendonça França2014-06-161-0/+12
|/ / / / / | | | | | | | | | | | | | | | [ci skip]
* | | | | Merge pull request #15684 from aditya-kapoor/add-tests-for-assert-valid-keysRafael Mendonça França2014-06-161-0/+15
|\ \ \ \ \ | | | | | | | | | | | | Add test cases for Hash#asset_valid_keys
| * | | | | Add test cases for Hash#asset_valid_keysAditya Kapoor2014-06-141-0/+15
| | | | | |
* | | | | | Merge pull request #13963 from lucas-clemente/pending_migrationsRafael Mendonça França2014-06-162-5/+58
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Skip migration check if adapter doesn't support it
| * | | | | skip migration check if adapter doesn't support itLucas Clemente2014-04-192-5/+58
| | | | | |
* | | | | | Merge pull request #15744 from mmozuras/special_keys_setYves Senn2014-06-161-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Change Http::Cache::SPECIAL_KEYS from Array to Set
| * | | | | | Change Http::Cache::SPECIAL_KEYS from Array to SetMindaugas Mozūras2014-06-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Slightly improves performance, for example, a simple benchmark: ```ruby require 'benchmark/ips' require 'set' SPECIAL_KEYS = %w[extras no-cache max-age public must-revalidate] SPECIAL_KEYS_SET = Set.new(SPECIAL_KEYS) directive = 'must-revalidate' Benchmark.ips do |x| x.report('array') { SPECIAL_KEYS.include?(directive) } x.report('set') { SPECIAL_KEYS_SET.include?(directive) } end ``` Output: ``` ------------------------------------- array 67926 i/100ms set 74054 i/100ms ------------------------------------- array 2318423.4 (±2.8%) i/s - 11615346 in 5.014899s set 3387981.8 (±4.7%) i/s - 16958366 in 5.019355s ```
* | | | | | | Merge pull request #15743 from tgxworld/remove_unused_parametersYves Senn2014-06-161-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Remove unused parameter.
| * | | | | | | Remove unused parameter.Guo Xiang Tan2014-06-151-1/+1
| |/ / / / / /
* | | | | | | Merge pull request #15722 from akshay-vishnoi/spell-correctZachary Scott2014-06-158-11/+11
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | [ci skip] /mysql/i -> MySQL, Spell correct in continuation to #15555
| * | | | | | | /mysql/i -> MySQL, Spell correct in continuation to #15555Akshay Vishnoi2014-06-148-11/+11
| | | | | | | |
* | | | | | | | Merge pull request #15723 from akshay-vishnoi/sql-correctionZachary Scott2014-06-152-3/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | [ci skip] Use `an` for SQL
| * | | | | | | | [ci skip] Use `an` for SQLAkshay Vishnoi2014-06-142-3/+3
| |/ / / / / / /
* | | | | | | | Merge pull request #15748 from akshay-vishnoi/activemodelRafael Mendonça França2014-06-151-3/+4
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Correct typo, add test for validates_absence_of
| * | | | | | | | Correct typo, add test for validates_absence_of, correct method namesAkshay Vishnoi2014-06-161-3/+4
|/ / / / / / / /
* | | | | | | | Merge pull request #15745 from mmozuras/remove_unused_separators_paramRafael Mendonça França2014-06-151-2/+2
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Remove unused param 'separators' from RouteSet#build_path
| * | | | | | | Remove unused param 'separators' from RouteSet#build_pathMindaugas Mozūras2014-06-151-2/+2
|/ / / / / / /
* | | | | | | Merge pull request #15741 from sgrif/sg-update-deprecation-messageYves Senn2014-06-151-8/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Change the deprecation warning on `serialized_attributes`
| * | | | | | | Change the deprecation warning on `serialized_attributes`Sean Griffin2014-06-151-8/+2
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | to "without replacement"
* | | | / / / allow preview interceptors to be registered through `config.action_mailer`.Yves Senn2014-06-154-1/+41
| |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was partially broken because `preview_interceptors=` just assigned the raw values, whithout going through `register_preview_interceptor`. Now the Action Mailer railtie takes care of the `preview_interceptors` option. This commit is a partial revert of: Revert "Merge pull request #15739 from y-yagi/correct_doc_for_action_mailer_base" This reverts commit a15704d7f35f17d34d0118546799141d6f853656, reversing changes made to 1bd12a8609d275ad75fcc4b622ca4f5b32dc76be. /cc @kuldeepaggarwal @y-yagi
* | | | | | Merge pull request #15739 from y-yagi/correct_doc_for_action_mailer_baseYves Senn2014-06-151-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | [ci skip] correct doc for `register_preview_interceptor`
| * | | | | | [ci skip] correct doc for `register_preview_interceptor`yuuji.yaginuma2014-06-151-1/+1
| | | | | | |
* | | | | | | Merge pull request #15738 from kuldeepaggarwal/remove-unwanted-codeYves Senn2014-06-152-13/+4
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | remove unnecessary calling of `I18n.backend.store_translations('empty', {})`
| * | | | | | remove unnecessary calling of `I18n.backend.store_translations(empty, {})`Kuldeep Aggarwal2014-06-152-13/+4
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Clean up I18n stored translations after test continuation with #15714
* | | | | | Merge pull request #15737 from kuldeepaggarwal/fix-buildYves Senn2014-06-141-0/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | reload I18n as we are defining `:ts` locale in `test_number_to_human_with_custom_translation_scope`
| * | | | | | reload I18n as we are defining `:ts` localeKuldeep Aggarwal2014-06-151-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | fix build- see https://travis-ci.org/rails/rails/jobs/27578159#L2913.
* | | | | | | Merge pull request #15735 from aditya-kapoor/update-wiki-mdZachary Scott2014-06-141-1/+1
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | [ci skip] update wiki link for MVC in README
| * | | | | | [ci skip] update wiki link for MVCAditya Kapoor2014-06-151-1/+1
|/ / / / / /
* | | | | | Merge pull request #15734 from aditya-kapoor/correct-linkZachary Scott2014-06-141-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | [ci-skip] correct wikipedia MVC link
| * | | | | | [ci-skip] correct wikipedia MVC linkAditya Kapoor2014-06-141-1/+1
| | | | | | |
* | | | | | | Edit pass on `has_secure_password` documentation [ci skip]Godfrey Chan2014-06-141-5/+6
|/ / / / / /
* | | | | | Merge pull request #15704 from sgrif/sg-deprecate-serializedYves Senn2014-06-143-18/+26
|\ \ \ \ \ \ | | | | | | | | | | | | | | Deprecate `serialized_attributes` without replacement
| * | | | | | Deprecate `serialized_attributes` without replacementSean Griffin2014-06-143-18/+26
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | We've stopped using it internally, in favor of polymorphism. So should you!
* | | | | | Merge pull request #15715 from ↵Yves Senn2014-06-143-4/+11
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | zuhao/refactor_actionview_cleanup_I18n_translations Clean up I18n stored translations after test.
| * | | | | | Clean up I18n stored translations after test.Zuhao Wan2014-06-143-4/+12
| | | | | | |
* | | | | | | docs, remove getting started guide sample application. [ci skip]Yves Senn2014-06-1489-1137/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This application always gets out of sync and doesn't add much value. Closes #15721 /cc @rafaelfranca
* | | | | | | Merge pull request #15724 from akshay-vishnoi/secure_password_docsEileen M. Uchitelle2014-06-141-6/+9
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | [ci skip] Update #has_secure_password docs
| * | | | | | [ci skip] Update #has_secure_password docsAkshay Vishnoi2014-06-141-6/+9
|/ / / / / /
* | | | | | Merge pull request #15717 from akshay-vishnoi/test-casesGodfrey Chan2014-06-141-6/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Use `@existing_user` while updating existing user, fixing - ee4e86
| * | | | | | Use `@existing_user` while updating existing user, fixing - #ee4e86Akshay Vishnoi2014-06-141-6/+6
| | |_|_|/ / | |/| | | |
* | | | | | pg guide, use the term `database views` to be specific. [ci skip]Yves Senn2014-06-141-3/+3
| | | | | |
* | | | | | Merge pull request #15712 from zuhao/refactor_actionview_javascript_helper_testYves Senn2014-06-141-3/+3
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Avoid hard-coded value in test setup and teardown.
| * | | | | Avoid hard-coded value in test setup and teardown.Zuhao Wan2014-06-141-3/+3
| | | | | |
* | | | | | Merge pull request #15710 from zuhao/refactor_actionview_digestor_testMatthew Draper2014-06-141-0/+1
|\ \ \ \ \ \ | |/ / / / / |/| / / / / | |/ / / / Make sure restoration always happen.
| * / / / Make sure restoration always happen.Zuhao Wan2014-06-141-0/+1
|/ / / /
* | | | Cleaned up the `has_secure_password` test casesGodfrey Chan2014-06-141-33/+29
| | | | | | | | | | | | | | | | | | | | | | | | * Grouped the valid test cases in one place * Make the length of the generated password obvious * Removed two wrong (copy-and-pasted) test cases
* | | | Added changelog for #15708 [ci skip]Godfrey Chan2014-06-141-0/+7
| | | |
* | | | Merge pull request #15708 from akshay-vishnoi/secure_passwordGodfrey Chan2014-06-142-0/+34
|\ \ \ \ | |_|_|/ |/| | | SecurePassword - Validate password must be less than or equal to 72