Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
documentation
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Nov 2008 at 09:59 UTC
Updated:
2 Jan 2014 at 23:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
webchickMoving to documentation component.
Comment #2
drewish commentedhere's a very basic starting point.
Comment #3
nicorac commentedDon't forget to mention that you need the SQLite extension enabled in php.ini:
extension=php_pdo_sqlite.dllor
extension=php_pdo_sqlite.soIf not enabled, the SQLite option does not appear in the Database configuration page.
Comment #4
dave reidSubscribing.
Comment #5
dave reidThe problem with installing on sqlite was that I was very confused about the "database name" field. It had a maxlength parameter that wouldn't allow me to put something like "sites/sqlite.drupalhead.local/files/.ht.drupalhead.sqlite" as the database name, which should be ok.
Comment #6
crashtest_ commentedI am attaching a patch that alters the description text for the Database Name field, and the Database User field in install.php to instruct SQLite users on how to correctly configure the Database Name field.
Comment #7
dave reidI think I'd prefer if the install.sqlite.txt was added first, and then we work on #346494: DB drivers need to be able to change the configure database form during install to allow DatabaseInstaller_sqlite to form_alter the database settings and remove the username/password fields and modify the descriptions. It seems a more Drupal-way to do things. Plus we'll future-proof future db backends.
Comment #8
webchickIMO this is unnecesasrily bloating the descriptions for 100% of our users, even though only maybe 5% might use SQLite.
I'd prefer to see this extra help moved to INSTALL.sqlite.txt, personally.
CrashTest, would you be willing to spiffy up drewish's patch so that it would've helped you be less confused here, and maybe add a general note to the installer that more specific instructions can be found in INSTALL.databasename.txt?
Comment #9
crashtest_ commentedOk, not much needed on the INSTALL.sqlite.txt file. I did add one small line to the install.php file to alert people to the fact that there are INSTALL.database.txt files that they may refer to.
Comment #10
dave reidBTW, I think it would be much more appropriate to actually link to the INSTALL.dbtype.txt like I have worked on in #346494: DB drivers need to be able to change the configure database form during install, so maybe we should keep that separate.
Comment #11
crashtest_ commentedActually, after reviewing a concern about the download-ability of the database file, we decided to reword the text file to include some precautions about that.
Comment #12
crashtest_ commentedNoticed the line breaks are not consistent in that patch, fixed here.
Comment #13
ultimateboy commentedTagging usability.
Comment #14
keith.smith commentedI think I saw one instance of double spaces between sentences in the INSTALL.sqlite.txt file.
(Also, the INSTALL file doesn't mention what PHP packages must be present before SQLite will work properly. This was an issue for me when I went to install an installation on SQLite the first time, and required both some hunting around and, eventually, asking chx what to do in #drupal.)
I'm not positive people will understand
INSTALL.databasetype.txt without an example -- I guess you could use the mysql installation instructions as an example, in italics, here. Also, its worth mentioning in this sentence that the INSTALL* files can be found in the Drupal root directory.
Comment #15
yoroy commentedone more tag for the copy writers :)
Comment #16
chrisshattuck commentedThis might be overkill, but what are thoughts on hiding extraneous information in a slide-down div which is triggered by a link that reads "Need help chosing?"
Comment #17
crashtest_ commentedFixed the double space, and added requirements for SQLite to the text file. Also clarified the INSTALL.databasetype.txt note.
Comment #18
keith.smith commentedGreat!
One small thing:
"To be able to use" can just simply be "To use"
Comment #19
crashtest_ commentedRemoved the "be able".
Comment #20
crashtest_ commentedRenamed patch, re-applied.
Comment #21
dries commentedI committed the help text to CVS, but not the install.php change. There is a better patch for that in the issue queue (see above). Thanks!
Comment #22
crashtest_ commentedAwesome!