podlove / podlove-subscribe-button-wp-plugin

inc/Helpers.php

<?php

Filenames should be all lowercase with hyphens as word separators. Expected helpers.php, but found Helpers.php.

Class file names should be based on the class name with "class-" prepended. Expected class-helpers.php, but found Helpers.php.

/**
 * @author    Podlove <podlove@podlove.org>
 * @copyright Copyright (c) 2014-2018, Podlove
 * @license   https://github.com/podlove/podlove-subscribe-button-wp-plugin/blob/master/LICENSE MIT
 * @package   Podlove\PodloveSubscribeButton
 */

namespace PodloveSubscribeButton;

Class Helpers {

As per coding-style, PHP keywords should be in lowercase; expected class, but found Class.

Missing doc comment for class Helpers

	public static function get_path( $folder = '' ) {

Method name "Helpers::get_path" is not in camel caps format

Missing doc comment for function get_path()

		$path = untrailingslashit( plugin_dir_path( \PodloveSubscribeButton::plugin_file() ) );

		return $path . $folder;

	} // End get_path()

	public static function get_url( $path = '' ) {

Method name "Helpers::get_url" is not in camel caps format

Missing doc comment for function get_url()

		$url = trailingslashit( plugins_url( $path, \PodloveSubscribeButton::plugin_file() ) );

		return $url;

	} // End get_url()

	/**
	 * Get plugin basename
	 *
	 * @since  1.0.0
	 *
	 * @return string
	 */
	public function get_basename() {

Method name "Helpers::get_basename" is not in camel caps format

		return plugin_basename( \PodloveSubscribeButton::plugin_file() );
	} // get_basename()

Expected 1 blank line before closing function brace; 0 found

67% of this comment could be valid code. Did you maybe forget this after debugging?

	/**
	 * Check if `Podlove Publisher` is installed + activated
	 *
	 * @return bool
	 */
	public static function is_podlove_publisher_active() {

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

		if ( is_plugin_active( "podlove-podcasting-plugin-for-wordpress/podlove.php" ) ) {
			return true;
		}

		return false;

	}

	/**
	 * Get button compatible language string.
	 *
	 * Examples:
	 *
	 *  language('de');    // => 'de'
	 *  language('de_DE'); // => 'de'
	 *  language('en_GB'); // => 'en'
	 *
	 * @param  string $language language identifier

Parameter comment must end with a full stop

	 *
	 * @return string
	 */
	static function language( $language ) {

It is generally recommended to explicitly declare the visibility for methods.

It is recommend to declare an explicit visibility for language.

		if ( empty( $language ) ) {
			$language = get_option( 'WPLANG' );
		}

		$lang_code = strtolower(explode('_', $language)[0]);

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		if ( in_array( $lang_code, \PodloveSubscribeButton\Defaults::button('language' ) ) ) {

Not using strict comparison for in_array; supply true for third argument.

Expected 1 spaces after opening bracket; 0 found

			return $lang_code;
		} else {
			return 'en';
		}

	}

	public static function for_every_podcast_blog( $callback ) {

Method name "Helpers::for_every_podcast_blog" is not in camel caps format

Missing doc comment for function for_every_podcast_blog()

		global $wpdb;

		/** @todo change to helper functionss */

Missing short description in doc comment

		$plugin  = basename( \PodloveSubscribeButton\PLUGIN_DIR ) . '/' . \PodloveSubscribeButton\PLUGIN_FILE_NAME;
		$blogids = $wpdb->get_col( "SELECT blog_id FROM " . $wpdb->blogs );

		if ( ! is_array( $blogids ) )
			return;

		foreach ( $blogids as $blog_id ) {
			switch_to_blog( $blog_id );
			if ( is_plugin_active( $plugin ) ) {
				$callback();
			}
			restore_current_blog();
		}

	}

} // END Class