{"action":"created","issue":{"url":"https://api.github.com/repos/rails/rails/issues/19598","labels_url":"https://api.github.com/repos/rails/rails/issues/19598/labels{/name}","comments_url":"https://api.github.com/repos/rails/rails/issues/19598/comments","events_url":"https://api.github.com/repos/rails/rails/issues/19598/events","html_url":"https://github.com/rails/rails/pull/19598","id":65439744,"number":19598,"title":"Fix default params for quote value on Rails 4.1","user":{"login":"nerdExpo","id":2749593,"avatar_url":"https://avatars.githubusercontent.com/u/2749593?v=3","gravatar_id":"","url":"https://api.github.com/users/nerdExpo","html_url":"https://github.com/nerdExpo","followers_url":"https://api.github.com/users/nerdExpo/followers","following_url":"https://api.github.com/users/nerdExpo/following{/other_user}","gists_url":"https://api.github.com/users/nerdExpo/gists{/gist_id}","starred_url":"https://api.github.com/users/nerdExpo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nerdExpo/subscriptions","organizations_url":"https://api.github.com/users/nerdExpo/orgs","repos_url":"https://api.github.com/users/nerdExpo/repos","events_url":"https://api.github.com/users/nerdExpo/events{/privacy}","received_events_url":"https://api.github.com/users/nerdExpo/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-31T12:34:27Z","updated_at":"2015-03-31T13:16:25Z","closed_at":"2015-03-31T13:16:25Z","pull_request":{"url":"https://api.github.com/repos/rails/rails/pulls/19598","html_url":"https://github.com/rails/rails/pull/19598","diff_url":"https://github.com/rails/rails/pull/19598.diff","patch_url":"https://github.com/rails/rails/pull/19598.patch"},"body":"### sanitization.rb\r\n\r\nWhile gems like `acts-as-taggable-on` heavily use `quote_value` in many cases, this will fix some problems for gems using the method without specifying a default value for `column`. \r\n\r\nThis should improve the compatibility of overall usage of ActiveRecord's sanitization methods across all the Rails developers.\r\n\r\nSince `connection.quote(value, column)` has already supports (1..2) params, I can't see why the `quote_value` is strictly needed for 2 parameters for the function to work."},"comment":{"url":"https://api.github.com/repos/rails/rails/issues/comments/88079732","html_url":"https://github.com/rails/rails/pull/19598#issuecomment-88079732","issue_url":"https://api.github.com/repos/rails/rails/issues/19598","id":88079732,"user":{"login":"matthewd","id":1034,"avatar_url":"https://avatars.githubusercontent.com/u/1034?v=3","gravatar_id":"","url":"https://api.github.com/users/matthewd","html_url":"https://github.com/matthewd","followers_url":"https://api.github.com/users/matthewd/followers","following_url":"https://api.github.com/users/matthewd/following{/other_user}","gists_url":"https://api.github.com/users/matthewd/gists{/gist_id}","starred_url":"https://api.github.com/users/matthewd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthewd/subscriptions","organizations_url":"https://api.github.com/users/matthewd/orgs","repos_url":"https://api.github.com/users/matthewd/repos","events_url":"https://api.github.com/users/matthewd/events{/privacy}","received_events_url":"https://api.github.com/users/matthewd/received_events","type":"User","site_admin":false},"created_at":"2015-03-31T13:16:25Z","updated_at":"2015-03-31T13:16:25Z","body":"We're not going to change this in 4.1"}}