aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #13335 from glorieux/favicon_link_tag_mimetypeRafael Mendonça França2014-04-223-6/+21
|\ | | | | Change favicon_link_tag helper mimetype from image/vnd.microsoft.icon to image/x-icon.
| * Change favicon_link_tag helper mimetype from image/vnd.microsoft.icon to ↵glorieux2014-04-213-6/+21
| | | | | | | | | | | | | | | | image/x-icon. Although the official IANA-registered MIME type for ICO files is image/vnd.microsoft.icon, registered in 2003, it was submitted to IANA by a third party and is not recognized by Microsoft products. The MIME type image/x-icon should be used since is the one recognized by the major browsers on the market.
* | Merge pull request #10764 from mokevnin/replace_class_eval_by_define_methodRafael Mendonça França2014-04-221-35/+29
|\ \ | | | | | | replace class_eval by define_method in abstract_controller/callbacks
| * | replace class_eval by define_method in abstract_controller/callbackskirill2014-04-201-35/+29
| | |
* | | Merge pull request #14835 from zzak/test_form_helper_label_with_block_and_htmlRafael Mendonça França2014-04-222-0/+8
|\ \ \ | | | | | | | | Add test for using ActionView::Helpers::FormHelper.label with block and html
| * | | Add test for using ActionView::Helpers::FormHelper.label with block and htmlZachary Scott2014-04-222-0/+8
|/ / /
* | / Rearrange deck chairs on the titanic. Organize connection handling test cases.Jeremy Kemper2014-04-214-249/+246
| |/ |/|
* | Merge pull request #14826 from stevegraham/action_controller_renderer_doc_fixArthur Nogueira Neves2014-04-201-2/+2
|\ \ | | | | | | ActionController::Renderers documentation fix
| * | ActionController::Renderers documentation fixStevie Graham2014-04-201-2/+2
| | | | | | | | | | | | | | | ActionController::Renderers::RENDERERS is an instance of Set. Docs incorrectly state that it's a Hash.
* | | Merge pull request #14825 from arunagw/aa-remove-missleading-commentArun Agrawal2014-04-201-1/+1
|\ \ \ | |/ / |/| | Removed missleading comment [ci skip]
| * | Changed miss-leading comment [ci skip]Arun Agrawal2014-04-201-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | After this 21dbe6f39b57f52967e92716dbd5e2b894e7a64c 2.1.1 :001 > 'business'.classify => "Business" 2.1.1 :004 > 'calculus'.classify => "Calculu" 2.1.1 :005 >
* | Merge pull request #14823 from kuldeepaggarwal/fix-inflector-exampleArun Agrawal2014-04-201-1/+1
|\ \ | |/ |/| correct the classify example's output.
| * [ci-skip] correct the classify example's outputKuldeep Aggarwal2014-04-211-1/+1
|/
* Merge pull request #14820 from akshay-vishnoi/doc_changesRichard Schneeman2014-04-206-8/+8
|\ | | | | [ci skip] builtin -> built-in
| * [ci skip] builtin -> built-inAkshay Vishnoi2014-04-206-8/+8
| |
* | Merge pull request #14821 from jonatack/patch-5Arthur Nogueira Neves2014-04-201-47/+63
|\ \ | |/ |/| Fix code indentation and improve formatting
| * Fix code indentation and improve formattingJon Atack2014-04-201-47/+63
|/ | | | | | | in the Getting Started Guide. This PR regroups #14817, #14818 and #14819 into one commit. [skip ci]
* Merge pull request #14816 from jonatack/patch-2Robin Dupret2014-04-201-1/+1
|\ | | | | Grammar fix in Getting Started Guide [ci skip]
| * Grammar fix in Getting Started GuideJon Atack2014-04-201-1/+1
|/
* Make URL escaping more consistentAndrew White2014-04-208-10/+89
| | | | | | | | | | | | | | | | | | | | | | 1. Escape '%' characters in URLs - only unescaped data should be passed to URL helpers 2. Add an `escape_segment` helper to `Router::Utils` that escapes '/' characters 3. Use `escape_segment` rather than `escape_fragment` in optimized URL generation 4. Use `escape_segment` rather than `escape_path` in URL generation For point 4 there are two exceptions. Firstly, when a route uses wildcard segments (e.g. *foo) then we use `escape_path` as the value may contain '/' characters. This means that wildcard routes can't be optimized. Secondly, if a `:controller` segment is used in the path then this uses `escape_path` as the controller may be namespaced. Fixes #14629, #14636 and #14070.
* Optimize URI escapingAndrew White2014-04-201-17/+42
| | | | | | | | | | | | | The URI::Parser#escape method is a general use method that has to deal with a variety of input however our use of it is limited in scope so we can increase the performance by implementing our specific needs within ActionDispatch::Journey::Router::Utils directly. If there is no encoding required then there is no change in performance or number of objects allocated, but for each character that needs to be encoded we save five object allocations and gain a performance boost. The performance boost seen varies from 20% when there is one character to over 50% when encoding ten characters.
* Always escape string passed to url helper.edogawaconan2014-04-202-3/+3
| | | | | | | | | | Makes it clear that anything passed with the helper must not be percent encoded. Fixes previous behavior which tricks people into believing passing non-percent-encoded will generate a proper percent-encoded path while in reality it doesn't ('%' isn't escaped). The intention is nice but the heuristic is broken.
* Merge pull request #14811 from qrush/masterGuillermo Iguaran2014-04-191-1/+1
|\ | | | | Remove statement assuming coffee shop/public space wifi is inherently insecure
| * Remove statement assuming coffee shop/public space wifi is inherently insecureNick Quaranto2014-04-191-1/+1
|/
* Merge pull request #14810 from robin850/patch-18Godfrey Chan2014-04-191-1/+0
|\ | | | | Don't rely on Arel master in bug report template
| * Don't rely on Arel master in bug report template [ci skip]Robin Dupret2014-04-191-1/+0
|/ | | | | | We should rely instead on the version supported by Active Record Closes #14809
* Merge pull request #14805 from akshay-vishnoi/structure-changeAndrew White2014-04-191-0/+0
|\ | | | | multibyte_conformance.rb --> multibyte_conformance_test.rb
| * multibyte_conformance.rb --> multibyte_conformance_test.rbAkshay Vishnoi2014-04-191-0/+0
| |
* | Merge pull request #14804 from akshay-vishnoi/doc_changesArthur Nogueira Neves2014-04-193-4/+4
|\ \ | | | | | | "subhash" --> "sub-hash"
| * | "subhash" --> "sub-hash"Akshay Vishnoi2014-04-193-4/+4
| |/
* | Merge pull request #14808 from ↵Xavier Noria2014-04-191-2/+2
|\ \ | |/ |/| | | | | akalyaev/few-corrections-in-rails-initialization-guides Few corrections in Rails Initialization Guides
| * wrap methods in backticks [ci skip]Anton Kalyaev2014-04-191-2/+2
|/
* Merge pull request #14799 from ernie/string-first-last-fixRafael Mendonça França2014-04-182-4/+16
|\ | | | | Fix inconsistent behavior from String#first/#last
| * Fix inconsistent behavior from String#first/#lastErnie Miller2014-04-182-4/+16
|/ | | | | | | While calling String#first or String#last with zero or a Fixnum < the string's length returns a new string, a Fixnum >= the string's length returns the string itself. This inconsistency can lead to inadvertent mutation of a string.
* Merge pull request #14796 from kuldeepaggarwal/fix-save-issueYves Senn2014-04-183-0/+21
|\ | | | | when a destroyed record is duped, it is no longer ´destroyed?`.
| * `@destroyed` should always be set to `false` when an object is duped.Kuldeep Aggarwal2014-04-193-0/+21
|/
* Merge pull request #14798 from kuldeepaggarwal/fix-warningYves Senn2014-04-181-1/+1
|\ | | | | remove warning while running TEST CASES
| * remove warningKuldeep Aggarwal2014-04-181-1/+1
|/ | | | `warning: ambiguous first argument; put parentheses or even spaces`
* Merge branch 'master' of github.com:rails/docrailsVijay Dev2014-04-1810-123/+161
|\
| * Fix spelling and proper nounsSteven Harman2014-04-172-5/+5
| | | | | | | | [ci skip]
| * [ci skip] document `statement_limit` schneems2014-04-151-1/+11
| | | | | | | | | | Per discussion at https://github.com/rails/rails/issues/14645#issuecomment-40499409
| * [ci skip] PP does not generate pass by defaultschneems2014-04-151-2/+0
| | | | | | | | This was changed in https://github.com/rails/rails/pull/13415/files
| * re-phrase how to run a testWaynn Lue2014-04-151-2/+2
| |
| * [ci skip] Use valid current config in exampleschneems2014-04-151-2/+2
| | | | | | `reaping_frequency` is used in Active Record `reap_frequency` is not
| * Fix spelling errors and grammar issuesSteven Harman2014-04-131-8/+11
| | | | | | | | | | | | Including replacing a few "smart quotes" with regular quotes (' and "), for consistency. [ci skip]
| * Correct docs as singlton true is not returned hereSteven Harman2014-04-111-1/+1
| | | | | | | | [ci skip]
| * Fix a few grammar issues. [ci skip]Steven Harman2014-04-111-5/+4
| |
| * Document LoadError core extensionsSteven Harman2014-04-111-1/+4
| | | | | | | | [ci skip]
| * Use path where extensions are defined, not usedSteven Harman2014-04-111-3/+3
| | | | | | | | | | | | | | For consistency with all other documentation, point to where these core extensions are defined, not where the example code comes from. [ci skip]
| * Instead boat load of libraries, simply say StdLibGaurish Sharma2014-04-091-1/+1
| |
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-08 15:53:38 +0000