Automattic\Jetpack\AdminPage\Page

Complexity

Total Complexity

9

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	4
A	admin_styles()	0	7	1
A	init()	0	7	1
A	render()	0	8	1
A	get_render_content()	0	13	2

<?php
/**
 * An admin page wrapper
 *
 * @package jetpack-admin-page
 */

namespace Automattic\Jetpack\AdminPage;

/**
 * Create and render a Jetpack logo.
 */
class Page {
	/**
	 * Absolute URL of the Jetpack logo.
	 *
	 * @var string
	 */
	private $header;

	private $footer;

	/**
	 * Constructor.
	 * You can optionally pass a URL to override the default one.
	 */
	public function __construct( $page_hook = null ) {
		if ( $page_hook ) {
			if ( is_array( $page_hook ) ) {
				foreach ( $page_hook as $hook ) {
					add_action( "admin_print_styles-$hook", array( $this, 'admin_styles' ) );
				}
			} else {
				add_action( "admin_print_styles-$page_hook", array( $this, 'admin_styles' ) );
			}
		}
	}

	public function admin_styles() {
		// TODO: implement min and rtl styles in some sort of build process?
		$version = time(); // where does this come from?
		wp_enqueue_style( 'jetpack-admin-page', plugins_url( 'assets/style.css', __DIR__ ), array(), $version );
		// wp_style_add_data( 'jetpack-admin', 'rtl', 'replace' );
		// wp_style_add_data( 'jetpack-admin', 'suffix', $min );
	}

	public function init() {
		$header       = new Header();
		$this->header = apply_filters( 'jetpack-admin-page-set-header', $header );

		$footer       = new Footer();
		$this->footer = apply_filters( 'jetpack-admin-page-set-footer', $footer );
	}

	/**
	 * Build and render an <img /> tag with the Jetpack logo.
	 *
	 * @return string
	 */
	public function render( $callback = null ) {
		$this->init();
		$html  = '';
		$html .= $this->header->render();

The method render cannot be called on $this->header (of type string).

		$html .= $this->get_render_content( $callback );
		$html .= $this->footer->render();
		return $html;
	}

	private function get_render_content( $callback ) {
		if ( is_callable( $callback ) ) {
			ob_start();
			?>
			<div class="jetpack-admin-page__container">
				<?php call_user_func( $callback ); ?>
			</div>
			<?php
			$html .= ob_get_contents();

The variable $html does not exist. Did you forget to declare it?

			ob_end_clean();
			return $html;
		}
	}
}