cfoellmann / podlove-subscribe-button-wp-plugin

widget.php

<?php

This file is missing a doc comment.

namespace PodloveSubscribeButton;

include_once( ABSPATH . 'wp-admin/includes/plugin.php' );

"include_once" is a statement not a function; no parentheses are required

File is being unconditionally included; use "require_once" instead

class Podlove_Subscribe_Button_Widget extends \WP_Widget {

This class is not in CamelCase format.

Missing class doc comment

	public function __construct() {

Missing function doc comment

		parent::__construct(
					'podlove_subscribe_button_wp_plugin_widget',

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 20.

					( self::is_podlove_publisher_active() ? 'Podlove Subscribe Button (WordPress plugin)' : 'Podlove Subscribe Button' ),

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 20.

					array( 'description' => __( 'Adds a Podlove Subscribe Button to your Sidebar', 'podlove-subscribe-button' ), )

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 20.

				);

This line of the multi-line function call does not seem to be indented correctly. Expected 8 spaces, but found 16.

	}

Expected 1 blank line before closing function brace; 0 found

	public static $widget_settings = array('infotext', 'title', 'size', 'style', 'format', 'autowidth', 'button', 'color');

Missing member variable doc comment

	public static function is_podlove_publisher_active() {

Missing function doc comment

Method name "Podlove_Subscribe_Button_Widget::is_podlove_publisher_active" is not in camel caps format

		if ( is_plugin_active("podlove-podcasting-plugin-for-wordpress/podlove.php") ) {

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

The string literal podlove-podcasting-plugin-for-wordpress/podlove.php does not require double quotes, as per coding-style, please use single quotes.

			return true;
		}

		return false;
	}

Expected 1 blank line before closing function brace; 0 found

	public function widget( $args, $instance ) {

Missing function doc comment

		// Fetch the (network)button by it's name

Inline comments must end in full-stops, exclamation marks, or question marks

		if ( ! $button = \PodloveSubscribeButton\Model\Button::get_button_by_name($instance['button']) )

It is generally a best practice to always use braces with control structures.

Assignments must be the first block of code on a line

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

			return sprintf( __('Oops. There is no button with the ID "%s".', 'podlove-subscribe-button'), $args['button'] );

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		echo $args['before_widget'];
		echo $args['before_title'] . apply_filters( 'widget_title', $instance['title'] ). $args['after_title'];

Concat operator must be surrounded by a single space

		echo $button->render(
				\PodloveSubscribeButton::get_array_value_with_fallback($instance, 'size'),

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 16.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

				\PodloveSubscribeButton::get_array_value_with_fallback($instance, 'autowidth'),

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 16.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

				\PodloveSubscribeButton::get_array_value_with_fallback($instance, 'style'),

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 16.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

				\PodloveSubscribeButton::get_array_value_with_fallback($instance, 'format'), 

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 16.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

				\PodloveSubscribeButton::get_array_value_with_fallback($instance, 'color')

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 16.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

			);

This line of the multi-line function call does not seem to be indented correctly. Expected 8 spaces, but found 12.

		
		if ( strlen($instance['infotext']) )

It is generally a best practice to always use braces with control structures.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

			echo wpautop($instance['infotext']);

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		echo $args['after_widget'];
	}	

Expected 1 blank line before closing function brace; 0 found

