aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Do not strip code blocks, otherwise we may get misaligned outputCarlos Antonio da Silva2012-11-071-1/+1
| | | | [ci skip]
* stop raising so many exceptionsAaron Patterson2012-11-071-1/+5
|
* adding requires for constant dependenciesAaron Patterson2012-11-072-0/+6
|
* cleaning up constantize testsAaron Patterson2012-11-071-44/+44
|
* fix warnings in Ruby 2.0Aaron Patterson2012-11-072-0/+4
|
* Merge pull request #8132 from bquorning/permitted_is_sticky_on_sliceCarlos Antonio da Silva2012-11-062-7/+34
|\ | | | | Permitted should be sticky on #slice
| * Set @permitted on new instance when slicing parameters hashBenjamin Quorning2012-11-061-1/+3
| |
| * Test that permitted? is sticky on accessors, mutators, and mergesBenjamin Quorning2012-11-061-0/+24
| |
| * Test that not permitted is sticky on #exceptBenjamin Quorning2012-11-061-0/+1
| |
| * Current tests are testing stickiness of non-permitted parametersBenjamin Quorning2012-11-061-3/+3
| |
| * Fix buggy testsBenjamin Quorning2012-11-061-4/+4
|/
* Update mocha github repo to new oneCarlos Antonio da Silva2012-11-061-1/+1
|
* mocha_standalone is now deprecated. Switch to mocha/api.Jeremy Kemper2012-11-052-2/+2
|
* Make the tests pass with minitest 4.2Rafael Mendonça França2012-11-051-0/+3
|
* Merge pull request #8118 from nashby/activemodel-errorsRafael Mendonça França2012-11-053-23/+41
|\ | | | | use Array() instead of flatten
| * use Array() instead flattenVasiliy Ermolovich2012-11-053-23/+41
| | | | | | | | | | * move ActiveModel::Errors tests to errors_test.rb * add spec coverage for add_on_empty and add_on_blank
* | Remove AS::Concern extension from Testing::TaggedLoggingCarlos Antonio da Silva2012-11-051-4/+0
|/
* Minor css style improvementsCarlos Antonio da Silva2012-11-041-47/+45
| | | | | | | Fix some indenting for some properties and closing }, remove tabs, and merge two pre/code definitions. [ci skip]
* Restore index link states to the way they are in current guideCarlos Antonio da Silva2012-11-041-2/+2
| | | | | | | | | | | | Current stable guides show the "Guides Index" links without underline, just showing it when it's hovered. This restores the same functionality for edge guides. Also remove annoying pointer cursor while accessing guides index, restoring previously functionality: now the pointer cursor only appears when hovering a link. Closes #8107 [ci skip]
* Use flat_map when building AR orderCarlos Antonio da Silva2012-11-041-6/+4
|
* Add ensure block to make sure the state is properly restoredCarlos Antonio da Silva2012-11-042-2/+2
|
* Remove some line breaks between array items that make the assert file tests ↵Carlos Antonio da Silva2012-11-044-42/+19
| | | | harder to read
* Refactor Inflector#ordinal to avoid converting the number twiceCarlos Antonio da Silva2012-11-041-2/+4
|
* Fix typo in test nameCarlos Antonio da Silva2012-11-042-3/+2
|
* Raise ArgumentError when no attribute is given to AMo::EachValidatorCarlos Antonio da Silva2012-11-042-2/+2
| | | | ArgumentError is better suited than RuntimeError for this.
* Set hash value instead of using merge!Carlos Antonio da Silva2012-11-041-1/+1
|
* Merge pull request #8111 from ↵Carlos Antonio da Silva2012-11-043-5/+9
|\ | | | | | | | | nikitug/use_tag_instead_of_method_missing_in_xml_builder Use `tag!` instead of `method_missing` in `to_xml` conversions.
| * Use `tag!` instead of `method_missing` in `to_xml` conversions.Nikita Afanasenko2012-11-043-5/+9
| | | | | | | | Since version `3.0.x` `Builder` caches method passed to `method_missing` each time. This commit replaces `method_missing` call with `tag!` call to prevent method redefinition on each `to_xml` call with the same builder.
* | Ensure nested attributes is restored in case of a test failureCarlos Antonio da Silva2012-11-031-1/+1
| |
* | Simplify query conditions a bit in nested attributes testCarlos Antonio da Silva2012-11-031-5/+3
| | | | | | | | Also refactor the test a bit.
* | Remove #permitted attr accessor from ParametersCarlos Antonio da Silva2012-11-031-1/+0
| | | | | | | | | | There is no need to expose this accessor since we already have the query method #permitted? that should handle this purpose.
* | Check if the options value is present before to send the deprecationRafael Mendonça França2012-11-032-6/+8
|/ | | | message
* Merge pull request #8108 from Casecommons/fix-multiple-and-index-in-instance-tagRafael Mendonça França2012-11-023-6/+17
|\ | | | | Support :multiple option on input tags that also have :index
| * Support :multiple option on input tags with :indexDaniel Fox, Grant Hutchins & Trace Wax2012-11-023-6/+17
|/ | | | | When you have an explicit index set, then when you build an input tag with :multiple => true, it doesn't add [] to the end of its name, although it should.
* Remove old commentRafael Mendonça França2012-11-021-1/+1
| | | | | | | | | | | | | | | This comment is not valid since that `if` is there to make possible to do: remove_index :users, :name Instead of: remove_index :users, column: :name What is a valid use case. [ci skip]
* Deprecate passing a string as third argument of `add_index`Rafael Mendonça França2012-11-023-0/+23
| | | | | | | This was there due historical reasons since 7dc45818dc43c163700efc9896a0f3feafa31138 to give the user the possibility to create unique indexes passing "UNIQUE" as the third argument
* Clear url helpers when reloading routesSantiago Pastorino2012-11-022-0/+5
|
* Raise an ArgumentError when passing an invalid option to add_indexRafael Mendonça França2012-11-023-1/+14
| | | | Closes #8104
* Merge pull request #8106 from steveklabnik/fix_grammarCarlos Antonio da Silva2012-11-021-1/+1
|\ | | | | Fixed grammar in migration pending error.
| * Fixed grammar in migration pending error.Steve Klabnik2012-11-021-1/+1
| |
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-11-03105-809/+873
|\ \ | | | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_controller/metal/mime_responds.rb activerecord/lib/active_record/attribute_methods.rb guides/source/working_with_javascript_in_rails.md
| * | copy edits [ci skip]Vijay Dev2012-11-032-7/+4
| | |
| * | Revert "Updated activerecord test comments to new hash syntax"Vijay Dev2012-11-032-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 3b89052014c2bf3b8fbbcfb93f657a17dd7e7ca3. Reason: changes made to a test, albeit in commented lines - non-uniform change. [ci skip]
| * | Revert "dont encourage AC::Parameters#permit_all_parameters usage [ci skip]"Vijay Dev2012-11-031-1/+21
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit cd17ee5598411728747011566abf6779166be9d3. Reason: Let's note that this is discouraged, not remove the docs [ci skip]
| * | This is comment for singular association.kennyj2012-11-021-1/+1
| | |
| * | 1.9 hash syntax changes to docsAvnerCohen2012-10-3118-169/+169
| | |
| * | Hash Syntax to 1.9 related changesAvnerCohen2012-10-2920-346/+346
| | |
| * | Merge branch 'master' of github.com:lifo/docrailsJuanjo Bazán2012-10-291-21/+1
| |\ \
| | * | dont encourage AC::Parameters#permit_all_parameters usage [ci skip]Francesco Rodriguez2012-10-281-21/+1
| | | |
| * | | 3.1 and up, in configuring guide.Juanjo Bazán2012-10-291-1/+1
| |/ /