<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"

"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />

<title>[6869] sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main: Main: About: Centralise the custom opengraph tags content into a singular function rather than spredding the filters through page templates.</title>

</head>

<body>

<style type="text/css"><!--

#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }

#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }

#msg dt:after { content:':';}

#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }

#msg dl a { font-weight: bold}

#msg dl a:link    { color:#fc3; }

#msg dl a:active  { color:#ff0; }

#msg dl a:visited { color:#cc6; }

h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }

#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }

#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }

#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }

#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }

#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }

#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }

#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }

#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }

#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }

#logmsg pre { background: #eee; padding: 1em; }

#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}

#logmsg dl { margin: 0; }

#logmsg dt { font-weight: bold; }

#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }

#logmsg dd:before { content:'\00bb';}

#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }

#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }

#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }

#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }

#logmsg table th.Corner { text-align: left; }

#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }

#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }

#patch { width: 100%; }

#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}

#patch .propset h4, #patch .binary h4 {margin:0;}

#patch pre {padding:0;line-height:1.2em;margin:0;}

#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}

#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}

#patch span {display:block;padding:0 10px;}

#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}

#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}

#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}

#patch .lines, .info {color:#888;background:#fff;}

--></style>

<div id="msg">

<dl class="meta" style="font-size: 105%">

<dt style="float: left; width: 6em; font-weight: bold">Revision</dt> <dd><a style="font-weight: bold" href="http://meta.trac.wordpress.org/changeset/6869">6869</a><script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","description":"Review this Commit","action":{"@type":"ViewAction","url":"http://meta.trac.wordpress.org/changeset/6869","name":"Review Commit"}}</script></dd>

<dt style="float: left; width: 6em; font-weight: bold">Author</dt> <dd>dd32</dd>

<dt style="float: left; width: 6em; font-weight: bold">Date</dt> <dd>2018-03-13 05:00:54 +0000 (Tue, 13 Mar 2018)</dd>

</dl>

<pre style='padding-left: 1em; margin: 2em 0; border-left: 2px solid #ccc; line-height: 1.25; font-size: 105%; font-family: sans-serif'>Main: About: Centralise the custom opengraph tags content into a singular function rather than spredding the filters through page templates.

This change also makes it possible to provide the page embeds with some content, where as previously it was blank, for example: https://test.wordpress.org/about/license/embed/

See <a href="http://meta.trac.wordpress.org/ticket/3046">#3046</a>.</pre>

<h3>Modified Paths</h3>

<ul>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainfunctionsphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/functions.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutdomainsphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-domains.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutetiquettephp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-etiquette.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutfeaturesphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-features.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageabouthistoryphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-history.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutlicensephp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-license.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutlogosphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-logos.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutphilosophyphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-philosophy.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutprivacyphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-privacy.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutrequirementsphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-requirements.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutroadmapphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-roadmap.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutsecurityphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-security.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutstatsphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-stats.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutswagphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-swag.php</a></li>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about.php</a></li>

</ul>

<h3>Added Paths</h3>

<ul>

<li><a href="#sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainincpagemetadescriptionsphp">sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/inc/page-meta-descriptions.php</a></li>

</ul>

</div>

<div id="patch">

<h3>Diff</h3>

<a id="sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainfunctionsphp"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/functions.php</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/functions.php      2018-03-13 03:40:28 UTC (rev 6868)

+++ sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/functions.php        2018-03-13 05:00:54 UTC (rev 6869)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -166,7 +166,30 @@

</span><span class="cx" style="display: block; padding: 0 10px"> }

</span><span class="cx" style="display: block; padding: 0 10px"> add_filter( 'page_template_hierarchy', __NAMESPACE__ . '\child_page_templates' );

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+function use_opengraph_data_for_embed_template() {

+       global $post;

+       if ( 'page' != $post->post_type || ! $post->page_template || 'default' == $post->page_template ) {

+               return;

+       }

+

+       $meta = custom_open_graph_tags();

+       if ( $meta ) {

+               add_filter( 'the_title', function( $title ) use( $meta ) {

+                       return $meta['og:title'] ?? $title;

+               } );

+               add_filter( 'the_content', function( $content ) use( $meta ) {

+                       return $meta['og:description'] ?? $content;

+               } );

+       }

+}

+add_action( 'embed_head', __NAMESPACE__ . '\use_opengraph_data_for_embed_template' );

+

</ins><span class="cx" style="display: block; padding: 0 10px"> /**

</span><span class="cx" style="display: block; padding: 0 10px">  * Custom template tags.

</span><span class="cx" style="display: block; padding: 0 10px">  */

</span><span class="cx" style="display: block; padding: 0 10px"> require_once get_stylesheet_directory() . '/inc/template-tags.php';

