| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
Style/SpaceBeforeBlockBraces
Style/SpaceInsideBlockBraces
Style/SpaceInsideHashLiteralBraces
Fix all violations in the repository.
|
| |
|
| |
|
|
|
|
|
|
| |
Reason: See https://github.com/rails/rails/pull/26163#issuecomment-239703322
This reverts commit 39effc857e5c774670d6fad1a26aebcc33c51f0a.
|
|\
| |
| | |
Clarify and fix typos in autoloading disabled section
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The Guides section about autoloading being disabled was slightly confusing
(#24724) and didn't directly reference the removed feature by name
(config.autoload_paths) making it much harder for someone to search the upgrade
guides for a mention or serendipitously find it via a Google search when running
into autoloading issues.
I also fixed some confusing turns of phrase and a missing word.
/cc @vipulnsward @jvanbaarsen
|
|/
|
|
| |
- It is Action Controller config, not Action View.
|
| |
|
| |
|
|\
| |
| | |
Minor doc fix related to ActiveModel::SecurePassword [ci skip]
|
| | |
|
| | |
|
|/
|
|
| |
Hash syntax auto-correcting breaks alignments. 411ccbdab2608c62aabdb320d52cb02d446bb39c
|
|\
| |
| | |
Document know limitation about using `references` in conjunction with custom select clauses [ci skip]
|
| |
| |
| |
| |
| |
| |
| |
| | |
loading causing it to ignore custom select clauses.
[ci skip]
Fixes #24314
|
|\ \
| |/
|/| |
Add link to 'Pry' in debugging guide [ci skip]
|
| | |
|
| |
| |
| |
| | |
Follow up to fe859a54219740fa8b4e09a592820d2ee12ba222
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implement naive partial caching mechanism.
Add test for LogSubscriber
Use ActionView::Base#log_payload to store log_subscriber's payload, so we can pass cache result into it.
Fixed tests
Remove useless settings
Check if #log_payload exists before calling it. Because other classes also includes CacheHelper but don't have is attribute
Use @log_payload_for_partial_reder instead of #log_payload to carry ActionView's payload.
Update test's hash syntax
Add configuration to enable/disable fragment caching logging
Remove unless test and add new test to ensure cache info won't effect next rendering's log
Move :enable_fragment_cache_logging config from ActionView to ActionPack
Apply new config to tests
Update actionview's changelog
Update configuration guide
Improve actionview's changelog
Refactor PartialRenderer#render and log tests
Mute subscriber's log instead of disabling instrumentation.
Fix typo, remove useless comment and use new hash syntax
Improve actionpack's log_subscriber test
Fix rebase mistake
Apply new config to all caching intstrument actions
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
The current code base is not uniform. After some discussion,
we have chosen to go with double quotes by default.
|
|\
| |
| | |
Correct usage for acceptance validator in guide
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Due to the limited support for inotify (Microsoft/BashOnWindows#216) in
WSL the best thing to do at moment is add a note about skipping spring
and listen when generating your application in the Getting Started guide.
[skip ci]
Fixes #26054.
|
| |
| |
| |
| |
| |
| |
| | |
Some Rails core members agreed to continue to maintain 4.2 until 5.1 is
released.
[ci skip]
|
|\ \
| |/
|/| |
|
| |
| |
| |
| | |
That file has been removed and replaced with new_framework_defaults.rb
|
| |
| |
| |
| | |
`_key_[]` [ci skip]
|
| | |
|
| |
| |
| |
| |
| |
| | |
guides [ci skip]
- Followup of #25692.
|
| | |
|
|\ \
| | |
| | | |
[ci skip] Add documentation for error using select with order
|
| | | |
|
|/ /
| |
| |
| |
| |
| | |
ActionController::UnknownFromat in ArticlesController#new
Restored texts about XML and JS template handlers. [ci skip]
|
|\ \
| | |
| | | |
Add proper description for `Rails.cache.fetch` [ci skip]
|
| | | |
|
|\ \ \
| | | |
| | | | |
Explain meaning of the code very well and Remove confusion. [ci skip]
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
Code like `render(topics) => render("topics/topic")` adds confusion
for the readers. On first impression, that code feels like hash kind
of data, but it's not. So make it more clear and meaningful.
|
|/ /
| |
| |
| |
| |
| | |
See the rationale in the documentation included in this patch.
We are going to gradually introduce this predicate in the code base.
|
|\ \
| | |
| | |
| | | |
Setup default session store internally, no longer through an initializer
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
initializer
- By default the session store will be set to cookie store with
application name as session key.
- Older apps are not affected as they will have the session store
initializer generated by Rails in older versions, and Rails will not
overwrite the session store if it is already set or disabled.
- But new apps will not have the initializer, instead the session store
will be set to cookie store by default.
- Based on comment by DHH here - https://github.com/rails/rails/issues/25181#issuecomment-222312764.
|
|\ \
| | |
| | | |
Update ActionCable Rebroadcasting a Message documentation
|
| |/
| |
| |
| | |
Replace broadcast_to with ActionCable.server.broadcast to be inline with its partner, #stream_from
|
| |
| |
| |
| | |
- Followup of https://github.com/rails/rails/commit/451437c6f57e66cc7586ec966e530493927098c7
|
| |
| |
| |
| |
| |
| | |
In #23179 the migration generator was changed to no longer output `index: true` for `references` migrations. This updates the migrations guide to remove `index: true` from relevant examples.
[ci skip]
|
| |
| |
| |
| | |
Also modify to use Ruby 1.9+ syntax.
|