Closed (fixed)
Project:
Drupal core
Version:
5.x-dev
Component:
base system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
30 Dec 2006 at 01:41 UTC
Updated:
9 Apr 2007 at 07:45 UTC
Jump to comment: Most recent file
DRUPAL_MINIMUM_MYSQL and a number of similar variables are defined in system.install where they aren't available after installation. I propose to move them to system.module below the Version string.
Currently, functions like db_status_report that use these defines won't work outside the installer's scope.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | move_defines2.patch.txt | 1.39 KB | asimmonds |
| #3 | move_defines.patch.txt | 2.8 KB | killes@www.drop.org |
Comments
Comment #1
dwwseems reasonable. currently, nothing in core calls db_status_report() except system.install, but i don't see why we'd want to end up breaking those functions outside the scope of update.php. the only *tiny* downside is 4 more lines of php to parse on every page load, but that's definitely not a good enough reason to avoid this change...
+1 from me.
Comment #2
ChrisKennedy commentedSounds good to me. I did some quick searching where system.install is loaded and system.module isn't (i.e. part of installation) but didn't find anything that used those variables.
Comment #3
killes@www.drop.org commentedok, here's the patch. it needs to be tested during an install.
Comment #4
drummComment #5
asimmonds commentedRerolled for HEAD.
Have also tested with a fresh install, which works OK.
Comment #6
moshe weitzman commentedlooks good, has been tested. RTBC.
i even made sure that killes didn't try to bump the minimum requirements :)
Comment #7
dries commentedCommitted to CVS HEAD. Thanks.
Comment #8
asimmonds commentedCommit http://drupal.org/cvs?commit=62004 doesn't show the defines being removed from system.install
Comment #9
killes@www.drop.org commentedhttp://drupal.org/cvs?commit=62009
Comment #10
(not verified) commented