Skip to:
Content

bbPress.org

Opened 11 years ago

Closed 11 years ago

#1297 closed defect (fixed)

bb_die wraps strings in two pairs of <p></p>

Reported by: Nightgunner5 Owned by:
Milestone: 1.0.3 Priority: high
Severity: major Version: 1.0.1
Component: Back-end Keywords: has-patch, tested
Cc:

Description

For example:

bb_die( '<pre>Test</pre>' );

Outputs the following:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en-US" lang="en-US">
<head>
	<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
	<title>bbPress &rsaquo; Error</title>
	<meta name="robots" content="noindex, nofollow" />
	<link rel="stylesheet" href="http://localhost/bbpress/bb-admin/install.css" type="text/css" />
</head>
<body>
	<div id="container">
	<p><p><pre>Test</pre></p></p>

	<p class="last">Back to <a href="http://localhost/bbpress/">bbPress Testsite</a>.</p>
	</div>
</body>
</html>

It would be much simpler to use bb_autop and skip the hard-coded <p></p>.

Attachments (1)

1297.patch (565 bytes) - added by Nightgunner5 11 years ago.

Download all attachments as: .zip

Change History (2)

@Nightgunner5
11 years ago

#1 @chrishajer
11 years ago

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

(In [2446]) Last commit really should have said "fixes #1297". Props Nightgunner5

Note: See TracTickets for help on using tickets.