	public function form( $instance ) {

Missing function doc comment

		foreach (self::$widget_settings as $setting) {
			$$setting = isset( $instance[$setting] ) ? $instance[$setting] : '';
		}

		$buttons = \PodloveSubscribeButton\Model\Button::all();

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

		if ( is_multisite() )

It is generally a best practice to always use braces with control structures.

			$network_buttons = \PodloveSubscribeButton\Model\NetworkButton::all();

		$buttons_as_options = function ($buttons) {

Expected 1 spaces between opening bracket and argument "$buttons"; 0 found

			foreach ($buttons as $subscribebutton) {
				echo "<option value='".$subscribebutton->name."' ".( $subscribebutton->name == $button ? 'selected=\"selected\"' : '' )." >".$subscribebutton->title." (".$subscribebutton->name.")</option>";

Concat operator must be surrounded by a single space

The string literal > does not require double quotes, as per coding-style, please use single quotes.

The string literal ( does not require double quotes, as per coding-style, please use single quotes.

The string literal )</option> does not require double quotes, as per coding-style, please use single quotes.

			}
		}
		?>
		<p>
			<label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php _e( 'Title', 'podlove-subscribe-button' ); ?></label> 
			<input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" value="<?php echo $title; ?>" />

			<label for="<?php echo $this->get_field_id( 'color' ); ?>"><?php _e( 'Color', 'podlove-subscribe-button' ); ?></label>
			<input class="podlove_subscribe_button_color" id="<?php echo $this->get_field_id( 'color' ); ?>" name="<?php echo $this->get_field_name( 'color' ); ?>" value="<?php echo $color; ?>" />
			<style type="text/css">
				.sp-replacer {
					display: flex;
				}
				.sp-preview {
					flex-grow: 10;
				}
			</style>

			<label for="<?php echo $this->get_field_id( 'button' ); ?>"><?php _e( 'Button', 'podlove-subscribe-button' ); ?></label> 
			<select class="widefat" id="<?php echo $this->get_field_id( 'button' ); ?>"
				      name="<?php echo $this->get_field_name( 'button' ); ?>">
				<?php if ( isset($network_buttons) && count($network_buttons) > 0 ) : ?>

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

					<optgroup label="<?php _e('Local', 'podlove-subscribe-button' ); ?>">

Expected 1 spaces after opening bracket; 0 found

						<?php $buttons_as_options($buttons); ?>
					</optgroup>
					<optgroup label="<?php _e('Network', 'podlove-subscribe-button' ); ?>">

Expected 1 spaces after opening bracket; 0 found

						<?php $buttons_as_options($network_buttons); ?>
					</optgroup>
				<?php else : 

Opening PHP tag must be on a line by itself

					$buttons_as_options($buttons);
				 endif; ?>

Closing brace indented incorrectly; expected 16 spaces, found 17

Closing PHP tag must be on a line by itself

			</select>

			<?php
			$customize_options = array(
				'size'      => array(
					'name'    => __( 'Size', 'podlove-subscribe-button' ),
					'options' => \PodloveSubscribeButton\Model\Button::$size
				),
				'style'     => array(
					'name'    => __( 'Style', 'podlove-subscribe-button' ),
					'options' => \PodloveSubscribeButton\Model\Button::$style
				),
				'format'    => array(
					'name'    => __( 'Format', 'podlove-subscribe-button' ),
					'options' => \PodloveSubscribeButton\Model\Button::$format
				),
				'autowidth' => array(
					'name'    => __( 'Autowidth', 'podlove-subscribe-button' ),
					'options' => \PodloveSubscribeButton\Model\Button::$width
				)
			);

			foreach ($customize_options as $slug => $properties) : ?>

Closing PHP tag must be on a line by itself

				<label for="<?php echo $this->get_field_id( $slug ); ?>"><?php echo $properties['name']; ?></label>
				<select class="widefat" id="<?php echo $this->get_field_id( $slug ); ?>" name="<?php echo $this->get_field_name( $slug ); ?>">
					<option value="default" <?php echo ( $$slug == 'default' ? 'selected="selected"' : '' ); ?>><?php printf( __( 'Default %s', 'podlove-subscribe-button' ), $properties['name'] ) ?></option>

Inline PHP statement must end with a semicolon

					<optgroup>
						<?php foreach ( $properties['options'] as $property => $name ) : ?>
						<option value="<?php echo $property; ?>" <?php echo ( $$slug == $property ? 'selected="selected"' : '' ); ?>><?php echo $name; ?></option>
						<?php endforeach; ?>
					</optgroup>
				</select>
			<?php endforeach; ?>
		
			<label for="<?php echo $this->get_field_id( 'infotext' ); ?>"><?php _e( 'Description', 'podlove-subscribe-button' ); ?></label> 
			<textarea class="widefat" rows="10" id="<?php echo $this->get_field_id( 'infotext' ); ?>" name="<?php echo $this->get_field_name( 'infotext' ); ?>"><?php echo $infotext; ?></textarea>
		</p>
		<?php 
	}

Expected 1 blank line before closing function brace; 0 found

	public function update( $new_instance, $old_instance ) {

Missing function doc comment

		$instance = array();

		foreach (self::$widget_settings as $setting) {
			$instance[$setting]  = ( ! empty( $new_instance[$setting] ) ) ? strip_tags( $new_instance[$setting] ) : '';

Equals sign not aligned correctly; expected 1 space but found 2 spaces

		}

		return $instance;
	}

Expected 1 blank line before closing function brace; 0 found

}
add_action( 'widgets_init', function(){

Expected 1 space before opening brace; found 0

     register_widget( '\PodloveSubscribeButton\Podlove_Subscribe_Button_Widget' );
});