aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #9104 from bemurphy/remove_bad_idea_parser_testCarlos Antonio da Silva2013-01-291-13/+0
|\ | | | | Remove yaml Proc param parser test
| * Remove yaml Proc param parser testBrendon Murphy2013-01-281-13/+0
| | | | | | | | | | | | | | | | I don't believe this test is exercising any explicit params_parser behavior that the other two Proc tests aren't already doing. Given that we now know it's a bad idea to load user input via YAML.load, somebody reading this test might get a dangerous idea about building out a YAML params parser.
* | Needs to explicitly require 'minitest/parallel_each'Akira Matsuda + Koichi Sasada2013-01-291-0/+2
| | | | | | | | Minitest has changed to require this since https://github.com/seattlerb/minitest/commit/728054b
* | s/ERb/ERB/Akira Matsuda2013-01-291-1/+1
|/
* Send paths argument as an array rather than converting it internallyCarlos Antonio da Silva2013-01-281-3/+3
|
* Refactor to use each_key, remove extra spacesCarlos Antonio da Silva2013-01-287-11/+5
|
* add API to pg for enabling / disabling hstoreAaron Patterson2013-01-283-7/+54
|
* Merge pull request #9096 from tricknotes/fix-code-commentRafael Mendonça França2013-01-281-4/+4
|\ | | | | Fix code comment
| * Fix JavaScript syntax in code comment [ci skip]Ryunosuke SATO2013-01-291-4/+4
| |
* | revamps the RDoc of Object#tryXavier Noria2013-01-281-22/+30
| |
* | Move AS changelog entry to the top, improve AP changelog a bitCarlos Antonio da Silva2013-01-282-6/+6
| | | | | | | | [ci skip]
* | no need for \Z, \z is more conciseXavier Noria2013-01-281-1/+1
| |
* | Merge pull request #8830 from antoinelyset/masterXavier Noria2013-01-284-5/+13
|\ \ | | | | | | Improve String#squish whitespaces matching
| * | Improve String#squish whitespaces matchingAntoine Lyset2013-01-224-5/+13
| | |
* | | Merge pull request #9098 from firmhouse/add-head-requestsRafael Mendonça França2013-01-284-5/+12
|\ \ \ | | | | | | | | Make current_url? work with HEAD requests and remove caching_allowed? since it doesn't do anything
| * | | Added a Changelog entry for fixing HEAD requests.Michiel Sikkes2013-01-281-0/+5
| | | |
| * | | Make current_url? work with a HEAD methodMichiel Sikkes2013-01-282-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | ActionDispatch::Head was removed in favor of Rack::Head. But Rack::Head does not convert GET requests to HEAD requests so we need to do checking for HEAD requests ourselves.
| * | | Remove caching_allowed? from ActionController::CachingMichiel Sikkes2013-01-281-4/+0
|/ / / | | | | | | | | | | | | Where is this used? No other code references to this method and it isn't being tested anywhere. No tests fail when commented out.
* | | updates a dynamic method heredoc docXavier Noria2013-01-281-2/+2
| | |
* | | explains why the query cache checks arel.lockedXavier Noria2013-01-281-0/+2
| |/ |/|
* | Merge pull request #9032 from firmhouse/head-breaks-csrfSantiago Pastorino2013-01-282-2/+6
|\ \ | | | | | | Make HEAD work / convert to GET once more
| * | Added request.head? to forgery protection codeMichiel Sikkes2013-01-221-2/+2
| | |
| * | Added a test that shows that a HEAD request does not normally pass CSRF ↵Michiel Sikkes2013-01-221-0/+4
| | | | | | | | | | | | protection
* | | Use Encoding::UTF_8 constant :do_not_litter:Akira Matsuda2013-01-2810-15/+15
| | |
* | | Use already defined Encoding constants rather than creating one-trip StringsAkira Matsuda2013-01-283-3/+3
| | |
* | | add fetch to CookieJarAaron Patterson2013-01-272-0/+34
| | |
* | | Prevent Relation#merge from collapsing wheres on the RHSJon Leighton2013-01-279-22/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This caused a bug with the new associations implementation, because now association conditions are represented as Arel nodes internally right up to when the whole thing gets turned to SQL. In Rails 3.2, association conditions get turned to raw SQL early on, which prevents Relation#merge from interfering. The current implementation was buggy when a default_scope existed on the target model, since we would basically end up doing: default_scope.merge(association_scope) If default_scope contained a where(foo: 'a') and association_scope contained a where(foo: 'b').where(foo: 'c') then the merger would see that the same column is representated on both sides of the merge and collapse the wheres to all but the last: where(foo: 'c') Now, the RHS of the merge is left alone. Fixes #8990
* | | Fix typo: adaptors => adapters [ci skip]Carlos Antonio da Silva2013-01-271-1/+1
| | |
* | | Fix cases where delete_records on a has_many association caused errorsDerek Kraan2013-01-277-5/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | because of an ambiguous column name. This happened if the association model had a default scope that referenced a third table, and the third table also referenced the original table (with an identical foreign_key). Mysql requires that ambiguous columns are deambiguated by using the full table.column syntax. Postgresql and Sqlite use a different syntax for updates altogether (and don't tolerate table.name syntax), so the fix requires always including the full table.column and discarding it later for Sqlite and Postgresql.
* | | Merge pull request #9087 from lucasmazza/silence-migration-checksJosé Valim2013-01-261-1/+1
|\ \ \ | | | | | | | | Use `silence` instead of `quietly` to silence the `CheckPending` middleware.
| * | | Use `silence` instead of `quietly` to silence the `CheckPending` middleware.Lucas Mazza2013-01-261-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | `Kernel.quietly` silences `STDOUT` and `STDERR`, which is useless if the logger is writing to a file, while `AS::Logger#silence` swaps the logger level to `ERROR`. Related to #8820 and #8052.
* | | Simplify type casting code for timezone aware attributesAndrew White2013-01-261-10/+5
| | | | | | | | | | | | | | | | | | With the addition of String#in_time_zone and Date#in_time_zone we can simplify the type casting code by checking if the value has an `in_time_zone` method.
* | | Fix handling of dirty time zone aware attributesLilibeth De La Cruz2013-01-263-2/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, when `time_zone_aware_attributes` were enabled, after changing a datetime or timestamp attribute and then changing it back to the original value, `changed_attributes` still tracked the attribute as changed. This caused `[attribute]_changed?` and `changed?` methods to return true incorrectly. Example: in_time_zone 'Paris' do order = Order.new original_time = Time.local(2012, 10, 10) order.shipped_at = original_time order.save order.changed? # => false # changing value order.shipped_at = Time.local(2013, 1, 1) order.changed? # => true # reverting to original value order.shipped_at = original_time order.changed? # => false, used to return true end
* | | Merge remote-tracking branch 'docrails/master'Xavier Noria2013-01-2626-49/+198
|\ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/form_options_helper.rb guides/code/getting_started/app/controllers/comments_controller.rb
| * | | generic pass before merging docrailsXavier Noria2013-01-265-423/+63
| | | |
| * | | Remove mass-assignment line from guide summaryAndrew Wilcox2013-01-251-1/+0
| | | |
| * | | :scissors: "raise" duplicationAkira Matsuda2013-01-261-2/+1
| | | |
| * | | fix Active Record/Action Pack Changelog linksJuanjo Bazán2013-01-251-2/+2
| | | |
| * | | [engines guide] reformatted some stuffWeston Platter2013-01-241-3/+3
| | | |
| * | | [engines guide] reworded a run onWeston Platter2013-01-241-3/+2
| | | |
| * | | Add new DB rake tasks to the contributing section.Vipul A M2013-01-231-0/+14
| | | |
| * | | Made indentation consistent.Philip Arndt2013-01-231-1/+1
| | | |
| * | | Typo, grammar, and comma love.Brian Littmann2013-01-221-4/+4
| | | |
| * | | Typo fix in guides/source/engines.mdBrian Littmann2013-01-221-1/+1
| | | |
| * | | Fix typoTatsuro Baba2013-01-221-1/+1
| | | |
| * | | adding comma to separate clausesvertigoclinic2013-01-211-1/+1
| | | |
| * | | Review #translate docs [ci skip]Carlos Antonio da Silva2013-01-201-8/+7
| | | |
| * | | Add documentation for abstract controller #translate and #localize method.Jens Bissinger2013-01-201-0/+10
| | | |
| * | | Add documentation to ActionDispatch::ResponseMatthew Stopa2013-01-201-0/+7
| | | |
| * | | consistently inherit from ApplicationController in guides exemplesCyril Jouve2013-01-203-4/+4
| | | |
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-16 00:48:39 +0000