Summary | Database migration triggered on non-database setup |
Queue | Horde Base |
Queue Version | Git master |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | jan (at) horde (dot) org |
Requester | thomas.jarosch (at) intra2net (dot) com |
Created | 04/26/2011 (5185 days ago) |
Due | |
Updated | 04/27/2011 (5184 days ago) |
Assigned | |
Resolved | 04/26/2011 (5185 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
Fix SQL configuration test (
Bug #9986).1 files changed, 1 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/33a12e8a0ac0c45d6ba4b7eb5fbb2c2268a0a9d6
[jan] Don't check for outdated DB schemas if database support is
disabled (
Bug #9986).of the same bug in horde/lib/Block/Metar.php:19
As Horde_Block_Metar is not working without a database anyway, it's
not worth fixing.
State ⇒ Resolved
[jan] Don't check for outdated DB schemas if database support is
disabled (
Bug #9986).3 files changed, 5 insertions(+), 3 deletions(-)
http://git.horde.org/horde-git/-/commit/6f67450baadec478c586b1e9814477e4dc194db9
Priority ⇒ 1. Low
Patch ⇒ No
Milestone ⇒
Queue ⇒ Horde Base
Summary ⇒ Database migration triggered on non-database setup
Type ⇒ Bug
State ⇒ Unconfirmed
I've updated my Kolab devel box to Horde4. The box is running without
a database server. I selected SQL type "none" in the horde prefs.
conf.php contained a "$conf['sql']['phptype'] = 'false' line IIRC.
Later on the LoginTasks for the admin try to trigger a DB upgrade
because of this code in horde/lib/LoginTasks/Task/AdminCheck.php:
if (!empty($GLOBALS['conf']['sql'])) {
/* Check for outdated DB schemas. */
$migration = new Horde_Core_Db_Migration();
foreach ($migration->apps as $app) {
$migrator = $migration->getMigrator($app);
if ($migrator->getTargetVersion() >
$migrator->getCurrentVersion()) {
$GLOBALS['notification']->push(_("At least one
database schema is outdated."), 'horde.warning');
Horde::url('admin/config', true, array('app' => 'horde'))
->redirect();
}
}
Should the [none] SQL database type completely remove
the $conf['sql'] key or does the code above need to handle the "false" case?
Cheers,
Thomas