aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* cleans the guides sources from fancy non-ASCII stuffXavier Noria2013-08-2321-96/+96
|
* Merge pull request #11996 from semaperepelitsa/masterYves Senn2013-08-231-1/+1
|\ | | | | Docs: fix typo in file path, should include extension. [ci skip]
| * Fix typo in file path, should include extension.Semyon Perepelitsa2013-08-231-1/+1
|/
* Merge pull request #11994 from rajcybage/fix_instanceCarlos Antonio da Silva2013-08-231-2/+1
|\ | | | | remove not used instance variable
| * remove unused instance variableRajarshi Das2013-08-231-2/+1
| |
* | Merge pull request #11991 from prathamesh-sonpatki/note-edge-guidesYves Senn2013-08-231-2/+4
|\ \ | | | | | | Add a note about edge guides in the feedback section [ci skip]
| * | Add a note about edge guides in the feedback section[ci skip]Prathamesh Sonpatki2013-08-231-2/+4
|/ /
* | Merge pull request #11971 from freerunningtechnologies/dynamic_respond_toAaron Patterson2013-08-222-1/+6
|\ \ | | | | | | Performance regression in AR::Base.respond_to?
| * | Avoid compiling regexs in AR::Base.respond_to?John Hawthorn2013-08-212-1/+6
| | | | | | | | | | | | | | | Caches the patterns of ActiveRecord::DynamicMatchers in a class instance variable.
* | | Merge pull request #11976 from etehtsea/infYves Senn2013-08-221-2/+2
|\ \ \ | | | | | | | | [PG adapter] Use infinity const
| * | | Use infinity constKonstantin Shabanov2013-08-221-2/+2
| |/ /
* | | Merge pull request #11943 from suginoy/ar-order-notationYves Senn2013-08-221-0/+10
|\ \ \ | | | | | | | | Add examples of AR order method's hash notation to Rails Guide [ci skip]
| * | | Add examples of AR order method's hash notation to Rails Guide [ci skip]Sugino Yasuhiro2013-08-221-0/+10
| | | |
* | | | Merge pull request #11967 from ↵Rafael Mendonça França2013-08-223-4/+99
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | jetthoughts/add_action_controller_bug_report_template Added bug report template for ActionController
| * | | | Added bug report template for ActionControllerPaul Nikitochkin2013-08-213-4/+99
| | |/ / | |/| | | | | | | | | | [ci skip]
* | | | Merge pull request #11978 from edogawaconan/fix_doc_collection_proxyYves Senn2013-08-221-2/+0
|\ \ \ \ | |/ / / |/| | | Stray nodoc causes rest of file not parsed [ci skip]
| * | | Stray nodoc causes rest of file not parsed [ci skip]edogawaconan2013-08-221-2/+0
|/ / /
* | | Merge pull request #11966 from jetthoughts/update_bug_report_templatesRafael Mendonça França2013-08-212-2/+2
|\ \ \ | | | | | | | | Use `Minitest::Test` instead of old `MiniTest::Unit::TestCase` for bug reports template
| * | | Updated bug report templatesPaul Nikitochkin2013-08-212-2/+2
|/ / / | | | | | | | | | | | | | | | * use `Minitest::Test` instead of deprectaed `MiniTest::Unit::TestCase` [ci skip]
* | | Merge pull request #11932 from gaurish/skip-encoding-convertorRafael Mendonça França2013-08-212-0/+16
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | [JRuby] Skip few of the ActiveSupport's inflector test cases Conflicts: activesupport/test/abstract_unit.rb
| * | | Define jruby_skip to skip test on JRubyGaurish Sharma2013-08-211-0/+5
| | | |
| * | | Skip few of the ActiveSupport'sinflector test on JRubyGaurish Sharma2013-08-211-0/+10
| | | |
* | | | Merge pull request #11850 from razielgn/mod-delegation-backtraceRafael Mendonça França2013-08-211-2/+1
|\ \ \ \ | | | | | | | | | | Generalize backtrace check on DelegationError.
| * | | | Rely on NoMethodError#name when deciding to raise DelegationError.Federico Ravasio2013-08-121-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Different Ruby implementations present backtraces differently, as it should be an information consumed by humans. A better implementation should use data from the error, in this case returned by NoMethodError#name. Fixes issues with Rubinius, which presents backtraces differently from MRI.
* | | | | Merge pull request #11959 from razielgn/skip-global-safe-tests-on-rubiniusRafael Mendonça França2013-08-212-0/+6
|\ \ \ \ \ | | | | | | | | | | | | Skip tests involving $SAFE on Rubinius, it's not supported.
| * | | | | Skip tests involving $SAFE, it's not supported on Rubinius.Federico Ravasio2013-08-211-0/+2
| | | | | |
| * | | | | Added method to skip tests on Rubinius to AS/abstract_unit.Federico Ravasio2013-08-211-0/+4
| | |_|_|/ | |/| | |
* | | | | Merge pull request #11961 from Intrepidd/debug-exception-htmlRafael Mendonça França2013-08-215-5/+5
|\ \ \ \ \ | |/ / / / |/| | | | Escape the message of an exception in debug_exceptions to avoid bad rendering
| * | | | Escape the message of an exception in debug_exceptions to avoid bad renderingAdrien Siami2013-08-215-5/+5
|/ / / /
* | | | Merge pull request #11952 from lmarcetic/masterGuillermo Iguaran2013-08-201-2/+2
|\ \ \ \ | | | | | | | | | | Remove set_primary_key, replace with primary_key=
| * | | | Remove set_primary_key, replace with primary_key=Luka Marčetić2013-08-211-2/+2
|/ / / / | | | | | | | | First deprecated, now removed: https://github.com/rails/rails/commit/9add7608f1acaa68b025470e7a38901d7e6161ca#activerecord/lib/active_record/attribute_methods/primary_key.rb
* | | | Fix the indentation :scissors:Rafael Mendonça França2013-08-201-6/+6
| | | |
* | | | write changelog entry for #11922. [ci skip]Yves Senn2013-08-201-0/+5
| | | |
* | | | Merge pull request #11922 from Empact/string-confusionYves Senn2013-08-202-2/+2
|\ \ \ \ | | | | | | | | | | Stop interpreting SQL 'string' columns as :string type.
| * | | | Stop interpreting SQL 'string' columns as :string type.Ben Woosley2013-08-172-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SQL doesn't have a string type, and interpreting 'string' as text is contrary to at least SQLite3's behavior: "Note that a declared type of 'STRING' has an affinity of NUMERIC, not TEXT." http://www.sqlite.org/datatype3.html
* | | | | Merge pull request #11919 from prathamesh-sonpatki/docRafael Mendonça França2013-08-191-1/+3
|\ \ \ \ \ | | | | | | | | | | | | Change the wording to explain following methods skip callbacks [ci skip]
| * | | | | Change the wording to explain following methods skip callbacks [ci skip]Prathamesh Sonpatki2013-08-191-1/+3
| | | | | |
* | | | | | Merge pull request #11940 from baroquebobcat/thread_var_patch_freezeGuillermo Iguaran2013-08-192-0/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | ensure freeze on Thread freezes locals (#11831)
| * | | | | | ensure freeze on Thread freezes localsNick Howard2013-08-192-0/+14
|/ / / / / /
* | | | | | Use each_with_objectSantiago Pastorino2013-08-191-4/+2
| | | | | |
* | | | | | Organize normalize_encoding_params' conditionals a bit better and use a case ↵Santiago Pastorino2013-08-191-15/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | statement for it Refactor of the work done in #11891
* | | | | | Merge pull request #11891 from robertomiranda/remove_upload_moduleSantiago Pastorino2013-08-193-14/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Normalize file parameters in same place as other parameters (ActionDispatch::Http::Parameters#normalize_encode_params)
| * | | | | | Normalize file parameters in same place as other parameters ↵robertomiranda2013-08-143-14/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | (ActionDispatch::Http::Parameters#normalize_encode_params)
* | | | | | | Merge pull request #11443 from wangjohn/sse_reloader_classSantiago Pastorino2013-08-192-0/+162
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | SSE class for ActionController::Live
| * | | | | | Creating an SSE class to be used with ActionController::Live.wangjohn2013-07-302-0/+162
| | | | | | |
* | | | | | | Fix the indentation on the from template of the scaffold generatorRafael Mendonça França2013-08-192-5/+5
| | | | | | |
* | | | | | | documents 565c367 in the CHANGELOGXavier Noria2013-08-191-0/+4
| | | | | | |
* | | | | | | let AR::FinderMethods#exists? return singletons in all cases [closes #11592]Xavier Noria2013-08-192-34/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a regression. The documentation said in its introduction paragraph that the method returns truthy/falsy, but then below it was said that if there were no arguments you'd get `true` or `false`. Also when the argument is exactly `false` a singleton is documented to be returned. The method was not returning the singletons so it didn't conform to those special cases. The best solution here seems to be to just return singletons in all cases. This solution is backwards compatible. Also, the contract has been revised because it has no sense that the predicate varies that way depending on the input. I bet the previous contract was just an accident, not something mixed on purpose. Conflicts: activerecord/lib/active_record/relation/finder_methods.rb activerecord/test/cases/finder_test.rb
* | | | | | | explain how to contribute to the documentation.Yves Senn2013-08-192-9/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow-up to #11934
* | | | | | | Merge pull request #11934 from ↵Yves Senn2013-08-191-1/+1
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | xta/clarify_instructions_in_contributing_to_ruby_on_rails clarify instructions in contributing_to_ruby_on_rails.md for committing ... [ci skip]