aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | 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
| | | | |
* | | | | Revert the change at ActiveModel::Errors#add_on_blank and fix in theRafael Mendonça França2012-12-263-14/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | right place. The EachValidator#validate already handle :allow_blank and :allow_nil, correctly. Closes #8622. Fix #8621.
* | | | | Tests and fix for validates_presence of :allow_nil, :allow_blankColin Kelley2012-12-262-1/+41
| | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activemodel/lib/active_model/errors.rb
* | | | | Note about migration timestamps.Steve Klabnik2012-12-261-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should mention that the ordering matters with regards to timestamps, and not imply that it's just for uniqueness purposes. Closes #8610.
* | | | | Consistent use of single and double quotesRafael Mendonça França2012-12-261-10/+10
| | | | |
* | | | | Merge pull request #8616 from senny/8612_respect_yaml_keywordsRafael Mendonça França2012-12-264-7/+48
|\ \ \ \ \ | | | | | | | | | | | | quote column names in generated fixture files
| * | | | | quote column names in generated fixture filesYves Senn2012-12-264-7/+48
| | | | | |
* | | | | | Fix CHANGELOG style [ci skip]Rafael Mendonça França2012-12-262-6/+6
| | | | | |
* | | | | | Merge pull request #5100 from ↵Rafael Mendonça França2012-12-263-6/+29
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | paukul/validate_on_condition_on_transaction_callbacks Validate :on option on after_commit and after_rollback callbacks
| * | | | | | validate :on option on after_commit and after_rollback callbacksPascal Friederich2012-12-263-6/+29
| | | | | | |
* | | | | | | We need to check explictly that env['action_dispatch.show_exceptions']Rafael Mendonça França2012-12-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | is false. If it is nil we can't raise the exception
* | | | | | | Merge pull request #8600 from shime/patch-1Rafael Mendonça França2012-12-261-4/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | more descriptive return parameters
| * | | | | | | more descriptive return parametersHrvoje Šimić2012-12-261-4/+3
| | | | | | | |
* | | | | | | | Revert "Ignore /bin on new apps" -- given the move to default binstubs, we ↵David Heinemeier Hansson2012-12-261-4/+3
| |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | want you to check those in! This reverts commit 35c554f0bf518e1068e79002a462c3deba649e9b.
* | | | | | | Merge pull request #8608 from ↵Rafael Mendonça França2012-12-252-1/+19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | senny/8576_cant_inherit_from_deprecated_buffered_logger deprecation warning when BufferedLogger is instantiated
| * | | | | | | deprecation warning when BufferedLogger is instantiatedYves Senn2012-12-252-1/+19
| | |/ / / / / | |/| | | | |
* | | | | | | Merge pull request #8609 from senny/buffered_logger_documentationGuillermo Iguaran2012-12-253-5/+5
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | replace AS::BufferedLogger doc mentions with AS::Logger
| * | | | | | replace AS::BufferedLogger doc mentions with AS::LoggerYves Senn2012-12-253-5/+5
|/ / / / / /
* | | | | | Merge pull request #8607 from ↵Rafael Mendonça França2012-12-252-2/+20
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | senny/8576_cant_inherit_from_deprecated_buffered_logger ActiveSupport::BufferedLogger can be subclassed
| * | | | | | ActiveSupport::BufferedLogger can be subclassedYves Senn2012-12-252-2/+20
|/ / / / / /
* | | | | | Merge pull request #8592 from rafaelfranca/warning_fixGuillermo Iguaran2012-12-241-1/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove warning of intance variable not initialized
| * | | | | | Remove warning of intance variable not initializedRafael Mendonça França2012-12-231-1/+5
| | | | | | |
* | | | | | | Merge pull request #8604 from zoltankiss/minor_fixesGuillermo Iguaran2012-12-241-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | minor grammar fix
| * | | | | | | minor grammar fixZoltan Kiss2012-12-241-1/+1
|/ / / / / / /
* | | / / / / silences "possibly useless use of :: in void context" warningsXavier Noria2012-12-241-7/+3
| |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The AS utility silence_warnings does not really silence this one, because it is issued at parse-time. It seemed to in some places because the constant was the only expression in the block and therefore it was its return value, that could potentially be used by silence_warnings are return value of the yield call. To bypass the warning we assign to a variable. The chosen variable is "_" because it is special-cased in parse.c not to issue an "assigned but unused variable" warning in turn.
* | | | | | Merge pull request #8601 from amatsuda/chmod_xRafael Mendonça França2012-12-242-0/+0
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | chmod -x from non-script files
| * | | | | chmod -x from non-script filesAkira Matsuda2012-12-242-0/+0
|/ / / / /
* | | | | Merge pull request #8595 from amatsuda/regex_warningRafael Mendonça França2012-12-231-1/+1
|\ \ \ \ \ | | | | | | | | | | | | "warning: ambiguous first argument; put parentheses or even spaces"
| * | | | | "warning: ambiguous first argument; put parentheses or even spaces"Akira Matsuda2012-12-241-1/+1
|/ / / / /
* | | | | Merge pull request #8594 from amatsuda/rm_sqlite2Guillermo Iguaran2012-12-231-13/+5
|\ \ \ \ \ | | | | | | | | | | | | I don't think we're supporting SQLite 2
| * | | | | we're not supporting SQLite 2Akira Matsuda2012-12-241-13/+5
| |/ / / /
* | | | | Explicit options should have precedence over railsrc on the `rails new`Rafael Mendonça França2012-12-242-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | command Closes #8545
* | | | | Merge pull request #8593 from shime/masterRafael Mendonça França2012-12-231-2/+0
|\ \ \ \ \ | |/ / / / |/| | | | another little documentation fix
| * | | | remove duplication from Rails::Application docsHrvoje Šimić2012-12-241-2/+0
|/ / / /
* | | | Merge pull request #8588 from marcandre/fix_reversibleCarlos Antonio da Silva2012-12-235-9/+24
|\ \ \ \ | | | | | | | | | | Fix Migration#reversible by not using `transaction`.
| * | | | Fixes for PR [#8267]Marc-Andre Lafortune2012-12-225-9/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix Migration#reversible by not using `transaction`. * Adapt mysql adapter to updated api for remove_column * Update test after aedcd683684d08eaf30623a4b48ce31a31426372
* | | | | Fix spacing regression in scaffold generatorYehuda Katz2012-12-221-1/+3
| | | | |
* | | | | Fix syntax error in controller templateGuillermo Iguaran2012-12-221-4/+12
| | | | |
* | | | | adds kindlerb to the :doc bundle group for guides generationXavier Noria2012-12-221-0/+1
| | | | |
* | | | | Ignore /bin on new appsJosé Valim2012-12-221-3/+4
| | | | |
* | | | | Forgot to check in railties fixture for binstubsYehuda Katz2012-12-221-0/+0
| | | | |
* | | | | Add --no-html to scaffold generatorYehuda Katz2012-12-225-8/+54
| | | | |
* | | | | Detect rbenv and update the shebangYehuda Katz2012-12-222-1/+18
| | | | |