GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.
Passed
Push — master ( 41de00...33ed65 )
by Christian
11:07
created

podlove_psb_deactivate()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 2
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 1
nc 1
nop 0
dl 0
loc 2
rs 10
c 0
b 0
f 0
1
<?php
0 ignored issues
show
introduced by
Class file names should be based on the class name with "class-" prepended. Expected class-podlovesubscribebutton.php, but found podlove.php.
Loading history...
2
/**
0 ignored issues
show
Coding Style introduced by
Missing short description in doc comment
Loading history...
3
 * @author    Podlove <[email protected]>
4
 * @copyright Copyright (c) 2014-2018, Podlove
5
 * @license   https://github.com/podlove/podlove-subscribe-button-wp-plugin/blob/master/LICENSE MIT
6
 * @package   Podlove\PodloveSubscribeButton
7
 * @version   1.4.0-beta
8
 */
9
10
/**
11
 * Plugin Name: Podlove Subscribe Button
12
 * Plugin URI:  https://wordpress.org/plugins/podlove-subscribe-button/
13
 * Description: Brings the Podlove Subscribe Button to your WordPress installation.
14
 * Version:     1.4.0-beta
15
 * Author:      Podlove
16
 * Author URI:  https://podlove.org/
17
 * License:     MIT
18
 * License URI: license.txt
19
 * Text Domain: podlove-subscribe-button
20
 */
21
22
/** Check if PHP version is sufficient */
23
if ( ! version_compare( phpversion(), '15.4', ">=" ) ) {
0 ignored issues
show
Coding Style Comprehensibility introduced by
The string literal >= does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
24
25
	function podlove_psb_php_notice() {
0 ignored issues
show
Coding Style Documentation introduced by
Missing doc comment for function podlove_psb_php_notice()
Loading history...
26
		?>
27
		<div id="message" class="error">
28
			<p>
29
				<strong>The Podlove Subscribe Button Plugin could not be activated</strong>
30
			</p>
31
			<p>
32
				The Podlove Subscribe Button Plugin requires <code>PHP 5.3</code> or higher.<br>
33
				You are running <code>PHP <?php echo phpversion(); ?></code>.<br>
0 ignored issues
show
introduced by
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'phpversion'.
Loading history...
34
				Please ask your hoster how to upgrade to an up-to-date PHP version.
35
			</p>
36
		</div>
37
		<?php
38
	}
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
39
40
	function podlove_psb_deactivate() {
0 ignored issues
show
Coding Style Documentation introduced by
Missing doc comment for function podlove_psb_deactivate()
Loading history...
41
		deactivate_plugins( plugin_basename( __FILE__ ) );
42
	}
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
43
44
	add_action( 'admin_notices', 'podlove_psb_php_notice' );
45
	add_action( 'admin_init', 'podlove_psb_deactivate' );
46
47
	return;
48
49
}
50
51
require_once __DIR__ . '/vendor/autoload.php';
52
53
// Constants
0 ignored issues
show
Coding Style introduced by
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
54
require( 'constants.php' );
0 ignored issues
show
Coding Style introduced by
"require" is a statement not a function; no parentheses are required
Loading history...
55
// Version control
0 ignored issues
show
Coding Style introduced by
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
56
require( 'version.php' );
0 ignored issues
show
Coding Style introduced by
"require" is a statement not a function; no parentheses are required
Loading history...
57
58
register_activation_hook(   __FILE__, array( 'PodloveSubscribeButton\Setup', 'activation' ) );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 3 found
Loading history...
59
register_deactivation_hook( __FILE__, array( 'PodloveSubscribeButton\Setup', 'deactivation' ) );
60
register_uninstall_hook(    __FILE__, array( 'PodloveSubscribeButton\Setup', 'uninstall' ) );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 4 found
Loading history...
61
62
PodloveSubscribeButton::run();
63
64
/**
65
 * Class PodloveSubscribeButton
66
 */
