aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Define [], []=, permanent, signed and encrypted as the only allowed methods ↵Santiago Pastorino2012-12-301-9/+57
| | | | | | | | | | | | for the non Raw Cookie classes
* | | Merge pull request #8546 from hsbt/fix-testcase-strict-warningSantiago Pastorino2012-12-301-1/+2
|\ \ \ | |/ / |/| | fix testcase: ruby-2.0.0 warned unused variables
| * | change spy of after invoked controller action. because ruby-2.0.0 waned ↵SHIBATA Hiroshi2012-12-201-1/+2
| | | | | | | | | | | | unused variables
* | | Revert "Rails 4.0 is going to depend on Rack 1.5.x so let's test against ↵Guillermo Iguaran2012-12-301-1/+0
| | | | | | | | | | | | | | | | | | Rack master" This reverts commit fba97c9620e709f0449ea9f3240b9e5174f72e1e.
* | | Point to guides.rubyonrails.org instead of edgeguides.rubyonrails.orgSantiago Pastorino2012-12-301-2/+2
| | |
* | | Merge pull request #8651 from tjouan/activerecord-structure_dump-ending_nlRafael Mendonça França2012-12-301-1/+3
|\ \ \ | | | | | | | | Fix missing ending newline in db structure dump:
| * | | Fix missing ending newline in db structure dump:Thibault Jouan2012-12-301-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When dumping database structure with `rake db:structure:dump` and using migrations, the resulting file will not end with a newline char. Although it's not mandatory, it breaks a lot of simple use cases with programs like cat, more, grep, etc. This changes use `puts' instead of `<<' to append migration versions data to the dump and also split the line where this is happening as it was a bit long.
* | | | Improve secret_token deprecation messageSantiago Pastorino2012-12-301-3/+4
|/ / /
* | | Merge pull request #8656 from arunagw/warning_removed_shadowing_varGuillermo Iguaran2012-12-301-2/+2
|\ \ \ | | | | | | | | warning removed: shadowing outer local variable
| * | | warning removed: shadowing outer local variableArun Agrawal2012-12-301-2/+2
|/ / /
* | | Merge pull request #8647 from nishantmodak/patch-2Vijay Dev2012-12-291-1/+1
|\ \ \ | | | | | | | | Update documentation for yml usage
| * | | Update documentation for yml usageNishant Modak2012-12-301-1/+1
| | | |
* | | | Rails 4.0 is going to depend on Rack 1.5.x so let's test against Rack masterSantiago Pastorino2012-12-291-0/+1
| | | |
* | | | Merge pull request #8650 from codeodor/patch-1Guillermo Iguaran2012-12-291-1/+1
|\ \ \ \ | | | | | | | | | | Fix up/down column change example code formatting
| * | | | Fix up/down column change example code formattingSammy Larbi2012-12-291-1/+1
|/ / / /
* | | | add missing require and fixturesAaron Patterson2012-12-291-0/+3
| | | |
* | | | fix time typcasting on group counts in PGAaron Patterson2012-12-282-2/+11
| | | |
* | | | mysql does not return alias names, so fall backAaron Patterson2012-12-281-4/+10
|/ / /
* | | Test that assert_not returns true. Use assert_raises instead of doing ↵Jeremy Kemper2012-12-281-17/+7
| | | | | | | | | | | | begin/rescue/else.
* | | fix PG typecasting errorsAaron Patterson2012-12-282-22/+12
| | |
* | | Merge pull request #8644 from acapilleri/abstract_mysqlRafael Mendonça França2012-12-281-2/+6
|\ \ \ | | | | | | | | small refactoring, added blob_or_text_colum? in AbstractMysqlAdapter
| * | | small refactoring, added blob_or_text_colum? in AbstractMysqlAdapterAngelo Capilleri2012-12-281-2/+6
|/ / /
* | | Merge pull request #8636 from senny/work_off_fixme_in_rename_column_testRafael Mendonça França2012-12-281-9/+22
|\ \ \ | | | | | | | | work off FIXME in rename_column_test
| * | | work off FIXME comments in AR rename_column_test.rbYves Senn2012-12-281-9/+22
| | | | | | | | | | | | | | | | | | | | | | | | There were a couple of tests, which had FIXME comments in palce of assertions. I replaced these FIXME comments with actual assertions to get more feedback from our test suite.
* | | | Revert "Install binstubs by default"Jeremy Kemper2012-12-282-4/+4
|/ / / | | | | | | | | | | | | | | | This reverts commit f34c27a452418d8aa17f92bb0fd7ae97b5f7e252. We'll be taking a different tack on this with new `bundle binstubs <gem>` support.
* | / Revert "Detect rbenv and update the shebang"Jeremy Kemper2012-12-283-18/+1
| |/ |/| | | | | | | This reverts commit 7173c4f931c9026bd3bc0b644d93dae744433f09. This reverts commit a562f9fa09967eba26dff1c3bc8aa85b6f0a72ff.
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-12-292-5/+1
|\ \
| * | Revert "Add rake test description"Vijay Dev2012-12-291-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit a3309caf338830849acd826bc6ad14f972e787db. Reason: docrails isn't the place to do this. [ci skip]
| * | Revert "Add rake default to description"Vijay Dev2012-12-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 0d8ca8124de9b3a5f6398a28d79baca935a1f96c. Reason: docrails isn't the place to do this. Also, I feel this is not really required. [ci skip]
| * | Revert "reminder to run bundle after setting up rails-dev-box"Vijay Dev2012-12-291-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 41381c19d1f3920ee9540aff0d35209cb43fb057. Reason: This doesn't quite fit the flow where it's added. Let's add this someplace else if it's explicitly required. [ci skip]
| * | Fixed couple of typosFabrizio Regini2012-12-271-1/+1
| | |
| * | removed TIP: :allow_nil/:allow_blank is ignored by the presence validatorColin Kelley2012-12-261-4/+0
| | | | | | | | | | | | These tips were documenting an inconsistency issue https://github.com/rails/rails/issues/8621. That issue is resolved by https://github.com/rails/rails/issues/8622.
| * | reminder to run bundle after setting up rails-dev-boxAce Suares2012-12-241-0/+7
| | |
| * | Add rake test descriptionsanemat2012-12-241-0/+2
| | |
| * | Add rake default to descriptionsanemat2012-12-241-1/+1
| | |
* | | Merge pull request #8641 from arunagw/warning_removed_unused_varSteve Klabnik2012-12-281-1/+2
|\ \ \ | | | | | | | | Remove 'assigned but unused variable' warning
| * | | Remove 'assigned but unused variable' warningArun Agrawal2012-12-281-1/+2
| | | |
* | | | Introduce assert_not to replace 'assert !foo'Jeremy Kemper2012-12-283-0/+39
|/ / /
* | | Consistent punctuation for @jaroslawr :bowtie:Jeremy Kemper2012-12-271-1/+1
| | |
* | | Merge pull request #8634 from senny/rewrite_order_dependent_test_caseRafael Mendonça França2012-12-271-1/+2
|\ \ \ | | | | | | | | rewrite order dependent test case
| * | | rewrite order dependent test case. #8185Yves Senn2012-12-271-1/+2
|/ / / | | | | | | | | | | | | | | | As reported (https://github.com/rails/rails/pull/8185#issuecomment-11702226) this test relied on the order a hash was serialized. Comparing the parsed hash makes the test no longer order dependent.
* | | Merge pull request #8627 from schneems/schneems/routes-1-9Rafael Mendonça França2012-12-261-1/+1
|\ \ \ | | | | | | | | Ruby 1.9 style hash in Route Inspector
| * | | Ruby 1.9 style hash in Route Inspectorschneems2012-12-271-1/+1
| | | |
* | | | Make test logs easier to read.Jeremy Kemper2012-12-262-13/+11
| | | | | | | | | | | | | | | | | | | | | | | | Tagging every message in tests makes the logs really wide. It's great for grepping, but annoying to open in an editor or a narrow terminal. Try out a different approach: spit out a heading before each test.
* | | | Extract the component rendering of collection_radio_buttons andRafael Mendonça França2012-12-262-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | collection_check_boxes to a private method. It will make easier to extend. Example: class CollectionRadioButtons < ActionView::Helpers::Tags::CollectionRadioButtons include CollectionExtensions def render wrap_rendered_collection(super, @options) end private def render_component(builder) builder.radio_button + builder.label(:class => "collection_radio_buttons") end end
* | | | Fix collection_radio_buttons with the option `:checked` with value ofRafael Mendonça França2012-12-262-1/+8
| | | | | | | | | | | | | | | | `false`
* | | | This change doesn't need a CHANGELOG entry.Rafael Mendonça França2012-12-261-5/+0
| | | | | | | | | | | | | | | | [ci skip]
* | | | Add CHANGELOG entry for #8622Rafael Mendonça França2012-12-262-2/+7
| | | |
* | | | Merge pull request #8598 from NARKOZ/refactor-highlightSantiago Pastorino2012-12-261-2/+2
|\ \ \ \ | | | | | | | | | | refactor highlight method: return fast if text or phrases are blank
| * | | | fetch highlighter option only when neededNihad Abbasov2012-12-241-2/+2
| | | | |