WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 5 years ago

#7754 closed defect (bug) (duplicate)

texturize bug with quotes inside parenthesis

Reported by: markjaquith Owned by: markjaquith
Milestone: 2.9 Priority: normal
Severity: normal Version: 2.7
Component: Formatting Keywords: wptexturize needs-patch
Focuses: Cc:

Description

I waved to a friend ("hi there!" he said).

Becomes:

I waved to a friend (”hi there!” he said).

That first quote should be an open quote, not a close quote.

Attachments (1)

7754.001.diff (1.3 KB) - added by markjaquith 6 years ago.

Download all attachments as: .zip

Change History (12)

@markjaquith6 years ago

comment:1 @markjaquith6 years ago

  • Keywords has-patch needs-testing added
  • Owner changed from anonymous to markjaquith
  • Status changed from new to assigned

Patch uploaded. Needs to be tested for regressions.

comment:2 @markjaquith6 years ago

  • Milestone changed from 2.7 to 2.8

Punting.

comment:4 @designsimply6 years ago

Here's a similar instance:

's'
becomes
’s’
but it should be
‘s’
http://img.skitch.com/20090218-e5w6144tru1wfejr9h59uws2h6.png

comment:5 @ryan6 years ago

  • Component changed from General to Formatting

comment:6 @Denis-de-Bernardy6 years ago

("foo") without a leading space doesn't get caught using that, does it?

comment:7 @Denis-de-Bernardy6 years ago

  • Keywords needs-patch added; has-patch needs-testing removed
  • Milestone changed from 2.8 to Future Release

patch is b0rke

comment:8 @Denis-de-Bernardy6 years ago

  • Milestone changed from Future Release to 2.9

comment:9 @voyou16 years ago

I believe this bug is a duplicate of #1418.

comment:10 @designsimply5 years ago

  • Cc designsimply added

comment:11 @markjaquith5 years ago

  • Resolution set to duplicate
  • Status changed from accepted to closed

Looks like a duplicate of #1418. Closing.

Note: See TracTickets for help on using tickets.