</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+

+/**

+ * Custom meta descriptions for page templates.

+ */

+require_once get_stylesheet_directory() . '/inc/page-meta-descriptions.php';

</ins></span></pre></div>

<a id="sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainincpagemetadescriptionsphp"></a>

<div class="addfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Added: sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/inc/page-meta-descriptions.php</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/inc/page-meta-descriptions.php                             (rev 0)

+++ sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/inc/page-meta-descriptions.php       2018-03-13 05:00:54 UTC (rev 6869)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -0,0 +1,104 @@

</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+<?php

+/**

+ * Custom template tags

+ *

+ * @package WordPressdotorg\MainTheme

+ */

+

+namespace WordPressdotorg\MainTheme;

+

+/**

+ * Add custom open-grapgh tags for page templates where the content is hard-coded.

+ *

+ * This is also defined here to allow it to be used on pages where the page template is not included for that page, sych as the embed template.

+ */

+function custom_open_graph_tags( $tags = array() ) {

+       $post = get_post();

+

+       switch ( $post->page_template ) {

+               default:

+                       return $tags;

+                       break;

+

+               case 'page-about-domains.php':

+                       $title = _esc_html__( 'WordPress Domains', 'wporg' );

+                       $desc  = _esc_html__( 'WordPress domains and site names can be very flexible; however, top-level domains can&#8217;t use the word WordPress. Find out what is allowed and what constitutes a trademark violation, as well as policies on subdomain use. Review the list of official WordPress sites to know how to recognize and advise violators.', 'wporg' );

+                       break;

+

+               case 'page-about-etiquette.php':

+                       $title = _esc_html__( 'Etiquette at WordPress', 'wporg' );

+                       $desc  = _esc_html__( 'We welcome the contributions of everyone who&#8217;s interested in joining the WordPress open source project, and every thriving, diverse community needs etiquette guidelines. Review our simple guidelines that focus on diversity, safety, and inclusion and foster a welcoming community for our contributors around the world.', 'wporg' );

+                       break;

+

+               case 'page-about-features.php':

+                       $title = _esc_html__( 'WordPress Features', 'wporg' );

+                       /* translators: WordPress market share: 30%; */

+                       $desc  = sprintf( _esc_html__( 'Discover why WordPress powers more than %s of the web. WordPress is a simple, flexible, user-friendly platform, with key features that include media management, SEO, and endless options for customization. More than 50,000 plugins extend the core functionality of WordPress even more. Build your site today.', 'wporg' ), WP_MARKET_SHARE . '%' );

+                       break;

+

+               case 'page-about-history.php':

+                       $title = _esc_html__( 'The History of WordPress', 'wporg' );

+                       /* translators: WordPress market share: 30%; */

+                       $desc  = sprintf( _esc_html__( 'WordPress currently powers more than %s of the web. How did it grow to become the world&#8217;s leading web publishing platform? Learn about the history of WordPress: an open source software project built by an active community of contributors who are passionate about collaboration, empowerment, and the open web.', 'wporg' ), WP_MARKET_SHARE . '%' );

+                       break;

+

+               case 'page-about-license.php':

+                       $title = _esc_html__( 'The GNU Public License', 'wporg' );

+                       $desc  = _esc_html__( 'WordPress is an open source software project, and a fierce believer in the values of the open web. WordPress uses the GNU Public License, which provides a platform for technical expansion and encourages adaptation and innovation. Learn more about this license, and discover what can and cannot be done under it.', 'wporg' );

+                       break;

+

+               case 'page-about-logos.php':

+                       $title = _esc_html__( 'Graphics &amp; Logos', 'wporg' );

+                       $desc  = sprintf( ___( 'When you need the official WordPress logo for a web site or publication, please use one of the following. Please only use logos in accordance with the <a href="%s">WordPress trademark&nbsp;policy</a>.', 'wporg' ), esc_url( 'http://wordpressfoundation.org/trademark-policy/' ) );

+                       break;

+

+               case 'page-about-philosophy.php':

+                       $title = _esc_html__( 'WordPress&rsquo; Philosophy', 'wporg' );

+                       $desc  = _esc_html__( 'At the core of the WordPress philosophy is our commitment to the open web and to building software that works for everyone, from a new user to an advanced developer. Our philosophy pushes WordPress to remain flexible, adaptable, and easy-to-use. Learn about WordPress&#8217;s philosophy and how it shapes our community.', 'wporg' );

+                       break;

+

+               case 'page-about-privacy.php':

+                       $title = _esc_html__( 'WordPress Privacy Policy', 'wporg' );

+                       $desc  = _esc_html__( 'Like other major software platforms, WordPress gathers and collects statistics and analytical data. Privacy is key in this endeavor and WordPress never discloses any personally identifiable data. Review the WordPress Privacy Policy to learn how, as a participant in this community, you&#8217;re privacy is protected.', 'wporg' );

+                       break;

+

+               case 'page-about-requirements.php':

+                       $title = _esc_html__( 'Hosting Requirements for WordPress', 'wporg' );

+                       $desc  = _esc_html__( 'Running WordPress doesn&#8217;t require a lot, but your host will still need to meet a few minimum requirements. Learn about the website hosting requirements to run WordPress, including our recommendation to support PHP 7.2+ and HTTPS. Not sure how to ask your host for these details? Use the sample email we include.', 'wporg' );

+                       break;

+

+               case 'page-about-roadmap.php':

+                       $title = _esc_html__( 'WordPress Development Roadmap', 'wporg' );

+                       $desc  = _esc_html__( 'The WordPress Roadmap lists major releases by date, includes details about the features of each release, and acknowledges the contributing community members. Learn about the status of upcoming releases, development cycles, issues, and milestones. Follow the progress of WordPress development week after week!', 'wporg' );

+                       break;

+

+               case 'page-about-security.php':

+                       $title = _esc_html__( 'WordPress is Secure', 'wporg' );

+                       /* translators: WordPress market share: 30%; */

+                       $desc  = sprintf( _esc_html__( 'Why is WordPress recommended as a secure website-building solution? With a passionate open source community and an extensible, easy-to-use platform, WordPress provides flexible and secure options for all levels of users, from beginners to pros. Learn how WordPress guarantees the security of %s of the web.', 'wporg' ), WP_MARKET_SHARE . '%' );

+                       break;

+

+               case 'page-about-stats.php':

+                       $title = _esc_html__( 'Key WordPress Statistics', 'wporg' );

+                       $desc  = _esc_html__( 'WordPress is committed to transparency, and you can get a better sense of its constant worldwide growth through the statistics we share. Review key WordPress stats including usage breakdown by WordPress versions, PHP and MySQL versions being run, and locales of use, and see how WordPress expands its global reach.', 'wporg' );

+                       break;

+

+               case 'page-about-swag.php':

+                       $title = _esc_html__( 'WordPress Swag', 'wporg' );

+                       $desc  = _esc_html__( 'Show your WordPress pride and run with the coolest swag! You&#8217;ll be surprised how widely recognized our logo is around the world, bringing people together through recognition and community. Choose your WordPress swag today (Wapuu t-shirt, anyone?) and your purchase will also support free swag at WordCamps and meetups.', 'wporg' );

+                       break;

+

+               case 'page-about.php':

+                       $title = _esc_html__( 'Democratize Publishing', 'wporg' );

+                       $desc  = _esc_html__( 'WordPress is software designed for everyone with emphasis on accessibility, performance, security, and usability.', 'wporg' );

+                       break;

+       }

+

+       $tags['og:title']            = $title;

+       $tags['twitter:text:title']  = $title;

+       $tags['og:description']      = $desc;

+       $tags['twitter:description'] = $desc;

+

+       return $tags;

+}

