aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* 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
| | | | | |
* | | | | | Merge pull request #11973 from prathamesh-sonpatki/nokogiri-warningYves Senn2013-08-261-0/+19
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add error message for including nokogiri in Gemfile for rake doc:guides
| * | | | | | Add error message for including nokogiri in Gemfile for rake doc:guides [ci ↵Prathamesh Sonpatki2013-08-221-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | skip] - When `rake doc:guides` is run from applications, it complains for presence of redcarpet if it not present in Gemfile - Similarly it should complain about nokogiri
* | | | | | | Merge pull request #11396 from strzalek/extract_renderersPiotr Sarnacki2013-08-26178-1592/+2073
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove dependency on Action View from Action Pack This set of changes removes the need of using Action View with Action Pack. Now you may use controllers without Action View, by rendering `:text` response or alternatively you can plug in your own rendering logic. This is especially handy when you're just dealing with APIs and don't need to include entire Action View just to render simple JSON responses.
| * | | | | | | Make AV dependency for ActionMailerŁukasz Strzałkowski2013-08-261-1/+1
| | | | | | | |
| * | | | | | | Fix formatting of error messageŁukasz Strzałkowski2013-08-251-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make it oneliner
| * | | | | | | Change description of BasicRendering#renderŁukasz Strzałkowski2013-08-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're not dealing with templates here [ci skip]
| * | | | | | | Simplify if statementsŁukasz Strzałkowski2013-08-251-6/+4
| | | | | | | |
| * | | | | | | Move abstract's controller tests to AVŁukasz Strzałkowski2013-08-2521-2/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ones that were actually testing AV functionality and should belong in there
| * | | | | | | Move actionpack's controller testsŁukasz Strzałkowski2013-08-254-12/+12
| | | | | | | |
| * | | | | | | Describe BasicRendering shortlyŁukasz Strzałkowski2013-08-251-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
| * | | | | | | Extend basic rendering, test it in railtiesŁukasz Strzałkowski2013-08-254-22/+84
| | | | | | | |
| * | | | | | | Load HTML in ActionView not ActionPackŁukasz Strzałkowski2013-08-252-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | HTML Scanner is part of ActionView and it should be loaded along with it
| * | | | | | | Update AP changelogŁukasz Strzałkowski2013-08-251-0/+9
| | | | | | | |
| * | | | | | | Basic rendering testŁukasz Strzałkowski2013-08-251-0/+19
| | | | | | | |