SFS\SemanticFormsSelectInput

Complexity

Total Complexity

33

Size/Duplication

Total Lines	225
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getName()	0	3	1
A	getParameters()	0	4	1
A	getResourceModuleNames()	0	10	1
A	getHtmlText()	0	8	1
F	getHTML()	0	167	28

<?php

namespace SFS;

use SMWQueryProcessor as QueryProcessor;
use Parser;
use PFFormInput;
use MWDebug;

/**
 * @license GNU GPL v2+
 * @since 1.3
 *
 * @author Jason Zhang
 * @author Toni Hermoso Pulido
 * @author Alexander Gesinn
 */
class SemanticFormsSelectInput extends PFFormInput {

	/**
	 * Internal data container
	 *
	 * @var array
	 */
	private static $data = array();

	public function __construct( $inputNumber, $curValue, $inputName, $disabled, $otherArgs ) {
		parent::__construct( $inputNumber, $curValue, $inputName, $disabled, $otherArgs );
	}

	public static function getName() {
		return 'SF_Select';
	}

	public static function getParameters() {
		$params = parent::getParameters();
		return $params;
	}

	public function getResourceModuleNames() {
		/**
		 * Loading modules this way currently fails with:
		 * "mw.loader.state({"ext.sf_select.scriptselect":"loading"});"
		 */

		return array(
			'ext.sf_select.scriptselect'
		);
	}

	/**
	 * Returns the HTML code to be included in the output page for this input.
	 * This is currently just a wrapper for getHTML().
	 */
	public function getHtmlText() {
		return self::getHTML(

The method SFS\SemanticFormsSelectInput::getHTML() has been deprecated with message: use getHtmlText() instead

			$this->mCurrentValue,
			$this->mInputName,
			$this->mIsMandatory,
			$this->mIsDisabled,
			$this->mOtherArgs );
	}

