Ajax\semantic\widgets\dataform\FormInstanceViewer

Complexity

Total Complexity

19

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	2
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
B	_beforeAddProperty()	0	12	5
A	hasHeader()	0	8	2
A	addSeparatorAfter()	0	5	2
A	addHeaderDividerBefore()	0	6	2
A	getSeparators()	0	3	1
A	removeSeparator()	0	3	1
A	removeField()	0	11	3
A	setSeparators()	0	4	1
A	getHeaders()	0	3	1

<?php

namespace Ajax\semantic\widgets\dataform;

use Ajax\semantic\widgets\base\InstanceViewer;
use Ajax\service\JString;
use Ajax\semantic\html\collections\form\HtmlFormInput;

class FormInstanceViewer extends InstanceViewer {
	protected $separators;
	protected $headers;

	public function __construct($identifier,$instance=NULL, $captions=NULL) {
		parent::__construct($identifier,$instance=NULL, $captions=NULL);
		$this->separators=[-1];
		$this->headers=[];
		$this->defaultValueFunction=function($name,$value,$index){
			$caption=$this->getCaption($index);
			$input=new HtmlFormInput($this->widgetIdentifier."-".$name,$caption,"text",$value);
			$input->setName($name);
			return $input;
		};
	}

	protected function _beforeAddProperty($index,&$field){
		if(JString::endswith($field, "\n")===true){
			$this->addSeparatorAfter($index);
		}
		if($index>1 && JString::startswith($field, "\n")===true){
			$this->addSeparatorAfter($index-1);
		}
		$field=\str_replace("\n", "", $field);
		if(($header=$this->hasHeader($field))!==false){
			$this->addHeaderDividerBefore($index, $header);
		}
	}

	protected function hasHeader(&$field){
		$matches=[];$result=false;
		if(\preg_match('/\{(.*?)\}/s', $field, $matches)===1){
			$result=$matches[1];
			$field=\str_replace("{".$result."}","", $field);
		}
		return $result;
	}



	public function addSeparatorAfter($fieldNum){
		if(\array_search($fieldNum, $this->separators)===false)
			$this->separators[]=$fieldNum;
		return $this;
	}

	public function addHeaderDividerBefore($fieldNum,$header){
		$this->headers[$fieldNum]=$header;
		if($fieldNum>0)
			$this->addSeparatorAfter($fieldNum-1);
		return $this;
	}

	public function getSeparators() {
		return $this->separators;
	}

	public function removeSeparator($index){
		\array_splice($this->separators,$index,1);
	}

	public function removeField($index){
		parent::removeField($index);
		$pos=\array_search($index, $this->separators);
		if($pos!==false){
			for($i=$pos+1;$i<\sizeof($this->separators);$i++){

It seems like you are calling the size function sizeof() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

				$this->separators[$i]--;
			}
			\array_splice($this->separators, $pos, 1);
		}
		return $this;
	}

	public function setSeparators($separators) {
		$this->separators=\array_merge([-1], $separators);
		return $this;
	}

	public function getHeaders() {
		return $this->headers;
	}


}