Skip to content

WP basic allows#6

Open
kaptinavenger wants to merge 1 commit into
uncatcrea:masterfrom
kaptinavenger:master
Open

WP basic allows#6
kaptinavenger wants to merge 1 commit into
uncatcrea:masterfrom
kaptinavenger:master

Conversation

@kaptinavenger

Copy link
Copy Markdown

These php plugins allow for the passing of the styling tags used by the
default WP tinyMCE editor.

White listing is good.
But
Should white list WP default style for WP apps, the fix should be modular so individual tags can be easily removed from the white-list.
This fixes that.

These php plugins allow for the passing of the styling tags used by the
default WP tinyMCE editor.
@mleroi

mleroi commented May 23, 2016

Copy link
Copy Markdown
Collaborator

Hi and thanks for your contribution!

Since WP-AppKit v0.5, there's no HTML tag whitelisting in post content anymore: uncatcrea/wp-appkit#140 > All tags are now allowed by default.

The "wpak_post_content_allowed_tags" filter is still there in case someone needs to allow only a given set of HTML tags in his app, but if nobody hooks on this filter to set tag restrictions, all tags are allowed: https://github.com/uncatcrea/wp-appkit/blob/master/wp-appkit/lib/components/components-utils.php#L97

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.

2 participants