| 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 (5295 days ago) |
| Due | |
| Updated | 04/27/2011 (5294 days ago) |
| Assigned | |
| Resolved | 04/26/2011 (5295 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