{"action":"created","issue":{"url":"https://api.github.com/repos/civicrm/civicrm-core/issues/8592","repository_url":"https://api.github.com/repos/civicrm/civicrm-core","labels_url":"https://api.github.com/repos/civicrm/civicrm-core/issues/8592/labels{/name}","comments_url":"https://api.github.com/repos/civicrm/civicrm-core/issues/8592/comments","events_url":"https://api.github.com/repos/civicrm/civicrm-core/issues/8592/events","html_url":"https://github.com/civicrm/civicrm-core/pull/8592","id":160797530,"number":8592,"title":"CRM-17917 - Fix D8 Civi/Install/Requirements","user":{"login":"totten","id":1336047,"avatar_url":"https://avatars.githubusercontent.com/u/1336047?v=3","gravatar_id":"","url":"https://api.github.com/users/totten","html_url":"https://github.com/totten","followers_url":"https://api.github.com/users/totten/followers","following_url":"https://api.github.com/users/totten/following{/other_user}","gists_url":"https://api.github.com/users/totten/gists{/gist_id}","starred_url":"https://api.github.com/users/totten/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/totten/subscriptions","organizations_url":"https://api.github.com/users/totten/orgs","repos_url":"https://api.github.com/users/totten/repos","events_url":"https://api.github.com/users/totten/events{/privacy}","received_events_url":"https://api.github.com/users/totten/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/civicrm/civicrm-core/labels/master","name":"master","color":"0052cc"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-06-17T01:58:05Z","updated_at":"2016-07-11T14:07:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/civicrm/civicrm-core/pulls/8592","html_url":"https://github.com/civicrm/civicrm-core/pull/8592","diff_url":"https://github.com/civicrm/civicrm-core/pull/8592.diff","patch_url":"https://github.com/civicrm/civicrm-core/pull/8592.patch"},"body":"This is a bit confusing because there are two nearly identical classes for\nchecking system requirements -- `install/index.php#InstallRequirements`\n(used by the graphical D6/D7/WP) and `Civi/Install/Requirements.php` (used\nby the headless D8 installer).  This looks like a code cleanup gone awry.\n\nIn any event, the DB config objects in these contexts have different\nproperties.  This fixes the D8 one to consistently use the D8 properties.\n\nWISHLIST: Use the same classes and array structures in each context!\n\n---\n\n * [CRM-17917: Add support for Drupal 8](https://issues.civicrm.org/jira/browse/CRM-17917)"},"comment":{"url":"https://api.github.com/repos/civicrm/civicrm-core/issues/comments/231744779","html_url":"https://github.com/civicrm/civicrm-core/pull/8592#issuecomment-231744779","issue_url":"https://api.github.com/repos/civicrm/civicrm-core/issues/8592","id":231744779,"user":{"login":"eileenmcnaughton","id":336308,"avatar_url":"https://avatars.githubusercontent.com/u/336308?v=3","gravatar_id":"","url":"https://api.github.com/users/eileenmcnaughton","html_url":"https://github.com/eileenmcnaughton","followers_url":"https://api.github.com/users/eileenmcnaughton/followers","following_url":"https://api.github.com/users/eileenmcnaughton/following{/other_user}","gists_url":"https://api.github.com/users/eileenmcnaughton/gists{/gist_id}","starred_url":"https://api.github.com/users/eileenmcnaughton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eileenmcnaughton/subscriptions","organizations_url":"https://api.github.com/users/eileenmcnaughton/orgs","repos_url":"https://api.github.com/users/eileenmcnaughton/repos","events_url":"https://api.github.com/users/eileenmcnaughton/events{/privacy}","received_events_url":"https://api.github.com/users/eileenmcnaughton/received_events","type":"User","site_admin":false},"created_at":"2016-07-11T14:07:38Z","updated_at":"2016-07-11T14:07:38Z","body":"Is this still valid? Should we open a new ticket since this one is closed https://issues.civicrm.org/jira/browse/CRM-17917"}}
{"actor":{"display_login":"eileenmcnaughton"}}