{"action":"created","issue":{"url":"https://api.github.com/repos/RubyMoney/money-rails/issues/430","repository_url":"https://api.github.com/repos/RubyMoney/money-rails","labels_url":"https://api.github.com/repos/RubyMoney/money-rails/issues/430/labels{/name}","comments_url":"https://api.github.com/repos/RubyMoney/money-rails/issues/430/comments","events_url":"https://api.github.com/repos/RubyMoney/money-rails/issues/430/events","html_url":"https://github.com/RubyMoney/money-rails/pull/430","id":141429398,"number":430,"title":"make monetized_attributes a hash_with_indifferent_access","user":{"login":"bsharpe","id":7917,"avatar_url":"https://avatars.githubusercontent.com/u/7917?v=3","gravatar_id":"","url":"https://api.github.com/users/bsharpe","html_url":"https://github.com/bsharpe","followers_url":"https://api.github.com/users/bsharpe/followers","following_url":"https://api.github.com/users/bsharpe/following{/other_user}","gists_url":"https://api.github.com/users/bsharpe/gists{/gist_id}","starred_url":"https://api.github.com/users/bsharpe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bsharpe/subscriptions","organizations_url":"https://api.github.com/users/bsharpe/orgs","repos_url":"https://api.github.com/users/bsharpe/repos","events_url":"https://api.github.com/users/bsharpe/events{/privacy}","received_events_url":"https://api.github.com/users/bsharpe/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-03-16T23:16:10Z","updated_at":"2016-03-24T19:58:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RubyMoney/money-rails/pulls/430","html_url":"https://github.com/RubyMoney/money-rails/pull/430","diff_url":"https://github.com/RubyMoney/money-rails/pull/430.diff","patch_url":"https://github.com/RubyMoney/money-rails/pull/430.patch"},"body":"`monetized_attributes` was getting created as a normal hash in one place and as a HWIA in another.  By making it always a HWIA, we don't care if the accessed attr in `validator` is a `symbol` or a `string.`\r\n\r\nWe noticed this when running our tests -- validation was failing because `@attr.to_s` was being used to look up the monetized_attribute but the key was a `symbol.`"},"comment":{"url":"https://api.github.com/repos/RubyMoney/money-rails/issues/comments/200993592","html_url":"https://github.com/RubyMoney/money-rails/pull/430#issuecomment-200993592","issue_url":"https://api.github.com/repos/RubyMoney/money-rails/issues/430","id":200993592,"user":{"login":"antstorm","id":118353,"avatar_url":"https://avatars.githubusercontent.com/u/118353?v=3","gravatar_id":"","url":"https://api.github.com/users/antstorm","html_url":"https://github.com/antstorm","followers_url":"https://api.github.com/users/antstorm/followers","following_url":"https://api.github.com/users/antstorm/following{/other_user}","gists_url":"https://api.github.com/users/antstorm/gists{/gist_id}","starred_url":"https://api.github.com/users/antstorm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/antstorm/subscriptions","organizations_url":"https://api.github.com/users/antstorm/orgs","repos_url":"https://api.github.com/users/antstorm/repos","events_url":"https://api.github.com/users/antstorm/events{/privacy}","received_events_url":"https://api.github.com/users/antstorm/received_events","type":"User","site_admin":false},"created_at":"2016-03-24T19:58:04Z","updated_at":"2016-03-24T19:58:04Z","body":"@bsharpe yeah, I agree, `monetize` method has grown too complex, but there's some work around refactoring it — #425.\r\n\r\nRe this PR — I'm happy to merge it, but it needs some tests to make sure others know that this is intentional."}}