{"action":"created","issue":{"url":"https://api.github.com/repos/Sylius/Sylius/issues/4674","repository_url":"https://api.github.com/repos/Sylius/Sylius","labels_url":"https://api.github.com/repos/Sylius/Sylius/issues/4674/labels{/name}","comments_url":"https://api.github.com/repos/Sylius/Sylius/issues/4674/comments","events_url":"https://api.github.com/repos/Sylius/Sylius/issues/4674/events","html_url":"https://github.com/Sylius/Sylius/pull/4674","id":145434271,"number":4674,"title":"[Bug Fix] Make sure doctrine removes orphan tax rates","user":{"login":"steffenbrem","id":1569156,"avatar_url":"https://avatars.githubusercontent.com/u/1569156?v=3","gravatar_id":"","url":"https://api.github.com/users/steffenbrem","html_url":"https://github.com/steffenbrem","followers_url":"https://api.github.com/users/steffenbrem/followers","following_url":"https://api.github.com/users/steffenbrem/following{/other_user}","gists_url":"https://api.github.com/users/steffenbrem/gists{/gist_id}","starred_url":"https://api.github.com/users/steffenbrem/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/steffenbrem/subscriptions","organizations_url":"https://api.github.com/users/steffenbrem/orgs","repos_url":"https://api.github.com/users/steffenbrem/repos","events_url":"https://api.github.com/users/steffenbrem/events{/privacy}","received_events_url":"https://api.github.com/users/steffenbrem/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Sylius/Sylius/labels/Awaiting%20Feedback","name":"Awaiting Feedback","color":"bfe5bf"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2016-04-03T00:13:05Z","updated_at":"2016-04-20T22:50:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Sylius/Sylius/pulls/4674","html_url":"https://github.com/Sylius/Sylius/pull/4674","diff_url":"https://github.com/Sylius/Sylius/pull/4674.diff","patch_url":"https://github.com/Sylius/Sylius/pull/4674.patch"},"body":"| Q               | A\r\n| --------------- | ---\r\n| Bug fix?        | yes\r\n| New feature?    | no\r\n| BC breaks?      | no\r\n| Deprecations?   | no\r\n| Related tickets | \r\n| License         | MIT\r\n\r\nThere is a bug now where you cannot remove existing tax rates from a tax category instance. It will try to set the tax rates category to `null`. We want to remove the tax rate entirely when it's not connected to a tax category anymore. This PR fixes this.\r\n"},"comment":{"url":"https://api.github.com/repos/Sylius/Sylius/issues/comments/212643207","html_url":"https://github.com/Sylius/Sylius/pull/4674#issuecomment-212643207","issue_url":"https://api.github.com/repos/Sylius/Sylius/issues/4674","id":212643207,"user":{"login":"steffenbrem","id":1569156,"avatar_url":"https://avatars.githubusercontent.com/u/1569156?v=3","gravatar_id":"","url":"https://api.github.com/users/steffenbrem","html_url":"https://github.com/steffenbrem","followers_url":"https://api.github.com/users/steffenbrem/followers","following_url":"https://api.github.com/users/steffenbrem/following{/other_user}","gists_url":"https://api.github.com/users/steffenbrem/gists{/gist_id}","starred_url":"https://api.github.com/users/steffenbrem/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/steffenbrem/subscriptions","organizations_url":"https://api.github.com/users/steffenbrem/orgs","repos_url":"https://api.github.com/users/steffenbrem/repos","events_url":"https://api.github.com/users/steffenbrem/events{/privacy}","received_events_url":"https://api.github.com/users/steffenbrem/received_events","type":"User","site_admin":false},"created_at":"2016-04-20T22:50:21Z","updated_at":"2016-04-20T22:50:21Z","body":"As you can see in this file, tax rates always needs a tax category: https://github.com/Sylius/Sylius/blob/master/src/Sylius/Bundle/TaxationBundle/Resources/config/doctrine/model/TaxRate.orm.xml#L29\r\n\r\n"}}