SFS\Output

Complexity

Total Complexity

2

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

33.33%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	addToHeadItem()	0	3	1
A	commitToParserOutput()	0	10	1

<?php

namespace SFS;

//use MWDebug;

/**
 * @license GNU GPL v2+
 * @since 1.3
 *
 * @author mwjames
 * @author Alexander Gesinn
 */
class Output {

	/**
	 * @var array
	 */
	private static $headItems = array();

	/**
	 * Add an array of SF_Select field parameters as defined in Page Form's field tag.
	 *
	 * This will later be added to $wgOut so that JS can access it via mw.config.get
	 *
	 * @param array $data
	 */
	public static function addToHeadItem( Array $data = [] ) {
		return self::$headItems[] = $data;
	}

	/**
	 * Commit all SF_Select field parameters to Output
	 *
	 */
	public static function commitToParserOutput() {
		global $wgOut;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

		// to be used in JS like:
		// var SFSelect_fobjs = $.parseJSON( mw.config.get( 'sf_select' ) );
		$wgOut->addJsConfigVars('sf_select', json_encode( self::$headItems ));

		//self::$resourceModules = array();

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

		//self::$headItems = array();

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

	}
}