Skip to content

Do not strip trailing semicolon out in safecss_filter_attr()#3579

Closed
Hug0-Drelon wants to merge 2 commits intoWordPress:trunkfrom
Hug0-Drelon:fix-57002
Closed

Do not strip trailing semicolon out in safecss_filter_attr()#3579
Hug0-Drelon wants to merge 2 commits intoWordPress:trunkfrom
Hug0-Drelon:fix-57002

Conversation

@Hug0-Drelon
Copy link
Copy Markdown

Ticket https://core.trac.wordpress.org/ticket/57002

Description

  • Add tests for safecss_filter_attr().
  • Add a ; before returning filtered CSS in safecss_filter_attr().

@Hug0-Drelon
Copy link
Copy Markdown
Author

I'm aware some tests are failing. This is due to the ; added in any cases...
I'll fix them once the ticket is validated.

@Hug0-Drelon
Copy link
Copy Markdown
Author

Won't fix, see this comment https://core.trac.wordpress.org/ticket/57002#comment:4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant