{"action":"created","issue":{"url":"https://api.github.com/repos/tbar0970/jethro-pmm/issues/238","repository_url":"https://api.github.com/repos/tbar0970/jethro-pmm","labels_url":"https://api.github.com/repos/tbar0970/jethro-pmm/issues/238/labels{/name}","comments_url":"https://api.github.com/repos/tbar0970/jethro-pmm/issues/238/comments","events_url":"https://api.github.com/repos/tbar0970/jethro-pmm/issues/238/events","html_url":"https://github.com/tbar0970/jethro-pmm/pull/238","id":169735767,"number":238,"title":"Convert MDB2 to PDO","user":{"login":"vanoudt","id":1248707,"avatar_url":"https://avatars2.githubusercontent.com/u/1248707?v=3","gravatar_id":"","url":"https://api.github.com/users/vanoudt","html_url":"https://github.com/vanoudt","followers_url":"https://api.github.com/users/vanoudt/followers","following_url":"https://api.github.com/users/vanoudt/following{/other_user}","gists_url":"https://api.github.com/users/vanoudt/gists{/gist_id}","starred_url":"https://api.github.com/users/vanoudt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vanoudt/subscriptions","organizations_url":"https://api.github.com/users/vanoudt/orgs","repos_url":"https://api.github.com/users/vanoudt/repos","events_url":"https://api.github.com/users/vanoudt/events{/privacy}","received_events_url":"https://api.github.com/users/vanoudt/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2016-08-06T08:04:50Z","updated_at":"2017-06-29T02:30:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tbar0970/jethro-pmm/pulls/238","html_url":"https://github.com/tbar0970/jethro-pmm/pull/238","diff_url":"https://github.com/tbar0970/jethro-pmm/pull/238.diff","patch_url":"https://github.com/tbar0970/jethro-pmm/pull/238.patch"},"body":"Introduces a helper class to switch from MDB2 to PDO. Tested for me, and it seems to work.\nAlso changes the configuration, to make it a lot easier - writing a DSN is more complex than I think we should expect people to think. Nevertheless, it does attempt to parse the old style DSN from MDB2, and convert it into something for PDO. (Thanks StackOverflow!) \n"},"comment":{"url":"https://api.github.com/repos/tbar0970/jethro-pmm/issues/comments/311845178","html_url":"https://github.com/tbar0970/jethro-pmm/pull/238#issuecomment-311845178","issue_url":"https://api.github.com/repos/tbar0970/jethro-pmm/issues/238","id":311845178,"user":{"login":"vanoudt","id":1248707,"avatar_url":"https://avatars2.githubusercontent.com/u/1248707?v=3","gravatar_id":"","url":"https://api.github.com/users/vanoudt","html_url":"https://github.com/vanoudt","followers_url":"https://api.github.com/users/vanoudt/followers","following_url":"https://api.github.com/users/vanoudt/following{/other_user}","gists_url":"https://api.github.com/users/vanoudt/gists{/gist_id}","starred_url":"https://api.github.com/users/vanoudt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vanoudt/subscriptions","organizations_url":"https://api.github.com/users/vanoudt/orgs","repos_url":"https://api.github.com/users/vanoudt/repos","events_url":"https://api.github.com/users/vanoudt/events{/privacy}","received_events_url":"https://api.github.com/users/vanoudt/received_events","type":"User","site_admin":false},"created_at":"2017-06-29T02:30:46Z","updated_at":"2017-06-29T02:30:46Z","body":"Hi Garrett,\r\n\r\nHmmm - that's strange. It works fine in production for me. I've thrown up a quick fix - which still works fine for me, but will hopefully make that error go away for you! Simply repeat all your steps, or just modify the file jethrodb.php in the include directory. Repace:\r\n`  public function quote($string) {\r\n\tif ($string === NULL) {\r\n\t\treturn 'NULL';\r\n\t} else {\r\n\t\treturn parent::quote($string);\r\n\t}\r\n  }`\r\n\r\nwith \r\n`  public function quote($string, $paramtype=NULL) {\r\n\tif ($string === NULL) {\r\n\t\treturn 'NULL';\r\n\t} else {\r\n\t\treturn parent::quote($string, $paramtype);\r\n\t}\r\n  }`"}}
{"actor":{"display_login":"vanoudt"}}