aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Make attributes output in html opt-in.Kasper Timm Hansen2016-12-181-1/+1
| | | | | | | | | | By using `except` any option provided to `form_with` would become an html attribute unless explicitly opted out. This proved annoying when adding new internal keys in the options hash and seeing them pop up in the html output. Prevent this by inverting the `except` to `slice` out the keys we want to allow becoming html attributes, like id and class.
* Merge pull request #27396 from bianjp/fix-kindlerbXavier Noria2016-12-184-13/+10
|\ | | | | Fix rake guides:generate:kindle error
| * Fix rake guides:generate:kindle errorBian Jiaping2016-12-184-13/+10
|/
* Merge pull request #27366 from utilum/avoid_fixnum_warningMatthew Draper2016-12-181-1/+1
|\ | | | | Fix Fixnum deprecated warning in Ruby 2.4+
| * Fix Fixnum deprecated warning in Ruby 2.4+utilum2016-12-161-1/+1
| |
* | Merge pull request #27329 from kamipo/simplify_unsigned_regexEileen M. Uchitelle2016-12-172-3/+2
|\ \ | | | | | | Simplify the regex for `unsigned?` method
| * | Simplify the regex for `unsigned?` methodRyuta Kamizono2016-12-112-3/+2
| | | | | | | | | | | | | | | It is enough to distinguish only the trailing `unsigned` and `unsigned zerofill`.
* | | Merge pull request #27339 from utilum/puts_adapterEileen M. Uchitelle2016-12-171-0/+1
|\ \ \ | | | | | | | | Improve clarity of AJ tests output
| * | | Output adapter being tested.utilum2016-12-121-0/+1
| | | |
* | | | How to disable scaffolds.scss in generators.md (#27383)Eike Send2016-12-171-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | * How to disable scaffolds.scss in generators.md * Improves workflow customization text in generators.md
* | | | Merge pull request #27393 from elpic/masterप्रथमेश Sonpatki2016-12-171-0/+6
|\ \ \ \ | | | | | | | | | | Add missing option on number helper
| * | | | Add missing option on numbers helperPablo Ifran2016-12-171-0/+6
|/ / / / | | | | | | | | | | | | | | | | The `delimiter_pattern` option was not present on the documentaion of the helper.
* | | | Merge pull request #27387 from MSP-Greg/masterRafael França2016-12-164-8/+8
|\ \ \ \ | | | | | | | | | | Change ActiveModel::Type::Helpers to :nodoc: [ci skip]
| * | | | Change ActiveModel::Type::Helpers to :nodoc: [ci skip]MSP-Greg2016-12-164-8/+8
| | | | |
* | | | | Merge pull request #27386 from kevinhughes27/add-generator-command-line-arg-docsRafael França2016-12-161-0/+20
|\ \ \ \ \ | | | | | | | | | | | | Docs Command Line Arguments for Generators
| * | | | | [ci skip] add a section explaining command line arguments for generatorsKevin Hughes2016-12-161-0/+20
| | | | | |
* | | | | | Merge pull request #27375 from kirs/fixture-error-messageRafael França2016-12-163-4/+20
|\ \ \ \ \ \ | | | | | | | | | | | | | | Throw friendly error message when fixture is not a hash
| * | | | | | Throw friendly error message when fixture is not a hashKir Shatrov2016-12-153-4/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Right now, when fixture is not a Hash we throw an error message saying "fixture is not a hash". This is not very user friendly because it's not saying which fixture is invalid.
* | | | | | | Bump coffee-script-source to fix bugeileencodes2016-12-161-1/+1
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Coffee script 1.12.1 was released and contained a bug where coffee script was undefined. 1.12.1 was yanked and 1.12.2 was released. This bumps coffee-script-source so that CI Railties tests pass. See: https://github.com/jashkenas/coffeescript/issues/4403
* | | | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2016-12-161-1/+1
|\ \ \ \ \ \
| * | | | | | Remove mention of SafeErb gem [ci skip]Prathamesh Sonpatki2016-11-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Followup of https://github.com/rails/rails/pull/27086
* | | | | | | Merge pull request #27368 from matthewd/doubled-callbacksMatthew Draper2016-12-162-18/+77
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Support double-yield inside an around callback
| * | | | | | Support double-yield inside an around callbackMatthew Draper2016-12-152-18/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's questionable whether this is a good thing -- it forces any later/ inner callback to handle multiple invocations, along with the actual wrapped action. But it worked prior to 871ca21f6a1d65c0ec78cb5a9641411e2210460b, so we shouldn't break it unintentionally.
* | | | | | | Merge pull request #27341 from richardmonette/fix-querycache-nil-dupRafael França2016-12-155-5/+34
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | fix QueryCache nil dup
| * | | | | | | fix QueryCache nil dupRichard Monette2016-12-155-5/+34
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | make sql statements frozen dup if arel is not our string expect runtime error dont wrap runtime error in invalid log errors will now be treated as runtime errors update changelog
* | | | | | | Merge pull request #27367 from georgeclaghorn/local-form-withKasper Timm Hansen2016-12-152-4/+10
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Avoid invalid attribute on local forms generated by `form_with`
| * | | | | | | Avoid invalid attribute on local forms generated by `form_with`George Claghorn2016-12-152-4/+10
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes that the following ERB template would result in invalid HTML output: <%= form_with model: Post.new, local: true do |form| %> <% end %> Specifically, the resulting form tag would have a spurious `remote` attribute: <form remote="false" ...>
* / | | | | | Add missing spaces to button_tag api doc [ci skip] (#27369)Andreas Maierhofer2016-12-151-1/+1
|/ / / / / /
* | | | | | Merge pull request #27358 from utilum/27347Jon Moss2016-12-141-0/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | Mention JS Runtime dependency in dev dependencies guide [ci skip].
| * | | | | | Mention JS Runtime dependency in dev dependencies guide [ci skip].utilum2016-12-141-0/+4
| | | | | | |
* | | | | | | Merge pull request #27354 from Shopify/fix-constantize-and-prepended-modulesRafael França2016-12-144-1/+16
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix constantize edge case involving prepend, autoloading and name conflicts
| * | | | | | | Fix constantize edge case involving prepend, autoloading and name conflictsJean Boussier2016-12-144-1/+16
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the following situation: ```ruby class Bar end module Baz end class Foo prepend Baz end class Foo::Bar end ``` Running `Inflector.constantize('Foo::Bar')` would blow up with a NameError. What is happening is that `constatize` was written before the introduction of prepend, and wrongly assume that `klass.ancestors.first == klass`. So it uses `klass.ancestors.inject` without arguments, as a result a prepended module is used in place of the actual class.
* | | | | | | Merge pull request #27353 from mtsmfm/remove-unused-methodEileen M. Uchitelle2016-12-141-4/+0
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Remove unused method `namespaced_file_path`
| * | | | | | Remove unused method `namespaced_file_path`Fumiaki MATSUSHIMA2016-12-141-4/+0
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``` $ git grep namespaced_file_path railties/lib/rails/generators/named_base.rb: def namespaced_file_path railties/lib/rails/generators/named_base.rb: @namespaced_file_path ||= namespaced_class_path.join("/") ```
* | | | | | Merge pull request #27350 from y-yagi/use_appropriate_type_for_rc_optionKasper Timm Hansen2016-12-141-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | use appropriate type for `rc` option
| * | | | | | use appropriate type for `rc` optionyuuji.yaginuma2016-12-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the following warning. ``` Expected boolean default value for '--rc'; got "~/.railsrc" (string) ```
* | | | | | | Missing require "active_support/testing/constant_lookup"Akira Matsuda2016-12-141-0/+1
| | | | | | |
* | | | | | | Merge pull request #27349 from y-yagi/fix_ruby_warning_in_ruby_2_4Rafael França2016-12-132-2/+2
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | fix new warning in ruby 2.4
| * | | | | | fix new warning in ruby 2.4yuuji.yaginuma2016-12-142-2/+2
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the following warning. ``` test/caching_test.rb:986: warning: parentheses after method name is interpreted as test/caching_test.rb:986: warning: an argument list, not a decomposed argument test/cases/adapters/mysql2/reserved_word_test.rb:146: warning: parentheses after method name is interpreted as test/cases/adapters/mysql2/reserved_word_test.rb:146: warning: an argument list, not a decomposed argument ``` Ref: https://github.com/ruby/ruby/commit/65e27c8b138d6959608658ffce2fa761842b8d24
* | | | | | form_with/fields: Don't output ids by defaultKasper Timm Hansen2016-12-134-206/+243
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Continuing 67f81cc where we decided not to output ids by default in the new form helpers. Went with @dhh's suggestion of just requiring ids on fields being labelled: https://github.com/rails/rails/issues/25197#issuecomment-231797117 Seems okay enough.
* | | | | | Add CHANGELOG entry to #27042Rafael Mendonça França2016-12-131-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | Merge pull request #27042 from kirs/yaml-schema-cacheRafael França2016-12-137-17/+435
|\ \ \ \ \ \ | | | | | | | | | | | | | | Schema cache in YAML
| * | | | | | Use YAML to serialize schema cacheKir Shatrov2016-11-277-17/+435
| | | | | | |
* | | | | | | Merge pull request #27293 from y-yagi/fix_duplicable_with_2_4Kasper Timm Hansen2016-12-133-35/+71
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | change return value of `duplicable?` with Ruby 2.4+
| * | | | | | change return value of `duplicable?` with Ruby 2.4+yuuji.yaginuma2016-12-133-35/+71
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `NilClass`, `FalseClass`, `TrueClass`, `Symbol` and `Numeric` can dup with Ruby 2.4+. Ref: https://bugs.ruby-lang.org/issues/12979
* | | | | | Don't assign a default to `webpack`.Kasper Timm Hansen2016-12-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unintentionally makes `--webpack` implied on `rails new apper`. If passed `--webpack` Thor assigns `"webpack"` to `options[:webpack]`, so we can check for that instead of `"base"`.
* | | | | | Add option to trigger JS framework-specific installation as part of webpack ↵David Heinemeier Hansson2016-12-122-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | setup
* | | | | | :nail_care:Akira Matsuda2016-12-131-2/+1
| | | | | |
* | | | | | nil, true, 1, etc. don't raise on #dup since Ruby 2.4Akira Matsuda2016-12-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | https://bugs.ruby-lang.org/issues/12979
* | | | | | Keep AS::XmlMini::PARSING["decimal"].call('') returning 0Akira Matsuda2016-12-131-1/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BigDecimal('an invalid string') has changed its behavior to raise an ArgumentError since 1.3.0 https://bugs.ruby-lang.org/issues/10286
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-18 14:32:59 +0000