67
class PodloveSubscribeButton {
68
69
	/**
0 ignored issues
show
Coding Style introduced by
Missing short description in doc comment
Loading history...
70
	 * @var string current plugin version
71
	 */
72
	public static $version = '1.4.0-beta';
73
74
	public static function run() {
0 ignored issues
show
Coding Style Documentation introduced by
Missing doc comment for function run()
Loading history...
75
		add_action( 'plugins_loaded', array( __CLASS__, 'load_translations' ) );
76
		add_action( 'init', array( __CLASS__, 'register_shortcode' ) );
77
		add_action( 'admin_init', array( __CLASS__, 'register_settings' ) );
78
		add_action( 'admin_init', array( 'PodloveSubscribeButton\Settings\Buttons', 'process_form' ) );
79
		add_action( 'admin_enqueue_scripts', array( __CLASS__, 'enqueue_assets' ) );
80
		add_action( 'widgets_init', array( __CLASS__, 'widgets' ) );
81
		self::menu();
82
83
	}
84
85
	public static function widgets() {
0 ignored issues
show
Coding Style Documentation introduced by
Missing doc comment for function widgets()
Loading history...
86
		register_widget( '\PodloveSubscribeButton\Widget' );
87
88
	}
89
90
	public static function menu() {
0 ignored issues
show
Coding Style Documentation introduced by
Missing doc comment for function menu()
Loading history...
91
		add_action( 'admin_menu', array( 'PodloveSubscribeButton', 'admin_menu' ) );
92
93
		if ( is_network_admin() ) {
94
			add_action( 'network_admin_menu', array( 'PodloveSubscribeButton', 'admin_network_menu' ) );
95
		}
96
97
	}
98
99
	public static function enqueue_assets( $hook ) {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::enqueue_assets" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function enqueue_assets()
Loading history...
100
101
		$pages = array( 'settings_page_podlove-subscribe-button', 'widgets.php' );
102
103
		if ( ! in_array( $hook, $pages ) ) {
0 ignored issues
show
introduced by
Not using strict comparison for in_array; supply true for third argument.
Loading history...
104
			return;
105
		}
106
107
		// CSS Stylesheet
0 ignored issues
show
Coding Style introduced by
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
108
		wp_register_style( 'podlove-subscribe-button', \PodloveSubscribeButton\Helpers::get_url( '' ) . 'css/style.css' , false, self::$version );
0 ignored issues
show
Bug introduced by
false of type false is incompatible with the type array expected by parameter $deps of wp_register_style(). ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

108
		wp_register_style( 'podlove-subscribe-button', \PodloveSubscribeButton\Helpers::get_url( '' ) . 'css/style.css' , /** @scrutinizer ignore-type */ false, self::$version );
Loading history...
Coding Style introduced by
Space found before comma in function call
Loading history...
109
		wp_enqueue_style( 'podlove-subscribe-button' );
110
111
		// Admin JS
0 ignored issues
show
Coding Style introduced by
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
112
		wp_enqueue_style( 'wp-color-picker' );
113
		wp_register_script( 'podlove-subscribe-button-admin-tools', \PodloveSubscribeButton\Helpers::get_url( '' ) . 'js/admin.js', array( 'jquery', 'wp-color-picker' ), self::$version );
0 ignored issues
show
introduced by
In footer ($in_footer) is not set explicitly wp_register_script; It is recommended to load scripts in the footer. Please set this value to true to load it in the footer, or explicitly false if it should be loaded in the header.
Loading history...
114
		$js_translations = array(
115
			'media_library' => __( 'Media Library', 'podlove-subscribe-button' ),
116
			'use_for'       => __( 'Use for Podcast Cover Art', 'podlove-subscribe-button' ),
117
		);
118
		wp_localize_script( 'podlove-subscribe-button-admin-tools', 'i18n', $js_translations );
119
		wp_enqueue_script( 'podlove-subscribe-button-admin-tools' );
120
121
	}
122
123
	public static function admin_menu() {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::admin_menu" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function admin_menu()
Loading history...
124
		add_options_page(
125
			__( 'Podlove Subscribe Button Options', 'podlove-subscribe-button' ),
126
			__( 'Podlove Subscribe Button', 'podlove-subscribe-button' ),
127
			'manage_options',
128
			'podlove-subscribe-button',
129
			array( 'PodloveSubscribeButton\Settings\Buttons', 'page' )
130
		);
131
132
	}
133
134
	public static function admin_network_menu() {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::admin_network_menu" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function admin_network_menu()
Loading history...
135
		add_submenu_page(
136
			'settings.php',
137
			__( 'Podlove Subscribe Button Options', 'podlove-subscribe-button' ),
138
			__( 'Podlove Subscribe Button', 'podlove-subscribe-button' ),
139
			'manage_options',
140
			'podlove-subscribe-button',
141
			array( 'PodloveSubscribeButton\Settings\Buttons', 'page' )
142
		);
143
144
	}
145
146
	public static function load_translations() {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::load_translations" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function load_translations()
Loading history...
147
		load_plugin_textdomain( 'podlove-subscribe-button' );
148
149
	}
150
151
	public static function register_settings() {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::register_settings" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function register_settings()
Loading history...
152
		$settings = array(
153
			'size',
154
			'autowidth',
155
			'style',
156
			'format',
157
			'color',
158
			'language',
159
		);
160
161
		foreach ( $settings as $setting ) {
162
			register_setting( 'podlove-subscribe-button', 'podlove_subscribe_button_default_' . $setting );
163
		}
164
165
	}
166
167
	static function get_option( $key, $default = false ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
Coding Style introduced by
Method name "PodloveSubscribeButton::get_option" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function get_option()
Loading history...
Comprehensibility Best Practice introduced by
It is recommend to declare an explicit visibility for get_option.

Generally, we recommend to declare visibility for all methods in your source code. This has the advantage of clearly communication to other developers, and also yourself, how this method should be consumed.

If you are not sure which visibility to choose, it is a good idea to start with the most restrictive visibility, and then raise visibility as needed, i.e. start with private, and only raise it to protected if a sub-class needs to have access, or public if an external class needs access.

Loading history...
168
169
		return \get_option( 'podlove_subscribe_button_default_' . $key, $default );
170
171
	}
172
173
	public static function register_shortcode() {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::register_shortcode" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function register_shortcode()
Loading history...
174
		add_shortcode( 'podlove-subscribe-button', array( 'PodloveSubscribeButton', 'shortcode' ) );
175
176
	}
177
178
179
180
	/**
0 ignored issues
show
Coding Style Documentation introduced by
Doc comment for parameter "$args" missing
Loading history...
181
	 * Add the shortcode
182
	 *
183
	 * @param $args
0 ignored issues
show
Coding Style introduced by
Missing parameter name
Loading history...
184
	 *
185
	 * @return string|void
186
	 */
187
	public static function shortcode( $args ) {
188
		if ( ! $args || ! isset( $args[ 'button' ] ) ) {
0 ignored issues
show
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
189
			return __( 'You need to create a Button first and provide its ID.', 'podlove-subscribe-button' );
190
		} else {
191
			$buttonid = $args[ 'button' ];
0 ignored issues
show
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
192
		}
193
194
		// Fetch the (network)button by it's name
0 ignored issues
show
Coding Style introduced by
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
195
		if ( ! $button = \PodloveSubscribeButton\Model\Button::get_button_by_name( $args[ 'button' ] ) )
0 ignored issues
show
Coding Style Best Practice introduced by
It is generally a best practice to always use braces with control structures.

Adding braces to control structures avoids accidental mistakes as your code changes:

// Without braces (not recommended)
if (true)
    doSomething();

// Recommended
if (true) {
    doSomething();
}
Loading history...
introduced by
Variable assignment found within a condition. Did you mean to do a comparison?
Loading history...
Coding Style introduced by
Assignments must be the first block of code on a line
Loading history...
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
196
			return sprintf( __( 'Oops. There is no button with the ID "%s".', 'podlove-subscribe-button' ), $args[ 'button' ] );
0 ignored issues
show
introduced by
A gettext call containing placeholders was found, but was not accompanied by a "translators:" comment on the line above to clarify the meaning of the placeholders.
Loading history...
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
197
198
		// Get button styling and options
0 ignored issues
show
Coding Style introduced by
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
199
		$autowidth = self::interpret_width_attribute( self::get_array_value_with_fallback( $args, 'width' ) );
0 ignored issues
show
Bug introduced by
It seems like self::get_array_value_wi...allback($args, 'width') can also be of type false; however, parameter $width_attribute of PodloveSubscribeButton::...rpret_width_attribute() does only seem to accept string, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

199
		$autowidth = self::interpret_width_attribute( /** @scrutinizer ignore-type */ self::get_array_value_with_fallback( $args, 'width' ) );
Loading history...
200
		$size      = self::get_attribute( 'size', self::get_array_value_with_fallback( $args, 'size' ) );
0 ignored issues
show
Bug introduced by
It seems like self::get_array_value_wi...fallback($args, 'size') can also be of type false; however, parameter $attribute_value of PodloveSubscribeButton::get_attribute() does only seem to accept string, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

200
		$size      = self::get_attribute( 'size', /** @scrutinizer ignore-type */ self::get_array_value_with_fallback( $args, 'size' ) );
Loading history...
201
		$style     = self::get_attribute( 'style', self::get_array_value_with_fallback( $args, 'style' ) );
202
		$format    = self::get_attribute( 'format', self::get_array_value_with_fallback( $args, 'format' ) );
203
204
		if ( isset( $args[ 'language' ] ) ) {
0 ignored issues
show
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
205
			$language = $args[ 'language' ];
0 ignored issues
show
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
206
		} else {
207
			$language = self::get_attribute( 'language', self::get_array_value_with_fallback( $args, 'language' ) );
208
		}
209
210
		if ( isset( $args[ 'color' ] ) ) {
0 ignored issues
show
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
211
			$color = $args[ 'color' ];
0 ignored issues
show
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
212
		} else {
213
			$color = self::get_attribute( 'color', self::get_array_value_with_fallback( $args, 'color' ) );
214
		}
215
216
		if ( isset( $args[ 'hide' ] ) && $args[ 'hide' ] == 'true' ) {
0 ignored issues
show
introduced by
Array keys must NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
introduced by
Found: ==. Use strict comparisons (=== or !==).
Loading history...
introduced by
Use Yoda Condition checks, you must.
Loading history...
217
			$hide = true;
218
		} else {
219
			$hide = false;
220
		}
221
222
		// Render button
0 ignored issues
show
Coding Style introduced by
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
223
		return $button->render( $size, $autowidth, $style, $format, $color, $hide, $buttonid, $language );
224
225
	}
226
227
	public static function get_array_value_with_fallback( $args, $key ) {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::get_array_value_with_fallback" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function get_array_value_with_fallback()
Loading history...
228
		if ( isset( $args[ $key ] ) ) {
229
			return $args[ $key ];
230
		}
231
232
		return false;
233
234
	}
235
236
	/**
0 ignored issues
show
Coding Style introduced by
Missing short description in doc comment
Loading history...
237
	 * @param  string $attribute
0 ignored issues
show
Coding Style Documentation introduced by
Missing parameter comment
Loading history...
238
	 * @param  string $attribute_value
0 ignored issues
show
Coding Style Documentation introduced by
Missing parameter comment
Loading history...
239
	 * @return string
240
	 */
241
	private static function get_attribute( $attribute = null, $attribute_value = null ) {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::get_attribute" is not in camel caps format
Loading history...
242
		if ( isset( $attribute_value ) && ctype_alnum( $attribute_value ) && key_exists( $attribute_value, \PodloveSubscribeButton\Model\Button::$$attribute ) ) {
243
			return $attribute_value;
244
		} else {
245
			return get_option( 'podlove_subscribe_button_default_' . $attribute, \PodloveSubscribeButton\Model\Button::$properties[ $attribute ] );
246
		}
247
248
	}
249
250
	/**
251
	 * Interprets the provided width attribute and return either auto- or a specific width
252
	 *
253
	 * @param  string $width_attribute
0 ignored issues
show
Coding Style Documentation introduced by
Missing parameter comment
Loading history...
254
	 * @return string
255
	 */
256
	private static function interpret_width_attribute( $width_attribute = null ) {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::interpret_width_attribute" is not in camel caps format
Loading history...
257
		if ( $width_attribute == 'auto' ) {
0 ignored issues
show
introduced by
Found: ==. Use strict comparisons (=== or !==).
Loading history...
introduced by
Use Yoda Condition checks, you must.
Loading history...
258
			return 'on';
259
		}
260
261
		if ( $width_attribute && $width_attribute !== 'auto' ) {
0 ignored issues
show
introduced by
Use Yoda Condition checks, you must.
Loading history...
262
			return 'off';
263
		}
264
265
		return self::get_option( 'autowidth', 'on' );
266
267
	}
268
269
	public static function plugin_file() {
0 ignored issues
show
Coding Style introduced by
Method name "PodloveSubscribeButton::plugin_file" is not in camel caps format
Loading history...
Coding Style Documentation introduced by
Missing doc comment for function plugin_file()
Loading history...
270
		return __FILE__;
271
	}
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
272
273
} // END class
274