Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix obsoleted method URI.unescape in activesupport/test | Vitor Oliveira | 2018-08-15 | 1 | -1/+1 |
| | |||||
* | e4e1b62 broke `to_param` handling: | Edouard CHIN | 2018-07-12 | 1 | -0/+14 |
| | | | | | | | | | | | | | | | | | | | - There was an issue inside controller tests where order params were not respected, the reason was because we were calling `Hash#to_query` which sorts the results lexicographically. 1e4e1b62 fixed that issue by not using `to_query` but instead a utility function provided by rack. - However with the fix came another issue where it's now no longer possible to do this ``` post :foo, params: { user: User.first } # Prior to the patch the controller will receive { "user" => "1" } # Whereas now you get { "user": "#<User: ...>" } ``` The fix in this PR is to modify `Hash#to_query` to sort only when it doesn't contain an array structure that looks something like "bar[]" Ref https://github.com/rails/rails/pull/33341#issuecomment-404039396 | ||||
* | [Active Support] `rubocop -a --only Layout/EmptyLineAfterMagicComment` | Koichi ITO | 2017-07-11 | 1 | -0/+1 |
| | |||||
* | Use frozen-string-literal in ActiveSupport | Kir Shatrov | 2017-07-09 | 1 | -0/+1 |
| | |||||
* | Revert "Merge pull request #29540 from kirs/rubocop-frozen-string" | Matthew Draper | 2017-07-02 | 1 | -1/+0 |
| | | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa. | ||||
* | Enforce frozen string in Rubocop | Kir Shatrov | 2017-07-01 | 1 | -0/+1 |
| | |||||
* | Add three new rubocop rules | Rafael Mendonça França | 2016-08-16 | 1 | -4/+4 |
| | | | | | | | | Style/SpaceBeforeBlockBraces Style/SpaceInsideBlockBraces Style/SpaceInsideHashLiteralBraces Fix all violations in the repository. | ||||
* | remove redundant curlies from hash arguments | Xavier Noria | 2016-08-06 | 1 | -5/+5 |
| | |||||
* | modernizes hash syntax in activesupport | Xavier Noria | 2016-08-06 | 1 | -5/+5 |
| | |||||
* | applies new string literal convention in activesupport/test | Xavier Noria | 2016-08-06 | 1 | -27/+27 |
| | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. | ||||
* | Merge pull request #15629 from akshay-vishnoi/test-to_param | Rafael Mendonça França | 2014-07-28 | 1 | -0/+10 |
|\ | | | | | Define Hash#to_query and set Hash#to_param as alias to it; with test cases | ||||
| * | Move to_param to to_query, also Improve tests | Akshay Vishnoi | 2014-06-20 | 1 | -0/+10 |
| | | |||||
* | | missing activesupport test coverage | Eugene Gilburg | 2014-07-19 | 1 | -0/+4 |
|/ | |||||
* | [Fixes #14948] Hash#to_query: Changed a way how empty hash and empty array ↵ | Bogdan Gusiev | 2014-05-03 | 1 | -3/+5 |
| | | | | | | | | | | | are serialized Empty Hash or Array should not present in serialization result {a: []}.to_query # => "" {a: {}}.to_query # => "" For more info see #14948. | ||||
* | Fix to_query with empty arrays too | Rafael Mendonça França | 2014-02-06 | 1 | -0/+2 |
| | |||||
* | Refatoring the method to avoid shot-circuit return | Rafael Mendonça França | 2014-02-05 | 1 | -0/+2 |
| | |||||
* | Test with a blank value | Rafael Mendonça França | 2014-02-05 | 1 | -2/+2 |
| | |||||
* | Move test to the right file | Rafael Mendonça França | 2014-02-05 | 1 | -0/+9 |
| | |||||
* | remove unused parameter passed to assert_query_equal method | Vipul A M | 2013-03-17 | 1 | -1/+1 |
| | |||||
* | Remove .rb from require sentences | José Corcuera Z | 2012-09-21 | 1 | -1/+1 |
| | |||||
* | remove usages of AS::OrderedHash | Vishnu Atrai | 2012-03-03 | 1 | -2/+2 |
| | |||||
* | use AS::TestCase as the base class | Aaron Patterson | 2012-01-05 | 1 | -1/+1 |
| | |||||
* | Fix Hash#to_query edge case with html_safe string on 1.8 ruby | brainopia | 2011-09-16 | 1 | -0/+9 |
| | |||||
* | Revert "Quick fix for not escaping []s (not ideal)" | Santiago Pastorino | 2010-11-18 | 1 | -4/+4 |
| | | | | | | | | | According to http://www.ietf.org/rfc/rfc2616.txt and http://tools.ietf.org/rfc/rfc3986.txt [ and ] are reserved characters and should be escaped using "%" HEX HEX This reverts commit 856d2fd874d72dd9f83204affff4edfef3308361 and 1ee9b40b18a0bed5bb10a0785f7e2730bac983f6.. | ||||
* | These tests are trusting in the order of the elements so use OrderedHash ↵ | Santiago Pastorino | 2010-07-21 | 1 | -2/+3 |
| | | | | instead of Hash | ||||
* | Failing tests for to_param/to_query not escaping "[]" | Joshua Peek | 2009-12-02 | 1 | -4/+4 |
| | |||||
* | Consolidate Object#to_param and #to_query core extensions | Jeremy Kemper | 2009-11-02 | 1 | -0/+43 |