2.1.0

Author: benkeen

code/Module.class.php

@@ -8,15 +8,14 @@
 
 class Module extends FormToolsModule
 {
-    // required properties
     protected $moduleName = "System Check";
+    protected $moduleDesc = "This module offers a few tests to analyze and repair your Form Tools installation.";
     protected $author = "Ben Keen";
     protected $authorEmail = "[email protected]";
     protected $authorLink = "http://formtools.org";
     protected $version = "2.1.0";
-    protected $date = "2017-09-23";
+    protected $date = "2017-09-24";
     protected $originLanguage = "en_us";
-
     protected $jsFiles = array("scripts/tests.js");
     protected $cssFiles = array("css/styles.css");
 

env.php

@@ -26,18 +26,11 @@
             "tab_link" => "{$same_page}?page=phpinfo",
             "pages" => array("phpinfo")
         )
-    ),
-    "css_files" => array("{$root_url}/modules/system_check/global/css/styles.css")
+    )
 );
 
-switch ($page) {
-    case "summary":
-        require_once("env_tab_summary.php");
-        break;
-    case "phpinfo":
-        require_once("env_tab_phpinfo.php");
-        break;
-    default:
-        require_once("env_tab_summary.php");
-        break;
+if ($page === "phpinfo") {
+    $module->displayPage("templates/env.tpl", $page_vars);
+} else {
+    require_once("env_tab_summary.php");
 }

env_tab_phpinfo.php

@@ -1,13 +1,14 @@
 <?php
 
+use FormTools\Core;
 use FormTools\Modules;
 use FormTools\Settings;
 use FormTools\Themes;
 
 $settings = Settings::get();
 
 $php_version = PHP_VERSION;
-$mysql_version = ""; // mysql_get_server_info(); TODO
+$mysql_version = Core::$db->getMysQLVersion();
 $suhosin_installed   = (extension_loaded("suhosin")) ? $L["word_installed"] : $L["phrase_not_installed"];
 $sessions_available  = (extension_loaded("session")) ? $L["word_available"] : $L["phrase_not_available"];
 $curl_available      = (extension_loaded("curl")) ? $L["word_available"] : $L["phrase_not_available"];
@@ -57,4 +58,4 @@
 
 $page_vars["report_card"] = $report_card;
 
-Themes::displayModulePage("templates/env.tpl", $page_vars);
+$module->displayPage("templates/env.tpl", $page_vars);

env_tab_summary.php

@@ -2,10 +2,6 @@
 
 require_once("../../global/library.php");
 
-use FormTools\Modules;
-use FormTools\Themes;
+$module = FormTools\Modules::initModulePage("admin");
 
-$module = Modules::initModule("admin");
-$L = $module->getLangStrings();
-
-Themes::displayModulePage("templates/help.tpl", array());
+$module->displayPage("templates/help.tpl", array());

help.php

@@ -48,7 +48,7 @@
 $L["text_hook_verification_intro"] = "Modules interact with the Core script through <i>hooks</i>. They attach their own functionality to act at particular junctions in the code. If this gets corrupted, it can prevent the module from working properly. This test examines your database to confirm that the hooks for all your modules are configured properly.";
 $L["text_problems_identified_and_fixed"] = "Problems are both identified and fixed.";
 $L["text_problems_identified_not_fixed"] = "Problems are only identified, not fixed.";
-$L["text_help"] = "For more information on this module, please see the <a href=\"http://modules.formtools.org/system_check/\" target=\"_blank\">help documentation</a> on the Form Tools site.";
+$L["text_help"] = "For more information on this module, please see the <a href=\"https://docs.formtools.org/modules/system_check/\" target=\"_blank\">help documentation</a> on the Form Tools site.";
 $L["text_file_check"] = "This examines all compatible components (Core, modules and themes) to confirm that the component's files exist.";
 $L["text_file_verification_intro"] = "This checks over your Core, modules and themes to confirm that all the necessary files have been uploaded properly. If any are missing, you will need to re-download the appropriate component and upload them to your server.";
 $L["text_orphan_record_check_intro"] = "This is a house-keeping test to examine the Core database tables for any unwanted orphaned records and references. Orphaned records are database entries that are no longer needed and should have been deleted along with their \"parents\". For example, when you delete a form, any references to that form ID should also be deleted. Orphaned records shouldn't cause problems, but add unnecessary clutter to your database. <b>If this test finds anything, we'd appreciate it if you <a href=\"http://forums.formtools.org/\">report them in the forums</a>!</b>";