zc_install - for issue #99

<a href='#crh-start'></a><a href='#crh-data-%7B%22processed%22%3A%20%5B%22https%3A//github.com/zencart/zc-v1-series/pull/100%23discussion_r14977316%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/100%23discussion_r14977318%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/100%23discussion_r14977323%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/100%23discussion_r14977365%22%5D%2C%20%22comments%22%3A%20%7B%22Pull%20a522f0ea17a4a7632ddb5f93153206f3c5cab21a%20zc_install/includes/classes/class.systemChecker.php%2010%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/zencart/zc-v1-series/pull/100%23discussion_r14977365%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22I%20don%27t%20really%20like%20this%20complex%20conditional%20-%20but%20at%20the%20very%20least%2C%20you%20can%20%60return%20TRUE%3B%60%20here.%22%2C%20%22created_at%22%3A%20%222014-07-16T01%3A12%3A14Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1590605%3F%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/texdc%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20zc_install/includes/classes/class.systemChecker.php%3AL231-243%22%7D%2C%20%22Pull%20a522f0ea17a4a7632ddb5f93153206f3c5cab21a%20zc_install/includes/classes/class.systemChecker.php%2046%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/zencart/zc-v1-series/pull/100%23discussion_r14977323%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22%5BUnnecessary%5D%28http%3A//c2.com/cgi/wiki%3FElseConsideredSmelly%29%20%60else%60.%22%2C%20%22created_at%22%3A%20%222014-07-16T01%3A09%3A57Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1590605%3F%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/texdc%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20zc_install/includes/classes/class.systemChecker.php%3AL525-565%22%7D%2C%20%22Pull%20a522f0ea17a4a7632ddb5f93153206f3c5cab21a%20zc_install/includes/languages/lngEnglish.php%2047%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/zencart/zc-v1-series/pull/100%23discussion_r14977316%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22%3A%2B1%3A%20%22%2C%20%22created_at%22%3A%20%222014-07-16T01%3A09%3A43Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1590605%3F%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/texdc%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20zc_install/includes/languages/lngEnglish.php%3AL140-147%22%7D%2C%20%22Pull%20a522f0ea17a4a7632ddb5f93153206f3c5cab21a%20zc_install/includes/classes/class.zcDatabaseInstaller.php%2017%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/zencart/zc-v1-series/pull/100%23discussion_r14977318%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22If%20this%20isn%27t%20being%20used%2C%20it%20can%20be%20removed.%20%20Source%20control%20will%20keep%20track%20of%20it.%20%20Otherwise%2C%20please%20condense%20it.%22%2C%20%22created_at%22%3A%20%222014-07-16T01%3A09%3A50Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1590605%3F%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/texdc%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20zc_install/includes/classes/class.zcDatabaseInstaller.php%3AL91-104%22%7D%7D%7D'></a>
<a href='https://www.codereviewhub.com/'><img src='http://www.codereviewhub.com/site/github-bar.png' height=40></a>
- [ ] <a href='#crh-comment-Pull a522f0ea17a4a7632ddb5f93153206f3c5cab21a zc_install/includes/classes/class.zcDatabaseInstaller.php 17'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/zencart/zc-v1-series/pull/100#discussion_r14977318'>File: zc_install/includes/classes/class.zcDatabaseInstaller.php:L91-104</a></b>
- <a href='https://github.com/texdc'><img border=0 src='https://avatars.githubusercontent.com/u/1590605?size=16' height=16 width=16'></a> If this isn't being used, it can be removed.  Source control will keep track of it.  Otherwise, please condense it.
- [ ] <a href='#crh-comment-Pull a522f0ea17a4a7632ddb5f93153206f3c5cab21a zc_install/includes/classes/class.systemChecker.php 46'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/zencart/zc-v1-series/pull/100#discussion_r14977323'>File: zc_install/includes/classes/class.systemChecker.php:L525-565</a></b>
- <a href='https://github.com/texdc'><img border=0 src='https://avatars.githubusercontent.com/u/1590605?size=16' height=16 width=16'></a> [Unnecessary](http://c2.com/cgi/wiki?ElseConsideredSmelly) `else`.
- [ ] <a href='#crh-comment-Pull a522f0ea17a4a7632ddb5f93153206f3c5cab21a zc_install/includes/classes/class.systemChecker.php 10'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/zencart/zc-v1-series/pull/100#discussion_r14977365'>File: zc_install/includes/classes/class.systemChecker.php:L231-243</a></b>
- <a href='https://github.com/texdc'><img border=0 src='https://avatars.githubusercontent.com/u/1590605?size=16' height=16 width=16'></a> I don't really like this complex conditional - but at the very least, you can `return TRUE;` here.


<a href='https://www.codereviewhub.com/zencart/zc-v1-series/pull/100?mark_as_completed=1'><img src='http://www.codereviewhub.com/site/github-mark-as-completed.png' height=26></a>&nbsp;<a href='https://www.codereviewhub.com/zencart/zc-v1-series/pull/100?approve=1'><img src='http://www.codereviewhub.com/site/github-approve.png' height=26></a>&nbsp;<a href='https://github.com/zencart/zc-v1-series/pull/100'><img src='http://www.codereviewhub.com/site/github-refresh.png' height=26></a>
<a href='#crh-end'></a>
I was just working on this. There's actually a bug here in that other parts of the code are expecting a return value. Some of that's legacy logic, and warrants turning into another issue to address that.