renderServerConfigurationReport(): handle InnoDB failures better
authorDenis Ovsienko <denis@ovsienko.info>
Fri, 9 Nov 2018 15:01:10 +0000 (15:01 +0000)
committerDenis Ovsienko <denis@ovsienko.info>
Fri, 9 Nov 2018 15:01:10 +0000 (15:01 +0000)
The InnoDB test fails trying to create a trigger when binary logging is
enabled in the database server (in particular, that is required for the
old-style master-slave replication setup). This used to result in an
unhandled PDOException, which effectively blinded all other tests.

Use the exception to generate a separate specific error message and to
skip the InnoDB test only.

wwwroot/inc/interface-reports.php

index e5b326a..9848059 100644 (file)
@@ -961,5 +961,14 @@ function renderDataIntegrityReport ()
 function renderServerConfigurationReport ()
 {
        echo '<br>';
-       platform_is_ok (TRUE);
+       try
+       {
+               $test_innodb = isInnoDBSupported();
+       }
+       catch (PDOException $e)
+       {
+               showError ('InnoDB test failed (is binary logging enabled?).');
+               $test_innodb = FALSE;
+       }
+       platform_is_ok ($test_innodb);
 }