	/**
	 * Returns the HTML code to be included in the output page for this input.
	 * @deprecated use getHtmlText() instead
	 *
	 * @param $cur_value
	 * @param $input_name
	 * @param $is_mandatory
	 * @param $is_disabled
	 * @param $other_args
	 * @return string
	 */
	public function getHTML( $cur_value, $input_name, $is_mandatory, $is_disabled, $other_args ) {
		global # $wgScriptSelectCount,

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

		$wgScriptSelectCount, $sfgFieldNum, $wgUser, $wgParser;

		$selectField = array();
		$values = null;
		$staticvalue = false;
		$data = array();

		if ( array_key_exists( "query", $other_args ) ) {
			$query = $other_args["query"];
			$query = str_replace( array( "~", "(", ")" ), array( "=", "[", "]" ), $query );

			$selectField["query"] = $query;

			// unparametrized query
			if ( strpos( $query, '@@@@' ) === false ) {
				$params = explode( ";", $query );

				// there is no need to run the parser, $query has been parsed already
				//$params[0] = $wgParser->replaceVariables( $params[0] );

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

				$values = QueryProcessor::getResultFromFunctionParams( $params, SMW_OUTPUT_WIKI );

The method SMWQueryProcessor::getResultFromFunctionParams() has been deprecated with message: Will vanish after release of SMW 1.8.
See SMW_Ask.php for example code on how to get query results from
#ask function parameters.

				$staticvalue = true;
			}

		} elseif ( array_key_exists( "function", $other_args ) ) {
			$query = $other_args["function"];
			$query = '{{#' . $query . '}}';
			$query = str_replace( array( "~", "(", ")" ), array( "=", "[", "]" ), $query );

			$selectField["function"] = $query;

			// unparametrized function
			if ( strpos( $query, '@@@@' ) === false ) {
				$f = str_replace( ";", "|", $query );

				$values = $wgParser->replaceVariables( $f );

				$staticvalue = true;
			}
		}

		if ( $staticvalue ) {
			$values = explode( ",", $values );
			$values = array_map( "trim", $values );
			$values = array_unique( $values );
		} else {

			if ( $wgScriptSelectCount == 0 ) {

This if statement is empty and can be removed.

				// this has been moved to getResourceModuleNames()
				//Output::addModule( 'ext.sf_select.scriptselect' );

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

			}
			// $wgScriptSelectCount ++;

			$data["selectismultiple"] = array_key_exists( "part_of_multiple", $other_args );

			$index = strpos( $input_name, "[" );
			$data['selecttemplate'] = substr( $input_name, 0, $index );

			// Does hit work for multiple template?
			$index = strrpos( $input_name, "[" );
			$data['selectfield'] = substr( $input_name, $index + 1, strlen( $input_name ) - $index - 2 );

			$valueField = array();

$valueField is not used, you could remove the assignment.

			$data["valuetemplate"] =
				array_key_exists( "sametemplate", $other_args ) ? $data['selecttemplate'] : $other_args["template"];
			$data["valuefield"] = $other_args["field"];

			$data['selectrm'] = array_key_exists( 'rmdiv', $other_args );
			$data['label'] = array_key_exists( 'label', $other_args );
			$data['sep'] = array_key_exists( 'sep', $other_args ) ? $other_args["sep"] : ',';

			if ( array_key_exists( "query", $selectField ) ) {
				$data['selectquery'] = $selectField['query'];
			} else {
				$data['selectfunction'] = $selectField['function'];
			}

			self::$data[] = $data;
		}

		$extraatt = "";
		$is_list = false;

		// TODO This needs clean-up

		if ( array_key_exists( 'is_list', $other_args ) && $other_args['is_list'] == true ) {
			$is_list = true;
		}

		if ( $is_list ) {
			$extraatt = ' multiple="multiple" ';
		}

		if ( array_key_exists( "size", $other_args ) ) {
			$extraatt .= " size=\"{$other_args['size']}\"";
		}

		$classes = array();
		if ( $is_mandatory ) {
			$classes[] = "mandatoryField";
		}
		if ( array_key_exists( "class", $other_args ) ) {
			$classes[] = $other_args['class'];
		}
		if ( $classes ) {

The expression $classes of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

			$cstr = implode( " ", $classes );
			$extraatt .= " class=\"$cstr\"";
		}

		$inname = $input_name;
		if ( $is_list ) {
			$inname .= '[]';
		}

		// TODO Use Html::

		$spanextra = $is_mandatory ? 'mandatoryFieldSpan' : '';
		$ret = "<span class=\"inputSpan $spanextra\"><select name='$inname' id='input_$sfgFieldNum' $extraatt>";
		$curvalues = null;

$curvalues is not used, you could remove the assignment.

		if ( $cur_value ) {
			if ( $cur_value === 'current user' ) {
				$cur_value = $wgUser->getName();
			}
			if ( is_array( $cur_value ) ) {
				$curvalues = $cur_value;
			} else {
				$curvalues = array_map( "trim", explode( ",", $cur_value ) );
			}

		} else {
			$curvalues = array();
		}

		// TODO handle empty value case.
		$ret .= "<option></option>";

		foreach ( $curvalues as $cur ) {
			$ret .= "<option selected='selected'>$cur</option>";
		}

		if ( $staticvalue ) {
			foreach ( $values as $val ) {
				if ( !in_array( $val, $curvalues ) ) {
					$ret .= "<option>$val</option>";
				}
			}
		}

		$ret .= "</select></span>";
		$ret .= "<span id=\"info_$sfgFieldNum\" class=\"errorMessage\"></span>";

		if ( $other_args["is_list"] ) {
			$hiddenname = $input_name . '[is_list]';
			$ret .= "<input type='hidden' name='$hiddenname' value='1' />";
		}

		if ( !$staticvalue ) {
			$item = Output::addToHeadItem( $data );

$item is not used, you could remove the assignment.

			//$wgOut->addJsConfigVars('sf_select', array(json_encode( $data )));

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

		}

		Output::commitToParserOutput();
		return $ret;
	}
}