markmarco16\git\components\Repository::formatDiff()

Complexity

Conditions	15
Paths	11

Size

Total Lines	52
Code Lines	36

Duplication

Lines	13
Ratio	25 %

Importance

Changes

0

<?php
	public function formatDiff($text) {
        $output = array();
        $output['info'] = '';
        /*
			old mode <mode>
       		new mode <mode>
       		new file mode <mode>
       		deleted file mode <mode>
       		copy from <path>
       		copy to <path>
       		rename from <path>
       		rename to <path>
       		similarity index <number>
       		dissimilarity index <number>
       		index <hash>..<hash> <mode>
        */
		foreach ($text as $item) {
            if ('diff' === substr($item, 0, 4)) {
				preg_match('#^diff --git a/(.*) b/(.*)$#', $item, $matches);
				$output['name'] = $matches[1];
				$output['info'] .= "<pre>".$item."</pre>";
			} elseif (('new file'===substr($item,0,8)) || ('old mode'===substr($item,0,8)) || 
					('new mode'===substr($item,0,8)) || ('deleted file'===substr($item,0,12)) || 
					('index' === substr($item, 0, 5))) {
				$output['info'] .= "<pre>".$item."</pre>";
			} elseif ('Binary files'===substr($item,0,12)) {

This code seems to be duplicated across your project.

                $output['contents'][] = array('lineNumOld'=> "-", 'lineNumNew'=> "-", 'lineCode' => "<pre class='chunk'>".$item."</pre>",);
			} elseif ('@@' === substr($item, 0, 2)) {
                preg_match('/@@ -([0-9]+)/', $item, $matches);
                $lineNumOld = $matches[1] - 1;
                $lineNumNew = $matches[1] - 1;
                $output['contents'][] = array('lineNumOld'=> "-", 'lineNumNew'=> "-", 'lineCode' => "<pre class='chunk'>".$item."</pre>",);
			} elseif ('---' === substr($item, 0, 3)) {

This code seems to be duplicated across your project.

				$output['info'] .= "<pre class='old'>".$item."</pre>";
			} elseif ('+++' === substr($item, 0, 3)) {
				$output['info'] .= "<pre class='new'>".$item."</pre>";
			} elseif ('\ ' === substr($item, 0, 2)) {

This code seems to be duplicated across your project.

                $output['contents'][] = array('lineNumOld'=> "-", 'lineNumNew'=> "-", 'lineCode'=> "<pre>".htmlspecialchars($item)."</pre>",);
			} elseif ('-' === substr($item, 0, 1)) {
                $lineNumOld++;

The variable $lineNumOld does not seem to be defined for all execution paths leading up to this point.

                $output['contents'][] = array('lineNumOld'=> $lineNumOld, 'lineNumNew'=> "-", 'lineCode'=> "<pre class='old'>".htmlspecialchars($item)."</pre>",);
            } elseif ('+' === substr($item, 0, 1)) {

This code seems to be duplicated across your project.

                $lineNumNew++;

The variable $lineNumNew does not seem to be defined for all execution paths leading up to this point.

                $output['contents'][] = array('lineNumOld'=> "-", 'lineNumNew'=> $lineNumNew, 'lineCode'=> "<pre class='new'>".htmlspecialchars($item)."</pre>",);
            } else {
                $lineNumNew++;
                $lineNumOld++;
                $output['contents'][] = array('lineNumOld'=> $lineNumOld, 'lineNumNew'=> $lineNumNew, 'lineCode'=> "<pre>".htmlspecialchars($item)."</pre>",);
			}
		}
		return $output;
	}

}