{"action":"created","issue":{"url":"https://api.github.com/repos/ruckus/ruckusing-migrations/issues/154","labels_url":"https://api.github.com/repos/ruckus/ruckusing-migrations/issues/154/labels{/name}","comments_url":"https://api.github.com/repos/ruckus/ruckusing-migrations/issues/154/comments","events_url":"https://api.github.com/repos/ruckus/ruckusing-migrations/issues/154/events","html_url":"https://github.com/ruckus/ruckusing-migrations/issues/154","id":60009192,"number":154,"title":"Unable to execute query for trigger","user":{"login":"tunder","id":1682645,"avatar_url":"https://avatars.githubusercontent.com/u/1682645?v=3","gravatar_id":"","url":"https://api.github.com/users/tunder","html_url":"https://github.com/tunder","followers_url":"https://api.github.com/users/tunder/followers","following_url":"https://api.github.com/users/tunder/following{/other_user}","gists_url":"https://api.github.com/users/tunder/gists{/gist_id}","starred_url":"https://api.github.com/users/tunder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tunder/subscriptions","organizations_url":"https://api.github.com/users/tunder/orgs","repos_url":"https://api.github.com/users/tunder/repos","events_url":"https://api.github.com/users/tunder/events{/privacy}","received_events_url":"https://api.github.com/users/tunder/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-05T20:15:41Z","updated_at":"2015-03-06T05:19:51Z","closed_at":null,"body":"After updating from version 0.1.2 to latest dev-master executing queries like this\r\n```php\r\n$this->execute(\"\r\n    CREATE TRIGGER `name` BEFORE UPDATE ON `table`\r\n        statement 1 goes here;\r\n        statement 2 goes here;\r\n    END;;\r\n\");\r\n```\r\ndoes not work anymore.\r\n\r\nCause:\r\nRuckusing_Migration_Base::split_query\r\nThis feature is useless. Having multiple sql queries inside the same string. Writing migrations is the developer job and we know that (at least) mysql doesn't support it. Why should you? This will be the source of many bugs.\r\n\r\nSome examples where this function will fail:\r\ntriggers, stored procedures, etc.\r\nquery like: ```insert into table set field = 'string \\'quote1; quote2\\'';```"},"comment":{"url":"https://api.github.com/repos/ruckus/ruckusing-migrations/issues/comments/77508316","html_url":"https://github.com/ruckus/ruckusing-migrations/issues/154#issuecomment-77508316","issue_url":"https://api.github.com/repos/ruckus/ruckusing-migrations/issues/154","id":77508316,"user":{"login":"ruckus","id":320,"avatar_url":"https://avatars.githubusercontent.com/u/320?v=3","gravatar_id":"","url":"https://api.github.com/users/ruckus","html_url":"https://github.com/ruckus","followers_url":"https://api.github.com/users/ruckus/followers","following_url":"https://api.github.com/users/ruckus/following{/other_user}","gists_url":"https://api.github.com/users/ruckus/gists{/gist_id}","starred_url":"https://api.github.com/users/ruckus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ruckus/subscriptions","organizations_url":"https://api.github.com/users/ruckus/orgs","repos_url":"https://api.github.com/users/ruckus/repos","events_url":"https://api.github.com/users/ruckus/events{/privacy}","received_events_url":"https://api.github.com/users/ruckus/received_events","type":"User","site_admin":false},"created_at":"2015-03-06T05:19:51Z","updated_at":"2015-03-06T05:19:51Z","body":"You are right, it has proven to be a source of bugs and confusion. I can no longer remember why it was included in the first place. I vaguely recall the use-case being that one should be able to slurp in a SQL file of commands and execute them via Ruckusing. \r\n\r\nBut I think at this point its causing more harm than good.\r\n\r\nAnyone else want to weigh in?"}}