aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | 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-175-5/+68
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-125-5/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | | | | Merge pull request #16526 from rails/jv-no-respondersJosé Valim2014-08-1714-1276/+16
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Move respond_with to the responders gem
| | * | | | | | Remove more references to respond_withJosé Valim2014-08-179-8/+7
| | | | | | | |
| | * | | | | | Remove usafe of respond_to in ActionView testsJosé Valim2014-08-171-3/+1
| | | | | | | |
| | * | | | | | 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.
| * | | | | | Merge pull request #16529 from y-yagi/patch-1Zachary Scott2014-08-171-1/+1
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | [ci skip] Fix sample code in Layouts and Rendering guide
| | * | | | | | [ci skip] Fix sample code in Layouts and Rendering guideyuuji.yaginuma2014-08-171-1/+1
| | | | | | | |
| * | | | | | | Allow usage of bundle local config for rack by specifying the branchCarlos Antonio da Silva2014-08-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
| * | | | | | | Merge pull request #16338 from robin850/rbx-safe-constantizeMatthew Draper2014-08-181-2/+2
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Avoid relying on error messages when rescuing
| | * | | | | | | Avoid relying on error messages when rescuingRobin Dupret2014-08-081-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we are rescuing from an error, it's a brittle approach to do checks with regular expressions on the raised message because it may change in in the future and error messages are different across implementations. The NameError API could be improved at the MRI level but for now we need to rely on its #name. A #== check will only pass for top level constants or only when the last constant of the path is missing so we need to rely on #include? instead. For instance: begin Namespace::Foo rescue NameError => e e.name # => :Namespace end However, if the name-space already exists, only the name of the first missing constant in the path is returned (e.g. for Math::PHI, the name would be :PHI). JRuby will return a fully qualified name (:"Math::PHI"). We need to keep the == check for 1.9 compatibility since const_get will raise a NameError with a name attribute set to the given string if it's one of "::" or "". See http://git.io/jnSN7g for further information.
| * | | | | | | | Merge pull request #16527 from jeremy/etag_with_template_digestJeremy Kemper2014-08-177-9/+136
| |\ \ \ \ \ \ \ \ | | |_|/ / / / / / | |/| | | | | | | When your templates change, browser caches bust automatically.
| | * | | | | | | 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
| * | | | | | | Merge pull request #16189 from sgrif/sg-attribute-was-in-placeGodfrey Chan2014-08-169-20/+67
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement `_was` and `changes` for in-place mutations of AR attributes
| | * | | | | | | Don't expose these new APIs yet (added in 877ea78 / #16189)Godfrey Chan2014-08-163-15/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | WARNING: don't use them! They might change or go away between future beta/RC/ patch releases! Also added a CHANGELOG entry for this.
| | * | | | | | | Implement `_was` and `changes` for in-place mutations of AR attributesSean Griffin2014-08-168-21/+62
| |/ / / / / / /
* / / / / / / / Fix rounding errors with #travel_to by resetting the usec on any passed time ↵David Heinemeier Hansson2014-08-173-4/+20
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | to zero, so we only travel with per-second precision, not anything deeper than that.
* | | | | | | Merge pull request #16517 from sgrif/sg-comparable-attributesRafael Mendonça França2014-08-164-0/+50
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Implement `==` on `Type::Value` and `Attribute`
| * | | | | | | Implement `==` on `Type::Value` and `Attribute`Sean Griffin2014-08-154-0/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was a small self contained piece of the refactoring that I am working on, which required these objects to be comparable.
* | | | | | | | Merge pull request #16523 from tomkadwill/cookie_store_rewordingRafael Mendonça França2014-08-161-3/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | [ci skip] re-worded section on CookieStore to make it more readable.
| * | | | | | | | [ci skip] re-worded section on CookieStore to make it more readable.Tom Kadwill2014-08-161-3/+3
|/ / / / / / / /
* | | | | | | | Avoid mutating the constants in a test caseAkira Matsuda2014-08-162-6/+4
| | | | | | | |
* | | | | | | | Avoid polluting String class globally in a test caseAkira Matsuda2014-08-161-8/+13
| | | | | | | |
* | | | | | | | Unused ivar in a test suiteAkira Matsuda2014-08-161-1/+0
| | | | | | | |
* | | | | | | | Merge pull request #16435 from ↵Godfrey Chan2014-08-154-4/+47
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | arthurnn/after_commit_run_on_top_level_trans_master Fix regression on after_commit in nested transactions. (master)
| * | | | | | | Fix regression on after_commit in nested transactions.Arthur Neves2014-08-153-0/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | after_commit should not run in nested transactions, however they should run once the outermost transaction gets committed. This patch fixes the problem copying the records from the Savepoint to its parent. So the RealTransaction will have all records that needs to run callbacks on it. [fixes #16425]
| * | | | | | | Use *_transaction methods in TransactionManagerArthur Neves2014-08-152-4/+26
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use `commit_transaction`/`rollback_transaction` on `within_new_transaction` method, so they make sure they `pop` the transaction from the stack before calling the methods `commit`/`rollback`.
* | | | | | | Point to rubygems instead of Rails GitHub. [ci skip]Rafael Mendonça França2014-08-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The rails repository is not the official plugin anymore
* | | | | | | Fixed link for in_place_editor [ci skip]Arun Agrawal2014-08-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | closes #16512
* | | | | | | fix typo in method name (broken build :sweat:)Yves Senn2014-08-151-1/+1
| | | | | | |
* | | | | | | prefer `has_attribute?` over `attributes.key?`.Yves Senn2014-08-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow up to the discussion on #16505.
* | | | | | | Merge pull request #16505 from sgrif/sg-attribute-internalsYves Senn2014-08-151-2/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Use the method for determining attribute methods rather than duplicating
| * | | | | | | Use the method for determining attribute methods rather than duplicatingSean Griffin2014-08-151-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've been trying to reduce the number of places that care about `attributes`, and its existence. We have a method for this check, let's use it instead.
* | | | | | | | create_table + transactional_fixtures = :bomb:Akira Matsuda2014-08-151-0/+4
| | | | | | | |
* | | | | | | | s/Dont'/Don't/Akira Matsuda2014-08-151-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #16508 from jbsmith86/masterSean Griffin2014-08-143-4/+4
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Spelling errors
| * | | | | | | Spelling errorsjbsmith862014-08-143-4/+4
|/ / / / / / /
* | | | | | | Merge pull request #16504 from schneems/schneems/fix_url_forAaron Patterson2014-08-141-1/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Perf optimization for `url_for` called w/ Hash
| * | | | | | | Perf optimization for `url_for` called w/ Hashschneems2014-08-141-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Benchmarking the existing code: ```ruby { :only_path => options[:host].nil? }.merge!(options.symbolize_keys)) ``` Against optimized code, that does not require a new hash or a merge: ```ruby options = options.symbolize_keys options[:only_path] = options[:host].nil? unless options.key?(:only_path) options ``` We see a statistically significant performance gain: ![](https://www.dropbox.com/s/onocpc0zfw4kjxl/Screenshot%202014-08-14%2012.45.30.png?dl=1) Updated to not mutate incoming parameters
* | | | | | | | Ignore MySQL "SHOW VARIABLES" when counting queriesAkira Matsuda2014-08-151-1/+1
| | | | | | | |
* | | | | | | | extract methods and metaprogram less.Aaron Patterson2014-08-141-11/+16
| | | | | | | |
* | | | | | | | Be sure to reset PK name renamed in the testAkira Matsuda2014-08-151-0/+3
| | | | | | | |
* | | | | | | | Use string for order argument, fixed from #16501 [ci skip]Zachary Scott2014-08-141-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #16501 from tomkadwill/update_delete_to_new_styleZachary Scott2014-08-141-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | [ci skip] updated 'where' in association documention to new style syntax
| * | | | | | | | [ci skip] updated 'where' in association documention to new style syntaxTom Kadwill2014-08-141-1/+1
| | | | | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-06 18:40:33 +0000