Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Classic Theme Helper: Copy Social Links code to Classic Theme Helper package #38593

Merged
merged 11 commits into from
Aug 1, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion projects/packages/classic-theme-helper/.phan/baseline.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,18 +10,21 @@
return [
// # Issue statistics:
// PhanTypeMismatchArgumentInternal : 10+ occurrences
// PhanUndeclaredClassMethod : 7 occurrences
// PhanUndeclaredClassReference : 4 occurrences
// PhanTypeInvalidDimOffset : 2 occurrences
// PhanTypeMismatchArgument : 2 occurrences
// PhanTypeComparisonToArray : 1 occurrence
// PhanTypeMismatchArgumentProbablyReal : 1 occurrence
// PhanTypeMismatchProperty : 1 occurrence
// PhanTypePossiblyInvalidDimOffset : 1 occurrence
// PhanUndeclaredFunction : 1 occurrence
// PhanUndeclaredTypeProperty : 1 occurrence

// Currently, file_suppressions and directory_suppressions are the only supported suppressions
'file_suppressions' => [
'_inc/lib/tonesque.php' => ['PhanTypeMismatchArgument', 'PhanTypeMismatchArgumentInternal', 'PhanTypeMismatchArgumentProbablyReal'],
'src/class-featured-content.php' => ['PhanTypeComparisonToArray', 'PhanTypeInvalidDimOffset', 'PhanTypeMismatchArgument', 'PhanTypeMismatchProperty', 'PhanTypePossiblyInvalidDimOffset'],
'src/social-links.php' => ['PhanUndeclaredClassMethod', 'PhanUndeclaredClassReference', 'PhanUndeclaredTypeProperty'],
],
// 'directory_suppressions' => ['src/directory_name' => ['PhanIssueName1', 'PhanIssueName2']] can be manually added if needed.
// (directory_suppressions will currently be ignored by subsequent calls to --save-baseline, but may be preserved in future Phan releases)
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Significance: patch
Type: added

Social Links: Added feature to Classic Theme Helper package.
2 changes: 1 addition & 1 deletion projects/packages/classic-theme-helper/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"private": true,
"name": "@automattic/jetpack-classic-theme-helper",
"version": "0.4.3",
"version": "0.4.4-alpha",
"description": "Features used with classic themes",
"homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/packages/classic-theme-helper/#readme",
"bugs": {
Expand Down
2 changes: 1 addition & 1 deletion projects/packages/classic-theme-helper/src/class-main.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
*/
class Main {

const PACKAGE_VERSION = '0.4.3';
const PACKAGE_VERSION = '0.4.4-alpha';

/**
* Modules to include.
Expand Down
279 changes: 279 additions & 0 deletions projects/packages/classic-theme-helper/src/social-links.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,279 @@
<?php // phpcs:ignore WordPress.Files.FileName.InvalidClassFileName
/**
* Theme Tools: Social Links.
*
* This feature will only be activated for themes that declare their support.
* This can be done by adding code similar to the following during the
* 'after_setup_theme' action:
*
* add_theme_support( 'social-links', array(
* 'facebook', 'twitter', 'linkedin', 'tumblr',
* ) );
*
* @package automattic/jetpack-classic-theme-helper
*/

namespace Automattic\Jetpack\Classic_Theme_Helper;

// phpcs:disable Universal.Files.SeparateFunctionsFromOO.Mixed -- TODO: Move classes to appropriately-named class files.

// @todo Un-comment and possibly move initialization when requiring the file.
// if ( ! function_exists( 'jetpack_theme_supports_social_links' ) ) {
// **
// * Init Social_Links if the theme declares support.
// *
// * @suppress PhanNoopNew
// */
// function jetpack_theme_supports_social_links() {
// if ( ! wp_is_block_theme() && current_theme_supports( 'social-links' ) && function_exists( 'publicize_init' ) ) {
// new \Automattic\Jetpack\Classic_Theme_Helper\Social_Links();
// }
// }
// add_action( 'init', 'jetpack_theme_supports_social_links', 30 );
// }

if ( ! class_exists( __NAMESPACE__ . '\Social_Links' ) ) {

/**
* Social_Links main class.
*/
class Social_Links {

/**
* The links the user set for each service.
*
* @var array
*/
private $links;

/**
* A Publicize object.
*
* @var Publicize
*/
private $publicize;

/**
* An array with all services that are supported by both Publicize and the
* currently active theme.
*
* @var array
*/
private $services = array();

/**
* An array of the services the theme supports
*
* @var array
*/
private $theme_supported_services = array();

/**
* Constructor.
*/
public function __construct() {
$theme_support = get_theme_support( 'social-links' );

/*
An array of named arguments must be passed as the second parameter
* of add_theme_support().
*/
if ( empty( $theme_support[0] ) ) {
return;
}

$this->theme_supported_services = $theme_support[0];
$this->links = class_exists( \Jetpack_Options::class ) ? \Jetpack_Options::get_option( 'social_links', array() ) : '';

$this->admin_setup();

add_filter( 'jetpack_has_social_links', array( $this, 'has_social_links' ) );
add_filter( 'jetpack_get_social_links', array( $this, 'get_social_links' ) );

foreach ( $theme_support[0] as $service ) {
add_filter( "pre_option_jetpack-$service", array( $this, 'get_social_link_filter' ) ); // - `get_option( 'jetpack-service' );`
add_filter( "theme_mod_jetpack-$service", array( $this, 'get_social_link_filter' ) ); // - `get_theme_mod( 'jetpack-service' );`
}
}

/**
* Init the admin setup.
*/
public function admin_setup() {
if ( ! current_user_can( 'manage_options' ) ) {
return;
}

if ( ! is_admin() && ! $this->is_customize_preview() ) {
return;
}

// @phan-suppress-next-line PhanUndeclaredFunction -- Function checked with function_exists - see https://github.com/phan/phan/issues/1204.
$this->publicize = function_exists( 'publicize_init' ) ? publicize_init() : null;
$publicize_services = $this->publicize->get_services( 'connected' );
Copy link
Contributor

Choose a reason for hiding this comment

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

What happens here if $this->publicize is null?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good question. I've no idea at this stage, but I'll make sure to look into that when requiring the package in the next PR.

$this->services = array_intersect( array_keys( $publicize_services ), $this->theme_supported_services );

add_action( 'publicize_connected', array( $this, 'check_links' ), 20 );
add_action( 'publicize_disconnected', array( $this, 'check_links' ), 20 );
add_action( 'customize_register', array( $this, 'customize_register' ) );
add_filter( 'sanitize_option_jetpack_options', array( $this, 'sanitize_link' ) );
}

/**
* Compares the currently saved links with the connected services and removes
* links from services that are no longer connected.
*
* @return void
*/
public function check_links() {
$active_links = array_intersect_key( $this->links, array_flip( $this->services ) );

if ( $active_links !== $this->links ) {
$this->links = $active_links;
if ( class_exists( \Jetpack_Options::class ) ) {
\Jetpack_Options::update_option( 'social_links', $active_links );
}
}
}

/**
* Add social link dropdown to the Customizer.
*
* @param \WP_Customize_Manager $wp_customize Theme Customizer object.
*/
public function customize_register( $wp_customize ) {
$wp_customize->add_section(
'jetpack_social_links',
array(
'title' => esc_html__( 'Connect', 'jetpack-classic-theme-helper' ),
'priority' => 35,
)
);

if ( class_exists( \Publicize::class ) ) {
foreach ( array_keys( $this->publicize->get_services( 'all' ) ) as $service ) {
$choices = $this->get_customize_select( $service );

if ( empty( $choices ) ) {
continue;
}

$wp_customize->add_setting(
"jetpack_options[social_links][$service]",
array(
'type' => 'option',
'default' => '',
)
);

$wp_customize->add_control(
"jetpack-$service",
array(
'label' => $this->publicize->get_service_label( $service ),
'section' => 'jetpack_social_links',
'settings' => "jetpack_options[social_links][$service]",
'type' => 'select',
'choices' => $choices,
)
);
}
}
}

/**
* Sanitizes social links.
*
* @param array $option The incoming values to be sanitized.
* @return array
*/
public function sanitize_link( $option ) {
foreach ( $this->services as $service ) {
if ( ! empty( $option['social_links'][ $service ] ) ) {
$option['social_links'][ $service ] = esc_url_raw( $option['social_links'][ $service ] );
} else {
unset( $option['social_links'][ $service ] );
}
}

return $option;
}

/**
* Returns whether there are any social links set.
*
* @return bool
*/
public function has_social_links() {
return ! empty( $this->links );
}

/**
* Return available social links.
*
* @return array
*/
public function get_social_links() {
return $this->links;
}

/**
* Short-circuits get_option and get_theme_mod calls.
*
* @param string $link The incoming value to be replaced.
* @return string $link The social link that we've got.
*/
public function get_social_link_filter( $link ) {
if ( preg_match( '/_jetpack-(.+)$/i', current_filter(), $matches ) && ! empty( $this->links[ $matches[1] ] ) ) {
return $this->links[ $matches[1] ];
}

return $link;
}

/**
* Puts together an array of choices for a specific service.
*
* @param string $service The social service.
* @return array An associative array with profile links and display names.
*/
private function get_customize_select( $service ) {
$choices = array(
'' => __( '&mdash; Select &mdash;', 'jetpack-classic-theme-helper' ),
);

if ( isset( $this->links[ $service ] ) ) {
$choices[ $this->links[ $service ] ] = $this->links[ $service ];
}

if ( class_exists( \Publicize::class ) ) {
$connected_services = $this->publicize->get_services( 'connected' );
if ( isset( $connected_services[ $service ] ) ) {
foreach ( $connected_services[ $service ] as $c ) {
$profile_link = $this->publicize->get_profile_link( $service, $c );

if ( false === $profile_link ) {
continue;
}

$choices[ $profile_link ] = $this->publicize->get_display_name( $service, $c );
}
}
}

if ( 1 === count( $choices ) ) {
return array();
}

return $choices;
}

/**
* Back-compat function for versions prior to 4.0.
*/
private function is_customize_preview() {
global $wp_customize;
return is_a( $wp_customize, 'WP_Customize_Manager' ) && $wp_customize->is_preview();
}
}

} // - end if ( ! class_exists( 'Social_Links' )
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Significance: patch
Type: other

Social Links: Adding a function_exists check within the social-links.php file, to preventconflicts with package version.
16 changes: 9 additions & 7 deletions projects/plugins/jetpack/modules/theme-tools/social-links.php
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,17 @@

// phpcs:disable Universal.Files.SeparateFunctionsFromOO.Mixed -- TODO: Move classes to appropriately-named class files.

/**
* Init Social_Links if the theme declares support.
*/
function jetpack_theme_supports_social_links() {
if ( ! wp_is_block_theme() && current_theme_supports( 'social-links' ) && function_exists( 'publicize_init' ) ) {
new Social_Links();
if ( ! function_exists( 'jetpack_theme_supports_social_links' ) ) {
/**
* Init Social_Links if the theme declares support.
*/
function jetpack_theme_supports_social_links() {
if ( ! wp_is_block_theme() && current_theme_supports( 'social-links' ) && function_exists( 'publicize_init' ) ) {
new Social_Links();
}
}
add_action( 'init', 'jetpack_theme_supports_social_links', 30 );
}
add_action( 'init', 'jetpack_theme_supports_social_links', 30 );

if ( ! class_exists( 'Social_Links' ) ) {

Expand Down
Loading