aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | fix request methods testVipul A M2013-03-311-6/+2
| | |_|/ / | |/| | |
* | | | | Merge pull request #10017 from vipulnsward/railties_cleanupsSteve Klabnik2013-03-313-6/+5
|\ \ \ \ \ | | | | | | | | | | | | cleanup railties test; fix typos
| * | | | | cleanup railties test; fix typosVipul A M2013-03-313-6/+5
| |/ / / /
* | | | | Merge pull request #10015 from vipulnsward/remove_unused_variable_in_getSteve Klabnik2013-03-312-2/+2
|\ \ \ \ \ | | | | | | | | | | | | remove unused variable and assignment
| * | | | | remove unused variable and assignmentVipul A M2013-03-312-2/+2
| |/ / / /
* | | | | Merge pull request #10014 from vipulnsward/remove_unused_match_varSteve Klabnik2013-03-311-8/+8
|\ \ \ \ \ | | | | | | | | | | | | remove unused match variables in selector
| * | | | | remove unused match variables in selectorVipul A M2013-03-311-8/+8
| |/ / / /
* | | | | Merge pull request #10012 from vipulnsward/fix_AP_error_message_errorSteve Klabnik2013-03-311-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | fix wrong argument error message [ci skip]
| * | | | fix wrong argument error messageVipul A M2013-03-311-1/+1
|/ / / /
* | | | Make comment about uglifier in generated Gemfile consistent between branchesCarlos Antonio da Silva2013-03-301-1/+1
| | | | | | | | | | | | | | | | Introduced in 59d20af2238f7ab553f41b96282722889d8d4393.
* | | | Fix indent and change "or" usage to "||" in schema dumper testsCarlos Antonio da Silva2013-03-301-5/+5
| | | |
* | | | Improve AP changelog entry about layout method with nil returnCarlos Antonio da Silva2013-03-301-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a note about getting the "no layout" behavior by returning "false" to make it easier for people that might need to change their code. Related to #8458. [ci skip]
* | | | Refactor index algorithm lookup so that it only builds the available options ↵Carlos Antonio da Silva2013-03-301-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | once This way the available options are only built for actually fetching the algorithm in case the option is given. The options are going to be necessary a second time only in case the option is given but does not exist, which is supposed to be due to a typo or something like that, so no problem.
* | | | Minor doc improvement about index algorithms [ci skip]Carlos Antonio da Silva2013-03-301-3/+2
| | | |
* | | | Fix typos in AP: "overriden" => "overridden"Carlos Antonio da Silva2013-03-305-36/+36
| | | |
* | | | Review some tests from AMo::Errors to remove "should" usageCarlos Antonio da Silva2013-03-301-55/+35
| | | | | | | | | | | | | | | | | | | | Also remove duplicated tests for Errors#as_json and minor improvements in some tests.
* | | | Improve changelog entry for ActiveModel::Errors#full_messages_for [ci skip]Carlos Antonio da Silva2013-03-302-4/+13
| | | | | | | | | | | | | | | | Add a working example of how to use this new method.
* | | | Do not calculate values if they are not going to be usedCarlos Antonio da Silva2013-03-301-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When building the indexes list in PostgreSQL, IndexDefinition objects are only created if the query for the related attributes really returns something matching the attributes. In case it does not, the variables for building the definition objects were being created but not used.
* | | | Make docs a little bit more consistent with colons [ci skip]Carlos Antonio da Silva2013-03-301-9/+9
| | | |
* | | | Fix code examples indent in new index options docs [ci skip]Carlos Antonio da Silva2013-03-301-3/+3
| | | |
* | | | Use new hash style in doc examples [ci skip]Carlos Antonio da Silva2013-03-301-2/+2
| | | |
* | | | Merge pull request #9991 from Passionate/patch-1Rafael Mendonça França2013-03-301-18/+50
|\ \ \ \ | | | | | | | | | | improve readability
| * | | | improve readabilityParitosh Piplewar2013-03-291-18/+50
| | | | |
* | | | | Minor reviews in AR changelog [ci skip]Carlos Antonio da Silva2013-03-301-10/+10
| | | | |
* | | | | Use last released version of thorGuillermo Iguaran2013-03-302-2/+1
| | | | |
* | | | | Merge pull request #10004 from choudhuryanupam/masterRafael Mendonça França2013-03-302-10/+0
|\ \ \ \ \ | | | | | | | | | | | | Refactored to remove unnecessary setup
| * | | | | Refactored to remove unnecessary setupAnupam Choudhury2013-03-302-10/+0
| | |_|_|/ | |/| | |
* | | | | Merge pull request #10003 from tkhr/refactor-scoping-test-filesCarlos Antonio da Silva2013-03-303-330/+333
|\ \ \ \ \ | | | | | | | | | | | | Reorganize scoping test files
| * | | | | split relation_scoping_test.rb's default scoping tests into another fileTakehiro Adachi2013-03-302-357/+360
| | | | | |
| * | | | | rename named_scope_test.rb to a proper file nameTakehiro Adachi2013-03-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The file name should be name_scoping_test.rb and the class should be `NamedScopingTest` according to ActiveRecord::Scoping::Name
| * | | | | move tests for NamedScope and DefaultScope under test/cases/scoping/Takehiro Adachi2013-03-302-0/+0
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | The scoping/default.rb and scoping/named.rb got moved under scoping/ in commit 2b22564c4efaa63d4bbc006762838c4025c1bdca, but the tests never did.
* | | | | Merge pull request #9997 from vipulnsward/hash_destructive_mergeCarlos Antonio da Silva2013-03-302-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Change merge to merge! to avoid creating new objects
| * | | | | change merge to merge!Vipul A M2013-03-292-2/+2
| | | | | |
* | | | | | Merge pull request #10007 from vipulnsward/fix_AS_typosCarlos Antonio da Silva2013-03-303-14/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix some typos in AS
| * | | | | | fix some typos in ASVipul A M2013-03-303-14/+14
| | |/ / / / | |/| | | |
* | | | | | configure API generation lazilyXavier Noria2013-03-301-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do the absolute bare minimum needed to define this task, no need to compute gem paths everytime you call rake.
* | | | | | be lazy configuring the RDoc filesXavier Noria2013-03-301-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This code is executed every time you run a rake task, there is no need to, specially adding a penalty to test suites.
* | | | | | be lazy loading SDocXavier Noria2013-03-304-6/+18
| | | | | |
* | | | | | fix upgrading guide [ci skip]Vijay Dev2013-03-301-23/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I see no need to list the Gemfile contents (twice!) to convey this message.
* | | | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-03-3044-208/+274
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb activerecord/test/cases/adapter_test.rb guides/source/testing.md [ci skip]
| * | | | | More typo / wording fixesJonathan Roes2013-03-291-6/+6
| | | | | |
| * | | | | Proofreading tweaksJonathan Roes2013-03-291-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | - Consistent hash structure - Rewording / typo fixes
| * | | | | explain method invertible_for [ci skip]Neeraj Singh2013-03-291-1/+2
| | | | | |
| * | | | | Merge pull request #130 from crafters/masterVinicius Quaiato2013-03-281-0/+26
| |\ \ \ \ \ | | | | | | | | | | | | | | Adding info that Gemfile group :assets doesn't exists anymore
| | * | | | | info that Gemfile *group :assets* was removedVinicius Quaiato2013-03-291-0/+26
| |/ / / / / | | | | | | | | | | | | Adding info that Gemfile *group :assets* was removed on upgrading_ruby_on_rails.md
| * | | | | Minor proofingJonathan Roes2013-03-271-13/+13
| | | | | |
| * | | | | Remove unnecessary hyphen.Jonathan Roes2013-03-271-1/+1
| | | | | |
| * | | | | Extraneous syntax on guides path.Jonathan Roes2013-03-271-1/+1
| | | | | |
| * | | | | ReCAPTCHA plug-in link now points to Github repo. Plug-in is no longer ↵sthollmann2013-03-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | available at the previous location
| * | | | | Fix typoCarson McDonald2013-03-261-1/+1
| | | | | |