aboutsummaryrefslogtreecommitdiffstats
path: root/guides
Commit message (Collapse)AuthorAgeFilesLines
* Moved all JSON core extensions into core_ext/object/jsonGodfrey Chan2013-09-131-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | TL;DR The primary driver is to remove autoload surprise. This is related to #12106. (The root cause for that ticket is that json/add defines Regexp#to_json among others, but here I'll reproduce the problem without json/add.) Before: >> require 'active_support/core_ext/to_json' => true >> //.as_json NoMethodError: undefined method `as_json' for //:Regexp from (irb):3 from /Users/godfrey/.rvm/rubies/ruby-2.0.0-p195/bin/irb:16:in `<main>' >> //.to_json => "\"(?-mix:)\"" >> //.as_json => "(?-mix:)" After: >> require 'active_support/core_ext/to_json' => true >> //.as_json => "(?-mix:)" This is because ActiveSupport::JSON is autoloaded the first time Object#to_json is called, which causes additional core extentions (previously defined in active_support/json/encoding.rb) to be loaded. When someone require 'active_support/core_ext', the expectation is that it would add certain methods to the core classes NOW. The previous behaviour causes additional methods to be loaded the first time you call `to_json`, which could cause nasty surprises and other unplesant side-effects. This change moves all core extensions in to core_ext/json. AS::JSON is still autoloaded on first #to_json call, but since it nolonger include the core extensions, it should address the aforementioned bug. *Requiring core_ext/object/to_json now causes a deprecation warnning*
* Use the Rails binary when generating taskRafael Mendonça França2013-09-121-2/+2
| | | | | | Copy-edits [ci skip]
* Merge pull request #12170 from siong1987/taskRafael Mendonça França2013-09-121-1/+3
|\ | | | | | | [ci skip] Documentation on how to create custom rake tasks.
| * [ci skip] Documentation on how to create custom rake tasks.Teng Siong Ong2013-09-101-1/+3
| |
* | Merge pull request #12175 from JuanitoFatas/doc-patchRafael Mendonça França2013-09-121-5/+16
|\ \ | | | | | | [ci skip] Add missing setting locale code snippet.
| * | [ci skip] Fix links in section 5.3 of i18n.mdJuanito Fatas2013-09-101-5/+5
| | | | | | | | | | | | Now all links point to the correct line number.
| * | [ci skip] Add missing setting locale code snippet.Juanito Fatas2013-09-101-0/+11
| |/ | | | | | | | | | | | | | | This is for 3 Internationalizing your Application of i18n.md. Missing the code as mentioned in 2.3, setting locale. Without this, the example to add config/locales/pirate.yml will not work.
* | Merge branch 'master' of github.com:rails/docrailsVijay Dev2013-09-1310-119/+283
|\ \
| * | Wrap the guides in 80 columns [ci skip]Rafael Mendonça França2013-09-121-101/+260
| | |
| * | Added quotes when defining the precision for the `decimal` fieldsmogetutu2013-09-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | before: ```bash $ rails generate migration AddDetailsToProducts price:decimal{5,2} supplier:references{polymorphic} ``` after: ```bash $ rails generate migration AddDetailsToProducts 'price:decimal{5,2}' supplier:references{polymorphic} ```
| * | Plural of Deer not deerSergio2013-09-101-1/+1
| | | | | | | | | Change the plural of Deer word in the naming conventions
| * | html generated for a search formSergio2013-09-091-1/+1
| | | | | | | | | There's a div that is generated in rails for a default search form and is not includen in the example.
| * | added # or commentSergio2013-09-051-1/+1
| | |
| * | Review the changes made on 90155b4e28a3887dce9428e9df150ede3d6c7465Rafael Mendonça França2013-09-041-5/+9
| | |
| * | Client.exists?(1,2,3) and Client.exists?([1,2,3]) does not workSergio2013-09-051-3/+3
| | | | | | | | | | | | The right command for doing that is Client.exists?id:[1,2,3] Exists does not work like find method, in find method you can do Person.find(1, 2, 6) or Person.find([7, 17]) but not Person.exists?(1,2,3) or Person.exists?([1,2,3])
| * | Example of Join modelsSergio2013-09-041-1/+1
| | | | | | | | | In the example of join models, there are five models but in the explanation only consider Category, Post, Comment and Guest.
| * | `implemention` => `implementation`Vipul A M2013-09-031-1/+1
| | |
| * | `attibutes` => `attributes`Vipul A M2013-09-031-1/+1
| | |
| * | Be assertive about adding `[ci skip]` messageVipul A M2013-09-031-1/+1
| | |
| * | 4.0 Release notes should point to 4.0 ChangelogVipul A M2013-09-031-6/+6
| | |
| * | Fix spacing issueVipul A M2013-09-031-0/+1
| | |
* | | Update 4_0_release_notes.mdHarshad Sabne2013-09-121-3/+3
| | | | | | | | | Active support --> Deprecations Code highlighted.
* | | remove sentence errmichaelminter2013-09-111-1/+1
| | |
* | | Update Rails 3.2.x guide link [ci skip]Tomas Varaneckas2013-09-101-1/+1
| | |
* | | Merge pull request #12167 from hannesfostie/grammar-improvementSteve Klabnik2013-09-091-1/+1
|\ \ \ | |_|/ |/| | [ci skip] Improves a sentence in guides/security
| * | Improves a sentence in guides/securityHannes Fostie2013-09-091-1/+1
| | | | | | | | | | | | Changed "... books make this wrong" to "... books get this wrong"
* | | Merge pull request #12157 from robin850/patch-9Yves Senn2013-09-091-1/+1
|\ \ \ | | | | | | | | Use MiniTest::Unit::TestCase instead of Minitest::Test [ci skip]
| * | | Use MiniTest::Unit::TestCase instead of Minitest::TestRobin Dupret2013-09-071-1/+1
| |/ / | | | | | | | | | | | | | | | | | | Since Rails 4.0.x is depending on Minitest 4.x, the constant won't be available so the gist won't run. [ci skip]
* / / [ci skip] Change link name of Rails i18n wiki.Juanito Fatas2013-09-091-1/+1
|/ / | | | | | | Actually it's Ruby i18n link.
* | Use Ruby on Rails Coding Conventions for code examples in the guidesPaul Nikitochkin2013-09-0611-90/+89
| | | | | | | | | | | | | | | | * Indent after private/protected * Ruby >= 1.9 syntax for hashes * Prefer method { do_stuff } instead of method{do_stuff} for single-line blocks. [ci skip]
* | Merge pull request #12130 from egilburg/patch-1Yves Senn2013-09-051-2/+33
|\ \ | | | | | | Document a difference between pluck and select [ci skip]
| * | Document a difference between pluck and select [ci skip]Eugene Gilburg2013-09-041-2/+33
| |/ | | | | | | Explain that `pluck` differs from `select` in that it does not construct `ActiveRecord` objects and thus model-level overrides are unavailable.
* / Adjusting 4.0 release notes to show more appropriate replacements for ↵Brock Trappitt2013-09-011-3/+3
|/ | | | dynamic finders [ci skip]
* Merge branch 'master' of github.com:rails/docrailsVijay Dev2013-08-313-11/+17
|\
| * Fix typos [ci skip]Carlos Antonio da Silva2013-08-301-2/+2
| |
| * Add documentation for rake db:setup taskJay Hayes2013-08-291-0/+5
| |
| * Note functional equivalence in reset taskJay Hayes2013-08-291-2/+2
| |
| * Document ability to run a single test.wangjohn2013-08-211-1/+8
| |
| * Removing merge conflict remnants in the guides.wangjohn2013-08-211-6/+0
| |
| * Moved "`mod_expires` to be enabled" to new line.Prathamesh Sonpatki2013-08-211-2/+2
| | | | | | | | | | | | | | | | | | | | - Right now the comment was not rendering properly. # The Expires* directives requires the Apache module `mod_expires` to be # enabled. - After this change - # The Expires* directives requires the Apache module # `mod_expires` to be enabled.
* | Fix broken link to Fixtures documentation on guidesDavid Padilla2013-08-292-2/+2
| | | | | | | | | | | | and activerecord tests [ci 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]
* | | 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]
* / / adding more finder migration information to upgrade guide [ci skip]Gary S. Weaver2013-08-271-0/+6
|/ /
* | 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
| | |