aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | Refactoring Generators::Base.wangjohn2013-08-281-14/+17
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The defaults hash isn't used unless the +class_options+ hash has a particular key, so we don't need to compute it unless this is true. Also moving some code for extracting a module into its own method.
* | | | | | | Merge pull request #12058 from gaurish/secure-pass-docRafael Mendonça França2013-08-281-1/+3
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | Add nodoc to ActiveModel::SecurePassword.min_cost
| * | | | | | Add :nodoc: to ActiveModel::SecurePassword.min_costGaurish Sharma2013-08-291-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | Merge pull request #12053 from schuetzm/doc_sqlite3_memYves Senn2013-08-281-0/+1
|\| | | | | | | | | | | | | | | | | | | | Mention the fast `test_sqlite3_mem` option in `RUNNING_UNIT_TESTS.rdoc` [ci skip]
| * | | | | | Mention the fast `test_sqlite3_mem` option in `RUNNING_UNIT_TESTS.rdoc` [ci ↵Marc Schütz2013-08-281-0/+1
|/ / / / / / | | | | | | | | | | | | | | | | | | skip]
* | | | | | Merge pull request #12033 from alexaltair/patch-1Yves Senn2013-08-271-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | change reference of `Test::Unit` to `Minitest` [ci skip]
| * | | | | Update Test::Unit to MinitestAlex Altair2013-08-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previous was misleading; current is more up to date. [ci skip]
* | | | | | use consistent keys between cache get / setAaron Patterson2013-08-271-1/+1
| | | | | |
* | | | | | remove intermediate variableAaron Patterson2013-08-271-3/+1
| | | | | |
* | | | | | cache association reset calculationAaron Patterson2013-08-271-4/+4
| | | | | |
* | | | | | query the association rather than send the method for the association nameAaron Patterson2013-08-272-11/+10
| | | | | |
* | | | | | Merge pull request #12048 from tjouan/app_generator-bin_perms-umaskAaron Patterson2013-08-271-1/+1
|\ \ \ \ \ \ | | |/ / / / | |/| | | | Comply with current umask when generating new app
| * | | | | Comply with current umask when generating new app:Thibault Jouan2013-08-271-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When creating a new application using the default rails generator (`rails new my_app`), it will apply some permissions bypassing the current process umask. The `bin' directory and any files inside it will always be set a mode of 0755. This change removes the current umask bits from the mode applied by the generator on the `bin' directory and its content.
* | | | | Improve CHANGELOG entry [ci skip]Rafael Mendonça França2013-08-271-4/+2
| | | | |
* | | | | Merge pull request #11768 from cloudspace/cache_template_loadingRafael Mendonça França2013-08-273-5/+24
|\ \ \ \ \ | | | | | | | | | | | | Only cache template digests if !config.cache_template_loading
| * | | | | Only cache template digests if config.cache_template_loadingJosh Lauer2013-08-063-5/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | since ActionView::Resolver.caching is set to the same value as config.cache_template_loading only cache template digests if config.cache_template_loading is not falsy fixes issues #10752 and #10791
* | | | | | Merge pull request #11805 from ↵Rafael Mendonça França2013-08-271-34/+36
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | jetthoughts/11803_update_move_forbidden_exception_description_before_show [Getting Started Guide] Move Forbidden attributes description before show post section [ci skip]
| * | | | | | #11803: Move Forbidden attributes description before show post sectionPaul Nikitochkin2013-08-271-34/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | Remove gist link from the codeRafael Mendonça França2013-08-271-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is not good to include links in the code. These links can be in the future outdated and nobody will upgrade they. [ci skip]
* | | | | | | Merge pull request #11945 from Mik-die/polymorphic-decoratorRafael Mendonça França2013-08-273-1/+32
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | check class hierarchy with is_a? in PredicateBuilder.expand
| * | | | | | | check class hierarchy with is_a? in PredicateBuilder.expandMikhail Dieterle2013-08-273-1/+32
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | add changelog entry for #11945
* | | | | | | Fix misspelling on README.mdPrem Sichanugrist2013-08-271-1/+1
| | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | Merge pull request #12028 from egilburg/patch-1Rafael Mendonça França2013-08-271-15/+20
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Updating Rails README.md to reflect current state [ci skip]
| * | | | | | | Updating README.md [ci skip]Eugene Gilburg2013-08-261-15/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updating the global Rails README following work by @strzalek et al which extracted Action View to a separate gem, and also some other improvements. Summary of changes: * Reordered the paragraphs from View-Model-Controller to Model-Controller-View, based on how (IMHO) most people think about the structure and hierarchy of the framework and their code in it. * Added references to ActionView's README in the _View_ paragraph * Reworded the _Controller_ paragraph to briefly mentioned routing as handled by Action Dispatch, and that both Action Dispatch and Action Controller work together to form the request-handling Action Pack * Removed paragraph saying Action Pack is coupled with Action View, as this is no longer the case * Added brief references to Action Mailer and Active Support, since they are both major Rails components. Briefly mentioned in the _View_ paragraph that Action View is often used to generate the body of the email, in addition to the other typical of using to build a response body.
* | | | | | | | Merge pull request #12030 from garysweaver/masterYves Senn2013-08-271-0/+6
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | adding more finder deprecation info in upgrade documentation [ci skip]
| * | | | | | | adding more finder migration information to upgrade guide [ci skip]Gary S. Weaver2013-08-271-0/+6
|/ / / / / / /
* | | | | | | Merge pull request #11636 from ↵Xavier Noria2013-08-274-1/+18
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | urbanautomaton/uppercase-nested-const-autoload-marking `ActiveSupport::Dependencies`: consts in uppercase nestings not marked as autoloaded
| * | | | | | | Ensure all-caps nested consts marked as autoloadedSimon Coffey2013-08-274-1/+18
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, an autoloaded constant `HTML::SomeClass` would not be marked as autoloaded by AS::Dependencies. This is because the `#loadable_constants_for_path` method uses `String#camelize` on the inferred file path, which in turn means that, unless otherwise directed, AS::Dependencies watches for loaded constants in the `Html` namespace. By passing the original qualified constant name to `#load_or_require`, this inference step is avoided, and the new constant is picked up in the correct namespace.
* | | | | | | Improve AP changelog regarding AV extraction [ci skip]Carlos Antonio da Silva2013-08-261-2/+2
| | | | | | |
* | | | | | | Merge pull request #12038 from SamSaffron/memoize_decorate_colsRafael Mendonça França2013-08-261-17/+27
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Perf: memoize serialized column list and time zone column list
| * | | | | | | Perf: memoize serialized column list and time zone column listSam2013-08-271-17/+27
| | | | | | | |
* | | | | | | | Merge pull request #12037 from SamSaffron/mem_allocAaron Patterson2013-08-262-3/+4
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Performance: Reduce memory allocations in Active Record
| * | | | | | | Perf: avoid array allocation where not neededSam2013-08-271-1/+1
| | | | | | | |
| * | | | | | | Perf: fields in pg gem causes an allocation, cache itSam2013-08-271-2/+3
|/ / / / / / /
* | | | | | | Merge pull request #12034 from wangjohn/fixtures_config_changesSantiago Pastorino2013-08-262-24/+27
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Add config to method calls in fixtures.
| * | | | | | Add config to method calls in fixtures.wangjohn2013-08-262-24/+27
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows you to change your configuration for calls to `table_name_prefix`, `table_name_suffix`, and `pluralize_table_names`. The default configuration is still ActiveRecord::Base, but you are now able to change the configuration easily.
* | | | | | Merge pull request #12032 from carolinemota/fixdocCarlos Antonio da Silva2013-08-261-19/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove whiny_nils config from the rails guides [ci skip]
| * | | | | | Remove whiny_nils config from the rails guidesCaroline Mota2013-08-261-19/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This option was removed from master. [ci skip]
* | | | | | | Merge pull request #11986 from fedesoria/patch-3Xavier Noria2013-08-261-0/+5
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Warning on doc for assets under lib|vendor not precompiling
| * | | | | | Warning on doc for assets under lib|vendor not precompilingFederico Soria2013-08-261-0/+5
| | | | | | |
* | | | | | | Add note about features vs. bugs.Steve Klabnik2013-08-261-3/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sometimes people ask what makes a feature and what makes a bug, so let's clarify that so we can show it to people later.
* | | | | | | Flesh out feedback portion of contributing guide.Steve Klabnik2013-08-261-1/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I don't know of anyone that hangs out in #rails-contrib, and we should also let people know that feedback can take some time.
* | | | | | | Soften wording about knowing other devs.Steve Klabnik2013-08-261-1/+7
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | Many people don't, so point them to a place where they can meet some.
* | | | | | Merge pull request #12015 from vipulnsward/add_dynamic_finders_to_upgradeSteve Klabnik2013-08-261-0/+12
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add Finder API changes to upgrading guide [ci skip]
| * | | | | | Add Finder API changes to upgrading guide [skip ci]Vipul A M2013-08-261-0/+12
| | | | | | |
* | | | | | | fix typo in the credit section of the guides. [ci skip]Yves Senn2013-08-261-1/+1
| | | | | | |
* | | | | | | Merge pull request #11997 from startupjockey/add_credits_ASYves Senn2013-08-262-0/+4
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Adds Akshay Surve to Rails Guides credits list [ci skip]
| * | | | | | | Adds Akshay Surve to Rails Guides credits listAkshay Surve2013-08-232-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I had written the original guide for "A Guide to Testing Rails Applications". The current documents maintains the same structure and examples I had drafted albeit with some minor changes to keep up with newer version of Rails. Older version which had credits included: http://guides.rubyonrails.org/v2.3.11/testing.html Github lifo/docrails History: https://github.com/rails/rails/commits/ff359af86505b4d317b7467e7f79aa2ef2c5e795/railties/doc/guides/source/testing_rails_applications.txt
* | | | | | | | Merge pull request #11827 from vipulnsward/remove_extra_caseSantiago Pastorino2013-08-261-1/+0
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | | Remove extra case.
| * | | | | | | Remove extra case.Vipul A M2013-08-101-1/+0
| | | | | | | |