[Sat Oct 24 02:19:07 2009] [error] [client 127.0.0.1] PHP Fatal error: Uncaught exception 'PDOException' with message
'SQLSTATE[42S22]: Column not found: 1054 Unknown column 'source' in 'field list'' in /home/catch/www/7/includes/database/database.inc:1786\nStack trace:\n#0 /home/catch/www/7/includes/database/database.inc(1786): PDOStatement->execute(Array)\n#1 /home/catch/www/7/includes/database/database.inc(568): DatabaseStatementBase->execute(Array, Array)\n#2 /home/catch/www/7/includes/database/database.inc(1872): DatabaseConnection->query('SELECT SUBSTRIN...', Array, Array)\n#3 /home/catch/www/7/includes/path.inc(370): db_query('SELECT SUBSTRIN...')\n#4 /home/catch/www/7/includes/path.inc(61): drupal_path_alias_whitelist_rebuild()\n#5 /home/catch/www/7/includes/path.inc(208): drupal_lookup_path('source', 'node', '')\n#6 /home/catch/www/7/includes/path.inc(21): drupal_get_normal_path('node')\n#7 /home/catch/www/7/includes/bootstrap.inc(1547): drupal_path_initialize()\n#8 /home/catch/www/7/includes/bootstrap.inc(1418): _drupal_bootstrap(7)\n#9 /home/catch/www/7/update.php(337): drupal_b in /home/catch/www/7/includes/database/database.inc on line 1786
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | path_wsod.patch | 4.27 KB | catch |
Comments
Comment #1
catchPatch isn't pretty, but removes the fatal errors. #585628: Cannot use update.php as anonymous user prevents testing beyond this for the overall upgrade, but at least no wsod with this one.
Comment #2
sun1) Couldn't we simply disable/skip path alias lookups when defined('MAINTENANCE_MODE') ?
2) If not 1), could we move the contents of that update into the d7 requirements function?
Comment #3
catchFor #1 unless we add !defined(MAINTENANCE_MODE) to every path lookup no, I'd rather keep the ugliness in update.php
For #2, the d7_requirements munging shouldn't leave you in a state where you can't revert to D6, so we can't rename columns there.
Comment #4
webchickI talked this over with catch on IRC. I also can't really envision a more svelte way to handle this. This moves ugly code to update.php as opposed to the critical path (+1) and doesn't automatically start converting anything just by virtue of visiting this page (also +1).
As a side note, it seems pretty odd that if you bootstrap to DRUPAL_BOOTSTRAP_LANGUAGE you still have to manually include unicode.inc. :P But obviously not for this patch to solve.
Committed to HEAD. Thanks!