+add_filter( 'jetpack_open_graph_tags', 'custom_open_graph_tags' );

</ins><span class="cx" style="display: block; padding: 0 10px">Property changes on: sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/inc/page-meta-descriptions.php

</span><span class="cx" style="display: block; padding: 0 10px">___________________________________________________________________

</span></span></pre></div>

<a id="svneolstyle"></a>

<div class="addfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Added: svn:eol-style</h4></div>

<ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+native

</ins><span class="cx" style="display: block; padding: 0 10px">\ No newline at end of property

</span><a id="sitestrunkwordpressorgpublic_htmlwpcontentthemespubwporgmainpageaboutdomainsphp"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-domains.php</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-domains.php     2018-03-13 03:40:28 UTC (rev 6868)

+++ sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-domains.php       2018-03-13 05:00:54 UTC (rev 6869)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -25,15 +25,8 @@

</span><span class="cx" style="display: block; padding: 0 10px">        return new \WP_Error();

</span><span class="cx" style="display: block; padding: 0 10px"> } );

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-add_filter( 'jetpack_open_graph_tags', function( $tags ) {

-       $tags['og:title']            = _esc_html__( 'WordPress Domains', 'wporg' );

-       $tags['og:description']      = _esc_html__( 'WordPress domains and site names can be very flexible; however, top-level domains can&#8217;t use the word WordPress. Find out what is allowed and what constitutes a trademark violation, as well as policies on subdomain use. Review the list of official WordPress sites to know how to recognize and advise violators.', 'wporg' );

-       $tags['twitter:text:title']  = $tags['og:title'];

-       $tags['twitter:description'] = $tags['og:description'];