Browse Source

Various feedback items

- Add logger call on theme install failure
- Add todo in Module\Admin\DBSync
pull/7044/head
Hypolite Petovan 3 years ago
parent
commit
6df37afb82
  1. 1
      src/Core/Theme.php
  2. 1
      src/Module/Admin/DBSync.php

1
src/Core/Theme.php

@ -177,6 +177,7 @@ class Theme
return true;
} catch (\Exception $e) {
Logger::error('Theme installation failed', ['theme' => $theme, 'error' => $e->getMessage()]);
return false;
}
}

1
src/Module/Admin/DBSync.php

@ -36,6 +36,7 @@ class DBSync extends BaseAdminModule
if ($a->argc > 2) {
if ($a->argv[2] === 'check') {
// @TODO Seems like a similar logic like Update::check()
$retval = DBStructure::update($a->getBasePath(), false, true);
if ($retval === '') {
$o .= L10n::t("Database structure update %s was successfully applied.", DB_UPDATE_VERSION) . "<br />";

Loading…
Cancel
Save