Opened 20 months ago

Closed 20 months ago

#18765 closed defect (bug) (fixed)

tag_escape() should not strip digits

Reported by: sirzooro Owned by:
Priority: normal Milestone: 3.3
Component: Formatting Version: 3.2.1
Severity: normal Keywords: has-patch
Cc:

Description

tag_escape() strips digits, but it shouldn't - they are needed for <h1>..<h6> tags. Attached patch fixes this.

Attachments (1)

formatting.php.diff​ (410 bytes) - added by sirzooro 20 months ago.

Download all attachments as: .zip

Change History (5)

sirzooro — 20 months ago

comment:1   SergeyBiryukov — 20 months ago

  • Milestone changed from Awaiting Review to 3.3

tag_escape() is only used in gallery_shortcode(), so the bug manifests if someone sets itemtag or captiontag attribute to one of the <h1>..<h6> tags. This seems like a rare situation, but still possible.

comment:2   olleicua — 20 months ago

Tested using revision 18784M and above formatting.php.diff. It works.

comment:3   nacin — 20 months ago

Realistically, 1-6 is the only required range. I haven't seen any specs for introducing more header tags. Unless, I guess, they're allowed in namespaced elements. Looks good.

comment:4   ryan — 20 months ago

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

[18955]

Note: See TracTickets for help on using tickets.