Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#11390 closed defect (bug) (fixed)

Press This entity encoding fix and blank title

Reported by: noel's profile noel Owned by: azaozz's profile azaozz
Milestone: 3.0 Priority: normal
Severity: normal Version: 2.9
Component: Press This Keywords: press this, has-patch
Focuses: Cc:

Description

Allows posts to have an empty title (for P2 and other themes) and fixes entities that were not decoded upon display in the editor.

Attachments (3)

press-this-fix-entities.patch (1.6 KB) - added by noel 14 years ago.
fixes entities and titles
no-&apos.patch (1.5 KB) - added by azaozz 14 years ago.
aposfix-remove.patch (1.6 KB) - added by noel 14 years ago.
Removes aposfix function calls completely.

Download all attachments as: .zip

Change History (15)

@noel
14 years ago

fixes entities and titles

#1 @lloydbudd
14 years ago

  • Milestone changed from Unassigned to 2.9
  • Type changed from task (blessed) to defect (bug)
  • Version set to 2.9

#2 @ryan
14 years ago

See also #10715

#3 @ryan
14 years ago

If the post ID is a WP_Error, you can't delete it.

#4 @azaozz
14 years ago

(In [12371]) Press This empty title fix, props noel, see #11390, fixes #10715

#5 @azaozz
14 years ago

Not sure about the HTML entities fix. If there's a selection we do stripslashes(), aposfix(), html_entity_decode(), htmlspecialchars() and trim() on it, then we add some <p> and then do html_entity_decode() again (in effect reverting htmlspecialchars)?

@azaozz
14 years ago

#6 @markjaquith
14 years ago

  • Milestone changed from 2.9 to 3.0

http://grafiqa.files.wordpress.com/2008/11/charlie_brown_lucy_football.jpg

#7 @hakre
14 years ago

  • Keywords reporter-feedback added

Can the original reporter please provide feedback wether or not this is fixed now with the current version?

#8 @noel
14 years ago

  • Status changed from new to reviewing

original reporter is still testing the patch. :)

#9 @Denis-de-Bernardy
14 years ago

  • Keywords reporter-feedback removed

@noel
14 years ago

Removes aposfix function calls completely.

#10 @noel
14 years ago

new patch is same as no-&apos patch except it removes erroneous calls to aposfix() function.

#11 @noel
14 years ago

  • Owner changed from noel to azaozz

#12 @azaozz
14 years ago

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

In [12592] Fix Press This entity encoding, props noel, fixes #11390

Note: See TracTickets for help on using tickets.