Ticket #1936 (closed defect (bug): fixed)

Opened 6 years ago

Last modified 5 years ago

Ugly bland error message upon missing wp-config file ... and a fix!

Reported by: chuyskywalker Owned by: markjaquith
Priority: normal Milestone: 2.1
Component: Administration Version: 1.6
Severity: normal Keywords: has-patch commit
Cc:

Description

I was going through the 2.0 beta1 to give it a run and after SNV CO'ing the files I just hit the website and got this ugly, no frills, plain, and scary error message about some missing files and some such.

So, doing my part, here's a patch that snatches the same layout as the database error screen and uses it to pretty up this error. Additionally, I added some basic write checks to see if a user can even use the setup-config.php before offering them the false hope that it'd work.

Finally, I've updated the "more help" link to point to the much more complete and helpful codex wp-config reference.

Attachments

nicer-no-config.diff Download (2.6 KB) - added by chuyskywalker 6 years ago.
patch to make the ugly error not so ugly, and much more helpful
1936.diff Download (2.5 KB) - added by Nazgul 5 years ago.
missing_wp-config.patch Download (2.4 KB) - added by Viper007Bond 5 years ago.
missing_wp-config.2.patch Download (2.4 KB) - added by Viper007Bond 5 years ago.

Change History

patch to make the ugly error not so ugly, and much more helpful

  • Keywords bg|has-patch added

comment:2   matt6 years ago

  • Milestone changed from 2.0 to 2.1

Nazgul5 years ago

  • Keywords has-patch added; bg|has-patch removed

Updated the patch to make use of wp_die.

  • Milestone changed from 2.1 to 2.0.5

2.0.5. Candidate

  • Owner changed from anonymous to markjaquith
  • Status changed from new to assigned
  • Milestone changed from 2.0.5 to 2.1

wp_die() is 2.1 only.

  • Status changed from assigned to closed
  • Resolution set to fixed

(In [4307]) missing wp-config.php error uses wp_die() now. props Nazgul. fixes #1936

  • Status changed from closed to reopened
  • Resolution fixed deleted

Needs some more work.

Bah, forgot to check the "overwrite" box.

comment:9   matt5 years ago

  • Milestone changed from 2.1 to 2.2

comment:10 follow-up: ↓ 11   Viper007Bond5 years ago

Why is changing the help link to the Codex something that should wait for 2.2?

comment:11 in reply to: ↑ 10   foolswisdom5 years ago

  • Keywords commit added
  • Milestone changed from 2.2 to 2.1

Replying to Viper007Bond:

Why is changing the help link to the Codex something that should wait for 2.2?

Because this ticket has become epic? ;-) But lets try again. Setting milestone to 2.1 . Please apply missing_wp-config.2.patch

epic: started out with a detailed problem and creative and complete solution by chuyskywalker, later Nazgul lay down a tight fix... time passed... Viper reopened for the missing important pieces, but the tale was probably forgotten. I often do the same thing, but I do try to remember that we have lots of ticket numbers available.

  • Status changed from reopened to closed
  • Resolution set to fixed

(In [4737]) Change editing wp-config codex link. fixes #1936

Note: See TracTickets for help on using tickets.