Closed (duplicate)
Project:
Drupal core
Version:
7.x-dev
Component:
update.module
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Feb 2010 at 18:28 UTC
Updated:
25 Feb 2012 at 18:38 UTC
Jump to comment: Most recent file
Comments
Comment #1
berdirThis is probably related to #706608: system_update_files_database() called from update_get_projects() breaks system table on cache clear since status is also set to 0. This also happened in the tests but has been fixed there by running cron as the installer does.
Comment #2
dave reidHave any idea how they got reset?
Comment #3
ctmattice1 commentedConfirm it has to do with cache.
On line 220 of update.php I change the cache_clear_all function from TRUE to FALSE
The update process works, modules retain their schema_version field, and remain enabled.
Comment #4
catchThat's equivalent to removing it entirely. I'm sure it needs to be cleared but let's see what happens if you don't.
Comment #5
berdir- *Something* seems to be causing this, right now, it's update.module but removing the clear_cache_all() is imho not a proper fix. This could as well happen in a contrib module and then it will be a lot harder to debug. How can a clear_cache_all() have any effect on the {system} table? That shouldn't be possible...
- Once fixed, this imho needs a test case to reproduce the breaking behavior to make sure that it doesn't happen again.
I'll try to go over this with an debugger...
Comment #6
catchYeah it's not a proper fix, I just want to see if that's actually what's causing it.
Comment #7
catchAlso, the presence of that cache_clear_all() is troubling in itself, as far as I know dww wrote a bunch of helper functions to avoid using cache_set() cache_get() and cache_clear_all() on {cache_update}, however it's now in system_flush_all_caches() and explicitly cleared in update.php, so something's dodgy there anyway. Changing component.
Comment #8
ctmattice1 commentedUploaded todays HEAD then ran update.php (D7). WSOD - System table trashed, same as on D5-D6-D7 database attempt, schema_version set to default.
Applied the patch and it works. I don't know enough about the cache system
Comment #9
catchThis has to be the same bug as #706608: system_update_files_database() called from update_get_projects() breaks system table on cache clear, so marking as duplicate. See you over there.
Comment #10
troks commentedComment #11
webchick