Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Update links to use https instead of http [ci skip] | Yoshiyuki Hirano | 2017-08-22 | 5 | -6/+6 |
| | |||||
* | Merge pull request #30352 from yhirano55/update_licenses_link | Ryuta Kamizono | 2017-08-22 | 12 | -15/+14 |
|\ | | | | | Update MIT licenses link [ci skip] | ||||
| * | Update MIT licenses link [ci skip] | Yoshiyuki Hirano | 2017-08-22 | 12 | -15/+14 |
| | | |||||
* | | Add frozen string literal pragma | George Claghorn | 2017-08-21 | 1 | -0/+2 |
| | | |||||
* | | Encode Content-Disposition filenames according to RFC 2231 | George Claghorn | 2017-08-21 | 6 | -4/+74 |
| | | | | | | | | Closes #30134. | ||||
* | | Merge pull request #30349 from ydakuka/patch-1 | Ryuta Kamizono | 2017-08-22 | 1 | -4/+4 |
|\ \ | | | | | | | Update debugging rails applications guide [ci skip] | ||||
| * | | Update debugging_rails_applications [ci skip] | Yauheni Dakuka | 2017-08-21 | 1 | -4/+4 |
|/ / | |||||
* | | Merge pull request #28912 from kaspth/getting-started-form-with | Kasper Timm Hansen | 2017-08-21 | 2 | -67/+100 |
|\ \ | | | | | | | [ci skip] form_with in the getting started guide. | ||||
| * | | [ci skip] form_with in the getting started guide. | Kasper Timm Hansen | 2017-08-21 | 2 | -67/+100 |
| | | | | | | | | | | | | Add back a bit about a resource oriented style of routing. | ||||
* | | | Merge pull request #30346 from ↵ | Ryuta Kamizono | 2017-08-22 | 2 | -2/+2 |
|\ \ \ | |_|/ |/| | | | | | | | | yhirano55/fix_broken_link_to_mysql_docs_in_database_yml Fix broken link to mysql docs in database.yml [ci skip] | ||||
| * | | Fix broken link to mysql docs in database.yml [ci skip] | Yoshiyuki Hirano | 2017-08-22 | 2 | -2/+2 |
|/ / | |||||
* | | Merge pull request #30345 from yhirano55/fix_link_contributing_guide | Eileen M. Uchitelle | 2017-08-21 | 1 | -7/+7 |
|\ \ | | | | | | | Fix link in Contributing Ruby on Rails guide [ci skip] | ||||
| * | | Fix link Contributing Ruby on Rails [ci skip] | Yoshiyuki Hirano | 2017-08-21 | 1 | -7/+7 |
|/ / | | | | | | | | | | | | | | | | | * Add Contributing to the Rails Code link. * Wiki link for Spanish is redirect. * The base branch of Polish repo is master. And Turkish is, too. * Unified expression `rails-dev-box` * The Clone url is wrong. * The mailing list url is old. | ||||
* | | Merge pull request #30344 from yhirano55/update_guide_contributing_ruby_on_rails | प्रथमेश Sonpatki | 2017-08-21 | 1 | -1/+1 |
|\ \ | | | | | | | Update Contributing to Ruby on Rails guide [ci skip] | ||||
| * | | Update Contributing to Ruby on Rails guide [ci skip] | Yoshiyuki Hirano | 2017-08-21 | 1 | -1/+1 |
|/ / | |||||
* | | Merge pull request #30343 from yhirano55/update_generator_guide | Vipul A M | 2017-08-21 | 1 | -4/+8 |
|\ \ | | | | | | | Update generator guide [ci skip] | ||||
| * | | Update generator guide [ci skip] | Yoshiyuki Hirano | 2017-08-21 | 1 | -4/+8 |
| | | | |||||
* | | | Prevent extra `SET time zone` in `configure_connection` (#28413) | Ryuta Kamizono | 2017-08-21 | 2 | -6/+15 |
| | | | | | | | | | | | | | | | | | | | | | | | | `SET time zone 'value'` is an alias for `SET timezone TO 'value'`. https://www.postgresql.org/docs/current/static/sql-set.html So if `variables["timezone"]` is specified, it is enough to `SET timezone` once. | ||||
* | | | Merge pull request #30341 from yhirano55/update_rails_on_rack_guide | प्रथमेश Sonpatki | 2017-08-21 | 1 | -1/+1 |
|\ \ \ | |/ / |/| | | Update Rails on Rack guide [ci skip] | ||||
| * | | Update Rails on Rack guide [ci skip] | Yoshiyuki Hirano | 2017-08-21 | 1 | -1/+1 |
|/ / | |||||
* | | Merge pull request #30332 from y-yagi/fix_29306 | George Claghorn | 2017-08-20 | 7 | -17/+44 |
|\ \ | | | | | | | Make `restart` and `dev:cache` tasks work when customizing pid file path | ||||
| * | | Make `restart` and `dev:cache` tasks work when customizing pid file path | yuuji.yaginuma | 2017-08-21 | 7 | -17/+44 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Originally, it hard-coded pid file path. It can not be removed when customizing pid file path. But rake task can not get pid file path. Therefore, do not remove file in rake task, makes it possible to judge whether it is restart from the argument of the command and removes the file in server command. Fixes #29306 | ||||
* | | | Merge pull request #30340 from ↵ | Ryuta Kamizono | 2017-08-21 | 1 | -1/+10 |
|\ \ \ | | | | | | | | | | | | | | | | | yhirano55/update_autoloading_and_reloading_constants Update Autoloading and Reloading Constants guide [ci skip] | ||||
| * | | | Update result of autoload_paths in Autoloading and Reloading Constants guide ↵ | Yoshiyuki Hirano | 2017-08-21 | 1 | -1/+10 |
|/ / / | | | | | | | | | | [ci skip] | ||||
* | | | Update regex | George Claghorn | 2017-08-20 | 1 | -1/+1 |
| | | | |||||
* | | | Fix tests | George Claghorn | 2017-08-20 | 4 | -4/+4 |
| | | | |||||
* | | | Fix syntax error | George Claghorn | 2017-08-20 | 1 | -1/+1 |
| | | | |||||
* | | | DRY | George Claghorn | 2017-08-20 | 8 | -16/+20 |
| | | | |||||
* | | | Rename ActiveStorage::Filename#extname to extension_with_delimiter | George Claghorn | 2017-08-20 | 2 | -9/+29 |
| | | | |||||
* | | | Merge pull request #30339 from bogdanvlviv/add-activestorage_bin_test-file | George Claghorn | 2017-08-20 | 1 | -0/+5 |
|\ \ \ | | | | | | | | | Add executable file `activestorage/bin/test` | ||||
| * | | | Add executable file `activestorage/bin/test` | bogdanvlviv | 2017-08-20 | 1 | -0/+5 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``` rails$ cd activestorage/ rails/activestorage$ bin/test ........(compressed).... rails/activestorage$ bin/test test/controllers/disk_controller_test.rb .(compressed).. rails/activestorage$ bin/test test/controllers/disk_controller_test.rb:42 . ``` | ||||
* | | | | Merge pull request #30121 from assain/add_expires_in_and_expires_at_to_cookies | Kasper Timm Hansen | 2017-08-20 | 3 | -5/+54 |
|\ \ \ \ | | | | | | | | | | | Add expiry metadata to Cookies and freshen expires option to support duration | ||||
| * | | | | freshen :expires option with duration support and add expiry metadata to cookies | Assain | 2017-08-20 | 3 | -5/+54 |
| | | | | | |||||
* | | | | | Require "active_support/core_ext/hash/compact" for `compact!` | Ryuta Kamizono | 2017-08-21 | 1 | -0/+2 |
| | | | | | |||||
* | | | | | Prefer formatted number as schema version [ci skip] | Ryuta Kamizono | 2017-08-21 | 1 | -1/+1 |
| | | | | | |||||
* | | | | | Place `update_table_definition` consistently in `SchemaStatements` | Ryuta Kamizono | 2017-08-21 | 6 | -12/+12 |
| | | | | | |||||
* | | | | | Don't expose `prepare_column_options` | Ryuta Kamizono | 2017-08-21 | 3 | -52/+30 |
| |/ / / |/| | | | | | | | | | | | | | | | This is only used for the internal `column_spec` and `column_spec_for_primary_key`. | ||||
* | | | | Merge pull request #30335 from ↵ | Ryuta Kamizono | 2017-08-20 | 1 | -13/+9 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | yhirano55/update_debugging_rails_applications_guide Update Debugging Rails Applications guide [ci skip] | ||||
| * | | | | Update Debugging Rails Applications guide [ci skip] | Yoshiyuki Hirano | 2017-08-20 | 1 | -13/+9 |
|/ / / / | |||||
* | | | | Register integer types limit correctly for postgresql adapter (#26386) | Ryuta Kamizono | 2017-08-20 | 1 | -15/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | currently integer types extracts the `limit` from `sql_type`. But the lookup key of type map is the `oid` in postgresql adapter. So in most case `sql_type` is passed to `extract_limit` as `""` and `limit` is extracted as `nil`. https://github.com/rails/rails/blob/v5.1.0.beta1/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb#L445 In mysql2 adapter, `limit` is registered correctly without extracting from `sql_type`. https://github.com/rails/rails/blob/v5.1.0.beta1/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb#L678-L682 Postgresql adapter should also be registered correctly. ``` ruby conn = ActiveRecord::Base.connection conn.select_all("SELECT 1::smallint, 2::integer, 3::bigint").column_types.map do |name, type| [name, type.limit] end ``` Before: ``` ruby # => [["int2", nil], ["int4", nil], ["int8", nil]] ``` After: ``` ruby # => [["int2", 2], ["int4", 4], ["int8", 8]] ``` | ||||
* | | | | Merge pull request #30331 from ↵ | George Claghorn | 2017-08-19 | 1 | -7/+10 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | yhirano55/update_rails_initialization_process_guide Update The Rails Initialization Process of guide [ci skip] | ||||
| * | | | | Update The Rails Initialization Process of guide [ci skip] | Yoshiyuki Hirano | 2017-08-20 | 1 | -7/+10 |
| | | | | | |||||
* | | | | | Merge pull request #30330 from ↵ | George Claghorn | 2017-08-19 | 1 | -4/+7 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | yhirano55/update_database_config_in_command_line_of_guide Update database config in The Rails Comamnd Line of guide [ci skip] | ||||
| * | | | | | Update database config in The Rails Comamnd Line of guide [ci skip] | Yoshiyuki Hirano | 2017-08-20 | 1 | -4/+7 |
| |/ / / / | |||||
* / / / / | Attributes are protected, not private, to avoid the warning | Matthew Draper | 2017-08-20 | 1 | -1/+2 |
|/ / / / | |||||
* | | | | Merge pull request #30325 from koic/fix_doc_format_in_active_storage_setup | Arun Agrawal | 2017-08-19 | 1 | -1/+1 |
|\ \ \ \ | | | | | | | | | | | Fix doc format in Active Storage Setup [ci skip] | ||||
| * | | | | Fix doc format in Active Storage Setup [ci skip] | Koichi ITO | 2017-08-19 | 1 | -1/+1 |
|/ / / / | | | | | | | | | | | | | Follow up of 2ebb284ef5bf2c36bd8ba0a4524cab83810f12ab. | ||||
* | | | | Merge pull request #30324 from koic/fix_cant_modify_frozen_string_in_ar_test | Matthew Draper | 2017-08-19 | 1 | -2/+2 |
|\ \ \ \ | | | | | | | | | | | Fix `can't modify frozen String` error in an AR test | ||||
| * | | | | Fix `can't modify frozen String` error in an AR test | Koichi ITO | 2017-08-19 | 1 | -2/+2 |
|/ / / / | |||||
* | | | | Merge pull request #30322 from zokioki/fix_action_dispatch_typo | Ryuta Kamizono | 2017-08-19 | 1 | -1/+1 |
|\ \ \ \ | | | | | | | | | | | Fix typo in "Ambiguous route definition" error message |