Opened 7 years ago
Closed 7 years ago
#2544 closed defect (bug) (fixed)
Running setup-config.php?step=2 (no wp-config.php) produces a 'Headers already sent' error
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 2.1 |
| Component: | General | Version: | 2.0.1 |
| Severity: | normal | Keywords: | wp-db.php bg|has-patch bg|commit |
| Cc: |
Description
Running the install script with wp-config.php not present, setup-config.php properly detects the missing file, and offers to create one. After filling in the necessary entries for hostname, user, password, database, and "submit"ing the data, setup-config.php fails at step 2 with this error.
Warning: Cannot modify header information - headers already sent by (output started at /storage/Debian/wordpress/wp-admin/setup-config.php:22) in /storage/Debian/wordpress/wp-includes/wp-db.php on line 308
This error is present in a 'clean' install of 2.0.1 and has been 'worked around' by commenting the following lines:
header( 'Content-Type: text/html; charset=utf-8');
<h1 id="logo"><img alt="WordPress" src="http://static.wordpress.org/logo.png" /></h1>
More in this thread: http://wordpress.org/support/topic/64003
Attachments (1)
Change History (14)
comment:2
davidhouse — 7 years ago
Yes, it's normal to both send an HTTP header and have said header in an HTML meta tag.
comment:3
davidhouse — 7 years ago
- Owner changed from anonymous to davidhouse
- Status changed from new to assigned
davidhouse — 7 years ago
- Resolution set to worksforme
- Status changed from assigned to closed
Fixed for me. Patch worked fine.
comment:6
davidhouse — 7 years ago
- Resolution worksforme deleted
- Status changed from closed to reopened
Thanks for your eagerness, but "worksforme" is for closing tickets that don't appear to be valid, i.e. no-one can replicate them. People tend to write patches then upload them onto bugs, which then eventually filter through into the core. When that is done (once the proposed code moves into the core), we then close the bug.
Sorry about the confusion.
comment:7
davidhouse — 7 years ago
- Keywords bg|commit added
On the other hand, now we've got verification, I'll push it for commit.
Not to worry at all, it does seem to work.
Afraid I'm pretty new to SVN so still finding my feet. I'll see if the codex has any info I can read up on the process on here. Thanks for the help.
comment:9
davidhouse — 7 years ago
Allow me to explain the bug process.
1) User finds a bug
2) User asks around to make sure it's actually a bug, perhaps posts to the testers list
3) User makes a ticket on trac (this site, trac.wordpress.org)
4) Various code monkeys find the ticket, test it does actually exist
5) Someone writes a patch
6) That person uploads the patch using the 'Attach file' button
7) Someone else verifies the patch works
8) A committor (one of Matt Mullenweg or Ryan Boren) comes along and 'commits' the patch: changes the core code in our svn repository to include the patch that someone coded back in stage 5.
9) Bug is closed with 'fixed'.
comment:10
davidhouse — 7 years ago
That was horribly formatted, let me try again.
- User finds a bug
- User asks around to make sure it's actually a bug, perhaps posts to the testers list
- User makes a ticket on trac (this site, trac.wordpress.org)
- Various code monkeys find the ticket, test it does actually exist
- Someone writes a patch
- That person uploads the patch using the 'Attach file' button
- Someone else verifies the patch works
- A committor (one of Matt Mullenweg or Ryan Boren) comes along and 'commits' the patch: changes the core code in our svn repository to include the patch that someone coded back in stage 5.
- Bug is closed with 'fixed'.
comment:11
MichaelH — 7 years ago
David's description of the bug process added to Codex at:
comment:12
ryan — 7 years ago
- Milestone set to 2.1
comment:13
ryan — 7 years ago
- Resolution set to fixed
- Status changed from reopened to closed
Just looking again and see the header is declared twice: