Opened 22 months ago

Closed 19 months ago

Last modified 19 months ago

#18312 closed defect (bug) (duplicate)

Bug when using WordPress should correct invalidly nested XHTML automatically with lists

Reported by: racqnhack Owned by: racqnhack
Priority: normal Milestone:
Component: Formatting Version: 3.2.1
Severity: normal Keywords:
Cc: racqnhack@…, kurtpayne

Description

To duplicate:

Enable "WordPress should correct invalidly nested XHTML automatically" in Settings -> Writing

Create new page.

Create new bullet list with nested list

Hit Preview and see that lists (html) are coming out different.

Then, disable and try again and see that lists work now.

I can duplicate this on all my wp sites.

Attachments (1)

18312.diff (8.1 KB) - added by kurtpayne 22 months ago.
New version of force_balance_tags function

Download all attachments as: .zip

Change History (11)

  • Keywords needs-patch added; reporter-feedback removed
  • Keywords needs-testing removed

New version of force_balance_tags function

  • Cc kurtpayne added
  • Keywords has-patch added; needs-patch removed

I can confirm the problem as well. I did some extra testing on this and found the problem with other nested types like dictionary lists, too. I have rewritten the force_balance_tags function to loop through nested tags differently to avoid this problem. I also added the ability to skip balancing tags inside certain tags (like script and style) where it may be problematic.

Thoughts?

  • Severity changed from critical to normal

Looks like this may be due to a TinyMCE upgrade to 3.2.1 not working with _balance_tags properly.

  • Keywords has-patch removed

Tested patch and it did not change the results, as mentioned before this is not just a problem with the function.

This appears to be something to be fixed in 3.3 with TinyMCE upgrade (in which this has been fixed).

  • Resolution set to duplicate
  • Status changed from new to closed
Last edited 19 months ago by racqnhack (previous) (diff)
  • Resolution duplicate deleted
  • Status changed from closed to reopened
  • Resolution set to duplicate
  • Status changed from reopened to closed

Duplicate (same issue with TinyMCE) of: http://core.trac.wordpress.org/ticket/18261

  • Milestone Awaiting Review deleted
Note: See TracTickets for help on using tickets.