lsx\sharing\classes\frontend\Button

Complexity

Total Complexity

19

Size/Duplication

Total Lines	118
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	4
A	get_link_twitter()	0	22	3
A	get_link_facebook()	0	6	1
A	get_link()	0	13	6
A	get_link_pinterest()	0	17	4
A	get_link_email()	0	1	1

<?php
/**
 * LSX_Sharing_Button
 *
 * @package lsx-sharing
 */

namespace lsx\sharing\classes\frontend;

/**
 * LSX Sharing buttons class.
 *
 * @package lsx-sharing
 */
class Button {


    /**
     * Services available.
     *
     * @var array
     */
    public $services = array(

Expected 1 blank line(s) before first member var; 2 found

		'facebook',
		'twitter',
		'pinterest',
    );

    /**
     * Current service.
     *
     * @var string
     */
    public $service = '';

    /**

Doc comment for parameter "$service" missing

Doc comment for parameter "$options" missing

Doc comment for parameter "$prefix" missing

     * Constructor.
     */
    public function __construct( $service, $options, $prefix = 'sharing' ) {

Expected 2 blank lines before function; 1 found

         $this->options = $options;

The property options does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

Found precision alignment of 1 spaces.

        if ( ! in_array($service, $this->services, true) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

Expected 0 spaces before closing bracket; 1 found

            return;
        }

No blank line found after control structure

        if ( \lsx\sharing\includes\functions\is_button_disabled('global', $service) || \lsx\sharing\includes\functions\is_button_disabled($prefix, $service) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

Expected 0 spaces before closing bracket; 1 found

            return '';
        }

No blank line found after control structure

        $this->service = $service;
    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

    /**

Doc comment for parameter "$post" missing

     * Get service link to share.
     */
    public function get_link( $post ) {
         if ( empty($post) ) {

Found precision alignment of 1 spaces.

Expected 0 spaces after opening bracket; 1 found

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

Expected 0 spaces before closing bracket; 1 found

            return '';
        }

Closing brace indented incorrectly; expected 9 spaces, found 8

        if ( 'email' === $this->service ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

            return $this->get_link_email($post);

The call to lsx\sharing\classes\frontend\Button::get_link_email() has too many arguments starting with $post.

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        } elseif ( 'facebook' === $this->service ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

            return $this->get_link_facebook($post);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        } elseif ( 'twitter' === $this->service ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

            return $this->get_link_twitter($post);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        } elseif ( 'pinterest' === $this->service ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

            return $this->get_link_pinterest($post);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        }
    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

    /**

Doc comment for parameter "$post" missing

     * Get Facebook link to share.
     */
    public function get_link_facebook( $post ) {
         $permalink = get_permalink($post->ID);

Found precision alignment of 1 spaces.

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        $permalink = apply_filters('lsx_sharing_facebook_url', $permalink, $post);

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        $title     = apply_filters('the_title', $post->post_title);

Equals sign not aligned with surrounding assignments; expected 6 spaces but found 5 spaces

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        return 'https://www.facebook.com/sharer.php?display=page&u=' . rawurlencode($permalink) . '&t=' . rawurlencode($title);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

It seems like $permalink can also be of type false; however, parameter $string of rawurlencode() does only seem to accept string, maybe add an additional type check?

    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

    /**

Doc comment for parameter "$post" missing

     * Get Twitter link to share.
     */
    public function get_link_twitter( $post ) {
         $permalink = get_permalink($post->ID);

Found precision alignment of 1 spaces.

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        $permalink = apply_filters('lsx_sharing_twitter_url', $permalink, $post);

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        $title     = apply_filters('the_title', $post->post_title);

Equals sign not aligned with surrounding assignments; expected 6 spaces but found 5 spaces

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        if ( function_exists('mb_stripos') ) {

Expected 0 spaces after opening bracket; 1 found

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

Expected 0 spaces before closing bracket; 1 found

            $strlen = 'mb_strlen';
            $substr = 'mb_substr';
        } else {
            $strlen = 'strlen';
            $substr = 'substr';
        }

        $short_url_length = 24;

        if ( ( $strlen($title) + $short_url_length ) > 140 ) {

Expected 0 spaces after opening bracket; 1 found

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

Expected 0 spaces before closing bracket; 1 found

            $text = $substr($title, 0, ( 140 - $short_url_length - 1 )) . "\xE2\x80\xA6";

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        } else {
            $text = $title;
        }

        return 'https://twitter.com/intent/tweet?text=' . rawurlencode($text) . '&url=' . rawurlencode($permalink);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

It seems like $permalink can also be of type false; however, parameter $string of rawurlencode() does only seem to accept string, maybe add an additional type check?

    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

    /**

Doc comment for parameter "$post" missing

     * Get Pinterest link to share.
     */
    public function get_link_pinterest( $post ) {
		$image = '';

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

        $permalink = get_permalink($post->ID);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        $permalink = apply_filters('lsx_sharing_pinterest_url', $permalink, $post);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        $title     = apply_filters('the_title', $post->post_title);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        if ( ! has_post_thumbnail($post) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

Expected 0 spaces before closing bracket; 1 found

            if ( class_exists('lsx\legacy\Placeholders') ) {

Expected 0 spaces after opening bracket; 1 found

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

Expected 0 spaces before closing bracket; 1 found

                $image = \lsx\legacy\Placeholders::placeholder_url(null, $post->post_type);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

            } elseif ( class_exists('LSX_Placeholders') ) {

Expected 0 spaces after opening bracket; 1 found

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

Expected 0 spaces before closing bracket; 1 found

                $image = \LSX_Placeholders::placeholder_url(null, $post->post_type);

The type LSX_Placeholders was not found. Maybe you did not declare it correctly or list all dependencies?

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

            }
        } else {
            $image = get_the_post_thumbnail_url($post->ID, 'large');

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

        }

        return 'https://www.pinterest.com/pin/create/button/?url=' . rawurlencode($permalink) . '&media=' . rawurlencode($image) . '&description=' . rawurlencode($title);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

It seems like $permalink can also be of type false; however, parameter $string of rawurlencode() does only seem to accept string, maybe add an additional type check?

    }

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

    /**
     * Get Email Link.
     */
    public function get_link_email() {     }

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 0 found

}