Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

WordPress coding standards fixes #331

Merged
merged 6 commits into from
Jul 6, 2016

Conversation

paulschreiber
Copy link
Contributor

Contains several small WordPress coding standards fixes:

  • use strict comparisons
  • use yoda conditions
  • use wp_json_encode()
    • use snake_case
  • don't use create_function()

This was branched off #328, which contains whitespace-only fixes.

@philipjohn philipjohn added this to the 2.12 milestone Jul 6, 2016
@philipjohn philipjohn merged commit dc1ec72 into Automattic:master Jul 6, 2016
@philipjohn philipjohn modified the milestones: 2.12, 3.0 Jul 6, 2016
@@ -35,7 +35,7 @@
<title><?php echo esc_html( $instant_article_post->get_the_title() ); ?></title>
<link><?php echo esc_url( $instant_article_post->get_canonical_url() ); ?></link>
<content:encoded>
<![CDATA[<?php echo $instant_article_post->to_instant_article()->render(); ?>]]>
<![CDATA[<?php echo wp_kses_post( $instant_article_post->to_instant_article()->render() ); ?>]]>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using wp_kses_post strips out any iframe tags insde of op-interactive elements.

philipjohn pushed a commit that referenced this pull request Jul 12, 2016
philipjohn added a commit that referenced this pull request Jul 12, 2016
Remove  to avoid breaking  elements. See #331
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants