aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | | | | | | Removed more redundant lines. Removed fix_content block that circumvented a ↵Timm2014-06-151-7/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug in html-scanner.
| | * | | | | | | Removed more lines of code in assert_select.Timm2014-06-151-7/+6
| | | | | | | | |
| | * | | | | | | Simplified the first delete_if loop in assert_select to use Loofah's text ↵Timm2014-06-151-14/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | method.
| | * | | | | | | Removed argument and root variable in assert_select proc.Timm2014-06-151-4/+2
| | | | | | | | |
| | * | | | | | | Changed name to selector. And a bunch of other things.Timm2014-06-151-60/+51
| | | | | | | | |
| | * | | | | | | The first attempt at abstracting argument parsing from selection methods.Timm2014-06-151-81/+114
| | | | | | | | |
| | * | | | | | | Removed tag.rb since it has been deprecated.Timm2014-06-154-321/+0
| | | | | | | | |
| | * | | | | | | Corrected documentation bug.Timm2014-06-151-10/+57
| | | | | | | | |
| | * | | | | | | Removed whitespace between dependencies.Timm2014-06-151-0/+1
| | | | | | | | |
| | * | | | | | | Added Loofah as a dependency in actionview.gemspec.Timm2014-06-112-9/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implemented ActionView: FullSanitizer, LinkSanitizer and WhiteListSanitizer in sanitizers.rb. Deprecated protocol_separator and bad_tags. Added new tests in sanitizers_test.rb and reimplemented assert_dom_equal with Loofah.
* | | | | | | | | Merge pull request #16533 from lucasmazza/lm-named-routes-helperRafael Mendonça França2014-08-171-0/+5
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | | Deprecate NamedRouteCollection#helpers.
| * | | | | | | | Deprecate NamedRouteCollection#helpers.Lucas Mazza2014-08-171-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method was removed at 210b338db20b1cdd0684f40bd78b52ed16148b99 but it is used by third party gems to check if a named route was defined. To help on the upgrade path on 4.2.0 we bring it back and emit a deprecation warning.
* | | | | | | | | Merge pull request #16294 from bf4/code_toolsRafael Mendonça França2014-08-171-21/+3
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | Update, unify, encapsulate, and fix various code tools in Rails
| * | | | | | | | Encapsulate rake lines from ActiveRecord/ActionPack as CodeTools::LineStatisticsBenjamin Fleischer2014-07-251-21/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | | | Use AS::JSON for (de)serializing cookiesGodfrey Chan2014-08-173-2/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the Active Support JSON encoder for cookie jars using the `:json` or `:hybrid` serializer. This allows you to serialize custom Ruby objects into cookies by defining the `#as_json` hook on such objects. Fixes #16520.
* | | | | | | | | `responders` 1.x won't do it. Told you to RTFM for details!Godfrey Chan2014-08-172-3/+8
| | | | | | | | |
* | | | | | | | | The gem is called 'responders'Godfrey Chan2014-08-172-4/+4
| | | | | | | | |
* | | | | | | | | Raise a more helpful error for people who are using these extracted featuresGodfrey Chan2014-08-172-1/+46
| | | | | | | | |
* | | | | | | | | Fixed broken reference caused by 14965baGodfrey Chan2014-08-171-2/+2
| | | | | | | | |
* | | | | | | | | Merge pull request #16467 from strzalek/cookies-digest-config-option2Godfrey Chan2014-08-173-3/+64
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cookies digest config option (pt. 2) Conflicts: actionpack/CHANGELOG.md actionpack/lib/action_dispatch/middleware/cookies.rb
| * | | | | | | | | Add config option for cookies digestŁukasz Strzałkowski2014-08-123-3/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | You can now configure custom digest for cookies in the same way as `serializer`: config.action_dispatch.cookies_digest = 'SHA256'
* | | | | | | | | | Merge pull request #16484 from strzalek/remove-redundant-null-serializerGodfrey Chan2014-08-171-16/+7
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Remove redundant NullSerializer
| * | | | | | | | | | Remove redundant NullSerializerŁukasz Strzałkowski2014-08-131-16/+7
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use one from ActiveSupport::MessageEncryptor module.
* | | | | | | | | | Remove more references to respond_withJosé Valim2014-08-177-6/+0
| | | | | | | | | |
* | | | | | | | | | Move respond_with to the responders gemJosé Valim2014-08-174-1265/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | respond_with (and consequently the class-level respond_to) are being removed from Rails. Instead of moving it to a 3rd library, the functionality will be moved to responders gem (at github.com/plataformatec/responders) which already provides some responders extensions.
* | | | | | | | | | When your templates change, browser caches bust automatically.Jeremy Kemper2014-08-177-9/+136
| |_|_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | New default: the template digest is automatically included in your ETags. When you call `fresh_when @post`, the digest for `posts/show.html.erb` is mixed in so future changes to the HTML will blow HTTP caches for you. This makes it easy to HTTP-cache many more of your actions. If you render a different template, you can now pass the `:template` option to include its digest instead: fresh_when @post, template: 'widgets/show' Pass `template: false` to skip the lookup. To turn this off entirely, set: config.action_controller.etag_with_template_digest = false
* | | | | | | | | extract methods and metaprogram less.Aaron Patterson2014-08-141-11/+16
| | | | | | | | |
* | | | | | | | | remove deprecated `MissingHelperError` proxy.Yves Senn2014-08-142-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The error was moved outside of the `ClassMethods` module.
* | | | | | | | | Merge pull request #16027 from tgxworld/template_assertionsYves Senn2014-08-143-0/+36
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fixes to ActionController::TemplateAssertions
| * | | | | | | | | Fix assert_template for files.Guo Xiang Tan2014-08-143-0/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test was not failing for `assert_template file: nil` when a file has been rendered.
* | | | | | | | | | ask the scope object if it is a resource_method_scopeAaron Patterson2014-08-131-10/+13
| | | | | | | | | |
* | | | | | | | | | ask the scope for the action nameAaron Patterson2014-08-131-14/+18
| | | | | | | | | |
* | | | | | | | | | reduce calls to scope_levelAaron Patterson2014-08-131-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this will help us to encapsulate magical symbols so hopefully we can eliminate hardcoded magic symbols
* | | | | | | | | | change to attr_readerAaron Patterson2014-08-131-5/+1
| | | | | | | | | |
* | | | | | | | | | scope_level is no longer a hash key, just use the ivarAaron Patterson2014-08-131-4/+9
| | | | | | | | | |
* | | | | | | | | | move the scope level key fully inside the scope objectAaron Patterson2014-08-131-1/+5
| | | | | | | | | |
* | | | | | | | | | move scope_level to a method on the scope objectAaron Patterson2014-08-131-7/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | now we don't have to have a hard coded key
* | | | | | | | | | only look up scope level onceAaron Patterson2014-08-131-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | avoid hash lookups and remove depency on the instance
* | | | | | | | | | only test `prefix` onceAaron Patterson2014-08-131-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | we don't need to repeat if statements
* | | | | | | | | | pass consistent parameters to canonical_action?Aaron Patterson2014-08-131-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | now we only have to look up @scope[:scope_level] once per call to canonical_action? and we don't have a variable named "flag"
* | | | | | | | | | fewer operations on the options hashAaron Patterson2014-08-132-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | since we pass `as` down, then we won't have to do an insert / delete dance with the options hash
* | | | | | | | | | this should be accessing the hash, not calling a methodAaron Patterson2014-08-131-1/+1
| | | | | | | | | |
* | | | | | | | | | UnexpectedErrors may reference exceptions that can't be dumpedAaron Patterson2014-08-131-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | UnexpectedError exceptions wrap the original exception, and the original exception may contain a reference to something that can't be marshal dumped which will cause the process to die.
* | | | | | | | | | [ci skip] correct default cache store classAditya Kapoor2014-08-131-1/+1
|/ / / / / / / / /
* | | | | | | | | Merge pull request #16491 from akshay-vishnoi/doc-changesYves Senn2014-08-131-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | [ci skip] fix spelling of overridden
| * | | | | | | | | [ci skip] fix spelling of overrideAkshay Vishnoi2014-08-131-1/+1
| | |_|_|_|/ / / / | |/| | | | | | |
* / | | | | | | | AM, AP, AV, and AMo tests are already order_independent!Akira Matsuda2014-08-131-5/+0
|/ / / / / / / /
* | | | | | | | Merge pull request #16476 from akshay-vishnoi/doc-changesRafael Mendonça França2014-08-122-2/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | [ci skip] use "based on" or "based off of"
| * | | | | | | | use 'based on' instead of 'based off' [ci skip]Akshay Vishnoi2014-08-122-2/+2
| | | | | | | | |
* | | | | | | | | Nobody sucks so nobody should call this awful method nameRafael Mendonça França2014-08-121-1/+1
|/ / / / / / / /