WordPress.org

Make WordPress Core

Opened 4 months ago

Closed 4 months ago

Last modified 4 months ago

#23516 closed defect (bug) (fixed)

Current 3.6 trunk wont install

Reported by: csixty4 Owned by: SergeyBiryukov
Priority: normal Milestone: 3.6
Component: Upgrade/Install Version: trunk
Severity: major Keywords: dev-feedback
Cc:

Description

Setup for a fresh WP 3.6 install using current trunk crashes with an empty white box on a gray background.

[18-Feb-2013 22:53:56] PHP Fatal error:  Call to undefined function wp_kses_normalize_entities() in /Applications/MAMP/htdocs/wp36/wp-includes/formatting.php on line 2797
[18-Feb-2013 22:53:56] PHP Stack trace:
[18-Feb-2013 22:53:56] PHP   1. {main}() /Applications/MAMP/htdocs/wp36/wp-admin/setup-config.php:0
[18-Feb-2013 22:53:56] PHP   2. setup_config_display_header() /Applications/MAMP/htdocs/wp36/wp-admin/setup-config.php:109
[18-Feb-2013 22:53:56] PHP   3. esc_url() /Applications/MAMP/htdocs/wp36/wp-admin/setup-config.php:103

Attachments (2)

23516.diff (496 bytes) - added by csixty4 4 months ago.
Patch assuming it's ok to load kses.php at this point
23516.1.diff (580 bytes) - added by JustinSainton 4 months ago.

Download all attachments as: .zip

Change History (9)

csixty44 months ago

Patch assuming it's ok to load kses.php at this point

comment:1 SergeyBiryukov4 months ago

  • Keywords dev-feedback added
  • Milestone changed from Awaiting Review to 3.6
  • Severity changed from normal to major

Broken in [23413]. We don't include kses.php in setup-config.php:
http://core.trac.wordpress.org/browser/tags/3.5.1/wp-admin/setup-config.php#L58

Should we revert to esc_attr_e() there, or perhaps include kses.php?

comment:2 nacin4 months ago

I think I am okay with esc_attr here.

JustinSainton4 months ago

comment:3 JustinSainton4 months ago

23516.1.diff replaces esc_url() with esc_attr().

comment:4 SergeyBiryukov4 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 23455:

Revert to esc_attr_e() in setup-config.php to prevent a fatal error.
kses.php is not loaded at this point, so esc_url() cannot be used.

props csixty4, JustinSainton.
fixes #23516. see #23334.

comment:5 follow-up: obenland4 months ago

Shouldn't it still be echo esc_attr() then, rather than using a translation function with esc_attr_e()? Or was that not part of the original ticket?

comment:6 kraftbj4 months ago

#23518 was marked as a duplicate.

comment:7 in reply to: ↑ 5 SergeyBiryukov4 months ago

Replying to obenland:

Shouldn't it still be echo esc_attr() then, rather than using a translation function with esc_attr_e()? Or was that not part of the original ticket?

The intention of #23334 was to use esc_url() where possible, while keeping the links translatable.

Note: See TracTickets for help on using tickets.